On Wed, Aug 4, 2010 at 8:14 PM, Zang Roy-R61911 <r61911@xxxxxxxxxxxxx> wrote:
>> > diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
>> > index c6d1bd8..a92566e 100644
>> > --- a/drivers/mmc/host/sdhci.c
>> > +++ b/drivers/mmc/host/sdhci.c
>> > @@ -817,8 +817,12 @@ static void
>> sdhci_set_transfer_mode(struct sdhci_host *host,
>> > WARN_ON(!host->data);
>> >
>> > mode = SDHCI_TRNS_BLK_CNT_EN;
>> > - if (data->blocks > 1)
>> > - mode |= SDHCI_TRNS_MULTI;
>> > + if (data->blocks > 1) {
>> > + if (host->quirks &
>> SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12)
>> > + mode |= SDHCI_TRNS_MULTI |
>> SDHCI_TRNS_ACMD12;
>> > + else
>> > + mode |= SDHCI_TRNS_MULTI;
>>
>> nit:
>> + mode |= SDHCI_TRNS_MULTI;
>> + if (host->quirks & SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12)
>> + mode |= SDHCI_TRNS_ACMD12;
>>
>> Clearer, no?
> why?
Shorter lines, fewer lines, and the SDHCI_TRNS_MULTI is more obviously
unconditional. But as I said, it is a nitpick.
g.
--
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
