On 09/01/2016 02:00 AM, Laurent Pinchart wrote: > 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 ?
It is: dt-bindings: drm/bridge: adv7511: Add regulators for ADV7533 https://patchwork.kernel.org/patch/9306945/ Thanks, Archit > >>>> 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(-) > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
