[PATCH v6 6/8] board: presidio: add LED support

[Simon Glass]

Hi Alex,

On Mon, 20 Apr 2020 at 23:30, Alex Nemirovsky
<alex.nemirovsky at cortina-access.com> wrote:
>
> From: Jway Lin <jway.lin at cortina-access.com>
>
> Add LED support for Cortina Access Presidio Engineering Board
>
> Signed-off-by: Jway Lin <jway.lin at cortina-access.com>
> Signed-off-by: Alex Nemirovsky <alex.nemirovsky at cortina-access.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> CC: Simon Glass <sjg at chromium.org>
>
> ---
>
> Changes in v6: None
> Changes in v5: None
> Changes in v4:
> - rename DT blink rate symbol
>
> Changes in v3: None
> Changes in v2: None
>
>  arch/arm/dts/ca-presidio-engboard.dts        | 31 ++++++++++++++++++++++++++++
>  configs/cortina_presidio-asic-emmc_defconfig |  2 ++
>  2 files changed, 33 insertions(+)
>

Do you have a new binding file for this from Linux? It seems to use
properties which I don't see in the existing binding in U-Boot.


> diff --git a/arch/arm/dts/ca-presidio-engboard.dts b/arch/arm/dts/ca-presidio-engboard.dts
> index 40c93d7..eef433e 100644
> --- a/arch/arm/dts/ca-presidio-engboard.dts
> +++ b/arch/arm/dts/ca-presidio-engboard.dts
> @@ -64,4 +64,35 @@
>                         spi-max-frequency = <108000000>;
>                 };
>         };
> +
> +       leds: led-controller at f43200f0 {
> +               compatible = "cortina,ca-leds";
> +               reg = <0x0 0xf43200f0 0x40>;
> +
> +               cortina,blink-rate1 = <256>;
> +               cortina,blink-rate2 = <512>;
> +
> +               led at 0 {
> +                       pin = <0>;
> +                       active-low;
> +                       blink-sel =<0>;
> +                       port = <0>;
> +                       off-event = <0>;
> +                       label = "led0";
> +               };
> +
> +               led at 1 {
> +                       pin = <1>;
> +                       active-low;
> +                       blink-sel =<1>;
> +                       label = "led1";
> +               };
> +
> +               led at 2 {
> +                       pin = <2>;
> +                       active-low;
> +                       label = "led2";
> +               };
> +
> +       };
>  };

Regards,
Simon