From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH 00/28] at24: remove at24_platform_data Date: Wed, 8 Aug 2018 18:44:02 +0200 Message-ID: <20180808164402.GH7275@lunn.ch> References: <20180808153150.23444-1-brgl@bgdev.pl> <20180808155548.s7p4xqsjywz3psrj@ninjato> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Bartosz Golaszewski Cc: Wolfram Sang , Sekhar Nori , Srinivas Kandagatla , linux-i2c , Mauro Carvalho Chehab , Rob Herring , Florian Fainelli , Kevin Hilman , Richard Weinberger , Bartosz Golaszewski , linux-doc , Russell King , Marek Vasut , Paolo Abeni , Dan Carpenter , Grygorii Strashko , David Lechner , Arnd Bergmann , Sven Van Asbroeck , Boris Brezillon , linux-mtd@lists.infradead.org, Linux-OMAP , arm-soc List-Id: linux-omap@vger.kernel.org On Wed, Aug 08, 2018 at 06:27:25PM +0200, Bartosz Golaszewski wrote: > 2018-08-08 17:55 GMT+02:00 Wolfram Sang : > > On Wed, Aug 08, 2018 at 05:31:22PM +0200, Bartosz Golaszewski wrote: > >> From: Bartosz Golaszewski > >> > >> This is a follow-up to the previously rejected series[1] which partially > >> removed the at24_platform_data structure. After further development and > >> taking reviews into account, this series finally removes that struct > >> completely but not without touching many different parts of the code > >> base. > >> > >> Since I took over maintainership of the at24 driver I've been working > >> towards removing at24_platform_data in favor for device properties. > > > > Wooha, nice work. I can't really comment on it but wondered how you want > > to upstream it (after reviews)? Pull request of an immutable branch for > > nvmem-tree sounds best to me. Then I could also pull it in if i2c needs > > it. Probably same situation for arm-soc... > > > > I initially wanted to merge small parts of it starting with v4.18, but > there were some voices against merging APIs without users. I'm not > sure how it should go in. There'll be a need for multiple immutable > branches most probably... Hi Bartosz What this series does is show all the different parts are now available, and can be reviewed as a whole. Once that review is completed, merging in parts then becomes possible. It looks like you could probably merge the nvmem, mtd and net parts independently via there maintainers for 4.20, since i don't think there are any dependencies. The arm-soc changes in 4.21, and the removal of the platform data in 4.22? Andrew