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

[Thierry Reding]

On Mon, Sep 21, 2026 at 12:47:23PM +0100, Kieran Bingham wrote:
> 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.

You already have my Acked-by and I'm fine with anyone picking up the
Tegra DT changes. I don't think there's going to be any conflict with
any other work in the Tegra tree. If they don't get applied, I'll pick
it up for the next release.

Thierry

Attachment: signature.asc
Description: PGP signature