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 E07C3C4321E for ; Mon, 5 Dec 2022 21:08:27 +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:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=En9SgoKhMv7Gr+HEnElD6BjI963JPFhbAHgPEM9sEgE=; b=bKaDFsmRxit3UK 6ZnB2+Bp+PJDt4+RxUMvzpcMKAmcDg8K6dlgPQ8M5aczK0ecM9SmxuSRzqTzle/WqrejtnifuGwtR lAiWFGDxD0QNDRfGf/Y52BRNpN3R+uqUThTJjQ9hWZV8CQuPUCAWaIXnAsLdTAHTaUzl5kLMUw9CL rTvBiJMER50/c0BscE8Iz2zgfbTGNMNpytJ//wkrq358wFLcE02qBl/ChE2cJqMkB6nQsB4TmL2fN lrxILkvU1+IwCk7XYlE6UHUF7ws9GQQ/4ItIJb8Rtbfn2bxAFatPwZ0iNujFNohEVLYaYetipmGUP 6iKYUqoQur80dPnWpu3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2IgR-00AbDu-JH; Mon, 05 Dec 2022 21:07:27 +0000 Received: from mail-ed1-x530.google.com ([2a00:1450:4864:20::530]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p2IgN-00AbCE-EB for linux-arm-kernel@lists.infradead.org; Mon, 05 Dec 2022 21:07:24 +0000 Received: by mail-ed1-x530.google.com with SMTP id e13so17547595edj.7 for ; Mon, 05 Dec 2022 13:07:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=5YyjWNhacttlG/+fLR1iZBtdi1pM5GJQVC7qwybysfY=; b=b5OX02cg7SErXkv4WY3gJwWSacUK9wlUWppABaoFuNN+gGzLCR6G0TWPiizgWwbRiH 4JnzvXpCbRbY+6kR8rzADqQ9u91D76LD794OjSyevq3Z8M5erx8f+VjYMhhMW70obv2k gi1rzWHjW2ROOIpiBPdqsArA6dMyQMMFOA5W7LDUR5bECF4xg7XC64dDtQEmQWthN4L2 Lcl3ugi1seZewMO0N0Zzjs0NWsbLomjuwsyylLroCl0b9io5Fu5Tte/FW+wxvtXLTSzH V5qfhCyLEasVrbKquMyUxCXTmbwhU/AW86pvoRyHeqBY8PN9DZX97OusD12jBFMhNVJc KE4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=5YyjWNhacttlG/+fLR1iZBtdi1pM5GJQVC7qwybysfY=; b=6O8LQ1nR40vDBYblM5uRj/qzAM56aGNGYmQ2GlWFfMFkSLC/f/+fE5Jt3YoiWG7Mrl a9xU5Fe7UjfoNsiviXIGKDGki7WJdVpEvK0Mn3jaA/tgVKnLMzGMlncqwTeD2LY+lJqi 5QT6Oi0a9nqMNCA48lkxvlAegn4EVtjj9WC+E1Cg9XCNgqPgrVDs/Rxc5+szHNFMEOrA HyAUGgahmCYBDTfGshM2y/LGQKKZMXbDYR50hsASrKoxlBv1I1dwmJxrCYtN+4l0Uu5D DpCVaCYO4WhGsk7eUhheDRqfikcdQqLOGXUEn+1N2iWmYYXs+eOXHmwjLegYbn5XuMC3 SubQ== X-Gm-Message-State: ANoB5pkikZS+sKSix9iCqLjt4zMSD/sMaZ/I2OGHx49e6IE9TDBJgt/l q5pn9kaw6yyEAo03W27EXNg= X-Google-Smtp-Source: AA0mqf5XnFwWpw+Bq5wUA3Blf/UMVsZH5iz85hcQ0084cyKJEKCGzQLnnYAQrR5En+CrMn2lauzzNQ== X-Received: by 2002:a05:6402:1c1a:b0:46c:74f0:c064 with SMTP id ck26-20020a0564021c1a00b0046c74f0c064mr8969352edb.85.1670274442174; Mon, 05 Dec 2022 13:07:22 -0800 (PST) Received: from kista.localnet (82-149-19-102.dynamic.telemach.net. [82.149.19.102]) by smtp.gmail.com with ESMTPSA id dk21-20020a0564021d9500b0045723aa48ccsm206135edb.93.2022.12.05.13.07.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Dec 2022 13:07:21 -0800 (PST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Andre Przywara , Samuel Holland Cc: Chen-Yu Tsai , Krzysztof Kozlowski , Rob Herring , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@lists.linux.dev Subject: Re: Re: [PATCH 1/2] ARM: dts: sunxi: Fix GPIO LED node names Date: Mon, 05 Dec 2022 22:07:20 +0100 Message-ID: <2324364.NG923GbCHz@kista> In-Reply-To: <2979be86-e561-e5ff-b348-367a7c20fab1@sholland.org> References: <20221125195401.61642-1-samuel@sholland.org> <20221125214050.711997f1@slackpad.lan> <2979be86-e561-e5ff-b348-367a7c20fab1@sholland.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221205_130723_508979_9D3EE55E X-CRM114-Status: GOOD ( 27.24 ) 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 Samuel, Dne petek, 25. november 2022 ob 22:50:07 CET je Samuel Holland napisal(a): > 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. I'm not a fan of that either, but binding really wants enumerator. So let's conform to that. Best regards, Jernej > > > 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