Re: [PATCH v6 0/6] dt-bindings: Orientation defines

[Kieran Bingham]

Quoting Thierry Reding (2026-09-21 12:33:54)
> On Mon, Sep 21, 2026 at 10:35:38AM +0300, Laurent Pinchart wrote:
> > On Mon, Sep 21, 2026 at 09:07:01AM +0200, Geert Uytterhoeven wrote:
> > > On Mon, 21 Sept 2026 at 09:05, Geert Uytterhoeven wrote:
> > > > On Fri, 18 Sept 2026 at 12:55, Kieran Bingham wrote:
> > > > > Add a new dt-bindings/media/video-interface-devices.h header that
> > > > > initially supports the Orientation types and convert existing users
> > > > > throughout the code base.
> > > >
> > > > Just noticed these are already in since commit 6ab5acfb8dd38729
> > > > ("dt-bindings: media: Add macros for video interface devices") in
> > > > media/master (next-20260903 and later)?
> > > >
> > > > > v3 collects tags, and fixes the typo, but drops all hunks that do not
> > > >
> > > > [...]
> > > >
> > > > > v2:
> > > >
> > > > Interesting changelog order ;-)
> > > >
> > > > [...]
> > > >
> > > > > v3:
> > > >
> > > > [...]
> > > >
> > > > > -v4:
> > > >
> > > > [...]
> > > >
> > > >
> > > > > -v5:
> > > > >  - Definitions now accepted in linux-media, rebased to get conversions in.
> > > >
> > > > Oh, you say so here ;-)
> > > >
> > > > >       arm64: dts: renesas: Convert to new media orientation definitions
> > > >
> > > > So it's been safe for me to pick this up since v5?
> > > 
> > > No it's not, there is a hard dependency.
> > 
> > The whole series needs to be postponed by one kernel cycle :-/ It's not
> > urgent so it's not a big deal. Kieran will have the priviledge of
> > rebasing and sending a v7 :-)
> 
> If all platform maintainers Ack this series, they could go through the
> media tree where the shared include was already merged. Alternatively
> the media maintainers could provide a shared branch for everyone to pull
> into their platform trees.
> 
> But given that this is simply replacing a literal with a preprocessor
> macro, maybe not worth the extra hassle.

Indeed, I'm not personally worried here. I thought that because Sakari
had accepted the main patches these could go in now, so I refreshed the
series.

I don't really care which route they go, or when. I'd rather not have to
re-re-rebase again in two months ... but I can cope whatever :D There's
certainly no urgency here.


--
Kieran


> 
> Thierry