James Hogan wrote:
> On 9 December 2011 05:49, Seungwon Jeon <[email protected]> wrote:
> > Signed-off-by: Seungwon Jeon <[email protected]>
>
> A commit message would be useful.
"This patch adds another capabilities field for MMC_CAPS2_XXX."
Is a commit message disappeared?
>
> > ---
> > drivers/mmc/host/dw_mmc.c | 6 ++++++
> > include/linux/mmc/dw_mmc.h | 1 +
> > 2 files changed, 7 insertions(+), 0 deletions(-)
> >
> > diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> > index 3aaeb08..366df6b 100644
> > --- a/drivers/mmc/host/dw_mmc.c
> > +++ b/drivers/mmc/host/dw_mmc.c
> > @@ -1681,6 +1681,12 @@ static int __init dw_mci_init_slot(struct dw_mci
> > *host, unsigned int id)
> > else
> > mmc->caps = 0;
> >
> > + if (host->pdata->caps2)
> > + mmc->caps2 = host->pdata->caps2;
> > + else
> > + mmc->caps2 = 0;
> > +
> > +
>
> There's really no need to check the value here, just copy it over.
> Same with the other caps field really.
Yes. No need to check.
It would be added rather than be copied.
Thanks,
Seungwon Jeon.
>
> > if (host->pdata->get_bus_wd)
> > if (host->pdata->get_bus_wd(slot->id) >= 4)
> > mmc->caps |= MMC_CAP_4_BIT_DATA;
> > diff --git a/include/linux/mmc/dw_mmc.h b/include/linux/mmc/dw_mmc.h
> > index 6dc9b80..e8779c6 100644
> > --- a/include/linux/mmc/dw_mmc.h
> > +++ b/include/linux/mmc/dw_mmc.h
> > @@ -214,6 +214,7 @@ struct dw_mci_board {
> > unsigned int bus_hz; /* Bus speed */
> >
> > unsigned int caps; /* Capabilities */
> > + unsigned int caps2; /* More capabilities */
> > /*
> > * Override fifo depth. If 0, autodetect it from the FIFOTH register,
> > * but note that this may not be reliable after a bootloader has used
> > --
> > 1.7.0.4
> >
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> > the body of a message to [email protected]
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> other than that it looks fine to me.
>
> --
> James Hogan
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to [email protected]
> 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
