[FAQ20676] [Android P] Camera Sensor Regulator Restricting voltage 的处理...

admin

[DESCRIPTION]

Camera Sensor 不同电压的引脚共用regulator 时，regulator 为了保护低电压引脚不被被烧坏，会把低电压引脚的电压设置成regulator的最高电压，从而使高电压引脚无法上电。
关键log为 "Restricting voltage"：

1. 01-01 00:00:22.329088 447 447 E [ 18.437858].(1)[447:camerahalserver]vcamd: Restricting voltage, 1200000-1100000uV

复制代码

[SOLUTION]

1. 对于共用regulator ，但电压需求不同的引脚：

(1) 下电后，调用 regulator_put 函数，释放掉 regulator。
(2) 上电前，调用 regulator_get 函数，重新申请 regulator。
(3) Android Q下该问题的解决办法可以参考FAQ22659 [Android Q] Camera Sensor Regulator Restricting voltage 的处理方法。

2. 示例代码如下（示例代码基于6739平台，其他平台可参考示例代码做修改）：

1. /drivers/misc/mediatek/imgsensor/src/mt6739/camera_hw/regulator/regulator.c
   
2. 
   
3. #include "regulator.h"
   
4. -
   
5. +static int regulator_status[REGULATOR_TYPE_MAX_NUM] = {0};
   
6. +static void check_for_regulator_get(struct REGULATOR *preg, struct device *pdevice, int index);
   
7. +static void check_for_regulator_put(struct REGULATOR *preg, int index);
   
8. +static struct device_node *of_node_record = NULL;
   
9. +static DEFINE_MUTEX(g_regulator_state_mutex);
   
10. 
    
11. 
    
12. static const int regulator_voltage[] = {
    
13. REGULATOR_VOLTAGE_0,
    
14. @@ -58,6 +61,7 @@ static enum IMGSENSOR_RETURN regulator_init(void *pinstance)
    
15. pdevice = gimgsensor_device;
    
16. pof_node = pdevice->of_node;
    
17. pdevice->of_node = of_find_compatible_node(NULL, NULL, "mediatek,camera_hw");
    
18. + of_node_record = pdevice->of_node;
    
19. 
    
20. if (pdevice->of_node == NULL) {
    
21. PK_PR_ERR("regulator get cust camera node failed!\n");
    
22. @@ -71,6 +75,7 @@ static enum IMGSENSOR_RETURN regulator_init(void *pinstance)
    
23. PK_PR_ERR("regulator[%d] %s fail!\n",
    
24. i, pregulator_ctrl->pregulator_type);
    
25. atomic_set(&preg->enable_cnt[i], 0);
    
26. + regulator_status[i] = 1;
    
27. }
    
28. 
    
29. 
    
30. @@ -116,7 +121,8 @@ static enum IMGSENSOR_RETURN regulator_set(
    
31. (sensor_idx == IMGSENSOR_SENSOR_IDX_SUB) ? REGULATOR_TYPE_SUB_VCAMA :
    
32. (sensor_idx == IMGSENSOR_SENSOR_IDX_MAIN2) ? REGULATOR_TYPE_MAIN2_VCAMA :
    
33. REGULATOR_TYPE_SUB2_VCAMA;
    
34. -
    
35. + //pr_err("regulator_dbg regulator_set sensor_idx %d, regulator %s, status %d\n", sensor_idx, regulator_control[(reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD)].pregulator_type,regulator_status[(reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD)]);
    
36. + check_for_regulator_get(preg, gimgsensor_device, (reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD));
    
37. pregulator = preg->pregulator[reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD];
    
38. enable_cnt = preg->enable_cnt + (reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD);
    
39. 
    
40. @@ -145,6 +151,7 @@ static enum IMGSENSOR_RETURN regulator_set(
    
41. PK_PR_ERR("[regulator]fail to regulator_disable, powertype: %d\n", pin);
    
42. return IMGSENSOR_RETURN_ERROR;
    
43. }
    
44. + check_for_regulator_put(preg, (reg_type_offset + pin - IMGSENSOR_HW_PIN_AVDD));
    
45. atomic_dec(enable_cnt);
    
46. }
    
47. } else {
    
48. @@ -170,3 +177,28 @@ enum IMGSENSOR_RETURN imgsensor_hw_regulator_open(
    
49. return IMGSENSOR_RETURN_SUCCESS;
    
50. }
    
51. 
    
52. +static void check_for_regulator_get(struct REGULATOR *preg, struct device *pdevice, int index)
    
53. +{
    
54. + struct device_node *pof_node;
    
55. + mutex_lock(&g_regulator_state_mutex);
    
56. + if(regulator_status[index]==0)
    
57. + {
    
58. + pof_node = pdevice->of_node;
    
59. + pdevice->of_node = of_node_record;
    
60. +
    
61. + preg->pregulator[index] = regulator_get(pdevice, regulator_control[index].pregulator_type);
    
62. +
    
63. + pdevice->of_node = pof_node;
    
64. + regulator_status[index] = 1;
    
65. + //pr_err("regulator_dbg regulator_get %s, of_node:%p\n", regulator_control[index].pregulator_type, of_node_record);
    
66. + }
    
67. + mutex_unlock(&g_regulator_state_mutex);
    
68. +}
    
69. +
    
70. +static void check_for_regulator_put(struct REGULATOR *preg, int index)
    
71. +{
    
72. + mutex_lock(&g_regulator_state_mutex);
    
73. + if(regulator_status[index]==1)
    
74. + {
    
75. + regulator_put(preg->pregulator[index]);
    
76. + preg->pregulator[index] = NULL;
    
77. + regulator_status[index]=0;
    
78. + //pr_err("regulator_dbg regulator_put %s\n", regulator_control[index].pregulator_type);
    
79. + }
    
80. + mutex_unlock(&g_regulator_state_mutex);
    
81. +}
    
82. --
    
83. 
    

复制代码