> > + priv = devm_kzalloc(dev, sizeof(struct gpio_dual_leds_priv),
> GFP_KERNEL);
> > + if (!priv)
> > + return -ENOMEM;
> > +
> > + priv->low_gpio = devm_gpiod_get(dev, "low", GPIOD_OUT_LOW);
> > + ret = PTR_ERR_OR_ZERO(priv->low_gpio);
> > + if (ret) {
> > +
On Fri, 12 Mar 2021 08:48:55 +
Hermes Zhang wrote:
> Hi Alexander,
>
> > Am Donnerstag, 11. März 2021, 14:04:08 CET schrieb Hermes Zhang:
> > > From: Hermes Zhang
> > >
> > > Introduce a new Dual GPIO LED driver. These two GPIOs LED will act as
> > > one LED as normal GPIO LED but give
Hi Alexander,
> Am Donnerstag, 11. März 2021, 14:04:08 CET schrieb Hermes Zhang:
> > From: Hermes Zhang
> >
> > Introduce a new Dual GPIO LED driver. These two GPIOs LED will act as
> > one LED as normal GPIO LED but give the possibility to change the
> > intensity in four levels: OFF, LOW,
Hallo Hermes,
thanks for your effort.
Am Donnerstag, 11. März 2021, 14:04:08 CET schrieb Hermes Zhang:
> From: Hermes Zhang
>
> Introduce a new Dual GPIO LED driver. These two GPIOs LED will act as
> one LED as normal GPIO LED but give the possibility to change the
> intensity in four levels:
On Fri, 12 Mar 2021 04:48:52 +
Hermes Zhang wrote:
> >
> > Sorry, leds-regulator has only a binary state LED.
> >
> > Maybe you could extend leds-regulator to be able to use all regulator
> > states?
> >
> > Or you can extend leds-gpio driver to support N states via log N gpios,
> >
>
> Sorry, leds-regulator has only a binary state LED.
>
> Maybe you could extend leds-regulator to be able to use all regulator states?
>
> Or you can extend leds-gpio driver to support N states via log N gpios,
> instead of adding new driver.
It seems a good idea to extend leds-gpio, so in
Hi!
> + priv = devm_kzalloc(dev, sizeof(struct gpio_dual_leds_priv),
> GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + priv->low_gpio = devm_gpiod_get(dev, "low", GPIOD_OUT_LOW);
> + ret = PTR_ERR_OR_ZERO(priv->low_gpio);
> + if (ret) {
> +
Hi!
> From: Hermes Zhang
>
> Introduce a new Dual GPIO LED driver. These two GPIOs LED will act as
> one LED as normal GPIO LED but give the possibility to change the
> intensity in four levels: OFF, LOW, MIDDLE and HIGH.
Do you have hardware that uses it?
Seems reasonably sane, but:
>
On Thu, 11 Mar 2021 16:38:14 +0100
Marek Behun wrote:
> LED_FULL, LED_HALF, LED_OFF are deprecated.
>
> And this driver is redundant. This can be done with leds-regulator,
> with a gpio-regulator.
>
> Marek
Sorry, leds-regulator has only a binary state LED.
Maybe you could extend
LED_FULL, LED_HALF, LED_OFF are deprecated.
And this driver is redundant. This can be done with leds-regulator,
with a gpio-regulator.
Marek
10 matches
Mail list logo