mbox series

[v3,0/6] dt-bindings: Add macros for video interface bus types

Message ID 20220615221410.27459-1-laurent.pinchart@ideasonboard.com (mailing list archive)
Headers show
Series dt-bindings: Add macros for video interface bus types | expand

Message

Laurent Pinchart June 15, 2022, 10:14 p.m. UTC
Hello,

This small patch series is the result of me getting a bus-type numerical
value wrong in a device tree file and spending too long debugging the
issue. Hopefully there's nothing controversial here.

Compared to v2, the PARALLEL bus type came back to replace BT601, as it
turned out that BT601 doesn't actually describe what is usually referred
to as the parallel bus type. Patch 3/3 has been split in per-vendor
patches at the request of Alexandre Torgue.

Laurent Pinchart (6):
  dt-bindings: media: Add macros for video interface bus types
  dt-bindings: Use new video interface bus type macros in examples
  ARM: dts: freescale: Use new media bus type macros
  ARM: dts: omap: Use new media bus type macros
  ARM: dts: renesas: Use new media bus type macros
  ARM: dts: stm32: Use new media bus type macros

 .../display/bridge/analogix,anx7625.yaml         |  1 +
 .../devicetree/bindings/media/i2c/mipi-ccs.yaml  |  3 ++-
 .../bindings/media/i2c/ovti,ov772x.yaml          |  3 ++-
 .../bindings/media/marvell,mmp2-ccic.yaml        |  3 ++-
 .../bindings/media/microchip,xisc.yaml           |  3 ++-
 .../devicetree/bindings/media/st,stm32-dcmi.yaml |  4 +++-
 arch/arm/boot/dts/imx6ul-14x14-evk.dtsi          |  4 +++-
 arch/arm/boot/dts/omap3-n900.dts                 |  5 +++--
 arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ca.dts  | 11 +++++++----
 .../r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi    |  4 +++-
 .../r8a7742-iwg21d-q7-dbcm-ov7725-single.dtsi    |  4 +++-
 arch/arm/boot/dts/stm32429i-eval.dts             |  3 ++-
 arch/arm/boot/dts/stm32mp157c-ev1.dts            |  3 ++-
 include/dt-bindings/media/video-interfaces.h     | 16 ++++++++++++++++
 14 files changed, 51 insertions(+), 16 deletions(-)
 create mode 100644 include/dt-bindings/media/video-interfaces.h


base-commit: b13baccc3850ca8b8cccbf8ed9912dbaa0fdf7f3

Comments

Sakari Ailus July 17, 2022, 6:54 a.m. UTC | #1
Folks,

> Laurent Pinchart (6):
>   dt-bindings: media: Add macros for video interface bus types
>   dt-bindings: Use new video interface bus type macros in examples
>   ARM: dts: freescale: Use new media bus type macros
>   ARM: dts: omap: Use new media bus type macros
>   ARM: dts: renesas: Use new media bus type macros
>   ARM: dts: stm32: Use new media bus type macros

What's the preference on the tree through which these would be merged?

The two first should probably go through the media tree but what about the
DTS? There's a dependency to the first patch. I can take these all if
people are fine with that.
Paul Elder Nov. 18, 2022, 9:23 a.m. UTC | #2
Hi Sakari,

Gentle ping.

On Sun, Jul 17, 2022 at 06:54:00AM +0000, Sakari Ailus wrote:
> Folks,
> 
> > Laurent Pinchart (6):
> >   dt-bindings: media: Add macros for video interface bus types
> >   dt-bindings: Use new video interface bus type macros in examples
> >   ARM: dts: freescale: Use new media bus type macros
> >   ARM: dts: omap: Use new media bus type macros
> >   ARM: dts: renesas: Use new media bus type macros
> >   ARM: dts: stm32: Use new media bus type macros
> 
> What's the preference on the tree through which these would be merged?
> 
> The two first should probably go through the media tree but what about the
> DTS? There's a dependency to the first patch. I can take these all if
> people are fine with that.

How is this going?


Thanks,

Paul
Laurent Pinchart Nov. 19, 2022, 7:15 p.m. UTC | #3
Hello,

