Re: [RFC PATCH 3/6] dt-bindings: display/bridge/nwl-dsi: Drop mux handling

2020-05-28 Thread Guido Günther
Hi Rob,
On Thu, May 28, 2020 at 01:59:14PM -0600, Rob Herring wrote:
> On Fri, May 15, 2020 at 03:12:12PM +0200, Guido Günther wrote:
> > No need to encode the SoC specifics in the bridge driver. For the
> > imx8mq we can use the mux-input-bridge.
> 
> You can't just change bindings like this. You'd still have to support 
> the "old" way. But IMO, this way is the right way.

My understanding is that binding stability only applies to released
kernels and this binding never was in released kernel yet. Does it still
apply in this case?
Cheers,
 -- Guido

> 
> > 
> > Signed-off-by: Guido Günther 
> > ---
> >  .../devicetree/bindings/display/bridge/nwl-dsi.yaml | 6 --
> >  1 file changed, 6 deletions(-)
> 


Re: [RFC PATCH 3/6] dt-bindings: display/bridge/nwl-dsi: Drop mux handling

2020-05-28 Thread Rob Herring
On Fri, May 15, 2020 at 03:12:12PM +0200, Guido Günther wrote:
> No need to encode the SoC specifics in the bridge driver. For the
> imx8mq we can use the mux-input-bridge.

You can't just change bindings like this. You'd still have to support 
the "old" way. But IMO, this way is the right way.

> 
> Signed-off-by: Guido Günther 
> ---
>  .../devicetree/bindings/display/bridge/nwl-dsi.yaml | 6 --
>  1 file changed, 6 deletions(-)


[RFC PATCH 3/6] dt-bindings: display/bridge/nwl-dsi: Drop mux handling

2020-05-15 Thread Guido Günther
No need to encode the SoC specifics in the bridge driver. For the
imx8mq we can use the mux-input-bridge.

Signed-off-by: Guido Günther 
---
 .../devicetree/bindings/display/bridge/nwl-dsi.yaml | 6 --
 1 file changed, 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml 
b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
index 8aff2d68fc33..d2c2d4e19a25 100644
--- a/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/nwl-dsi.yaml
@@ -46,10 +46,6 @@ properties:
   - const: phy_ref
   - const: lcdif
 
-  mux-controls:
-description:
-  mux controller node to use for operating the input mux
-
   phys:
 maxItems: 1
 description:
@@ -151,7 +147,6 @@ required:
   - clocks
   - compatible
   - interrupts
-  - mux-controls
   - phy-names
   - phys
   - ports
@@ -180,7 +175,6 @@ examples:
< IMX8MQ_CLK_LCDIF_PIXEL>;
   clock-names = "core", "rx_esc", "tx_esc", "phy_ref", "lcdif";
   interrupts = ;
-  mux-controls = < 0>;
   power-domains = <_mipi>;
   resets = < IMX8MQ_RESET_MIPI_DSI_RESET_BYTE_N>,
< IMX8MQ_RESET_MIPI_DSI_DPI_RESET_N>,
-- 
2.26.1