From: Marek Vasut <marek.vasut@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: akpm@linux-foundation.org, sameo@linux.intel.com, dtor@mail.ru,
adobriyan@gmail.com, tj@kernel.org
Subject: Re: [PATCH] UCB1400: Pass ucb1400-gpio data through ac97 bus
Date: Mon, 9 Aug 2010 06:43:14 +0200 [thread overview]
Message-ID: <201008090643.14777.marek.vasut@gmail.com> (raw)
In-Reply-To: <1281327394-6716-1-git-send-email-marek.vasut@gmail.com>
Dne Po 9. srpna 2010 06:16:34 Marek Vasut napsal(a):
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> ---
> drivers/gpio/ucb1400_gpio.c | 19 ++++++-------------
> drivers/mfd/ucb1400_core.c | 5 +++++
> include/linux/ucb1400.h | 19 ++++++-------------
> 3 files changed, 17 insertions(+), 26 deletions(-)
Ah right, guys, if possible, let this go through Eric Miao's tree (as I have
changes that depend on this). Thanks!
Cheers
>
> diff --git a/drivers/gpio/ucb1400_gpio.c b/drivers/gpio/ucb1400_gpio.c
> index 50e6bd1..fba45a5 100644
> --- a/drivers/gpio/ucb1400_gpio.c
> +++ b/drivers/gpio/ucb1400_gpio.c
> @@ -12,8 +12,6 @@
> #include <linux/module.h>
> #include <linux/ucb1400.h>
>
> -struct ucb1400_gpio_data *ucbdata;
> -
> static int ucb1400_gpio_dir_in(struct gpio_chip *gc, unsigned off)
> {
> struct ucb1400_gpio *gpio;
> @@ -50,7 +48,7 @@ static int ucb1400_gpio_probe(struct platform_device
> *dev) struct ucb1400_gpio *ucb = dev->dev.platform_data;
> int err = 0;
>
> - if (!(ucbdata && ucbdata->gpio_offset)) {
> + if (!(ucb && ucb->gpio_offset)) {
> err = -EINVAL;
> goto err;
> }
> @@ -58,7 +56,7 @@ static int ucb1400_gpio_probe(struct platform_device
> *dev) platform_set_drvdata(dev, ucb);
>
> ucb->gc.label = "ucb1400_gpio";
> - ucb->gc.base = ucbdata->gpio_offset;
> + ucb->gc.base = ucb->gpio_offset;
> ucb->gc.ngpio = 10;
> ucb->gc.owner = THIS_MODULE;
>
> @@ -72,8 +70,8 @@ static int ucb1400_gpio_probe(struct platform_device
> *dev) if (err)
> goto err;
>
> - if (ucbdata && ucbdata->gpio_setup)
> - err = ucbdata->gpio_setup(&dev->dev, ucb->gc.ngpio);
> + if (ucb && ucb->gpio_setup)
> + err = ucb->gpio_setup(&dev->dev, ucb->gc.ngpio);
>
> err:
> return err;
> @@ -85,8 +83,8 @@ static int ucb1400_gpio_remove(struct platform_device
> *dev) int err = 0;
> struct ucb1400_gpio *ucb = platform_get_drvdata(dev);
>
> - if (ucbdata && ucbdata->gpio_teardown) {
> - err = ucbdata->gpio_teardown(&dev->dev, ucb->gc.ngpio);
> + if (ucb && ucb->gpio_teardown) {
> + err = ucb->gpio_teardown(&dev->dev, ucb->gc.ngpio);
> if (err)
> return err;
> }
> @@ -113,11 +111,6 @@ static void __exit ucb1400_gpio_exit(void)
> platform_driver_unregister(&ucb1400_gpio_driver);
> }
>
> -void __init ucb1400_gpio_set_data(struct ucb1400_gpio_data *data)
> -{
> - ucbdata = data;
> -}
> -
> module_init(ucb1400_gpio_init);
> module_exit(ucb1400_gpio_exit);
>
> diff --git a/drivers/mfd/ucb1400_core.c b/drivers/mfd/ucb1400_core.c
> index dbe2801..3e24632 100644
> --- a/drivers/mfd/ucb1400_core.c
> +++ b/drivers/mfd/ucb1400_core.c
> @@ -75,6 +75,11 @@ static int ucb1400_core_probe(struct device *dev)
>
> /* GPIO */
> ucb_gpio.ac97 = ac97;
> + if (pdata) {
> + ucb_gpio.gpio_setup = pdata->gpio_setup;
> + ucb_gpio.gpio_teardown = pdata->gpio_teardown;
> + ucb_gpio.gpio_offset = pdata->gpio_offset;
> + }
> ucb->ucb1400_gpio = platform_device_alloc("ucb1400_gpio", -1);
> if (!ucb->ucb1400_gpio) {
> err = -ENOMEM;
> diff --git a/include/linux/ucb1400.h b/include/linux/ucb1400.h
> index 1b47909..1a6bfdd 100644
> --- a/include/linux/ucb1400.h
> +++ b/include/linux/ucb1400.h
> @@ -83,15 +83,12 @@
> #define UCB_ID 0x7e
> #define UCB_ID_1400 0x4304
>
> -struct ucb1400_gpio_data {
> - int gpio_offset;
> - int (*gpio_setup)(struct device *dev, int ngpio);
> - int (*gpio_teardown)(struct device *dev, int ngpio);
> -};
> -
> struct ucb1400_gpio {
> struct gpio_chip gc;
> struct snd_ac97 *ac97;
> + int gpio_offset;
> + int (*gpio_setup)(struct device *dev, int ngpio);
> + int (*gpio_teardown)(struct device *dev, int ngpio);
> };
>
> struct ucb1400_ts {
> @@ -112,6 +109,9 @@ struct ucb1400 {
>
> struct ucb1400_pdata {
> int irq;
> + int gpio_offset;
> + int (*gpio_setup)(struct device *dev, int ngpio);
> + int (*gpio_teardown)(struct device *dev, int ngpio);
> };
>
> static inline u16 ucb1400_reg_read(struct snd_ac97 *ac97, u16 reg)
> @@ -163,11 +163,4 @@ static inline void ucb1400_adc_disable(struct snd_ac97
> *ac97)
>
> unsigned int ucb1400_adc_read(struct snd_ac97 *ac97, u16 adc_channel,
> int adcsync);
> -
> -#ifdef CONFIG_GPIO_UCB1400
> -void __init ucb1400_gpio_set_data(struct ucb1400_gpio_data *data);
> -#else
> -static inline void ucb1400_gpio_set_data(struct ucb1400_gpio_data *data)
> {} -#endif
> -
> #endif
next prev parent reply other threads:[~2010-08-09 4:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-09 4:16 [PATCH] UCB1400: Pass ucb1400-gpio data through ac97 bus Marek Vasut
2010-08-09 4:43 ` Marek Vasut [this message]
2010-08-26 23:07 ` Andrew Morton
2010-08-27 5:50 ` Marek Vasut
2010-08-27 6:14 ` Andrew Morton
2010-08-27 6:17 ` Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201008090643.14777.marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=adobriyan@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=dtor@mail.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=sameo@linux.intel.com \
--cc=tj@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.