[MTK]Camera方向与颜色异常排查复盘

最初现象

- 相机画面方向不对
- 必须把画面旋转 180 度，显示才正常
- 怀疑点放在 sensor driver 的 `mirror/flip` 配置

一开始修改方向的点放到这个函数上

static struct imgsensor_struct imgsensor = {
	.mirror = IMAGE_NORMAL,

后面画面一直是需要旋转180°的，是有地方覆盖掉了。

在这个驱动里，几个场景函数原来都在调用：

set_mirror_flip(IMAGE_V_MIRROR);

包括：

- `preview()`
- `capture()`
- `normal_video()`
- `hs_video()`
- `slim_video()`

- 现在跑的是 `IMAGE_V_MIRROR`
- 画面需要再转 180 度才对
- 那是不是应该改成 `IMAGE_H_MIRROR` 或 `IMAGE_HV_MIRROR`

进一步推导：

- 如果当前运行的是 `IMAGE_V_MIRROR`
- 而你观察到“在当前画面基础上再旋转 180 度才对”
- 那目标值并不是 `IMAGE_HV_MIRROR`
- 而是 `IMAGE_H_MIRROR`

因为：

- 当前状态是 `V`
- 再叠加一个 180 度，相当于 `V + HV = H`

排查

第二阶段验证：为什么改了 IMAGE_ 也没有效果*

- 试过 `IMAGE_V_MIRROR`
- 试过 `IMAGE_HV_MIRROR`
- 都没有解决问题
- 后来甚至改成 `IMAGE_NORMAL`
- 画面仍然和 `IMAGE_HV_MIRROR` 的表现一样

- 场景函数里写入的 `set_mirror_flip()`，不是最终生效值

如果 `IMAGE_NORMAL` 都和 `IMAGE_HV_MIRROR` 看起来一样，就不能再继续怀疑“选错了哪个枚举”，而应该立刻转向：

- 后面是不是还有地方把 `0x0101` 覆盖掉了

**真正的可疑点：streaming_control()**

驱动中的 `streaming_control()` 有这样一段代码：
```c
if (enable)
{
    while (1)
    {
        write_cmos_sensor(0x0100, 0x0103);
        framecnt = read_cmos_sensor_byte(0x0005);
        ...
    }
}
else
{
    while (1)
    {
        write_cmos_sensor_byte(0x0100, 0x00);
        framecnt = read_cmos_sensor_byte(0x0005);
        ...
    }
}

这里最可疑的地方有两个：

stream off 用的是 write_cmos_sensor_byte(0x0100, 0x00)
stream on 却用的是 write_cmos_sensor(0x0100, 0x0103)

也就是说，开流和关流的寄存器写法不对称。

```text

为什么 write_cmos_sensor(0x0100, 0x0103) 很可疑

1 两种写寄存器接口的区别

本驱动里：

static void write_cmos_sensor_byte(kal_uint32 addr, kal_uint32 para)
{
    char pu_send_cmd[3] = {
        (char)(addr >> 8),
        (char)(addr & 0xFF),
        (char)(para & 0xFF)
    };
    iWriteRegI2C(pu_send_cmd, 3, imgsensor.i2c_write_id);
}

这是 8-bit data 写法。

static void write_cmos_sensor(kal_uint16 addr, kal_uint16 para)
{
    char pusendcmd[4] = {
        (char)(addr >> 8),
        (char)(addr & 0xFF),
        (char)(para >> 8),
        (char)(para & 0xFF)
    };
    iWriteRegI2C(pusendcmd, 4, imgsensor.i2c_write_id);
}

这是 16-bit data 写法。

2 为什么这会影响 mirror/flip

```text

set_mirror_flip()` 写的是：

- `0x0101`

而 `stream on` 写的是：

- `0x0100, 0x0103`

如果这颗 sensor 的 `0x0100`、`0x0101` 是连续 8-bit 寄存器，那么：
```c
write_cmos_sensor(0x0100, 0x0103);

非常像是在一次 transaction 里顺手写成：

0x0100 = 0x01
0x0101 = 0x03

这就意味着：

刚刚通过 set_mirror_flip() 设置的方向
会在 stream on 时被强制改回 0x03
也就是 IMAGE_HV_MIRROR

这正好解释了为什么：

你改成 IMAGE_NORMAL
实际画面却还是 HV 的效果

这是本次排查最关键的根因判断。

```text

## 方向修复

## 1 方向修复

最终的思路不是继续尝试 `IMAGE_H`、`IMAGE_V`、`IMAGE_HV`，而是先保证：

- `stream on` 只控制 `0x0100`
- `mirror/flip` 只由 `set_mirror_flip()` 控制

因此开流逻辑改成只写：
```c
write_cmos_sensor_byte(0x0100, 0x01);

然后场景函数统一保留：

set_mirror_flip(IMAGE_NORMAL);

验证结果

方向恢复正常

```text

## 2 颜色修复

方向修好以后，颜色异常。

这时要联想到第二条经验：

- 改了 mirror/flip
- Bayer 首像素顺序也可能变了

驱动中原来配置的是：

```c
.sensor_output_dataformat = SENSOR_OUTPUT_FORMAT_RAW_Gb

但根据 set_mirror_flip() 的注释映射：

IMAGE_NORMAL 对应 0x0101 = 0x00，注释是 GR

所以最终把：

SENSOR_OUTPUT_FORMAT_RAW_Gb

改成：

SENSOR_OUTPUT_FORMAT_RAW_Gr

验证结果

颜色恢复正常

```text