RE: [PATCH v3 2/2] mmc: sdhci-s3c: add platform data for the second capability

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sorry, please discard my previous reply.

> -----Original Message-----
> From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Huang Changming-R66093
> Sent: Wednesday, February 08, 2012 11:32 AM
> To: Chris Ball; Jaehoon Chung
> Cc: linux-mmc; linux-samsung-soc@xxxxxxxxxxxxxxx; Kyungmin Park; 'kgene
> kim'; Girish K S
> Subject: RE: [PATCH v3 2/2] mmc: sdhci-s3c: add platform data for the
> second capability
> 
> Hi, Chris,
> I have reworked it and send it to the maillist, please review it.
> 
> 
> > -----Original Message-----
> > From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc-
> > owner@xxxxxxxxxxxxxxx] On Behalf Of Chris Ball
> > Sent: Monday, February 06, 2012 7:20 PM
> > To: Jaehoon Chung
> > Cc: linux-mmc; linux-samsung-soc@xxxxxxxxxxxxxxx; 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.chung@xxxxxxxxxxx>
> > > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> > > ---
> > > 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 <cjb@xxxxxxxxxx>
> >
> > - Chris.
> > --
> > Chris Ball   <cjb@xxxxxxxxxx>   <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 majordomo@xxxxxxxxxxxxxxx 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at
> http://vger.kernel.org/majordomo-info.html


--
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


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux