cfi_flash cleanups
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 of 6 patches 0 repliesNew
5 of 6 patches are in this archive. The cover letter announces 6.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | cfi_flash: Break long lines | +0 | 0 replies | |
| 4/6 | cfi_flash: Introduce read and write accessors | +0 | 0 replies | |
| 6/6 | cfi_flash: Use map_physmem() and unmap_physmem() | +0 | 1 reply | |
| 6/6 | cfi_flash: Use map_physmem() and unmap_physmem() | +0 | 0 replies | |
| 6/6 | cfi_flash: Use map_physmem() and unmap_physmem() | +0 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [U-Boot-Users] [PATCH 0/6] cfi_flash cleanups
Hi Haavard,
Quoted 10 lines · Haavard Skinnemoen
This is a series of cleanups to the CFI driver I did while debugging why it refused to work on my AVR32 boards, rebased on top of the latest CFI custodian tree with "[PATCH] CFI: synchronize command offsets with Linux CFI driver" applied as well.
I'm not completely done yet, as I need more fixes to make the CFI driver work on boards like the ATNGW100. But these initial patches are seriously painful to rebase (especially #3 in this series,) so I'd like some feedback from others if I'm moving in an acceptable direction with this.
Great work so far. Thanks.
Quoted 4 lines
I've compile-tested the whole thing using MAKEALL on ppc (no point in testing on avr32 since no boards use this driver yet.) mcc200 failed, but it complains about lots of undefined symbols that has nothing to do with the CFI driver, so I'm pretty sure it isn't my fault.
No, I have seen this here today too. That's not related with your patch series.
Quoted 3 lines
Please let me know which parts of this are acceptable, and if there's something that can be done in a better way. And if you merge parts 1-3 (or even more of them), I'd be _very_ happy :-)
You have my ack at least on 1-3. And with a little rework I would gladly accept all of your patches. Looks like a great improvement to me.
Quoted 3 lines
There are a couple of remaining cleanups I'd like to make that go on top of this stuff, mostly related to refactoring the command set handling and adding manufacturer-specific fixups.
Good. Let's focus on your current patchset for now.
Thanks.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================