首页 > 代码库 > Camera兼容引起的打开失败问题

Camera兼容引起的打开失败问题

平台:展讯SC7730

Camera IC:前GC0329后GC0328

近日,在解决项目BUG的过程中,测试部门有报:在进入Camera的时候,第一次第二次点击Camera图标都是打开失败(无法连接的相机),第三次就可以正常打开了。

 

解决思路:

1、首先是硬件主板电路存在问题,给Camera上电电压不对,但经过示波器测量,发现供给电压(AVDD,IOVDD,DVDD)都是正确的,主板没有问题。

2、怀疑Camera 没有焊好,重新焊接,此问题还是会复现,说明不是焊接问题。

3、得出结论:问题出自软件的可能性最大。

 

软件解决思路:

1、首先打log,查看log,看是否能读到Camera的ID,来判断主板是否给Camera上电成功。通过log,看到能正确读到Camera的ID。

  说明Camera需要的几个电压(AVDD,IOVDD,DVDD)都正确配置,之前用示波器测量电压也再次说明主板供电没问题。

2、是不是兼容出现问题。其它IC引起了GC0328+GC0329的不能正常打开。在sensor_cfg.c里,先临时把兼容的其它IC去掉,后Camera只保留GC0328前只保留  GC0329,经验证发现,此种方法是有效的,不存在前2次打不开Camera问题。

3、问题就出在了兼容上,问题还是集中在上电函数上,因为尽管电压设置正确,但影响Camera打开的还有PWN和RESET这个两个引脚。而这两个引脚恰恰在上电函  数中进行配置。

4、首先怀疑PWN设置不对,尝试在Camera打开之前,先把PWN拉高,在sensor_drv_k.c的probe函数里,在申请MAIN Camera和SUB Camera的PWN的GPIO  时,顺便把PWN拉高,发现此种方法没有作用。

5、回到IC的上电函数,去排查兼容GC0328的前一个IC GC0308的上电函数,POWEROFF的代码,在PWN的设置上没有问题。兼容GC0329的前一个IC GC0309  的上电函数,POWEROFF的代码,在PWN的设置上也没有问题。

6、期间还尝试了加大上电的延时,看Camera的上电timing等,PWN的设置确实没有什么问题。

7、这时就开始怀疑Reset引脚,经对比发现GC038和GC0328的对reset引脚的配置函数不一样:

  GC0328:

Sensor_SetResetLevel((BOOLEAN)!reset_level);

 GC0308:

Sensor_Reset(reset_level);    

继续跟踪这两个函数:

第一个Reset函数:sensor_drv_u.c定义:

BOOLEAN Sensor_SetResetLevel(BOOLEAN plus_level)
{
    if (-1 ==_Sensor_Device_ResetLevel((uint32_t)plus_level))
        return SENSOR_FAIL;

    return SENSOR_SUCCESS;
}
LOCAL int _Sensor_Device_ResetLevel(uint32_t level)
{
    int ret = SENSOR_SUCCESS;
    SENSOR_DRV_CHECK_ZERO(s_p_sensor_cxt);

    ret = xioctl(s_p_sensor_cxt->fd_sensor, SENSOR_IO_RST_LEVEL, &level);
    if (0 != ret)
    {
        CMR_LOGE("_Sensor_Device_Reset failed,  level = %d, ret=%d \n", level, ret);
        ret = -1;
    }

    return ret;
}

最后通过ioctl进入kernel:sensor_drv_k.c

    case SENSOR_IO_RST_LEVEL:
        {
            uint32_t level;
            ret = copy_from_user(&level, (uint32_t *) arg, sizeof(uint32_t));
            if (0 == ret)
                ret = _sensor_k_set_rst_level(level);
        }
        break;
