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@lists.freedesktop.org Cc: Laurent Pinchart laurent.pinchart@ideasonboard.com Signed-off-by: Archit Taneja architt@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(-)