lkml.org 
[lkml]   [2017]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCH v2 1/2] doc: bindings: Add bindings documentation for mtd nvmem
    On Tue,  7 Mar 2017 09:26:03 +0100
    Alban <albeu@free.fr> wrote:

    > Config data for drivers, like MAC addresses, is often stored in MTD.
    > Add a binding that define how such data storage can be represented in
    > device tree.
    >
    > Signed-off-by: Alban <albeu@free.fr>
    > ---
    > Changelog:
    > v2: * Added a "Required properties" section with the nvmem-provider
    > property
    > ---
    > .../devicetree/bindings/nvmem/mtd-nvmem.txt | 33 ++++++++++++++++++++++
    > 1 file changed, 33 insertions(+)
    > create mode 100644 Documentation/devicetree/bindings/nvmem/mtd-nvmem.txt
    >
    > diff --git a/Documentation/devicetree/bindings/nvmem/mtd-nvmem.txt b/Documentation/devicetree/bindings/nvmem/mtd-nvmem.txt
    > new file mode 100644
    > index 0000000..8ed25e6
    > --- /dev/null
    > +++ b/Documentation/devicetree/bindings/nvmem/mtd-nvmem.txt
    > @@ -0,0 +1,33 @@
    > += NVMEM in MTD =
    > +
    > +Config data for drivers, like MAC addresses, is often stored in MTD.
    > +This binding define how such data storage can be represented in device tree.
    > +
    > +An MTD can be defined as an NVMEM provider by adding the `nvmem-provider`
    > +property to their node.

    If everyone agrees that this is actually needed, then it should
    definitely go in the nvmem binding doc, and we should patch all nvmem
    providers to define this property (even if we keep supporting nodes
    that are not defining it). I'm not fully convinced yet, but I might be
    wrong.

    I also think we should take the "nvmem under flash node without partitions"
    into account now, or at least have a clear plan on how we want to represent
    it.

    Something like that?

    flash {
    partitions {
    part@X {
    nvmem {
    #address-cells = <1>;
    #size-cells = <1>;

    cell@Y {
    };
    };
    };
    };

    nvmem {
    #address-cells = <1>;
    #size-cells = <1>;

    cell@X {
    };
    };
    };

    Note that patching nvmem core to support the subnode case should be
    pretty easy (see below).

    --->8---
    diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c
    index 408b521ee520..507c6190505b 100644
    --- a/drivers/nvmem/core.c
    +++ b/drivers/nvmem/core.c
    @@ -465,7 +465,7 @@ struct nvmem_device *nvmem_register(const struct nvmem_config *config)
    nvmem->priv = config->priv;
    nvmem->reg_read = config->reg_read;
    nvmem->reg_write = config->reg_write;
    - np = config->dev->of_node;
    + np = config->of_node ? : config->dev->of_node;
    nvmem->dev.of_node = np;
    dev_set_name(&nvmem->dev, "%s%d",
    config->name ? : "nvmem", config->id);
    diff --git a/include/linux/nvmem-provider.h b/include/linux/nvmem-provider.h
    index cd93416d762e..ec2f5116d62d 100644
    --- a/include/linux/nvmem-provider.h
    +++ b/include/linux/nvmem-provider.h
    @@ -21,6 +21,7 @@ typedef int (*nvmem_reg_write_t)(void *priv, unsigned int offset,

    struct nvmem_config {
    struct device *dev;
    + struct device_node *of_node;
    const char *name;
    int id;
    struct module *owner;
    \
     
     \ /
      Last update: 2017-03-07 22:12    [W:8.764 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site