summaryrefslogtreecommitdiff
path: root/include/media
diff options
context:
space:
mode:
authorJohannes Obermaier <johannes.obermaier@gmail.com>2011-06-02 13:03:41 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2011-07-27 17:52:25 -0300
commit32127363eebdf63be2f375ed94838a4cdb1d6fe0 (patch)
treee75be3ac94e53092657ba68d4bbd6ed101edf218 /include/media
parent590929f32adc3aaa702c287b624a0d0382730088 (diff)
downloadlinux-3.10-32127363eebdf63be2f375ed94838a4cdb1d6fe0.tar.gz
linux-3.10-32127363eebdf63be2f375ed94838a4cdb1d6fe0.tar.bz2
linux-3.10-32127363eebdf63be2f375ed94838a4cdb1d6fe0.zip
[media] mt9v011: Fixed gain calculation
The implementation of the gain calculation for this sensor is incorrect. It is only working for the first 127 values. The reason is, that the gain cannot be set directly by writing a value into the gain registers of the sensor. The gain register work this way (see datasheet page 24): bits 0 to 6 are called "initial gain". These are linear. But bits 7 and 8 ("analog multiplicative factors") and bits 9 and 10 ("digital multiplicative factors") work completely different: Each of these bits increase the gain by the factor 2. So if the bits 7-10 are 0011, 0110, 1100 or 0101 for example, the gain from bits 0-6 is multiplied by 4. The order of the bits 7-10 is not important for the resulting gain. (But there are some recommended values for low noise) The current driver doesn't do this correctly: If the current gain is 000 0111 1111 (127) and the gain is increased by 1, you would expect the image to become brighter. But the image is completly dark, because the new gain is 000 1000 0000 (128). This means: Initial gain of 0, multiplied by 2. The result is 0. This patch adds a new function which does the gain calculation and also fixes the same bug for red_balance and blue_balance. Additionally, the driver follows the recommendation from the datasheet, which says, that the gain should always be above 0x0020. Tested-by: Mauro Carvalho Chehab <mchehab@redhat.com> Signed-off-by: Johannes Obermaier <johannes.obermaier@gmail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'include/media')
0 files changed, 0 insertions, 0 deletions