Hi Alex,

On Mon, 20 Apr 2020 at 23:30, Alex Nemirovsky
<[email protected]> wrote:
>
> From: Jway Lin <[email protected]>
>
> Add LED support for Cortina Access Presidio Engineering Board
>
> Signed-off-by: Jway Lin <[email protected]>
> Signed-off-by: Alex Nemirovsky <[email protected]>
> Reviewed-by: Simon Glass <[email protected]>
>
> CC: Simon Glass <[email protected]>
>
> ---
>
> 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@f43200f0 {
> +               compatible = "cortina,ca-leds";
> +               reg = <0x0 0xf43200f0 0x40>;
> +
> +               cortina,blink-rate1 = <256>;
> +               cortina,blink-rate2 = <512>;
> +
> +               led@0 {
> +                       pin = <0>;
> +                       active-low;
> +                       blink-sel =<0>;
> +                       port = <0>;
> +                       off-event = <0>;
> +                       label = "led0";
> +               };
> +
> +               led@1 {
> +                       pin = <1>;
> +                       active-low;
> +                       blink-sel =<1>;
> +                       label = "led1";
> +               };
> +
> +               led@2 {
> +                       pin = <2>;
> +                       active-low;
> +                       label = "led2";
> +               };
> +
> +       };
>  };

Regards,
Simon

Reply via email to