[PATCH 2/3] drm/bridge: adv7533: Initialize regulators

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Aug 31 20:30:41 UTC 2016


Hi Archit,

On Wednesday 31 Aug 2016 22:24:30 Archit Taneja wrote:
> On 8/31/2016 9:23 PM, Laurent Pinchart wrote:
> > On Wednesday 31 Aug 2016 16:22:09 Archit Taneja wrote:
> >> ADV7533 requires supply to the AVDD, V1P2 and V3P3 pins for proper
> >> functionality.
> >> 
> >> Initialize and enable the regulators during probe itself. Controlling
> >> these dynamically is left for later.
> > 
> > You should document the power supplies in the DT bindings.
> 
> The DT bindings doc update was a part of the same series. I accidentally
> Cc'd you only for this patch.

No worries. What's the patch's subject ?

> >> Cc: dri-devel at lists.freedesktop.org
> >> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> >> Signed-off-by: Archit Taneja <architt at codeaurora.org>
> >> ---
> >> 
> >>   drivers/gpu/drm/bridge/adv7511/adv7511.h     | 16 ++++++++++
> >>   drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 34 +++++++++++++++------
> >>   drivers/gpu/drm/bridge/adv7511/adv7533.c     | 45 +++++++++++++++++++++
> >>   3 files changed, 86 insertions(+), 9 deletions(-)

-- 
Regards,

Laurent Pinchart



More information about the dri-devel mailing list