[PATCH] ARM: Samsung: Add helper function for setup gpios
Kukjin Kim
kgene.kim at samsung.com
Thu Sep 30 03:10:04 EDT 2010
Kyungmin Park wrote:
>
> From: Kyungmin Park <kyungmin.park at samsung.com>
>
> There are many places setup GPIOs for configuring the pin and pull state.
> With this helper function we can make it simple codes.
>
> Before:
> /* Set all the necessary GPK0[0:1] pins to special-function 2 */
> for (gpio = S5PV310_GPK0(0); gpio < S5PV310_GPK0(2); gpio++) {
> s3c_gpio_cfgpin(gpio, S3C_GPIO_SFN(2));
> s3c_gpio_setpull(gpio, S3C_GPIO_PULL_NONE);
> }
>
> switch (width) {
> case 8:
> /* GPK1[3:6] special-funtion 3 */
> for (gpio = S5PV310_GPK1(3); gpio <= S5PV310_GPK1(6);
> gpio++) {
> s3c_gpio_cfgpin(gpio, S3C_GPIO_SFN(3));
> s3c_gpio_setpull(gpio, S3C_GPIO_PULL_NONE);
> }
> case 4:
> /* GPK0[3:6] special-funtion 2 */
> for (gpio = S5PV310_GPK0(3); gpio <= S5PV310_GPK0(6);
> gpio++) {
> s3c_gpio_cfgpin(gpio, S3C_GPIO_SFN(2));
> s3c_gpio_setpull(gpio, S3C_GPIO_PULL_NONE);
> }
> default:
> break;
> }
> After:
>
> /* Set all the necessary GPK0[0:1] pins to special-function 2 */
> s3c_setup_gpios(S5PV310_GPK0(0), S5PV310_GPK0(1),
> S3C_GPIO_SFN(2),
> S3C_GPIO_PULL_NONE);
> switch (width) {
> case 8:
> /* GPK1[3:6] special-funtion 3 */
> s3c_setup_gpios(S5PV310_GPK1(3), S5PV310_GPK1(6),
> S3C_GPIO_SFN(3),
> S3C_GPIO_PULL_NONE);
> case 4:
> /* GPK0[3:6] special-funtion 2 */
> s3c_setup_gpios(S5PV310_GPK0(3), S5PV310_GPK0(6),
> S3C_GPIO_SFN(2),
> S3C_GPIO_PULL_NONE);
> default:
> break;
> }
>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> ---
> diff --git a/arch/arm/plat-samsung/gpio-config.c
b/arch/arm/plat-samsung/gpio-
> config.c
> index e3d41ea..66b838f 100644
> --- a/arch/arm/plat-samsung/gpio-config.c
> +++ b/arch/arm/plat-samsung/gpio-config.c
> @@ -80,6 +80,20 @@ int s3c_gpio_setpull(unsigned int pin, s3c_gpio_pull_t
pull)
> }
> EXPORT_SYMBOL(s3c_gpio_setpull);
>
> +int s3c_setup_gpios(unsigned int start, unsigned int end,
> + unsigned int config, s3c_gpio_pull_t pull)
> +{
> + unsigned int gpio;
> +
> + for (gpio = start; gpio <= end; gpio++) {
> + s3c_gpio_cfgpin(gpio, config);
> + s3c_gpio_setpull(gpio, pull);
> + }
> +
> + return 0;
> +}
> +EXPORT_SYMBOL(s3c_setup_gpios);
> +
> #ifdef CONFIG_S3C_GPIO_CFG_S3C24XX
> int s3c_gpio_setcfg_s3c24xx_a(struct s3c_gpio_chip *chip,
> unsigned int off, unsigned int cfg)
> diff --git a/arch/arm/plat-samsung/include/plat/gpio-cfg.h
b/arch/arm/plat-
> samsung/include/plat/gpio-cfg.h
> index 1c6b929..6ac9091 100644
> --- a/arch/arm/plat-samsung/include/plat/gpio-cfg.h
> +++ b/arch/arm/plat-samsung/include/plat/gpio-cfg.h
> @@ -133,6 +133,19 @@ extern unsigned s3c_gpio_getcfg(unsigned int pin);
> extern int s3c_gpio_setpull(unsigned int pin, s3c_gpio_pull_t pull);
>
> /**
> + * s3c_setup_gpios() - Change & set gpios pin from start to end
> + * @start: The start pin number to configure
> + * @end: The end pin number to configure
> + * @config: The configuration for the pin's function.
> + * @pull: The configuration for the pull resistor.
> + *
> + * Helper functions to do both s3c_gpio_cfgpin and s3c_gpio_setpull
> + * with range
> + */
> +extern int s3c_setup_gpios(unsigned int start, unsigned int end,
> + unsigned int config, s3c_gpio_pull_t pull);
> +
> +/**
> * s3c_gpio_getpull() - get the pull resistor state of a gpio pin
> * @pin: The pin number to get the settings for
> *
Hmm...this has been submitted by Ben.
http://lists.infradead.org/pipermail/linux-arm-kernel/2010-May/016629.html
If possible, I'd like to merge Ben's full patches except several patches
which have some problem.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
More information about the linux-arm-kernel
mailing list