跟踪G-Sensor有时不起作用的问题

       最近在生产反馈中，有提到批量机器中有大约2%的机器遇到不转屏的问题。笔者通过反复跟踪，重现了这个问题，通过分析lsmod，看到驱动又已经加载，但是就是不起作用，再通过getevent看信息，发现没有数据报上来，这说明有问题，G-sensor没有报数据肯定就不转屏了，那怎么解决呢？

      仔细分析了G-sensor的驱动，流程上也没看出什么问题来，当然如果流程有问题就不可能是小概率出问题了， 那应该是大面积有问题了。层层分析，还是觉得在detect处理上有疑点，全志平台提供的驱动，读一遍chip ID后，就判断是不是BMA250，或者BMA250E，如果都不是就返回了，上层也就认为没有找到G-sensor，就不会去读数据。通过加了一些打印后，通过大量的测试跟踪，发现G-sensor没有的时候，在detect的时候都没有读成功，于是笔者又从硬件原理图上分析了一下，原来在同一组IIC上，不止它一个，而它就是相对比较晚加载的一个，因此在加载它驱动的时候，前一个驱动使用iic可能有忙的问题，这样就可能造成超时，最终读不出chip ID。那么，我们可以在读chip ID的时候，可以尝试多读几次，一次不行，休息一会再去读，这样就尽了最多限度的努力去读chip ID了，当然极端一点的还可以在读不出chip ID的时候，也强制认为读到了，这样有一个不好地方就是，不能做多种G-sensor的兼容，如果你只有一种G-sensor，这样做也没什么。笔者通过这样一修改后，通过专项测试，以及近段时间的批量生产，没有复现了这个问题。

      因此，在靠读外设的chip ID来判断是否存在这个硬件的时候，我们可以尝试多读几次，这样软件的兼容性会好一点，有一个不好的地方就是，系统的启动时间会稍微增加一点。下面就是对G-sensor detect改造的代码：

int gsensor_detect(struct i2c_client *client, struct i2c_board_info *info)
{
	struct i2c_adapter *adapter = client->adapter;
	int ret;
	int retrytimes = 5;
	
    if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA))
            return -ENODEV;
            
	if(twi_id == adapter->nr){
	    for(i2c_num = 0; i2c_num < (sizeof(i2c_address)/sizeof(i2c_address[0]));i2c_num++)
	    {	    client->addr = i2c_address[i2c_num];
    	       pr_info("%s:addr= 0x%x,i2c_num:%d\n",__func__,client->addr,i2c_num);
    	       
    	       do {
		    ret = i2c_smbus_read_byte_data(client,BMA250_CHIP_ID_REG);
		    if (ret < 0) {
		              printk("FAIL read value is 0x%x, retrytimes =%d\n", ret, retrytimes);
		            
		              retrytimes--;
		              msleep(5);/*5ms*/
		            }
    	       } while((retrytimes > 0) && (ret < 0));
    	       
    	       pr_info("Read ID value is :0x%x\n",ret);
    	       if ((ret &0x00FF) == BMA250_CHIP_ID){
            		pr_info("Bosch Sensortec Device detected!\n" );
    				strlcpy(info->type, SENSOR_NAME, I2C_NAME_SIZE);
                    return 0; 
    
               }else if((ret &0x00FF) == BMA150_CHIP_ID){
            	  	
            	   pr_info("Bosch Sensortec Device detected!\n" \
    			"BMA150 registered I2C driver!\n");  
    			 strlcpy(info->type, SENSOR_NAME, I2C_NAME_SIZE);
                     return 0; 
	      } else if((ret &0x00FF) == BMA250E_CHIP_ID) { 
		strlcpy(info->type, SENSOR_NAME, I2C_NAME_SIZE);
		return 0; 	
	      }  else { 
		/*worearound we just one gsensor*/
		pr_info("set force!!!\n");
		strlcpy(info->type, SENSOR_NAME, I2C_NAME_SIZE);
		return 0; 	
					}                                                                                                                                
        }
        
        pr_info("%s:Bosch Sensortec Device not found, \
                 maybe the other gsensor equipment! \n",__func__);
        return -ENODEV;
    }else{
		return -ENODEV;
	     }
}