Hi, Chris,
I have reworked it and send it to the maillist, please review it.


> -----Original Message-----
> From: linux-mmc-ow...@vger.kernel.org [mailto:linux-mmc-
> ow...@vger.kernel.org] On Behalf Of Chris Ball
> Sent: Monday, February 06, 2012 7:20 PM
> To: Jaehoon Chung
> Cc: linux-mmc; linux-samsung-...@vger.kernel.org; Kyungmin Park; 'kgene
> kim'; Girish K S
> Subject: Re: [PATCH v3 2/2] mmc: sdhci-s3c: add platform data for the
> second capability
> 
> Hi,
> 
> On Mon, Feb 06 2012, Jaehoon Chung wrote:
> > This patch is added host_caps2 in sdhci-s3c.c It's necessary that use
> > the second capabilities.
> > And removed the duplicated host_caps.
> >
> > Signed-off-by: Jaehoon Chung <jh80.ch...@samsung.com>
> > Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
> > ---
> > Changelog v3:
> >     - based-on latest mmc-next tree
> > Changelog v2:
> >     - remove the duplicated host_caps.
> >
> >  drivers/mmc/host/sdhci-s3c.c |    6 +++---
> >  1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/mmc/host/sdhci-s3c.c
> > b/drivers/mmc/host/sdhci-s3c.c index d065e37..46b9782 100644
> > --- a/drivers/mmc/host/sdhci-s3c.c
> > +++ b/drivers/mmc/host/sdhci-s3c.c
> > @@ -521,9 +521,6 @@ static int __devinit sdhci_s3c_probe(struct
> platform_device *pdev)
> >     if (pdata->cd_type == S3C_SDHCI_CD_PERMANENT)
> >             host->mmc->caps = MMC_CAP_NONREMOVABLE;
> >
> > -   if (pdata->host_caps)
> > -           host->mmc->caps |= pdata->host_caps;
> > -
> >     if (pdata->pm_caps)
> >             host->mmc->pm_caps |= pdata->pm_caps;
> >
> > @@ -547,6 +544,9 @@ static int __devinit sdhci_s3c_probe(struct
> platform_device *pdev)
> >     if (pdata->host_caps)
> >             host->mmc->caps |= pdata->host_caps;
> >
> > +   if (pdata->host_caps2)
> > +           host->mmc->caps2 |= pdata->host_caps2;
> > +
> >     ret = sdhci_add_host(host);
> >     if (ret) {
> >             dev_err(dev, "sdhci_add_host() failed\n");
> 
> In case you'd rather merge both of these via the samsung-soc tree:
> 
> Acked-by: Chris Ball <c...@laptop.org>
> 
> - Chris.
> --
> Chris Ball   <c...@laptop.org>   <http://printf.net/>
> One Laptop Per Child
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majord...@vger.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html


--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to