[U-Boot-Users] [PATCH] NAND FSL UPM: driver re-write using the hwcontrol callback

Wolfgang Grandegger wg at grandegger.com
Mon Jun 2 15:02:45 CEST 2008


Anton Vorontsov wrote:
> On Mon, Jun 02, 2008 at 12:11:11PM +0200, Wolfgang Grandegger wrote:
>> NAND FSL UPM: driver re-write using the hwcontrol callback
>>
>> This is a re-write of the NAND FSL UPM driver using the more universal
>> hwcontrol callback (instead of the cmdfunc callback). Here is a brief
>> list of furher modifications:
>>
>> - For the time being, the UPM setup writing the UPM array has been
>>   removed from the driver and must now be done by the board specific
>>   code.
>>
>> - The bus width definition in "struct fsl_upm_nand" is now in bits to
>>   comply with the corresponding Linux driver.
>>
>> - chip->dev_read is only set if fun->dev_ready != NULL, which is
>>   required for boards not connecting the R/B pin.
>>
>> - A few issue have been fixed with MxMR bit manipulation like in the
>>   corresponding Linux driver.
>>
>> Note: I think the "io_addr" field of "struct fsl_upm" could be removed
>>       as well, because the address is already determined by
>>       "nand->IO_ADDR_[RW]", but I'm not 100% sure.
>>
>> This patch has been tested on a TQM8548 modules with the NAND chip
>> Micron MT29F8G08FABWP.
>>
>> This patch is based on the following patches posted to this list a few
>> minutes ago:
>>
>>   [PATCH] PPC: add accessor macros to clear and set bits in one shot
>>   [PATCH] 83xx/85xx/86xx: add more MxMR local bus definitions
>>
>> Anton, could you please verify if it works on your MPC8360ERDK board as
>> well. A patch will follow. In principle, the NAND driver of the TQM8272
>> should work with it as well.
> 
> Works great here (tested on MPC8360E-RDK). Thanks!

Great.

>> Signed-off-by: Wolfgang Grandegger <wg at grandegger.com>
> 
> Acked-by: Anton Vorontsov <avorontsov at ru.mvista.com>

All 4 patches are maintained by different people. What is the fastest
way to get these patches in? Maybe Stefan could pick them up directly?

Wolfgang.




More information about the U-Boot mailing list