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

From: Thierry Reding <thierry.reding@kernel.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	 Kieran Bingham <kieran.bingham@ideasonboard.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	 Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	 Conor Dooley <conor+dt@kernel.org>,
	Jacopo Mondi <jacopo@jmondi.org>,
	 Sakari Ailus <sakari.ailus@linux.intel.com>,
	Jimmy Su <jimmy.su@intel.com>,
	 Matthias Fend <matthias.fend@emfend.at>,
	Mikhail Rudenko <mike.rudenko@gmail.com>,
	 Daniel Scally <dan.scally@ideasonboard.com>,
	Jacopo Mondi <jacopo.mondi@ideasonboard.com>,
	 Michael Riesch <michael.riesch@collabora.com>,
	Benjamin Mugnier <benjamin.mugnier@foss.st.com>,
	 Sylvain Petinot <sylvain.petinot@foss.st.com>,
	Paul Elder <paul.elder@ideasonboard.com>,
	 Martin Kepplinger <martin.kepplinger@puri.sm>,
	Quentin Schulz <quentin.schulz@theobroma-systems.com>,
	 Tommaso Merciai <tomm.merciai@gmail.com>,
	Svyatoslav Ryhel <clamor95@gmail.com>,
	 Richard Acayan <mailingradian@gmail.com>,
	Frank Li <Frank.Li@nxp.com>,
	 Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	 Fabio Estevam <festevam@gmail.com>,
	Bjorn Andersson <andersson@kernel.org>,
	 Konrad Dybcio <konradybcio@kernel.org>,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	 Magnus Damm <magnus.damm@gmail.com>,
	Heiko Stuebner <heiko@sntech.de>,
	 Jonathan Hunter <jonathanh@nvidia.com>,
	linux-media@vger.kernel.org, devicetree@vger.kernel.org,
	 linux-kernel@vger.kernel.org, linux@ew.tq-group.com,
	imx@lists.linux.dev,  linux-arm-kernel@lists.infradead.org,
	linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	 linux-rockchip@lists.infradead.org, linux-tegra@vger.kernel.org,
	 Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>,
	Alexander Stein <alexander.stein@ew.tq-group.com>,
	 David Heidelberg <david@ixit.cz>,
	Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>,
	 Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
	Quentin Schulz <quentin.schulz@cherry.de>
Subject: Re: [PATCH v6 0/6] dt-bindings: Orientation defines
Date: Mon, 21 Sep 2026 13:33:54 +0200	[thread overview]
Message-ID: <arEVs0FDleyHeeLj@orome> (raw)
In-Reply-To: <20260921073538.GA1466037@killaraus.ideasonboard.com>

[-- Attachment #1: Type: text/plain, Size: 1827 bytes --]

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.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]