On 24 October 2011 16:39, Chris Ball <cjb@xxxxxxxxxx> wrote:
> Hi Seungwon,
>
> On Mon, Oct 24 2011, Seungwon Jeon wrote:
>> Code cleanup. The codes of related to eMMC4.5 are scattered.
>> This patch removes a duplicate if-statement and assembles all.
>>
>> Signed-off-by: Seungwon Jeon <tgih.jun@xxxxxxxxxxx>
>> ---
>> drivers/mmc/core/mmc.c | 20 +++++++++-----------
>> 1 files changed, 9 insertions(+), 11 deletions(-)
>>
>> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
>> index fb5bf01..3627044 100644
>> --- a/drivers/mmc/core/mmc.c
>> +++ b/drivers/mmc/core/mmc.c
>> @@ -467,29 +467,27 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
>> card->ext_csd.rst_n_function = ext_csd[EXT_CSD_RST_N_FUNCTION];
>> }
>>
>> - /* eMMC v4.5 or later */
>> - if (card->ext_csd.rev >= 6)
>> - card->ext_csd.feature_support |= MMC_DISCARD_FEATURE;
>> -
>> card->ext_csd.raw_erased_mem_count = ext_csd[EXT_CSD_ERASED_MEM_CONT];
>> if (ext_csd[EXT_CSD_ERASED_MEM_CONT])
>> card->erased_byte = 0xFF;
>> else
>> card->erased_byte = 0x0;
>>
>> + /* eMMC v4.5 or later */
>> if (card->ext_csd.rev >= 6) {
>> + card->ext_csd.feature_support |= MMC_DISCARD_FEATURE;
>> +
>> card->ext_csd.generic_cmd6_time = 10 *
>> ext_csd[EXT_CSD_GENERIC_CMD6_TIME];
>> card->ext_csd.power_off_longtime = 10 *
>> ext_csd[EXT_CSD_POWER_OFF_LONG_TIME];
>> - } else
>> - card->ext_csd.generic_cmd6_time = 0;
>
> Your patch removes this line completely. Why is that? You should
> explain it in the commit message.
By tracing the code what i understand is mmc_read_ext_csd is called
from mmc_card_init. In the init function the card instance is created.
When the card instance is created kzalloc is used. This will
initialize all the card structure members to 0.
So the deleted line is implicit.
>
>>
>> - card->ext_csd.cache_size =
>> - ext_csd[EXT_CSD_CACHE_SIZE + 0] << 0 |
>> - ext_csd[EXT_CSD_CACHE_SIZE + 1] << 8 |
>> - ext_csd[EXT_CSD_CACHE_SIZE + 2] << 16 |
>> - ext_csd[EXT_CSD_CACHE_SIZE + 3] << 24;
>> + card->ext_csd.cache_size =
>> + ext_csd[EXT_CSD_CACHE_SIZE + 0] << 0 |
>> + ext_csd[EXT_CSD_CACHE_SIZE + 1] << 8 |
>> + ext_csd[EXT_CSD_CACHE_SIZE + 2] << 16 |
>> + ext_csd[EXT_CSD_CACHE_SIZE + 3] << 24;
>> + }
>>
>> out:
>> return err;
>
> The rest looks good, thanks,
>
> - 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
