On Mon, Oct 12, 2015 at 08:53:26AM +0200, Geert Uytterhoeven wrote:
> On Mon, Oct 12, 2015 at 2:04 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote:
> >> + flash@0 {
> >> + #address-cells = <1>;
> >> + #size-cells = <1>;
> >> + compatible = "spansion,s25fl512s", "jedec,spi-nor";
> >
> > spansion,s25fl512s is used in several dt files but it seems to be
> > undocumented. Do we have a plan to resolve this?
>
> Documentation/devicetree/bindings/mtd/jedec,spi-nor.txt:
>
> - compatible : May include a device-specific string consisting of the
> manufacturer and name of the chip. Bear in mind the DT binding
> is not Linux-only, but in case of Linux, see the "m25p_ids"
> table in drivers/mtd/devices/m25p80.c for the list of supported
> chips.
> Must also include "jedec,spi-nor" for any SPI NOR flash that can
> be identified by the JEDEC READ ID opcode (0x9F).
>
> Hence it's OK.
Thanks, I guess checkpatch needs to learn about that rule.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
