On Wed, Nov 27, 2019 at 12:31:37PM +0100, Ahmad Fatoum wrote:
> Hello Sascha,
>
> On 11/27/19 12:16 PM, Sascha Hauer wrote:
> > GPIOs can be given names with the gpio-line-names device tree property.
> > This patch adds support for it.
> >
> > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> > ---
> > drivers/gpio/gpiolib.c | 16 +++++++++++++++-
> > 1 file changed, 15 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
> > index 057cea43cc..9764ddf0f0 100644
> > --- a/drivers/gpio/gpiolib.c
> > +++ b/drivers/gpio/gpiolib.c
> > @@ -428,11 +428,25 @@ static int of_hog_gpio(struct device_node *np, struct gpio_chip *chip,
> > static int of_gpiochip_scan_hogs(struct gpio_chip *chip)
> > {
> > struct device_node *np;
> > - int ret, i;
> > + int ret, i, count;
> >
> > if (!IS_ENABLED(CONFIG_OFDEVICE) || !chip->dev->device_node)
> > return 0;
> >
> > + count = of_property_count_strings(chip->dev->device_node, "gpio-line-names");
> > +
> > + if (count > 0) {
> > + const char **arr = xzalloc(count * sizeof(char *));
> > +
> > + of_property_read_string_array(chip->dev->device_node,
> > + "gpio-line-names", arr, count);
> > +
> > + for (i = 0; i < chip->ngpio && i < count; i++)
> > + gpio_desc[chip->base + i].name = xstrdup(arr[i]);
>
> There's no need to duplicate memory here. You could just directly point into the
> unflattened DTB.
barebox contains functions and commands to change and delete properties
and nodes. We shouldn't store pointers into the live tree anywhere.
Sascha
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
