lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 19 May 2010 01:23:53 +0200 From: Samuel Ortiz <sameo@...ux.intel.com> To: Rabin VINCENT <rabin.vincent@...ricsson.com> Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, STEricsson_nomadik_linux <STEricsson_nomadik_linux@...t.st.com>, Linus WALLEIJ <linus.walleij@...ricsson.com>, Srinidhi KASAGAR <srinidhi.kasagar@...ricsson.com> Subject: Re: [PATCH] mfd: new AB8500 driver Hi Rabin, On Mon, May 17, 2010 at 09:30:06PM +0530, Rabin VINCENT wrote: > Hi Samuel, > > On Mon, May 17, 2010 at 00:53:55 +0200, Samuel Ortiz wrote: > > On Tue, May 11, 2010 at 10:15:31AM +0530, Rabin Vincent wrote: > > > Add a new driver to support the AB8500 Power Management chip, replacing > > > the current AB4500. The new driver replaces the old one, instead of an > > > incremental modification, because this is a substantial overhaul > > > including: > > > > > > - Split of the driver into -core and -spi portions, to allow another > > > interface layer to be added > > > > > > - Addition of interrupt support > > > > > > - Switch to MFD core API for handling subdevices > > > > > > - Simplification of the APIs to remove a redundant block parameter > > > > > > - Rename of the APIs and macros from ab4500_* to ab8500_* > > > > > > - Rename of the files from ab4500* to ab8500* > > > > > > The driver still identifes itself as "ab4500" > > The patch looks good, but why not changing that to "ab8500" and integrating > > the relevant board file changes to this patch ? > > I originally didn't do this because I thought that this would create > merge conflicts with some code movement in the arm tree, but looks like > I was wrong -- there won't be conflicts for this change. > > So, the updated patch below also includes the board file change to > change the driver name from ab4500 to ab8500. (It also includes a small > fix to the interrupt handling, found after the previous version was > sent.) Thanks, patch applied to my for-next branch. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists