[Skiboot] [PATCH 1/2] opal-prd: Disable pnor access interface on FSP system
Jeremy Kerr
jk at ozlabs.org
Fri Aug 11 14:49:19 AEST 2017
Hi Vasant,
>> Here you're duplicating the check for the ibm,system-flash property in
>> the libflash code. Why not just use the return value of pnor_init there?
>> We'd want the following logic:
>
> Yes. its kind of duplicate. But get_dev_mtd and file_init_path() is in
> common code and the error message we print inside these functions may
> confuse in this context.
OK, that makes sense.
One day, we might want to make the error logging in those common
functions a bit more flexible. But for now, this sounds like the best
approach.
Acked-by: Jeremy Kerr <jk at ozlabs.org>
Cheers,
Jeremy
More information about the Skiboot
mailing list