From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH v2 6/7] gpio_msm: Move Qualcomm v6 MSM driver into drivers Date: Sat, 4 Jun 2011 00:44:00 -0600 Message-ID: <20110604064400.GA15846@ponder.secretlab.ca> References: <1305755453-31112-1-git-send-email-davidb@codeaurora.org> <1307141059-22431-1-git-send-email-davidb@codeaurora.org> <1307141059-22431-7-git-send-email-davidb@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pz0-f46.google.com ([209.85.210.46]:42453 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751876Ab1FDGoD (ORCPT ); Sat, 4 Jun 2011 02:44:03 -0400 Content-Disposition: inline In-Reply-To: <1307141059-22431-7-git-send-email-davidb@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: David Brown Cc: Daniel Walker , Bryan Huntsman , Russell King , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org On Fri, Jun 03, 2011 at 03:44:18PM -0700, David Brown wrote: > Migrate the driver for the v6-based MSM chips into drivers/gpio. The > driver is unchanged, only moved. > > Signed-off-by: David Brown > Acked-by: Linus Walleij > Acked-by: Nicolas Pitre Hi David, Comments below. When you repost, please cc me on the entire series, not just the last 2. > --- > arch/arm/mach-msm/Kconfig | 3 +++ > arch/arm/mach-msm/Makefile | 2 -- > drivers/gpio/Kconfig | 8 ++++++++ > drivers/gpio/Makefile | 1 + > arch/arm/mach-msm/gpio.c => drivers/gpio/msm_v1.c | 7 +++---- drivers/gpio/gpio-msm-v1.c please. I'm now enforcing a gpio-*.c naming convention in this directory. The other drivers will be renamed soon. > .../mach-msm/gpio_hw.h => drivers/gpio/msm_v1.h | 9 ++++----- > 6 files changed, 19 insertions(+), 11 deletions(-) > rename arch/arm/mach-msm/gpio.c => drivers/gpio/msm_v1.c (98%) > rename arch/arm/mach-msm/gpio_hw.h => drivers/gpio/msm_v1.h (98%) > > diff --git a/arch/arm/mach-msm/Kconfig b/arch/arm/mach-msm/Kconfig > index 1516896..cc7c0b2 100644 > --- a/arch/arm/mach-msm/Kconfig > +++ b/arch/arm/mach-msm/Kconfig > @@ -11,6 +11,7 @@ config ARCH_MSM7X00A > select MSM_SMD > select MSM_SMD_PKG3 > select CPU_V6 > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > @@ -22,6 +23,7 @@ config ARCH_MSM7X30 > select MSM_VIC > select CPU_V7 > select MSM_GPIOMUX > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > @@ -33,6 +35,7 @@ config ARCH_QSD8X50 > select MSM_VIC > select CPU_V7 > select MSM_GPIOMUX > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > diff --git a/arch/arm/mach-msm/Makefile b/arch/arm/mach-msm/Makefile > index 9519fd2..c0709af 100644 > --- a/arch/arm/mach-msm/Makefile > +++ b/arch/arm/mach-msm/Makefile > @@ -34,6 +34,4 @@ ifndef CONFIG_ARCH_MSM8960 > # TODO: TLMM Mapping issues need to be resolved > obj-y += gpio-v2.o > endif > -else > -obj-y += gpio.o > endif > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig > index 4a7f631..c19ecd6a 100644 > --- a/drivers/gpio/Kconfig > +++ b/drivers/gpio/Kconfig > @@ -164,6 +164,14 @@ config GPIO_VX855 > additional drivers must be enabled in order to use the > functionality of the device. > > +config GPIO_MSM_V1 > + tristate "Qualcomm MSM GPIO v1" > + depends on GPIOLIB && ARCH_MSM > + help > + Say yes here to support the GPIO interface on ARM v6 based > + Qualcomm MSM chips. Most of the pins on the MSM can be > + selected for GPIO, and are controlled by this driver. > + The kconfig entries are pseudo-alphabetically sorted. This entry should be higher in the list. > comment "I2C GPIO expanders:" > > config GPIO_MAX7300 > diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile > index b605f8e..2a7c24c 100644 > --- a/drivers/gpio/Makefile > +++ b/drivers/gpio/Makefile > @@ -45,6 +45,7 @@ obj-$(CONFIG_GPIO_RDC321X) += rdc321x-gpio.o > obj-$(CONFIG_GPIO_JANZ_TTL) += janz-ttl.o > obj-$(CONFIG_GPIO_SX150X) += sx150x.o > obj-$(CONFIG_GPIO_VX855) += vx855_gpio.o > +obj-$(CONFIG_GPIO_MSM_V1) += msm_v1.o Ditto > obj-$(CONFIG_GPIO_ML_IOH) += ml_ioh_gpio.o > obj-$(CONFIG_AB8500_GPIO) += ab8500-gpio.o > obj-$(CONFIG_GPIO_TPS65910) += tps65910-gpio.o > diff --git a/arch/arm/mach-msm/gpio.c b/drivers/gpio/msm_v1.c > similarity index 98% > rename from arch/arm/mach-msm/gpio.c > rename to drivers/gpio/msm_v1.c > index 6767deb..89ed35b 100644 > --- a/arch/arm/mach-msm/gpio.c > +++ b/drivers/gpio/msm_v1.c > @@ -1,7 +1,6 @@ > -/* linux/arch/arm/mach-msm/gpio.c > - * > +/* > * Copyright (C) 2007 Google, Inc. > - * Copyright (c) 2009-2010, Code Aurora Forum. All rights reserved. > + * Copyright (c) 2009-2011, Code Aurora Forum. All rights reserved. > * > * This software is licensed under the terms of the GNU General Public > * License version 2, as published by the Free Software Foundation, and > @@ -22,7 +21,7 @@ > #include > #include > #include > -#include "gpio_hw.h" > +#include "msm_v1.h" What is the purpose of this header? Does anything else include it? If not, then it should be squashed into the .c file. > > #define FIRST_GPIO_IRQ MSM_GPIO_TO_INT(0) > > diff --git a/arch/arm/mach-msm/gpio_hw.h b/drivers/gpio/msm_v1.h > similarity index 98% > rename from arch/arm/mach-msm/gpio_hw.h > rename to drivers/gpio/msm_v1.h > index ba7972a..a3e3cb2 100644 > --- a/arch/arm/mach-msm/gpio_hw.h > +++ b/drivers/gpio/msm_v1.h > @@ -1,8 +1,7 @@ > -/* arch/arm/mach-msm/gpio_hw.h > - * > +/* > * Copyright (C) 2007 Google, Inc. > * Author: Brian Swetland > - * Copyright (c) 2008-2010, Code Aurora Forum. All rights reserved. > + * Copyright (c) 2008-2011, Code Aurora Forum. All rights reserved. > * > * This software is licensed under the terms of the GNU General Public > * License version 2, as published by the Free Software Foundation, and > @@ -15,8 +14,8 @@ > * > */ > > -#ifndef __ARCH_ARM_MACH_MSM_GPIO_HW_H > -#define __ARCH_ARM_MACH_MSM_GPIO_HW_H > +#ifndef _MSM_GPIO_V1_H > +#define _MSM_GPIO_V1_H > > #include > > -- > Sent by an employee of the Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. > From mboxrd@z Thu Jan 1 00:00:00 1970 From: grant.likely@secretlab.ca (Grant Likely) Date: Sat, 4 Jun 2011 00:44:00 -0600 Subject: [PATCH v2 6/7] gpio_msm: Move Qualcomm v6 MSM driver into drivers In-Reply-To: <1307141059-22431-7-git-send-email-davidb@codeaurora.org> References: <1305755453-31112-1-git-send-email-davidb@codeaurora.org> <1307141059-22431-1-git-send-email-davidb@codeaurora.org> <1307141059-22431-7-git-send-email-davidb@codeaurora.org> Message-ID: <20110604064400.GA15846@ponder.secretlab.ca> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Jun 03, 2011 at 03:44:18PM -0700, David Brown wrote: > Migrate the driver for the v6-based MSM chips into drivers/gpio. The > driver is unchanged, only moved. > > Signed-off-by: David Brown > Acked-by: Linus Walleij > Acked-by: Nicolas Pitre Hi David, Comments below. When you repost, please cc me on the entire series, not just the last 2. > --- > arch/arm/mach-msm/Kconfig | 3 +++ > arch/arm/mach-msm/Makefile | 2 -- > drivers/gpio/Kconfig | 8 ++++++++ > drivers/gpio/Makefile | 1 + > arch/arm/mach-msm/gpio.c => drivers/gpio/msm_v1.c | 7 +++---- drivers/gpio/gpio-msm-v1.c please. I'm now enforcing a gpio-*.c naming convention in this directory. The other drivers will be renamed soon. > .../mach-msm/gpio_hw.h => drivers/gpio/msm_v1.h | 9 ++++----- > 6 files changed, 19 insertions(+), 11 deletions(-) > rename arch/arm/mach-msm/gpio.c => drivers/gpio/msm_v1.c (98%) > rename arch/arm/mach-msm/gpio_hw.h => drivers/gpio/msm_v1.h (98%) > > diff --git a/arch/arm/mach-msm/Kconfig b/arch/arm/mach-msm/Kconfig > index 1516896..cc7c0b2 100644 > --- a/arch/arm/mach-msm/Kconfig > +++ b/arch/arm/mach-msm/Kconfig > @@ -11,6 +11,7 @@ config ARCH_MSM7X00A > select MSM_SMD > select MSM_SMD_PKG3 > select CPU_V6 > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > @@ -22,6 +23,7 @@ config ARCH_MSM7X30 > select MSM_VIC > select CPU_V7 > select MSM_GPIOMUX > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > @@ -33,6 +35,7 @@ config ARCH_QSD8X50 > select MSM_VIC > select CPU_V7 > select MSM_GPIOMUX > + select GPIO_MSM_V1 > select MSM_PROC_COMM > select HAS_MSM_DEBUG_UART_PHYS > > diff --git a/arch/arm/mach-msm/Makefile b/arch/arm/mach-msm/Makefile > index 9519fd2..c0709af 100644 > --- a/arch/arm/mach-msm/Makefile > +++ b/arch/arm/mach-msm/Makefile > @@ -34,6 +34,4 @@ ifndef CONFIG_ARCH_MSM8960 > # TODO: TLMM Mapping issues need to be resolved > obj-y += gpio-v2.o > endif > -else > -obj-y += gpio.o > endif > diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig > index 4a7f631..c19ecd6a 100644 > --- a/drivers/gpio/Kconfig > +++ b/drivers/gpio/Kconfig > @@ -164,6 +164,14 @@ config GPIO_VX855 > additional drivers must be enabled in order to use the > functionality of the device. > > +config GPIO_MSM_V1 > + tristate "Qualcomm MSM GPIO v1" > + depends on GPIOLIB && ARCH_MSM > + help > + Say yes here to support the GPIO interface on ARM v6 based > + Qualcomm MSM chips. Most of the pins on the MSM can be > + selected for GPIO, and are controlled by this driver. > + The kconfig entries are pseudo-alphabetically sorted. This entry should be higher in the list. > comment "I2C GPIO expanders:" > > config GPIO_MAX7300 > diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile > index b605f8e..2a7c24c 100644 > --- a/drivers/gpio/Makefile > +++ b/drivers/gpio/Makefile > @@ -45,6 +45,7 @@ obj-$(CONFIG_GPIO_RDC321X) += rdc321x-gpio.o > obj-$(CONFIG_GPIO_JANZ_TTL) += janz-ttl.o > obj-$(CONFIG_GPIO_SX150X) += sx150x.o > obj-$(CONFIG_GPIO_VX855) += vx855_gpio.o > +obj-$(CONFIG_GPIO_MSM_V1) += msm_v1.o Ditto > obj-$(CONFIG_GPIO_ML_IOH) += ml_ioh_gpio.o > obj-$(CONFIG_AB8500_GPIO) += ab8500-gpio.o > obj-$(CONFIG_GPIO_TPS65910) += tps65910-gpio.o > diff --git a/arch/arm/mach-msm/gpio.c b/drivers/gpio/msm_v1.c > similarity index 98% > rename from arch/arm/mach-msm/gpio.c > rename to drivers/gpio/msm_v1.c > index 6767deb..89ed35b 100644 > --- a/arch/arm/mach-msm/gpio.c > +++ b/drivers/gpio/msm_v1.c > @@ -1,7 +1,6 @@ > -/* linux/arch/arm/mach-msm/gpio.c > - * > +/* > * Copyright (C) 2007 Google, Inc. > - * Copyright (c) 2009-2010, Code Aurora Forum. All rights reserved. > + * Copyright (c) 2009-2011, Code Aurora Forum. All rights reserved. > * > * This software is licensed under the terms of the GNU General Public > * License version 2, as published by the Free Software Foundation, and > @@ -22,7 +21,7 @@ > #include > #include > #include > -#include "gpio_hw.h" > +#include "msm_v1.h" What is the purpose of this header? Does anything else include it? If not, then it should be squashed into the .c file. > > #define FIRST_GPIO_IRQ MSM_GPIO_TO_INT(0) > > diff --git a/arch/arm/mach-msm/gpio_hw.h b/drivers/gpio/msm_v1.h > similarity index 98% > rename from arch/arm/mach-msm/gpio_hw.h > rename to drivers/gpio/msm_v1.h > index ba7972a..a3e3cb2 100644 > --- a/arch/arm/mach-msm/gpio_hw.h > +++ b/drivers/gpio/msm_v1.h > @@ -1,8 +1,7 @@ > -/* arch/arm/mach-msm/gpio_hw.h > - * > +/* > * Copyright (C) 2007 Google, Inc. > * Author: Brian Swetland > - * Copyright (c) 2008-2010, Code Aurora Forum. All rights reserved. > + * Copyright (c) 2008-2011, Code Aurora Forum. All rights reserved. > * > * This software is licensed under the terms of the GNU General Public > * License version 2, as published by the Free Software Foundation, and > @@ -15,8 +14,8 @@ > * > */ > > -#ifndef __ARCH_ARM_MACH_MSM_GPIO_HW_H > -#define __ARCH_ARM_MACH_MSM_GPIO_HW_H > +#ifndef _MSM_GPIO_V1_H > +#define _MSM_GPIO_V1_H > > #include > > -- > Sent by an employee of the Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. >