Ben Dooks wrote:
>
> On Tue, Jun 15, 2010 at 08:27:46PM +0900, Kukjin Kim wrote:
> > From: Lee Hyuk <hyuk1.lee@xxxxxxxxxxx>
> >
> > This patch adds the members of platdata which are cfg_wp and get_ro.
> > The cfg_wp is the function for setting the specific GPIO for WP pin
> > and get_ro is the function for getting data from the assigned GPIO.
>
> Why not just pass gpio numbers, and have the sdhci-s3c driver claim them
> if they are valid?
>
Ok..fixed it as your comments.
> > Signed-off-by: Hyuk Lee <hyuk1.lee@xxxxxxxxxxx>
> > Signed-off-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
> > ---
> > arch/arm/plat-samsung/dev-hsmmc.c | 4 ++++
> > arch/arm/plat-samsung/dev-hsmmc1.c | 4 ++++
> > arch/arm/plat-samsung/dev-hsmmc2.c | 4 ++++
> > arch/arm/plat-samsung/dev-hsmmc3.c | 4 ++++
> > arch/arm/plat-samsung/include/plat/sdhci.h | 6 ++++++
> > 5 files changed, 22 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/plat-samsung/dev-hsmmc.c
b/arch/arm/plat-samsung/dev-
> hsmmc.c
> > index 4c05b39..f2bae6f 100644
> > --- a/arch/arm/plat-samsung/dev-hsmmc.c
> > +++ b/arch/arm/plat-samsung/dev-hsmmc.c
> > @@ -65,4 +65,8 @@ void s3c_sdhci0_set_platdata(struct s3c_sdhci_platdata
> *pd)
> > set->cfg_gpio = pd->cfg_gpio;
> > if (pd->cfg_card)
> > set->cfg_card = pd->cfg_card;
> > + if (pd->cfg_wp)
> > + set->cfg_wp = pd->cfg_wp;
> > + if (pd->get_ro)
> > + set->get_ro = pd->get_ro;
> > }
> > diff --git a/arch/arm/plat-samsung/dev-hsmmc1.c
b/arch/arm/plat-samsung/dev-
> hsmmc1.c
> > index e49bc4c..b326e0d 100644
> > --- a/arch/arm/plat-samsung/dev-hsmmc1.c
> > +++ b/arch/arm/plat-samsung/dev-hsmmc1.c
> > @@ -65,4 +65,8 @@ void s3c_sdhci1_set_platdata(struct s3c_sdhci_platdata
> *pd)
> > set->cfg_gpio = pd->cfg_gpio;
> > if (pd->cfg_card)
> > set->cfg_card = pd->cfg_card;
> > + if (pd->cfg_wp)
> > + set->cfg_wp = pd->cfg_wp;
> > + if (pd->get_ro)
> > + set->get_ro = pd->get_ro;
> > }
> > diff --git a/arch/arm/plat-samsung/dev-hsmmc2.c
b/arch/arm/plat-samsung/dev-
> hsmmc2.c
> > index 824580b..3474ef2 100644
> > --- a/arch/arm/plat-samsung/dev-hsmmc2.c
> > +++ b/arch/arm/plat-samsung/dev-hsmmc2.c
> > @@ -66,4 +66,8 @@ void s3c_sdhci2_set_platdata(struct s3c_sdhci_platdata
> *pd)
> > set->cfg_gpio = pd->cfg_gpio;
> > if (pd->cfg_card)
> > set->cfg_card = pd->cfg_card;
> > + if (pd->cfg_wp)
> > + set->cfg_wp = pd->cfg_wp;
> > + if (pd->get_ro)
> > + set->get_ro = pd->get_ro;
> > }
> > diff --git a/arch/arm/plat-samsung/dev-hsmmc3.c
b/arch/arm/plat-samsung/dev-
> hsmmc3.c
> > index 05d5ea4..1da8ea7 100644
> > --- a/arch/arm/plat-samsung/dev-hsmmc3.c
> > +++ b/arch/arm/plat-samsung/dev-hsmmc3.c
> > @@ -69,4 +69,8 @@ void s3c_sdhci3_set_platdata(struct s3c_sdhci_platdata
> *pd)
> > set->cfg_gpio = pd->cfg_gpio;
> > if (pd->cfg_card)
> > set->cfg_card = pd->cfg_card;
> > + if (pd->cfg_wp)
> > + set->cfg_wp = pd->cfg_wp;
> > + if (pd->get_ro)
> > + set->get_ro = pd->get_ro;
> > }
> > diff --git a/arch/arm/plat-samsung/include/plat/sdhci.h b/arch/arm/plat-
> samsung/include/plat/sdhci.h
> > index 1314ffa..da198d8 100644
> > --- a/arch/arm/plat-samsung/include/plat/sdhci.h
> > +++ b/arch/arm/plat-samsung/include/plat/sdhci.h
> > @@ -29,6 +29,8 @@ struct mmc_ios;
> > * is necessary the controllers and/or GPIO blocks require
the
> > * changing of driver-strength and other controls dependant on
> > * the card and speed of operation.
> > + * @cfg_wp: Configure the GPIO in order to allocate WP pin.
> > + * @get_ro: Call back function for getting data from the allocated
GPIO.
>
> if you just used gpio calls and passed a gpio number here,
> you'd not need cfg_wp.
>
OK.
> > *
> > * Initialisation data specific to either the machine or the platform
> > * for the device driver to use or call-back when configuring gpio or
> > @@ -45,6 +47,8 @@ struct s3c_sdhci_platdata {
> > void __iomem *regbase,
> > struct mmc_ios *ios,
> > struct mmc_card *card);
> > + void (*cfg_wp)(int dev_id);
> > + int (*get_ro)(struct mmc_host *mmc);
> > };
> >
> > /**
> > @@ -234,6 +238,8 @@ extern void s5pv210_setup_sdhci_cfg_card(struct
> platform_device *dev,
> > struct mmc_ios *ios,
> > struct mmc_card *card);
> >
> > +extern void s3c_sdhci_set_platdata(void);
> > +
> > #ifdef CONFIG_S3C_DEV_HSMMC
> > static inline void s5pv210_default_sdhci0(void)
> > {
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
