Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

From: Pavel Machek <pavel@ucw.cz>
To: Steve Longerbeam <slongerbeam@gmail.com>
Cc: "Tim Harvey" <tharvey@gateworks.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Sascha Hauer" <kernel@pengutronix.de>,
	"Fabio Estevam" <fabio.estevam@nxp.com>,
	"Russell King - ARM Linux" <linux@armlinux.org.uk>,
	mchehab@kernel.org, "Hans Verkuil" <hverkuil@xs4all.nl>,
	"Nick Dyer" <nick@shmanahar.org>,
	markus.heiser@darmarit.de,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	laurent.pinchart+renesas@ideasonboard.com, bparrot@ti.com,
	geert@linux-m68k.org, "Arnd Bergmann" <arnd@arndb.de>,
	"Sudip Mukherjee" <sudipm.mukherjee@gmail.com>,
	minghsiu.tsai@mediatek.com,
	"Tiffany Lin" <tiffany.lin@mediatek.com>,
	"Jean-Christophe TROTIN" <jean-christophe.trotin@st.com>,
	"Simon Horman" <horms+renesas@verge.net.au>,
	"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
	robert.jarzmik@free.fr, songjun.wu@microchip.com,
	andrew-ct.chen@mediatek.com,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	shuah@kernel.org, sakari.ailus@linux.intel.com,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	linux-media <linux-media@vger.kernel.org>,
	devel@driverdev.osuosl.org,
	"Steve Longerbeam" <steve_longerbeam@mentor.com>
Subject: Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder
Date: Sat, 10 Jun 2017 01:23:58 +0200	[thread overview]
Message-ID: <20170609232358.GA14181@amd> (raw)
In-Reply-To: <cb9d79e9-fa36-d0ad-3df6-27f1b8790f78@gmail.com>

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

Hi!

> >>>Steve,
> >>>
> >>>You need to remove the fim node now that you've moved this to V4L2 controls.
> >>>
> >>
> >>Yep, I caught this just after sending the v8 patchset. I'll send
> >>a v9 of this patch.
> >
> >This needs ack from devicetree people, then it can be merged. Can you
> >be a bit more forceful getting the ack?
> 
> OK, I need an Ack please, he said, in a forceful way. :)

I'd tune the force up a tiny bit more. This is not FreeBSD ;-). You
can read some emails from Linus for inspiration. Or drink few beers
and look at Al Viro's emails.

> In fact Ack's are needed for all the changes to dts sources,
> patches 4-14.

Actually, are they? Those should not need acks from device tree
people, just acks from ARM people, and those are easier to get... in
fact they should not need any acks, you should just send them to arm
people and get them merged.

1-4 is just a documentation, and you have acks there. (Except 2?)
That's ready to be merged, probably via the media tree? Just make it
clear you want it merged.

15,16 should be ready to. Media tree, too, I guess.

drivers/staging is greg. Advantage is these do _not_ need to go after
the dts changes. It is a driver. Actually I'd normally add dts support
after the driver. So you can push it now.

> >I don't think it makes sense to resubmit v9 before that. This is not a
> >rocket science.
> 
> I guess that makes sense, I'll wait for Ack's from all these patches
> before submitting the entire patchset as v9.

You may want to split the series, according to mainainters, or just
ask the maintainers to merge the relevant parts. I believe most of it
can be pushed now. 

Good luck,
								Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]