linux-kernel - Re: [PATCH v6 6/7] arm64: dts: meson: Fix schema warnings for pwm-leds

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 2 Oct 2020 11:23:11 +0200
From:   Neil Armstrong <narmstrong@...libre.com>
To:     Alexander Dahl <post@...pocky.de>, linux-leds@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-samsung-soc@...r.kernel.org, linux-omap@...r.kernel.org,
        linux-stm32@...md-mailman.stormreply.com,
        linux-amlogic@...ts.infradead.org, linux-mips@...r.kernel.org
Cc:     Alexander Dahl <ada@...rsis.com>,
        Peter Ujfalusi <peter.ujfalusi@...com>,
        Rob Herring <robh+dt@...nel.org>, Dan Murphy <dmurphy@...com>,
        Pavel Machek <pavel@....cz>,
        Jacek Anaszewski <jacek.anaszewski@...il.com>
Subject: Re: [PATCH v6 6/7] arm64: dts: meson: Fix schema warnings for
 pwm-leds

On 01/10/2020 01:46, Alexander Dahl wrote:
> The node names for devices using the pwm-leds driver follow a certain
> naming scheme (now).
> 
> Signed-off-by: Alexander Dahl <post@...pocky.de>
> ---
> 
> Notes:
>     v6:
>       * added this patch to series
> 
>  .../arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts | 4 ++--
>  arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts     | 4 ++--
>  arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts          | 8 ++++----
>  3 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts b/arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts
> index 8bcdffdf55d0..adfc72500e66 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts
> @@ -42,10 +42,10 @@
>  		};
>  	};
>  
> -	pwmleds {
> +	led-controller {
>  		compatible = "pwm-leds";
>  
> -		power {
> +		led-1 {
>  			label = "vim:red:power";
>  			pwms = <&pwm_AO_ab 1 7812500 0>;
>  			max-brightness = <255>;
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
> index bff8ec2c1c70..dcb435af4e0b 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
> @@ -81,10 +81,10 @@
>  		};
>  	};
>  
> -	pwmleds {
> +	led-controller {
>  		compatible = "pwm-leds";
>  
> -		power {
> +		led-1 {
>  			label = "vim:red:power";
>  			pwms = <&pwm_AO_ab 1 7812500 0>;
>  			max-brightness = <255>;
> diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts
> index 5ab139a34c01..039a8d0d1e9b 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts
> @@ -101,20 +101,20 @@
>  		};
>  	};
>  
> -	leds {
> +	led-controller-1 {
>  		compatible = "gpio-leds";
>  
> -		led-bluetooth {
> +		led-1 {
>  			label = "sei610:blue:bt";
>  			gpios = <&gpio GPIOC_7 (GPIO_ACTIVE_LOW | GPIO_OPEN_DRAIN)>;
>  			default-state = "off";
>  		};
>  	};
>  
> -	pwmleds {
> +	led-controller-2 {
>  		compatible = "pwm-leds";
>  
> -		power {
> +		led-2 {
>  			label = "sei610:red:power";
>  			pwms = <&pwm_AO_ab 0 30518 0>;
>  			max-brightness = <255>;
> 

Reviewed-by: Neil Armstrong <narmstrong@...libre.com>