<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><div>About  04/17/2019 08:55AM in some timezone, Cédric Le Goater wrote:</div><br><div><br></div><div>>Here is a little series providing cleanups on the Aspeed SMC</div>>Controller<br>>driver and adding support for the 4B opcodes which were recently<br>>added<br>>to the Linux kernels 5.0.x.<br>><br>>The use of the 4B opcodes was breaking the read of the golden buffer<br>>done at slow speed in the optimization read sequence. The code<br>>assumed<br>>that the chip was in 4B address mode, as if a EN4B opcode had been<br>>sent, but this is not the case anymore with 4B opcodes. The golden<br>>buffer is now read with a SPINOR_OP_READ_4B (0x13) when the chip<br>>supports 4B opcodes.<br>><br>>It should fix accesses to the palmetto PNOR and the Witherspoon BMC<br>>flash modules.<br>><br>><br>>*Please test !*<br><div><br></div><div><br></div><div>Cedric,</div><div><br></div><div>did you notice that spi_nor_read_sfdp is now calling the nor->read method?</div><div><br></div><div>Since the read method is hard-coding the opcode, this means that we are</div><div>returning random flash contents instead of the flash parameter block, which</div><div>probably confuses the kernel.</div><div><br></div><div>milton</div><div><br></div></font><BR>