Menu

#7 Parallel flash operation not supported by jedec.c

0.10
closed-postponed
UrJTAG (101)
1
2008-02-16
2007-11-13
No

In contrast to cfi_detect() of cfi.c, jedec_detect() in jedec.c does not handle the case when two or more chips are connected in parallel to a bus. The function simply ignores the result of bw/8 and builds the cfi_array->cfi_chips array for one single chip only. cfi_detect() is more clever and loops through all attached chips.
This is probably no issue as long as only one single chip is connected to a bus. Might be easy to fix if jedec_detect() is extended with a looping detection behavior for all attached chips based on the bus width (as cfi_detect() does).

Should be fixed upon request when there's hardware available to test & debug this modification.

Discussion

  • Arnim Läuger

    Arnim Läuger - 2007-11-18
    • status: open --> open-postponed
     
  • Arnim Läuger

    Arnim Läuger - 2007-11-18

    Logged In: YES
    user_id=156321
    Originator: YES

    Missing functionality for a subset of AMD flash devices is included in amd_flash.c. When fixing this issue, consider the code in amd_flash.c as well. Preferably merge the two drivers with the goal to delete one of them completely.
    Proposal: Update jedec.c/amd.c and remove amd_flash.c.

     
  • Kolja Waschk

    Kolja Waschk - 2008-02-16

    Logged In: YES
    user_id=478715
    Originator: NO

    I think the Enhancement "1869650 Improve flash detection and support" is similar enough to this item, so I add a reference there and close this.

     
  • Kolja Waschk

    Kolja Waschk - 2008-02-16
    • assigned_to: nobody --> kawk
    • status: open-postponed --> closed-postponed
     
  • Kolja Waschk

    Kolja Waschk - 2017-02-12
    • Group: 0.x --> 0.10
     

Log in to post a comment.