Re: [PATCH v4 0/2] Move plat-mxc gpio driver into drivers/gpio
From: Shawn Guo
Date: Mon Jun 06 2011 - 08:17:19 EST
On Mon, Jun 06, 2011 at 10:50:51AM +0200, Sascha Hauer wrote:
> On Mon, Jun 06, 2011 at 12:07:53AM +0800, Shawn Guo wrote:
> > The patch set moves plat-mxc gpio driver into drivers/gpio, and
> > migrates mach-imx and mach-mx5 to the new driver.
> > Changes since v3:
> > * Reorganize the patch set per Grant's comments
> > Shawn Guo (2):
> > gpio/mxc: Move Freescale MXC gpio driver to drivers/gpio
> > gpio/mxc: Change gpio-mxc into an upstanding gpio driver
> > arch/arm/plat-mxc/include/mach/common.h | 12 +-
> > arch/arm/plat-mxc/include/mach/devices-common.h | 2 +
> > arch/arm/plat-mxc/include/mach/gpio.h | 27 ---
> > drivers/gpio/Kconfig | 4 +
> > drivers/gpio/Makefile | 1 +
> > .../arm/plat-mxc/gpio.c => drivers/gpio/gpio-mxc.c | 217 +++++++++++++-------
> While at renaming I'd prefer naming this file gpio-imx.c. We currently
> have inconsistent naming between mxc vs. imx and I'd like to get rid
> of the former.
It's not just a file name. The "mxc" is embedded in the variable
and function name all over the driver. I would currently keep it as
it is for the reasons below.
* The driver was moved out from plat-mxc, so naturally the name
gpio-mxc is picked.
* Changing name to gpio-imx would require replace all "mxc" with
"imx" all over the file, IMO. This makes the diff of the patch
* i.MX23 and i.MX28 are also part of imx family, but architecture
name "imx" does not cover it. Taking "mxs" as the architecture name
for i.MX23 and i.MX28, "mxc" stands well what it stands right now.
I would start renaming the driver with another patch, probably when
I see mach-mx5 start getting merged into mach-imx and plat-mxc renamed
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/