Re: [PATCH 2/2] media: ov5645: Use regulator_bulk() functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Fabio,

On Thu, Jun 27, 2019 at 9:24 PM Fabio Estevam <festevam@xxxxxxxxx> wrote:
>
> Hi Sakari,
>
> On Thu, Jun 27, 2019 at 1:27 PM Sakari Ailus <sakari.ailus@xxxxxx> wrote:
>
> > This appears to change the order in which the regulators are enabled. Is
> > that intentional? Does the sensor support this order as well?
>
> Good catch! I have sent a v2 that preserves the regulator enable order.

Thank you for the patch.
The question about using the regulator_bulk API seems to come
regularly from time to time.
This has been discussed on [1] and I believe the conclusion has been
that the regulator_bulk API doesn't guarantee the order of enabling of
the regulators. So in theory this is possible to cause problems in
some corner cases and we have agreed to leave the order explicit.

Best regards,
Todor

[1] https://patchwork.linuxtv.org/patch/36918/

P.S. Sorry for previous emails, hopefully my client is set up correctly now.



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux