From patchwork Mon Jul 1 16:49:52 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 13718409 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BECBBC2BD09 for ; Mon, 1 Jul 2024 16:50:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-Type: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=6WYsTFcHc1U6T4hl6euHdIy2ChhnfJYSvBlY2eCJ30o=; b=yQhy38wJjoG9pBVjWDj9ZdAT59 6RigZumhT1j92FDjnE5FvQnajInbPW0HgE8N1srQdrIFJBTkbFAMr89U1Zx1CRZg/9yWk24CLGLja gDgN34wRYto5zxSRueBlDE9Tu1sAYRBdX1j94z6e784ZVH9ASXPszcOPJuU5iCy2A4rV5mi6PiDu9 l06r+Uvpk478bVXzNUaDErzYog7kaa84NaRZPJKGgVzCLM9zJTzEp7rQO0C7msf2GwPR+BQ5YdYrb kVUp/exyaV8t0eDySB/5hfycab7ls1CJlatZ33BvYK5wJFi4Fq6U7Aaki8kXKA51QKTi7DWCq+2ss yBakm/3g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOKEK-00000004And-3E0C; Mon, 01 Jul 2024 16:50:16 +0000 Received: from mail-wr1-x430.google.com ([2a00:1450:4864:20::430]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOKE0-00000004AeB-3Jwd for linux-arm-kernel@lists.infradead.org; Mon, 01 Jul 2024 16:49:59 +0000 Received: by mail-wr1-x430.google.com with SMTP id ffacd0b85a97d-3658197cdbbso1804642f8f.3 for ; Mon, 01 Jul 2024 09:49:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1719852595; x=1720457395; darn=lists.infradead.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=6WYsTFcHc1U6T4hl6euHdIy2ChhnfJYSvBlY2eCJ30o=; b=jTlFhY1ax4fym1sFEDoJ06rXa1kbDf2eWnqUit6OFKqvVc4DRsi8QPS0y7ibKqYryb H2pkk1nYGXJ80SYeGr8dBTdVzmgsnDZa//rw649S6687YIVAmczo8aHyBN3mFCYuHy7T 3PBn2dRAbh314NKGCrHV86ghjjdAauMaW68QJBJb26MQGTGekZy92APN3eDe1i+0YJ0K RnnWpQj2g2NGdCWi9ncftrw9dwXpLghRk7AVTRlJjJhYtx4psXpYdrhdclMNCA45xDOp hflApR5DSDngeCvSPGIhXkemKA25cNhlkCMi6fbPbkvxSYyeGD42eN/TBN6xufcfal5N 6fyw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1719852595; x=1720457395; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=6WYsTFcHc1U6T4hl6euHdIy2ChhnfJYSvBlY2eCJ30o=; b=Sf7ApBL24HtQDch6ilQ2/PxqFd253Qwnp48YWGdwN9pQsfipYYmWz0AYwwe/jI8soc g+dQoTfO72a4X/0XFL9y8F3b37bNtFO+NOpDkeSbHkgGC/cATqWBCS8YMhqQEiy5AS5v TrtzsRlaXm4oQuY4Lq/XQUUkcx3dGmEROMlW7dokt7LJyKVaN+rxAptLS3WlV4hBeDus S+sTCW40udcdH0tTkTESwJPWjk12q3i2ATF9xoDGcsUhW5iVjXB1lG9BDSU6F3PAcFqC J8/m2j9z9O97ZaEYt/UsaheFO3wTb60M3LV+EwiJYQ7jVen3E7XpJ5QFfWzbsyOEtdqq CZMg== X-Forwarded-Encrypted: i=1; AJvYcCUaQukJKb/+wMenGdwX0dqi27nxiYHKLIB7i1M/MFm7qEIsE4G3y8uFHx46z5nYRr0v5qsJMBRq5a8tK2kKKXT3dAbRx/g5lopCrWMipqhMaQ5UmMw= X-Gm-Message-State: AOJu0Yy7uWPlR2xMS2aLDEWvkyFKDwgM3dl7VMsXISPruBS1Kpy5SH5o PHAjfk7FmUG/rO6YRE8PvOZ+AvprG3ONG8Ya6FFMGHX0Lj0GgEawG4OktIBE72Y= X-Google-Smtp-Source: AGHT+IGpvak2bY7pflYuNHWyul4+lpbEDkugsotfcFTVhB4nkxQ+OR3E6SnvILUjQvg31iUOjT89qg== X-Received: by 2002:a05:6000:1449:b0:35f:2a02:6038 with SMTP id ffacd0b85a97d-36775725989mr5314902f8f.47.1719852595216; Mon, 01 Jul 2024 09:49:55 -0700 (PDT) Received: from krzk-bin.. ([178.197.219.137]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3675a0fba2dsm10559667f8f.73.2024.07.01.09.49.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Jul 2024 09:49:54 -0700 (PDT) From: Krzysztof Kozlowski To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski Subject: [PATCH] ARM: dts: microchip: at91: align LED node name with bindings Date: Mon, 1 Jul 2024 18:49:52 +0200 Message-ID: <20240701164952.577277-1-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240701_094956_948906_26CBC553 X-CRM114-Status: GOOD ( 11.62 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Bindings expect the LED node names to follow certain pattern, see dtbs_check warnings: at91sam9g15ek.dtb: leds: 'pb18', 'pd21' do not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+' Signed-off-by: Krzysztof Kozlowski Reviewed-by: Alexander Dahl --- .../boot/dts/microchip/at91sam9g20ek_2mmc.dts | 4 ++-- .../at91sam9g25-gardena-smart-gateway.dts | 24 +++++++++---------- arch/arm/boot/dts/microchip/at91sam9n12ek.dts | 6 ++--- arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi | 4 ++-- 4 files changed, 19 insertions(+), 19 deletions(-) diff --git a/arch/arm/boot/dts/microchip/at91sam9g20ek_2mmc.dts b/arch/arm/boot/dts/microchip/at91sam9g20ek_2mmc.dts index 172af6ff4b18..3e5eab57d1a5 100644 --- a/arch/arm/boot/dts/microchip/at91sam9g20ek_2mmc.dts +++ b/arch/arm/boot/dts/microchip/at91sam9g20ek_2mmc.dts @@ -40,13 +40,13 @@ pinctrl_board_mmc0_slot0: mmc0_slot0-board { leds { compatible = "gpio-leds"; - ds1 { + led-ds1 { label = "ds1"; gpios = <&pioB 9 GPIO_ACTIVE_HIGH>; linux,default-trigger = "heartbeat"; }; - ds5 { + led-ds5 { label = "ds5"; gpios = <&pioB 8 GPIO_ACTIVE_LOW>; }; diff --git a/arch/arm/boot/dts/microchip/at91sam9g25-gardena-smart-gateway.dts b/arch/arm/boot/dts/microchip/at91sam9g25-gardena-smart-gateway.dts index af70eb8a3a02..e0c1e8df81b1 100644 --- a/arch/arm/boot/dts/microchip/at91sam9g25-gardena-smart-gateway.dts +++ b/arch/arm/boot/dts/microchip/at91sam9g25-gardena-smart-gateway.dts @@ -37,71 +37,71 @@ button { leds { compatible = "gpio-leds"; - power_blue { + led-power-blue { label = "smartgw:power:blue"; gpios = <&pioC 21 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - power_green { + led-power-green { label = "smartgw:power:green"; gpios = <&pioC 20 GPIO_ACTIVE_HIGH>; default-state = "on"; }; - power_red { + led-power-red { label = "smartgw:power:red"; gpios = <&pioC 19 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - radio_blue { + led-radio-blue { label = "smartgw:radio:blue"; gpios = <&pioC 18 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - radio_green { + led-radio-green { label = "smartgw:radio:green"; gpios = <&pioC 17 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - radio_red { + led-radio-red { label = "smartgw:radio:red"; gpios = <&pioC 16 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - internet_blue { + led-internet-blue { label = "smartgw:internet:blue"; gpios = <&pioC 15 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - internet_green { + led-internet-green { label = "smartgw:internet:green"; gpios = <&pioC 14 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - internet_red { + led-internet-red { label = "smartgw:internet:red"; gpios = <&pioC 13 GPIO_ACTIVE_HIGH>; default-state = "off"; }; - heartbeat { + led-heartbeat { label = "smartgw:heartbeat"; gpios = <&pioB 8 GPIO_ACTIVE_HIGH>; linux,default-trigger = "heartbeat"; }; - pb18 { + led-pb18 { status = "disabled"; }; - pd21 { + led-pd21 { status = "disabled"; }; }; diff --git a/arch/arm/boot/dts/microchip/at91sam9n12ek.dts b/arch/arm/boot/dts/microchip/at91sam9n12ek.dts index 4c644d4c6be7..643c3b2ab97e 100644 --- a/arch/arm/boot/dts/microchip/at91sam9n12ek.dts +++ b/arch/arm/boot/dts/microchip/at91sam9n12ek.dts @@ -207,19 +207,19 @@ bl_reg: backlight_regulator { leds { compatible = "gpio-leds"; - d8 { + led-d8 { label = "d8"; gpios = <&pioB 4 GPIO_ACTIVE_LOW>; linux,default-trigger = "mmc0"; }; - d9 { + led-d9 { label = "d9"; gpios = <&pioB 5 GPIO_ACTIVE_LOW>; linux,default-trigger = "nand-disk"; }; - d10 { + led-d10 { label = "d10"; gpios = <&pioB 6 GPIO_ACTIVE_HIGH>; linux,default-trigger = "heartbeat"; diff --git a/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi b/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi index cdd37f67280b..fb3c19bdfcb6 100644 --- a/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi +++ b/arch/arm/boot/dts/microchip/at91sam9x5cm.dtsi @@ -120,13 +120,13 @@ rootfs@800000 { leds { compatible = "gpio-leds"; - pb18 { + led-pb18 { label = "pb18"; gpios = <&pioB 18 GPIO_ACTIVE_LOW>; linux,default-trigger = "heartbeat"; }; - pd21 { + led-pd21 { label = "pd21"; gpios = <&pioD 21 GPIO_ACTIVE_HIGH>; };