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@xxxxxxxxxxxxxxxxxxxxx > >> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >> Signed-off-by: Archit Taneja <architt@xxxxxxxxxxxxxx> > >> --- > >> > >> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
