Re: [PATCH] MMC: ext_csd.raw_* used in comparison but never set

2011-09-24 Thread Chris Ball
Hi, On Fri, Sep 23 2011, Andrei Warkentin wrote: f39b2dd9d065151a04f5996656d1f27a7eb32d45 added code to only compare read-only ext_csd fields in bus width testing code, yet it's comparing some fields that are never set. The affected fields are ext_csd.raw_erased_mem_count and

Re: [PATCH] MMC: ext_csd.raw_* used in comparison but never set

2011-09-23 Thread Philip Rakity
Acked-by: Philip Rakity prak...@marvell.com On Sep 22, 2011, at 9:25 PM, Andrei Warkentin wrote: f39b2dd9d065151a04f5996656d1f27a7eb32d45 added code to only compare read-only ext_csd fields in bus width testing code, yet it's comparing some fields that are never set. The affected fields

[PATCH] MMC: ext_csd.raw_* used in comparison but never set

2011-09-22 Thread Andrei Warkentin
f39b2dd9d065151a04f5996656d1f27a7eb32d45 added code to only compare read-only ext_csd fields in bus width testing code, yet it's comparing some fields that are never set. The affected fields are ext_csd.raw_erased_mem_count and ext_csd.raw_partition_support. Signed-off-by: Andrei Warkentin