Re: [U-Boot] [PATCH] Add some second source legacy flash chips 256x8

Dear Niklaus Giger,

In message 1248244909-7085-1-git-send-email-niklaus.giger@member.fsf.org you wrote:

Signed-off-by: Niklaus Giger niklaus.giger@member.fsf.org

We tested some boot flashes as possible second source. As some had ids above 0x7f, we had to implement JEDEC JEP106Z in cfi_flash.c.

Please provide at least this statement as part of the commit message.

I now that I just missed the merge window, but the patches are not urgent and I was away for a three week

Stefan is probably willin gto consider this as a bug fix (at least I would not reprimand him if he did :-)

static void cmdset_amd_read_jedec_ids(flash_info_t *info) {

• flash_write_cmd(info, 0, 0, AMD_CMD_RESET);

• ushort bankId = 0;
• uchar manuId;

•    flash_write_cmd(info, 0, 0, AMD_CMD_RESET);
  

Please use TAB for indentation.

Best regards,

Wolfgang Denk