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

Neil Armstrong
2020-10-02T09:23:17Z

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@lespocky.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@baylibre.com>