Re: [PATCH] mtd: spi-nor: fix options for mx66l51235f

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



28.08.2018 14:43, Marek Vasut wrote:
> On 08/28/2018 01:29 PM, Alexander Amelkin wrote:
>> 27.08.2018 14:34, Marek Vasut wrote:
>>>>>> According to the datasheet for mx66l51235f, "The device default is in
>>>>>> 24-bit address mode" (section 9-10). Having option SPI_NOR_4B_OPCODES
>>>>>> makes the code act as if the device was already in 4B mode and didn't
>>>>>> need the EN4B command. That prevents this chip from functioning on
>>>>>> systems where the boot loader left the chip in 3B mode (e.g. if the
>>>>>> chip wasn't used during the boot process).
>>>>>>
>>>>>> Hence, this commit removes the SPI_NOR_4B_OPCODES option for
>>>>>> mx66l51235f (added previously by commit d342b6a973af).
>>>>> Could it be there are two variants of the chip, one which supports the
>>>>> 4B opcodes and one which does not ? Wouldn't be the first time I saw
>>>>> this. If this chip supports the SFDP tables, you can check those.
>>>> I was unable to find another variant of the chip. There is only one specified in the datasheet:
>>>> http://www.macronix.com/Lists/Datasheet/Attachments/7401/MX66L51235F,%203V,%20512Mb,%20v1.1.pdf
>>>> and it says that the device supports both 3B and 4B modes, but defaults to 3B (24-bit) mode.
>>> So keep the 4B part in. Linux will just not reconfigure the device to 4B
>>> mode using register write, but will instead issue the 4B opcode
>>> directly, without any stateful change.
>> Marek, thank you for asking all these questions. They made me conduct a deeper investigation than just finding the commit that broke it and reverting it. It appears that OpenPOWER P8's SBE code (the thing that P8 CPU runs first from its built-in memory) expects the PNOR flash chip to be in EN4B mode. It reads the rest of OpenPOWER Firmware using a usual READ (0x03) command with 4 bytes of address. Always. That's why, I believe, Roman's commit broke host booting for us.
> Uh oh, I seem to remember some flashes had this weird quirk where they
> were in EN4B mode by default, and if U-Boot/Linux reset the flash into
> "sane default" state (3B addressing etc), some systems failed to boot.
> Furthermore, this EN4B is non-volatile bit on this flash, right ? That
> means it is retained across power-cycles, which sucks even more.
No, EN4B is volatile. It doesn't survive power cycling. A quote from datasheet:
>>> The 4BYTE bit may be cleared by power-off or writing EX4B instruction to reset the state to be "0".
>
>> Probably it would be best not to merge this patch into Linux kernel. I'll discuss it with colleagues from OpenPOWER community and we'll probably fix it on SBE side.
> Can you fix the firmware ? If so, that'd be amazing.
It turns out that it is impossible to fix that on firmware side mainly because the firmware does not send any opcodes over SPI at all. It relies on the SPI flash controller to do the address mapping so the firmware accesses the chip as a usual memory address range. The flash controller is located in our case inside Aspeed AST2400 BMC SoC and does not support 4B opcodes. It expects the chip to respond to generic READ and WRITE opcodes with (configurable) either 3 or 4 address byte cycles. So the only way to make it work is put the chip in EN4B mode when BMC boots. As this is quite a specific case, I guess you won't be willing to remove the SPI_NOR_4B_OPCODES option globally. We'll have to keep the patch local to openbmc project
> But still, there can be a software which does rogue transmission on the
> SPI bus and flips the flash into 3B mode before reset, so the system
> will not boot anyway. Do you have a way to deal with that somehow ?
Sure. As I said, there is no way to turn on the host without BMC knowing/controlling it.
The BMC ensures the chip is in the right mode. That's actually the reason why this patch appeared in the first place.
Also, there is no rogue software on BMC.
>> As for renaming the chip, I can provide a patch just for that. Or we can leave it as is.
> A patch would be nice, yes.
Will do a bit later.

Best regards,
Alexander.

Attachment: signature.asc
Description: OpenPGP digital signature

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux