From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933772Ab3GWRpz (ORCPT ); Tue, 23 Jul 2013 13:45:55 -0400 Received: from cassiel.sirena.org.uk ([80.68.93.111]:45361 "EHLO cassiel.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933361Ab3GWRpv (ORCPT ); Tue, 23 Jul 2013 13:45:51 -0400 Date: Tue, 23 Jul 2013 18:44:56 +0100 From: Mark Brown To: Greg KH Cc: Tomasz Figa , Kishon Vijay Abraham I , Alan Stern , Tomasz Figa , Laurent Pinchart , Sylwester Nawrocki , Sascha Hauer , kyungmin.park@samsung.com, balbi@ti.com, jg1.han@samsung.com, s.nawrocki@samsung.com, kgene.kim@samsung.com, grant.likely@linaro.org, tony@atomide.com, arnd@arndb.de, swarren@nvidia.com, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-omap@vger.kernel.org, linux-usb@vger.kernel.org, linux-media@vger.kernel.org, linux-fbdev@vger.kernel.org, akpm@linux-foundation.org, balajitk@ti.com, george.cherian@ti.com, nsekhar@ti.com, olof@lixom.net, Stephen Warren , b.zolnierkie@samsung.com, Daniel Lezcano Message-ID: <20130723174456.GM9858@sirena.org.uk> References: <51EE9EC0.6060905@ti.com> <20130723161846.GD2486@kroah.com> <1446965.6APW5ZgLBW@amdc1227> <20130723173710.GB28284@kroah.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="HQbpMUFNRY4iYVZ3" Content-Disposition: inline In-Reply-To: <20130723173710.GB28284@kroah.com> X-Cookie: You will be awarded some great honor. User-Agent: Mutt/1.5.21 (2010-09-15) X-SA-Exim-Connect-IP: 94.175.92.69 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH 01/15] drivers: phy: add generic PHY framework X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:57:07 +0000) X-SA-Exim-Scanned: Yes (on cassiel.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --HQbpMUFNRY4iYVZ3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 23, 2013 at 10:37:11AM -0700, Greg KH wrote: > On Tue, Jul 23, 2013 at 06:50:29PM +0200, Tomasz Figa wrote: > > I fully agree that a simple, single string will not scale even in some,= not=20 > > so uncommon cases, but there is already a lot of existing lookup soluti= ons=20 > > over the kernel and so there is no point in introducing another one. > I'm trying to get _rid_ of lookup "solutions" and just use a real > pointer, as you should. I'll go tackle those other ones after this one > is taken care of, to show how the others should be handled as well. What are the problems you are seeing with doing things with lookups? Having to write platform data for everything gets old fast and the code duplication is pretty tedious... --HQbpMUFNRY4iYVZ3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (GNU/Linux) iQIcBAEBAgAGBQJR7sEVAAoJELSic+t+oim9SUcP/ieysuGxUWbhqKHNUQJMzk22 s5EUHq9mXmZnv8GmwmuFlQZ3ntlWPuahPB8SUVgbwmCG8726u7z5qLSI+6RLPuTm n1cTd1UqtKMpg3LlFky+4HaLE844ZE6fGxdQetxxDUj6D3YhENnPyqVOM+4qI9eO 9FpL2tZArEw1Ob0aFB8LJHL5t9zK2FFLyLqqa+LxctmgN71t/U8uiEXBwsqIxYVS 0aVhjm5L9FnnUT3uSsu8dznALPGUG5rkdWDe9FzMzyUpV8d+mn7iHHc8h8+0HNQE EwtT9OrmBDORDVhq7QaRMejgKyCxTZKorqOB8Z1QmK5BBQZdsQE9Tp7nyFgHcqmC +Dny2OE2rITlcpj5R/Q93Nv9r+0y/Uf4dOGymxzIMB/cbyAndxvN0EGOzKFvTU9+ fudz+arQaUOB29jxTN7vlnW+OSExX6zJ9uERObfOG+im8vagKcNvRok88hJOpC+7 4lEzcZzR5T4C2uvd7Se4YZylZeEW6TnWB9yMln9TW57u1DN5AeZuAimY2GEEc+AF 1/tvRfkNARDQmR5bianOpNPgDCoO1g+MuPyDOZYWhT/5bfuhei7Y5iy8+522HF65 CJLlQt9GMlSKcwjre8PS97jTcscmd6CXRxHqhYNowZlbhO+IT0zTXREHuDgCRvZi Q63fF0aCpz1Y9Qh3iDYL =K4vT -----END PGP SIGNATURE----- --HQbpMUFNRY4iYVZ3-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Date: Tue, 23 Jul 2013 17:44:56 +0000 Subject: Re: [PATCH 01/15] drivers: phy: add generic PHY framework Message-Id: <20130723174456.GM9858@sirena.org.uk> MIME-Version: 1 Content-Type: multipart/mixed; boundary="HQbpMUFNRY4iYVZ3" List-Id: References: <51EE9EC0.6060905@ti.com> <20130723161846.GD2486@kroah.com> <1446965.6APW5ZgLBW@amdc1227> <20130723173710.GB28284@kroah.com> In-Reply-To: <20130723173710.GB28284@kroah.com> To: linux-arm-kernel@lists.infradead.org --HQbpMUFNRY4iYVZ3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 23, 2013 at 10:37:11AM -0700, Greg KH wrote: > On Tue, Jul 23, 2013 at 06:50:29PM +0200, Tomasz Figa wrote: > > I fully agree that a simple, single string will not scale even in some,= not=20 > > so uncommon cases, but there is already a lot of existing lookup soluti= ons=20 > > over the kernel and so there is no point in introducing another one. > I'm trying to get _rid_ of lookup "solutions" and just use a real > pointer, as you should. I'll go tackle those other ones after this one > is taken care of, to show how the others should be handled as well. What are the problems you are seeing with doing things with lookups? Having to write platform data for everything gets old fast and the code duplication is pretty tedious... --HQbpMUFNRY4iYVZ3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (GNU/Linux) iQIcBAEBAgAGBQJR7sEVAAoJELSic+t+oim9SUcP/ieysuGxUWbhqKHNUQJMzk22 s5EUHq9mXmZnv8GmwmuFlQZ3ntlWPuahPB8SUVgbwmCG8726u7z5qLSI+6RLPuTm n1cTd1UqtKMpg3LlFky+4HaLE844ZE6fGxdQetxxDUj6D3YhENnPyqVOM+4qI9eO 9FpL2tZArEw1Ob0aFB8LJHL5t9zK2FFLyLqqa+LxctmgN71t/U8uiEXBwsqIxYVS 0aVhjm5L9FnnUT3uSsu8dznALPGUG5rkdWDe9FzMzyUpV8d+mn7iHHc8h8+0HNQE EwtT9OrmBDORDVhq7QaRMejgKyCxTZKorqOB8Z1QmK5BBQZdsQE9Tp7nyFgHcqmC +Dny2OE2rITlcpj5R/Q93Nv9r+0y/Uf4dOGymxzIMB/cbyAndxvN0EGOzKFvTU9+ fudz+arQaUOB29jxTN7vlnW+OSExX6zJ9uERObfOG+im8vagKcNvRok88hJOpC+7 4lEzcZzR5T4C2uvd7Se4YZylZeEW6TnWB9yMln9TW57u1DN5AeZuAimY2GEEc+AF 1/tvRfkNARDQmR5bianOpNPgDCoO1g+MuPyDOZYWhT/5bfuhei7Y5iy8+522HF65 CJLlQt9GMlSKcwjre8PS97jTcscmd6CXRxHqhYNowZlbhO+IT0zTXREHuDgCRvZi Q63fF0aCpz1Y9Qh3iDYL =K4vT -----END PGP SIGNATURE----- --HQbpMUFNRY4iYVZ3-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH 01/15] drivers: phy: add generic PHY framework Date: Tue, 23 Jul 2013 18:44:56 +0100 Message-ID: <20130723174456.GM9858@sirena.org.uk> References: <51EE9EC0.6060905@ti.com> <20130723161846.GD2486@kroah.com> <1446965.6APW5ZgLBW@amdc1227> <20130723173710.GB28284@kroah.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============4371443843795486829==" Return-path: In-Reply-To: <20130723173710.GB28284@kroah.com> 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: Greg KH Cc: linux-fbdev@vger.kernel.org, Daniel Lezcano , linux-doc@vger.kernel.org, tony@atomide.com, Tomasz Figa , nsekhar@ti.com, Tomasz Figa , Laurent Pinchart , s.nawrocki@samsung.com, kgene.kim@samsung.com, swarren@nvidia.com, jg1.han@samsung.com, Kishon Vijay Abraham I , Alan Stern , grant.likely@linaro.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, george.cherian@ti.com, arnd@arndb.de, Stephen Warren , b.zolnierkie@samsung.com, Sascha Hauer , linux-samsung-soc@vger.kernel.org, Sylwester Nawrocki , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, balajitk@ti.com, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, balbi@ List-Id: linux-omap@vger.kernel.org --===============4371443843795486829== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="HQbpMUFNRY4iYVZ3" Content-Disposition: inline --HQbpMUFNRY4iYVZ3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 23, 2013 at 10:37:11AM -0700, Greg KH wrote: > On Tue, Jul 23, 2013 at 06:50:29PM +0200, Tomasz Figa wrote: > > I fully agree that a simple, single string will not scale even in some,= not=20 > > so uncommon cases, but there is already a lot of existing lookup soluti= ons=20 > > over the kernel and so there is no point in introducing another one. > I'm trying to get _rid_ of lookup "solutions" and just use a real > pointer, as you should. I'll go tackle those other ones after this one > is taken care of, to show how the others should be handled as well. What are the problems you are seeing with doing things with lookups? Having to write platform data for everything gets old fast and the code duplication is pretty tedious... --HQbpMUFNRY4iYVZ3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (GNU/Linux) iQIcBAEBAgAGBQJR7sEVAAoJELSic+t+oim9SUcP/ieysuGxUWbhqKHNUQJMzk22 s5EUHq9mXmZnv8GmwmuFlQZ3ntlWPuahPB8SUVgbwmCG8726u7z5qLSI+6RLPuTm n1cTd1UqtKMpg3LlFky+4HaLE844ZE6fGxdQetxxDUj6D3YhENnPyqVOM+4qI9eO 9FpL2tZArEw1Ob0aFB8LJHL5t9zK2FFLyLqqa+LxctmgN71t/U8uiEXBwsqIxYVS 0aVhjm5L9FnnUT3uSsu8dznALPGUG5rkdWDe9FzMzyUpV8d+mn7iHHc8h8+0HNQE EwtT9OrmBDORDVhq7QaRMejgKyCxTZKorqOB8Z1QmK5BBQZdsQE9Tp7nyFgHcqmC +Dny2OE2rITlcpj5R/Q93Nv9r+0y/Uf4dOGymxzIMB/cbyAndxvN0EGOzKFvTU9+ fudz+arQaUOB29jxTN7vlnW+OSExX6zJ9uERObfOG+im8vagKcNvRok88hJOpC+7 4lEzcZzR5T4C2uvd7Se4YZylZeEW6TnWB9yMln9TW57u1DN5AeZuAimY2GEEc+AF 1/tvRfkNARDQmR5bianOpNPgDCoO1g+MuPyDOZYWhT/5bfuhei7Y5iy8+522HF65 CJLlQt9GMlSKcwjre8PS97jTcscmd6CXRxHqhYNowZlbhO+IT0zTXREHuDgCRvZi Q63fF0aCpz1Y9Qh3iDYL =K4vT -----END PGP SIGNATURE----- --HQbpMUFNRY4iYVZ3-- --===============4371443843795486829== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============4371443843795486829==-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: broonie@kernel.org (Mark Brown) Date: Tue, 23 Jul 2013 18:44:56 +0100 Subject: [PATCH 01/15] drivers: phy: add generic PHY framework In-Reply-To: <20130723173710.GB28284@kroah.com> References: <51EE9EC0.6060905@ti.com> <20130723161846.GD2486@kroah.com> <1446965.6APW5ZgLBW@amdc1227> <20130723173710.GB28284@kroah.com> Message-ID: <20130723174456.GM9858@sirena.org.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jul 23, 2013 at 10:37:11AM -0700, Greg KH wrote: > On Tue, Jul 23, 2013 at 06:50:29PM +0200, Tomasz Figa wrote: > > I fully agree that a simple, single string will not scale even in some, not > > so uncommon cases, but there is already a lot of existing lookup solutions > > over the kernel and so there is no point in introducing another one. > I'm trying to get _rid_ of lookup "solutions" and just use a real > pointer, as you should. I'll go tackle those other ones after this one > is taken care of, to show how the others should be handled as well. What are the problems you are seeing with doing things with lookups? Having to write platform data for everything gets old fast and the code duplication is pretty tedious... -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: