Hi Rob,

On 26-01-20, Rob Herring wrote:
> On Thu, Jan 15, 2026 at 04:24:51PM +0100, Marco Felsch wrote:
> > From: Liu Ying <[email protected]>
> > 
> > i.MX93 SoC mediamix blk-ctrl contains one DISPLAY_MUX register which
> > configures parallel display format by using the "PARALLEL_DISP_FORMAT"
> > field. Document the Parallel Display Format Configuration(PDFC) subnode
> > and add the subnode to example.
> > 
> > Signed-off-by: Liu Ying <[email protected]>
> > [[email protected]: port to v6.18-rc1]
> > [[email protected]: add bus-width]
> > Signed-off-by: Marco Felsch <[email protected]>
> > ---
> >  .../bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml | 78 
> > ++++++++++++++++++++++
> >  1 file changed, 78 insertions(+)
> > 
> > diff --git 
> > a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml 
> > b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml
> > index 
> > 34aea58094e55365a2f9c86092f637e533f954ff..d828c2e82965c7a4cd69a67136047d83c96b0a35
> >  100644
> > --- 
> > a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml
> > +++ 
> > b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml
> > @@ -40,6 +40,58 @@ properties:
> >      minItems: 8
> >      maxItems: 10
> >  
> > +  dpi-bridge:
> > +    type: object
> > +    additionalProperties: false
> > +
> > +    properties:
> > +      compatible:
> > +        enum:
> > +          - nxp,imx91-pdfc
> > +          - nxp,imx93-pdfc
> 
> Looks like there's a register for this at 0x60. That should be a 'reg' 
> entry in the DT.

We discussed the usage of 'reg' here [1]. In short: This single
hw-register contains not only bits for the parallel output configuration
but also for the mipi-dsi configuration. So there should be no 'reg'
usage and instead let the syscon and driver do the correct handling.

Regards,
  Marco

[1] 
https://lore.kernel.org/all/paxpr04mb84599d676ec1e3879694579688...@paxpr04mb8459.eurprd04.prod.outlook.com/#t

Reply via email to