Hi Brian,
On Wed, Mar 11, 2015 at 10:57 PM, Brian Norris
<[email protected]> wrote:
> diff --git a/Documentation/devicetree/bindings/mtd/m25p80.txt
> b/Documentation/devicetree/bindings/mtd/m25p80.txt
> index 4611aa83531b..1b2997d4cee4 100644
> --- a/Documentation/devicetree/bindings/mtd/m25p80.txt
> +++ b/Documentation/devicetree/bindings/mtd/m25p80.txt
> @@ -3,9 +3,12 @@
> Required properties:
> - #address-cells, #size-cells : Must be present if the device has sub-nodes
> representing partitions.
> -- compatible : Should be the manufacturer and the name of the chip. Bear in
> mind
> +- compatible : Should be "nor-jedec" for any SPI NOR flash that can be
> + identified by the JEDEC READ ID opcode (0x95).
> + Additionally, may include a device-specific string consisting
> of
> + the manufacturer and name of the chip. Bear in mind
For the casual reader, this suggests putting "nor-jedec" first, which is not
what we want. So I would write it like e.g.
"Should be the manufacturer and the name of the chip. Additionally,
should contain "nor-jedec" for any SPI NOR flash that can be
identified by the JEDEC READ ID opcode (0x95)."
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
