[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-06 Thread Laurent Pinchart
Hi Mark, On Tuesday 06 Dec 2016 13:20:20 Mark Brown wrote: > On Tue, Dec 06, 2016 at 02:46:55PM +0200, Laurent Pinchart wrote: > > On Tuesday 06 Dec 2016 10:05:17 Mark Brown wrote: > > > On Mon, Dec 05, 2016 at 11:16:22PM +0200, Laurent Pinchart wrote: > > > > This has been discussed previously, a

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-06 Thread Mark Brown
On Tue, Dec 06, 2016 at 06:08:55PM +0200, Laurent Pinchart wrote: > On Tuesday 06 Dec 2016 13:20:20 Mark Brown wrote: > > The tiny amount of extra typing involved doesn't seem like much of a > > cost for keeping things consistent with every other regulator user out > > there. > I'm not concerned

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-06 Thread Laurent Pinchart
Hi Mark, On Tuesday 06 Dec 2016 10:05:17 Mark Brown wrote: > On Mon, Dec 05, 2016 at 11:16:22PM +0200, Laurent Pinchart wrote: > > On Monday 05 Dec 2016 13:11:51 Bjorn Andersson wrote: > >> Further more, a DT binding for a particular block should describe that > >> block; so if we have three diffe

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-06 Thread Mark Brown
On Tue, Dec 06, 2016 at 02:46:55PM +0200, Laurent Pinchart wrote: > On Tuesday 06 Dec 2016 10:05:17 Mark Brown wrote: > > On Mon, Dec 05, 2016 at 11:16:22PM +0200, Laurent Pinchart wrote: > > > This has been discussed previously, and Rob agreed that if the datasheet > > > recommends to power all s

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-06 Thread Mark Brown
On Mon, Dec 05, 2016 at 11:16:22PM +0200, Laurent Pinchart wrote: > On Monday 05 Dec 2016 13:11:51 Bjorn Andersson wrote: > > Further more, a DT binding for a particular block should describe that > > block; so if we have three different 1.8V pins then the DT binding > > should reflect this - even

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-05 Thread Laurent Pinchart
Hi Bjorn, On Monday 05 Dec 2016 13:11:51 Bjorn Andersson wrote: > On Tue 29 Nov 01:11 PST 2016, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 13:41:33 Archit Taneja wrote: > >> On 11/29/2016 12:03 PM, Laurent Pinchart wrote: > >>> On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: > Ad

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-05 Thread Bjorn Andersson
On Tue 29 Nov 01:11 PST 2016, Laurent Pinchart wrote: > Hi Archit, > > (CC'ing Mark Brown) > > On Tuesday 29 Nov 2016 13:41:33 Archit Taneja wrote: > > On 11/29/2016 12:03 PM, Laurent Pinchart wrote: > > > On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: > > >> Add the regulator supply prope

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-30 Thread Mark Brown
On Tue, Nov 29, 2016 at 09:37:31PM +0200, Laurent Pinchart wrote: > On Tuesday 29 Nov 2016 11:01:25 Mark Brown wrote: > > Please note that if you're going to CC me into a graphics thread there's > > a good chance I will miss it, I get copied on quite a lot of graphics > > related mail that's not r

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Laurent Pinchart
Hi Mark, On Tuesday 29 Nov 2016 11:01:25 Mark Brown wrote: > On Tue, Nov 29, 2016 at 11:11:03AM +0200, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 13:41:33 Archit Taneja wrote: > >> I thought we couldn't add mandatory properties once the device is > >> already present in DT for one or more

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Archit Taneja
On 11/29/2016 12:03 PM, Laurent Pinchart wrote: > Hi Archit, > > Thank you for the patch. > > On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: >> Add the regulator supply properties needed by ADV7511 and ADV7533. >> >> The regulators are specified as optional properties since there can >> be

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Archit Taneja
Add the regulator supply properties needed by ADV7511 and ADV7533. The regulators are specified as optional properties since there can be boards which have a fixed supply directly routed to the pins, and these may not be modelled as regulator supplies. Cc: devicetree at vger.kernel.org Acked-by:

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Laurent Pinchart
Hi Archit, (CC'ing Mark Brown) On Tuesday 29 Nov 2016 13:41:33 Archit Taneja wrote: > On 11/29/2016 12:03 PM, Laurent Pinchart wrote: > > On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: > >> Add the regulator supply properties needed by ADV7511 and ADV7533. > >> > >> The regulators are spec

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Mark Brown
On Tue, Nov 29, 2016 at 11:11:03AM +0200, Laurent Pinchart wrote: > On Tuesday 29 Nov 2016 13:41:33 Archit Taneja wrote: > > I thought we couldn't add mandatory properties once the device is already > > present in DT for one or more platforms. > You can, as long as you treat them as optional in t

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Laurent Pinchart
Hi Archit, Thank you for the patch. On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: > Add the regulator supply properties needed by ADV7511 and ADV7533. > > The regulators are specified as optional properties since there can > be boards which have a fixed supply directly routed to the pins,