Elad,

I would like to discuss this problem with someone at sandisk.  Could you let me 
know what part you are using.

Also, since you have managed to track down the problem this far perhaps you can 
print out the fields values in 1, 4, and 8 bit modes and let me know what is 
present.

regards,

Philip

On Mar 22, 2013, at 6:56 PM, Philip Rakity <prak...@nvidia.com> wrote:

> 
> Elad,
> 
> Please supply the name of the spec which shows that this field is dependent 
> on bit width.
> 
> I checked my sd / emmc specs and do not see this.
> 
> regards,
> 
> Philip
> 
> 
> On Mar 17, 2013, at 5:07 AM, Chris Ball <c...@laptop.org> wrote:
> 
>> Hi Elad, I'm forwarding your bug report to the linux-mmc@ list.
>> 
>> - Chris.
>> 
>> 
>> I've been working on a bringup of an new board in Harmonic Inc.
>> This board uses Sandisk iNand eMMC flash.
>> I've noticed that the mmc driver keeps detecting 1 bit width although HW
>> supports 4 bit, looked into it, and found the problem. In line 571 at mmc.c:
>> There is:
>>   /* only compare read only fields */
>>   err = !((card->ext_csd.raw_partition_support ==
>>           bw_ext_csd[EXT_CSD_PARTITION_SUPPORT]) &&
>>       (card->ext_csd.raw_erased_mem_count ==
>>           bw_ext_csd[EXT_CSD_ERASED_MEM_CONT]) &&
>>       (card->ext_csd.rev ==
>>           bw_ext_csd[EXT_CSD_REV]) &&
>>       (card->ext_csd.raw_ext_csd_structure ==
>>               ...
>> 
>> The problem is that raw_partition_support is not the same when setting width 
>> of
>> 4 bits instead of 1 bit, and shouldn't be compared at all although it is read
>> only.
>> this should be changed to:
>>   /* only compare read only fields */
>>   err = !((card->ext_csd.raw_erased_mem_count ==
>>           bw_ext_csd[EXT_CSD_ERASED_MEM_CONT]) &&
>>       (card->ext_csd.rev ==
>>           bw_ext_csd[EXT_CSD_REV]) &&
>>       (card->ext_csd.raw_ext_csd_structure ==
>>               ...
>> 
>> -- 
>> 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