[Skiboot] Patch to support additional Barreleye nor flash chips
Stewart Smith
stewart at linux.vnet.ibm.com
Fri Jan 22 13:33:39 AEDT 2016
Andrew Donnellan <andrew.donnellan at au1.ibm.com> writes:
> Hi Adriana
>
> On 14/01/16 06:18, Adriana Kobylak wrote:
> > { 0xc22019, 0x02000000, FL_ERASE_ALL | FL_CAN_4B, "Macronix
> MXxxL25635F"},
> > { 0xc2201a, 0x04000000, FL_ERASE_ALL | FL_CAN_4B, "Macronix
> MXxxL51235F"},
> > { 0xef4018, 0x01000000, FL_ERASE_ALL, "Winbond
> W25Q128BV" },
> > + { 0xef4019, 0x02000000, FL_ERASE_ALL | FL_ERASE_64K | FL_CAN_4B |
> > + FL_ERASE_BULK,
> > + "Winbond
> W25Q256BV"},
>
> The general coding style in skiboot is to use tabs, rather than spaces
> for indentation (similar to the Linux kernel) - could you resubmit your
> patch using tabs set to 8 spaces?
>
> Also, it's good practice to use the git send-email tool to send patches,
> or to import the .patch file into your mail client as an entire email
> message (alas, neither of these options are particularly easy if you're
> using Notes-based email...). This helps us in reviewing patches, so if
> you're planning on submitting patches regularly in future it'll be worth
> setting all that up.
I'd go a bit further and say it's impossible with Notes/IBM Verse based
email.
IBMers will have to get an LTC IMAP account or sign up to GMail (or similar).
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list