Am Dienstag, den 26.06.2018, 20:58 -0700 schrieb Andrey Smirnov:
> Introduce nvmem_cell_get_and_read() that combines getting a NVMEM cell
> by name and reading its contents.
>
> > Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> ---
> drivers/nvmem/core.c | 23 +++++++++++++++++++++++
> include/linux/nvmem-consumer.h | 10 ++++++++++
> 2 files changed, 33 insertions(+)
>
> diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c
> index 53b934bb3..c785de125 100644
> --- a/drivers/nvmem/core.c
> +++ b/drivers/nvmem/core.c
> @@ -750,3 +750,26 @@ int nvmem_device_write(struct nvmem_device *nvmem,
> > return bytes;
> }
> EXPORT_SYMBOL_GPL(nvmem_device_write);
> +
> +void *nvmem_cell_get_and_read(struct device_node *np, const char *cell_name,
> > + size_t bytes)
> +{
> > + struct nvmem_cell *cell;
> > + void *value;
> > + size_t len;
> +
> > + cell = of_nvmem_cell_get(np, cell_name);
> > + if (IS_ERR(cell))
> > + return cell;
> +
> > + value = nvmem_cell_read(cell, &len);
> > + if (!IS_ERR(value) && len != bytes) {
> > + kfree(value);
> > + value = ERR_PTR(-EINVAL);
> > + }
> +
> > + nvmem_cell_put(cell);
> +
> > + return value;
> +}
> +EXPORT_SYMBOL_GPL(nvmem_cell_get_and_read);
> \ No newline at end of file
I'm sure you are tired by me nitpicking on this. ;)
I really like this helper, as it makes the consumer code look much
nicer, as seen in patch 2.
Regards,
Lucas
> diff --git a/include/linux/nvmem-consumer.h b/include/linux/nvmem-consumer.h
> index 606cadb49..0ec2f05b9 100644
> --- a/include/linux/nvmem-consumer.h
> +++ b/include/linux/nvmem-consumer.h
> @@ -32,6 +32,9 @@ struct nvmem_cell_info {
> struct nvmem_cell *nvmem_cell_get(struct device_d *dev, const char *name);
> void nvmem_cell_put(struct nvmem_cell *cell);
> void *nvmem_cell_read(struct nvmem_cell *cell, size_t *len);
> +void *nvmem_cell_get_and_read(struct device_node *np, const char *cell_name,
> > + size_t bytes);
> +
> int nvmem_cell_write(struct nvmem_cell *cell, void *buf, size_t len);
>
> /* direct nvmem device read/write interface */
> @@ -55,6 +58,13 @@ static inline char *nvmem_cell_read(struct nvmem_cell *cell, size_t *len)
> > return ERR_PTR(-ENOSYS);
> }
>
> +static inline void *nvmem_cell_get_and_read(struct device_node *np,
> > + const char *cell_name,
> > + size_t bytes)
> +{
> > + return ERR_PTR(-ENOSYS);
> +}
> +
> static inline int nvmem_cell_write(struct nvmem_cell *cell,
> > const char *buf, size_t len)
> {
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
