From mboxrd@z Thu Jan 1 00:00:00 1970 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 E97D3C4332F for ; Fri, 25 Nov 2022 21:51:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Subject:From:References:Cc: To:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=2DbkrUPs7mAYy+TzIRQP1gbYUHPFTd2lvPbUxsCQYaM=; b=klArIk7uoBt0zy i7K4fhclpOesULCUgFOueLsOpllS2qm3rfZ3imCxg3aI7henKAJU4DsZkGmxdK6Fq16Rq7yXhmFKZ qobuksKpJCDkbb9/+vQLTx4OPvrZh5GThLNlIwxlXjIZcIkx6EGlmKLj09HXPPO5Y2FbuXgOxwEj3 7eiTnsORE/tEP3WkoKevvVDNfjxQWKoC5R6Vqg8T7p34nhdWXUJfkwQbSmu7bscdfnQGcrP/ck4oL 18cT111BmIXBElIOzjbWPMycQIOORGhOVPnoON4fxozS1hsWmdufnvsVcJczJ0I1Tq2duqrL8Ezqy Epj4q/l9Wf7dmWmVXlMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oygaQ-002Kj8-TE; Fri, 25 Nov 2022 21:50:19 +0000 Received: from out3-smtp.messagingengine.com ([66.111.4.27]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oygaN-002Kgv-4N for linux-arm-kernel@lists.infradead.org; Fri, 25 Nov 2022 21:50:17 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.nyi.internal (Postfix) with ESMTP id D1D9C5C00B3; Fri, 25 Nov 2022 16:50:09 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute4.internal (MEProxy); Fri, 25 Nov 2022 16:50:09 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sholland.org; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to; s=fm2; t=1669413009; x= 1669499409; bh=S1O8VZU+Rduz5/f4O46rBMJK70Funpw/H3+pweGpMJ0=; b=Y S2avTfkHFGy8sgHxyQ7JLwvolqnfDfK3or1D9V5RS3HRK5jyCYf26riMYJOaWP3w vTHhVlAtaVg6RBRfjeRaLrCABUycfuonOjFWbVpX0bawYLnGNRXSb5aij3sEht0G e6iT1XJJjECgcVE8CoVGyXrm7/QPGeDas0UEXlfA00rS94jy26298MIOoBPLrmWy 9FcGSsb3MmhVQfP2Xl9frjLO+sANB0WgIagJExMQITNx0b0rPxc7XQvBBVmFPfcW veyx4zjPnWKrHTVaZbI6FTJ/NlQmKuOZ/cO2FD6O+ttNmbqz0htXLbtVhioUoUzf Oo3C+9BP4b94949hx4How== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1669413009; x= 1669499409; bh=S1O8VZU+Rduz5/f4O46rBMJK70Funpw/H3+pweGpMJ0=; b=w IsWJCIPr8U6zbCg/ci5FWsELdhw8kYbvT0EDWaJYz6LLS6mPG3lFQpkiS1b9xJ5r TMhZhCIMXxjuTJa9ZiZ0f2RyT3VZkPXLmT0iZ+TlycbqviEQ747g63BIDUSecJX+ srcuv1AoHBUuEpneFeuylMsuSym+ADwi+1Jj+dAC1vRfT2135S8b0xe2gPZWcTZS LqMMWURqyOTLjhCvjMIdspPuTgvqY7jBcIB1RNjXxtqhs/siCdpVRkpuDYsF5eQ+ kSIMK2eJ0U7WsmO7IHvCW/FZBl+No/5kibR/eUyWUK8Dk+MH5we5QUxCcKsR3Vu2 8TrC61xlWVMgIXzhplpdw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvgedrieehgdduheegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfvfevfhfhufgjtgfgsehtjeertddtfeejnecuhfhrohhmpefurghm uhgvlhcujfholhhlrghnugcuoehsrghmuhgvlhesshhhohhllhgrnhgurdhorhhgqeenuc ggtffrrghtthgvrhhnpeejgfffhfdujeeftdeuudeguedttefgieetffffheejuefguedv heejteeftdfftdenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfh hrohhmpehsrghmuhgvlhesshhhohhllhgrnhgurdhorhhg X-ME-Proxy: Feedback-ID: i0ad843c9:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 25 Nov 2022 16:50:08 -0500 (EST) Message-ID: <2979be86-e561-e5ff-b348-367a7c20fab1@sholland.org> Date: Fri, 25 Nov 2022 15:50:07 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux ppc64le; rv:102.0) Gecko/20100101 Thunderbird/102.3.3 Content-Language: en-US To: Andre Przywara Cc: Chen-Yu Tsai , Jernej Skrabec , Krzysztof Kozlowski , Rob Herring , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@lists.linux.dev References: <20221125195401.61642-1-samuel@sholland.org> <20221125214050.711997f1@slackpad.lan> From: Samuel Holland Subject: Re: [PATCH 1/2] ARM: dts: sunxi: Fix GPIO LED node names In-Reply-To: <20221125214050.711997f1@slackpad.lan> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221125_135015_812327_617F6F6B X-CRM114-Status: GOOD ( 20.36 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Andre, On 11/25/22 15:40, Andre Przywara wrote: > On Fri, 25 Nov 2022 13:54:00 -0600 > Samuel Holland wrote: > > Hi Samuel, > >> These board devicetrees fail to validate because the gpio-leds schema >> requires its child nodes to have "led" in the node name. >> >> Signed-off-by: Samuel Holland > > That looks alright, though the comment in the binding says that we > should just have led-0, led-1 instead, so just (hex) numbers. The > "status" name is also in the label, so we wouldn't lose information. I am not a fan of giving the LEDs meaningless enumerators, but I can do that if the maintainers insist. > Actually, also "label" is deprecated, in favour of "color" and > "function", shall this be fixed on the way? Or is there anything that > breaks (older kernels) when removing the label property? The label is exposed to userspace as the path in sysfs, so we cannot change it. There is no way to construct that exact label using function and color -- see led_compose_name(). Regards, Samuel >> --- >> >> arch/arm/boot/dts/sun5i-gr8-chip-pro.dts | 2 +- >> arch/arm/boot/dts/sun5i-r8-chip.dts | 2 +- >> arch/arm/boot/dts/sun6i-a31s-sina31s.dts | 2 +- >> 3 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/boot/dts/sun5i-gr8-chip-pro.dts b/arch/arm/boot/dts/sun5i-gr8-chip-pro.dts >> index a32cde3e32eb..3222f1490716 100644 >> --- a/arch/arm/boot/dts/sun5i-gr8-chip-pro.dts >> +++ b/arch/arm/boot/dts/sun5i-gr8-chip-pro.dts >> @@ -70,7 +70,7 @@ chosen { >> leds { >> compatible = "gpio-leds"; >> >> - status { >> + led-status { >> label = "chip-pro:white:status"; >> gpios = <&axp_gpio 2 GPIO_ACTIVE_HIGH>; >> default-state = "on"; >> diff --git a/arch/arm/boot/dts/sun5i-r8-chip.dts b/arch/arm/boot/dts/sun5i-r8-chip.dts >> index 4bf4943d4eb7..303191c926c2 100644 >> --- a/arch/arm/boot/dts/sun5i-r8-chip.dts >> +++ b/arch/arm/boot/dts/sun5i-r8-chip.dts >> @@ -70,7 +70,7 @@ chosen { >> leds { >> compatible = "gpio-leds"; >> >> - status { >> + led-status { >> label = "chip:white:status"; >> gpios = <&axp_gpio 2 GPIO_ACTIVE_HIGH>; >> default-state = "on"; >> diff --git a/arch/arm/boot/dts/sun6i-a31s-sina31s.dts b/arch/arm/boot/dts/sun6i-a31s-sina31s.dts >> index 0af48e143b66..b84822453381 100644 >> --- a/arch/arm/boot/dts/sun6i-a31s-sina31s.dts >> +++ b/arch/arm/boot/dts/sun6i-a31s-sina31s.dts >> @@ -67,7 +67,7 @@ hdmi_con_in: endpoint { >> leds { >> compatible = "gpio-leds"; >> >> - status { >> + led-status { >> label = "sina31s:status:usr"; >> gpios = <&pio 7 13 GPIO_ACTIVE_HIGH>; /* PH13 */ >> }; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel