From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751640AbdFIXYE (ORCPT ); Fri, 9 Jun 2017 19:24:04 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:51826 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751545AbdFIXYC (ORCPT ); Fri, 9 Jun 2017 19:24:02 -0400 Date: Sat, 10 Jun 2017 01:23:58 +0200 From: Pavel Machek To: Steve Longerbeam Cc: Tim Harvey , Rob Herring , Mark Rutland , Shawn Guo , Sascha Hauer , Fabio Estevam , Russell King - ARM Linux , mchehab@kernel.org, Hans Verkuil , Nick Dyer , markus.heiser@darmarit.de, Philipp Zabel , laurent.pinchart+renesas@ideasonboard.com, bparrot@ti.com, geert@linux-m68k.org, Arnd Bergmann , Sudip Mukherjee , minghsiu.tsai@mediatek.com, Tiffany Lin , Jean-Christophe TROTIN , Simon Horman , Niklas =?iso-8859-1?Q?S=F6derlund?= , robert.jarzmik@free.fr, songjun.wu@microchip.com, andrew-ct.chen@mediatek.com, Greg Kroah-Hartman , shuah@kernel.org, sakari.ailus@linux.intel.com, "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , linux-media , devel@driverdev.osuosl.org, Steve Longerbeam Subject: Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder Message-ID: <20170609232358.GA14181@amd> References: <1496860453-6282-1-git-send-email-steve_longerbeam@mentor.com> <1496860453-6282-15-git-send-email-steve_longerbeam@mentor.com> <20170609213843.GB28596@amd> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="W/nzBZO5zC0uMSeA" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --W/nzBZO5zC0uMSeA Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi! > >>>Steve, > >>> > >>>You need to remove the fim node now that you've moved this to V4L2 con= trols. > >>> > >> > >>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? >=20 > 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. >=20 > 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.=20 Good luck, Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blo= g.html --W/nzBZO5zC0uMSeA Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlk7Lg4ACgkQMOfwapXb+vI96QCfbAkWGGNMgk0kjjjJL3Pz074D RpQAoMRL06s7rqtS3+j+TImQTfriXHKD =DiOG -----END PGP SIGNATURE----- --W/nzBZO5zC0uMSeA-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder Date: Sat, 10 Jun 2017 01:23:58 +0200 Message-ID: <20170609232358.GA14181@amd> References: <1496860453-6282-1-git-send-email-steve_longerbeam@mentor.com> <1496860453-6282-15-git-send-email-steve_longerbeam@mentor.com> <20170609213843.GB28596@amd> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="W/nzBZO5zC0uMSeA" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Steve Longerbeam Cc: Tim Harvey , Rob Herring , Mark Rutland , Shawn Guo , Sascha Hauer , Fabio Estevam , Russell King - ARM Linux , mchehab@kernel.org, Hans Verkuil , Nick Dyer , markus.heiser@darmarit.de, Philipp Zabel , laurent.pinchart+renesas@ideasonboard.com, bparrot@ti.com, geert@linux-m68k.org, Arnd Bergmann , Sudip Mukherjee , minghsiu.tsai@mediatek.com, Tiffany Lin , Jean-Christophe TROTIN , Simon Horman , Niklas List-Id: devicetree@vger.kernel.org --W/nzBZO5zC0uMSeA Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi! > >>>Steve, > >>> > >>>You need to remove the fim node now that you've moved this to V4L2 con= trols. > >>> > >> > >>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? >=20 > 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. >=20 > 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.=20 Good luck, Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blo= g.html --W/nzBZO5zC0uMSeA Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlk7Lg4ACgkQMOfwapXb+vI96QCfbAkWGGNMgk0kjjjJL3Pz074D RpQAoMRL06s7rqtS3+j+TImQTfriXHKD =DiOG -----END PGP SIGNATURE----- --W/nzBZO5zC0uMSeA-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: pavel@ucw.cz (Pavel Machek) Date: Sat, 10 Jun 2017 01:23:58 +0200 Subject: [PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder In-Reply-To: References: <1496860453-6282-1-git-send-email-steve_longerbeam@mentor.com> <1496860453-6282-15-git-send-email-steve_longerbeam@mentor.com> <20170609213843.GB28596@amd> Message-ID: <20170609232358.GA14181@amd> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 181 bytes Desc: Digital signature URL: