Skip to content

Commit 14a6078

Browse files
Umang JainHans Verkuil
authored andcommitted
media: imx335: Set reserved register to default value
0x3a00 is a reserved register as per the datasheet. The driver currently sets to it 0x01 while the datasheet claims 0x00 to be the default value. On one of the setup based on i.MX8MP platform with IMX335, this register write broke the CSI-2 configuration. Setting 0x3a00 to its default value (i.e. 0x00) fixed the issue. It is not clear from the datasheet what this register write is responsible for but setting it to its default value seems the right thing to do, provided it has been found responsible to break CSI-2 configuration on one of the platform. Signed-off-by: Umang Jain <[email protected]> Reported-by: Matthias Fend <[email protected]> Signed-off-by: Sakari Ailus <[email protected]> Signed-off-by: Hans Verkuil <[email protected]>
1 parent 9d00cca commit 14a6078

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/media/i2c/imx335.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ static const struct imx335_reg mode_2592x1940_regs[] = {
249249
{0x3794, 0x7a},
250250
{0x3796, 0xa1},
251251
{0x37b0, 0x36},
252-
{0x3a00, 0x01},
252+
{0x3a00, 0x00},
253253
};
254254

255255
static const struct imx335_reg raw10_framefmt_regs[] = {

0 commit comments

Comments
 (0)