On Fri, Nov 18, 2022 at 06:23:38PM +0900, Paul Elder wrote:
> Hi Sakari,
> 
> Gentle ping.
> 
> On Sun, Jul 17, 2022 at 06:54:00AM +0000, Sakari Ailus wrote:
> > Folks,
> > 
> > > Laurent Pinchart (6):
> > >   dt-bindings: media: Add macros for video interface bus types
> > >   dt-bindings: Use new video interface bus type macros in examples
> > >   ARM: dts: freescale: Use new media bus type macros
> > >   ARM: dts: omap: Use new media bus type macros
> > >   ARM: dts: renesas: Use new media bus type macros
> > >   ARM: dts: stm32: Use new media bus type macros
> > 
> > What's the preference on the tree through which these would be merged?
> > 
> > The two first should probably go through the media tree but what about the
> > DTS? There's a dependency to the first patch. I can take these all if
> > people are fine with that.
> 
> How is this going?

Sakari, if there's a concern taking patches 3/6 to 6/6 in the media
tree, could you merge 1/6 and 2/6 ? If they can still be included in a
pull request for v6.2, I'll work on getting the dts changes in v6.3
through their respective trees.
Sakari Ailus Nov. 21, 2022, 10:54 a.m. UTC | #4
Hi Laurent,

On Sat, Nov 19, 2022 at 09:15:04PM +0200, Laurent Pinchart wrote:
> Hello,
> 
> On Fri, Nov 18, 2022 at 06:23:38PM +0900, Paul Elder wrote:
> > Hi Sakari,
> > 
> > Gentle ping.
> > 
> > On Sun, Jul 17, 2022 at 06:54:00AM +0000, Sakari Ailus wrote:
> > > Folks,
> > > 
> > > > Laurent Pinchart (6):
> > > >   dt-bindings: media: Add macros for video interface bus types
> > > >   dt-bindings: Use new video interface bus type macros in examples
> > > >   ARM: dts: freescale: Use new media bus type macros
> > > >   ARM: dts: omap: Use new media bus type macros
> > > >   ARM: dts: renesas: Use new media bus type macros
> > > >   ARM: dts: stm32: Use new media bus type macros
> > > 
> > > What's the preference on the tree through which these would be merged?
> > > 
> > > The two first should probably go through the media tree but what about the
> > > DTS? There's a dependency to the first patch. I can take these all if
> > > people are fine with that.
> > 
> > How is this going?
> 
> Sakari, if there's a concern taking patches 3/6 to 6/6 in the media
> tree, could you merge 1/6 and 2/6 ? If they can still be included in a
> pull request for v6.2, I'll work on getting the dts changes in v6.3
> through their respective trees.

I think I was expecting v4 from you. But I can address the comments, too,
they were minor matters.

For applying the DTS patches I'd like to have ack from respective
treemaintainers it's fine to take these through the media tree.
Laurent Pinchart Nov. 21, 2022, 11:06 a.m. UTC | #5
Hi Sakari,

On Mon, Nov 21, 2022 at 10:54:01AM +0000, Sakari Ailus wrote:
> On Sat, Nov 19, 2022 at 09:15:04PM +0200, Laurent Pinchart wrote:
> > On Fri, Nov 18, 2022 at 06:23:38PM +0900, Paul Elder wrote:
> > > On Sun, Jul 17, 2022 at 06:54:00AM +0000, Sakari Ailus wrote:
> > > > Folks,
> > > > 
> > > > > Laurent Pinchart (6):
> > > > >   dt-bindings: media: Add macros for video interface bus types
> > > > >   dt-bindings: Use new video interface bus type macros in examples
> > > > >   ARM: dts: freescale: Use new media bus type macros
> > > > >   ARM: dts: omap: Use new media bus type macros
> > > > >   ARM: dts: renesas: Use new media bus type macros
> > > > >   ARM: dts: stm32: Use new media bus type macros
> > > > 
> > > > What's the preference on the tree through which these would be merged?
> > > > 
> > > > The two first should probably go through the media tree but what about the
> > > > DTS? There's a dependency to the first patch. I can take these all if
> > > > people are fine with that.
> > > 
> > > How is this going?
> > 
> > Sakari, if there's a concern taking patches 3/6 to 6/6 in the media
> > tree, could you merge 1/6 and 2/6 ? If they can still be included in a
> > pull request for v6.2, I'll work on getting the dts changes in v6.3
> > through their respective trees.
> 
> I think I was expecting v4 from you. But I can address the comments, too,
> they were minor matters.

That would be nice :-) Could it still be done for v6.2 ?

> For applying the DTS patches I'd like to have ack from respective
> treemaintainers it's fine to take these through the media tree.

Works for me, this can be done in v6.3.