Commit 24c8ac89 authored Jun 28, 2019 by Fabio Estevam Committed by Mauro Carvalho Chehab Jul 25, 2019

media: i2c: ov5640: Fix the order for enabling regulators



According to the OV5640 datasheet the following sequence needs to be
followed when powering the OV5640 supplies:

- DOVDD
- AVDD
- DVDD

So follow this order inside the ov5640_supply_name[] array, so that
the regulator_bulk() functions can enable the regulator in the
correct sequence.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

parent 8791a102

drivers/media/i2c/ov5640.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -158,8 +158,8 @@ static const int ov5640_framerates[] = {
		/* regulator supplies */
		static const char * const ov5640_supply_name[] = {
		"DOVDD", /* Digital I/O (1.8V) supply */
		"DVDD", /* Digital Core (1.5V) supply */
		"AVDD", /* Analog (2.8V) supply */
		"DVDD", /* Digital Core (1.5V) supply */
		};

		#define OV5640_NUM_SUPPLIES ARRAY_SIZE(ov5640_supply_name)

Admin message