On 12/10/2016 3:41 AM, Rob Herring wrote: > On Mon, Dec 05, 2016 at 01:23:54PM +0530, Archit Taneja wrote: >> Add the regulator supply properties needed by ADV7511 and ADV7533. >> >> Cc: devicetree at vger.kernel.org >> Acked-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> >> Signed-off-by: Archit Taneja <architt at codeaurora.org> >> --- >> Documentation/devicetree/bindings/display/bridge/adi,adv7511.txt | 8 >> ++++++++ >> 1 file changed, 8 insertions(+) > > Didn't I ack this already? Anyway,
You had Acked it before. This set moved the regulator bindings from optional to mandatory, so I thought it may need you review again. Mark Brown insisted that we have a regulator supply per pin even if the chip's specs recommend using a common supply for them, so I'm going to bring that back again. I'll keep your Ack for the next revision since that's something you'd originally recommended. Thanks, Archit > > Acked-by: Rob Herring <robh at kernel.org> > -- > To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project