On 9/23/24 7:04 PM, Tudor Ambarus wrote: >> * Always read Macronix chips SFDP, as Macronix replaced all old chips >> in the Manufacture table. > I'll NACK it unless you prove that the old flavors of flashes are not > used anymore in the kernel. Even if you can prove that the older flashes are not used in the kernel anymore, we can't just switch to parsing SFDP, because we have seen in the past flashes with wrong SFDP data that made the flashes misbehave. The recommended way is to update just the flashes that you can test.