LOCAL int _sensor_k_set_rst_level(uint32_t plus_level)
{
    SENSOR_PRINT("sensor set rst lvl: lvl %d, rst pin %d \n", plus_level, GPIO_SENSOR_RESET);

    gpio_direction_output(GPIO_SENSOR_RESET, plus_level);
    gpio_set_value(GPIO_SENSOR_RESET, plus_level);


    return SENSOR_K_SUCCESS;
}

而第二个reset函数:

void Sensor_Reset(uint32_t level)
{
    int err = 0xff;
    uint32_t rst_val[2];
    SENSOR_IOCTL_FUNC_PTR reset_func;

    CMR_LOGI("in.\n");
    SENSOR_DRV_CHECK_ZERO_VOID(s_p_sensor_cxt);

    if (PNULL == s_p_sensor_cxt->sensor_info_ptr) {
        CMR_LOGE("Sensor_SetI2CClock: No sensor info \n");
        return;
    }

    reset_func = s_p_sensor_cxt->sensor_info_ptr->ioctl_func_tab_ptr->reset;

    if (PNULL != reset_func) {
        reset_func(level);
    } else {
        rst_val[0] = level;
        rst_val[1] = s_p_sensor_cxt->sensor_info_ptr->reset_pulse_width;
        if (rst_val[1] < SENSOR_RESET_PULSE_WIDTH_DEFAULT) {
            rst_val[1] = SENSOR_RESET_PULSE_WIDTH_DEFAULT;
        } else if (rst_val[1] > SENSOR_RESET_PULSE_WIDTH_MAX) {
            rst_val[1] = SENSOR_RESET_PULSE_WIDTH_MAX;
        }
        _Sensor_Device_Reset(rst_val); 
    }
    CMR_LOGI("OK out.\n");
}
LOCAL int _Sensor_Device_Reset(uint32_t *reset_val)
{
    int ret = SENSOR_SUCCESS;
    SENSOR_DRV_CHECK_ZERO(s_p_sensor_cxt);

    CMR_LOGI("level %d, width %d",reset_val[0],reset_val[1]);

    ret = xioctl(s_p_sensor_cxt->fd_sensor, SENSOR_IO_RST, reset_val);
    if (ret) {
        ret = -1;
    }

    return ret;
}
    case SENSOR_IO_RST:
        {
            uint32_t rst_val[2];
            ret = copy_from_user(rst_val, (uint32_t *) arg, 2*sizeof(uint32_t));
            if (0 == ret)
                ret = _sensor_k_reset(rst_val[0], rst_val[1]);
        }
        break;
LOCAL int _sensor_k_reset(uint32_t level, uint32_t width)
{
    SENSOR_PRINT("SENSOR:_sensor_k_reset, reset_val=%d  camera:%d (0:main 1:sub)\n",level, _sensor_K_get_curId());

    switch (_sensor_K_get_curId()) {
    case SENSOR_MAIN:
    {
        gpio_direction_output(GPIO_SENSOR_RESET, level);
        gpio_set_value(GPIO_SENSOR_RESET, level);
        SLEEP_MS(width);
        gpio_set_value(GPIO_SENSOR_RESET, !level);
        mdelay(1);
        break;
    }
    case SENSOR_SUB:
    {
        gpio_direction_output(GPIO_SUB_SENSOR_RESET, level);
        gpio_set_value(GPIO_SUB_SENSOR_RESET, level);
        SLEEP_MS(width);
        gpio_set_value(GPIO_SUB_SENSOR_RESET, !level);
        mdelay(1);
        break;
    }
    default:
        break;
    }

    return SENSOR_K_SUCCESS;
}
Sensor_SetResetLevel((BOOLEAN)!reset_level); //就是把reset 设置成想要的高或低
Sensor_Reset(reset_level);    //反而把reset 设置成相反的高低位了

问题就是出在

Sensor_Reset(reset_level);    

然后把GC0308和GC0329的上电函数里的 (同时检查其它IC GC5004, OV5648都有此问题)

Sensor_Reset(reset_level);    

换成 

Sensor_SetResetLevel(reset_level);

就解决了此BUG。