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 D015AC43334 for ; Sun, 12 Jun 2022 17:08:00 +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:From:References:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Cc:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=PrrSrKQOJm4qCG3MSjmQ3EfEt8o5+8Kc0ht99nMsDlw=; b=o2eDeCXE+H9Sww Spl+yqCQvQ4PGMVM5bA63lXtIH9loxUeCGDJ/ApNOIrk1BChjXocG1gSzFidp/zVI62JjVfULqzEp 7VicD453bydmfWHigYzym5HBENpAm2ee2zFehHYFjDwwI72BUo75Cq+a5nFisjafITcxsqneN+MUv YxWZFGBTA3Mzc6J5L0+CN3xHhZGl2BgdmPvBTQVWpXsluVifr5lfqv/NQ7hgqTZ4mXaiZgGkZB81E WHr7XfhymvbrvHwckbos+3TEuQ5VMy5yAQjHZvd6C2xmlSJLpTk5QXBlpB+9FDw4sJeq2sVWNDhFE yxmrodXesMnZr5ImTBiA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o0R2Z-00H9si-9M; Sun, 12 Jun 2022 17:06:19 +0000 Received: from mail-ej1-x62c.google.com ([2a00:1450:4864:20::62c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1o0R2V-00H9r5-Ny for linux-arm-kernel@lists.infradead.org; Sun, 12 Jun 2022 17:06:17 +0000 Received: by mail-ej1-x62c.google.com with SMTP id m20so6923187ejj.10 for ; Sun, 12 Jun 2022 10:06:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :references:from:in-reply-to:content-transfer-encoding; bh=e1j5oNyWe6LjPGibCsOkD4pe/7KGmuvi3WqbEeyX52A=; b=axtd06CitJ9XTEQlj/DgAkzQO7gwwugoWwZY1AE+3Zgrmx9g1bJNWCNMLWlwF3iEl4 GhYCjZ1F7rhF6iZMCV3zMH+89SbmhW1HYE2LWqQReDf2b8zTolL4VWga094dShEhTncU rXBUN7y3Juwybt8Udriw5j+nE2KAu+rQwc9it6aEmgN54efusB7QLYAaiLredlUnFdpx o8Zp+1zZeXtNqmAXKQQzRBNOqMx9wixdGTeA+zOfuR7gU+lFX57Rn3kpULfWzeHw4sfP riHzAZNZ4FYDkUtThXbRlJuV1Huhu/47lyl7C8BI7SiPecrv8yOniALBcDn26/A9LB+s c3PA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:references:from:in-reply-to :content-transfer-encoding; bh=e1j5oNyWe6LjPGibCsOkD4pe/7KGmuvi3WqbEeyX52A=; b=C4ona6hQP3E+yX/DJY0bwSK/dfJLsO5vrE2mohddm7Ra+AmlYWCAFAKcQzyFCkTdbC RjQsnit/EpQId36b0JcBVCZdMVnBcqbM2zhwLNhhCfnEweQ4aHk2DrB1PthTWzl6Sglu tJkrvdvN3QKGnr5WfYiweuzTfk6QJeBxUJu4cWncp3KxdVg86f2E0D20F5q6Ta5hJcp9 OxlCk0ukNM5v/skoJ4taHe+yinf4Qh/cHvrfHMIFl6lnTbhoh/AL6wp7hwO0/hsU5u5l yt4n6uY9cgE14VHKI/gma5mD+b0Vs4oHGmHPH61fgVjuA7XF53lcrfaaHwBRVUP+eDYg VnHg== X-Gm-Message-State: AOAM531kbyGG8FrXt9446MgBP9zgc8/KW9YkDG2PmnTDCjuOaMsyIF1N Gu6k5+0cSG31MznbtMXouuv34w== X-Google-Smtp-Source: ABdhPJywtatBoFfH74/PESsNXnL7SJhTfWWSGNB3lg0vls4R1pXgQ7dz3i5/bNXUiIxLtCTPyo5tTQ== X-Received: by 2002:a17:906:804a:b0:70c:1dac:9bb8 with SMTP id x10-20020a170906804a00b0070c1dac9bb8mr46957960ejw.92.1655053571275; Sun, 12 Jun 2022 10:06:11 -0700 (PDT) Received: from [192.168.0.203] (xdsl-188-155-176-92.adslplus.ch. [188.155.176.92]) by smtp.gmail.com with ESMTPSA id pv17-20020a170907209100b00704757b1debsm2675740ejb.9.2022.06.12.10.06.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 12 Jun 2022 10:06:10 -0700 (PDT) Message-ID: Date: Sun, 12 Jun 2022 19:06:09 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH 3/3] ARM: dts: exynos: add function and color to aat1290 flash LED node in Galaxy S3 Content-Language: en-US To: Jacek Anaszewski , Pavel Machek , Rob Herring , Krzysztof Kozlowski , Alim Akhtar , linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Simon Shields , =?UTF-8?Q?Martin_J=c3=bccker?= References: <20220607085343.72414-1-krzysztof.kozlowski@linaro.org> <20220607085343.72414-3-krzysztof.kozlowski@linaro.org> <4a7f8ab6-c061-3861-5790-b6c0fbd7cad1@gmail.com> From: Krzysztof Kozlowski In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220612_100615_904603_337D0229 X-CRM114-Status: GOOD ( 19.05 ) 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 On 12/06/2022 17:09, Jacek Anaszewski wrote: > On 6/10/22 12:14, Krzysztof Kozlowski wrote: >> On 09/06/2022 22:31, Jacek Anaszewski wrote: >>> Hi Krzysztof, >>> >>> On 6/7/22 10:53, Krzysztof Kozlowski wrote: >>>> Add common LED properties - the function and color - to aat1290 flash >>>> LED node in Galaxy S3. >>>> >>>> Signed-off-by: Krzysztof Kozlowski >>>> --- >>>> arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi | 3 +++ >>>> 1 file changed, 3 insertions(+) >>>> >>>> diff --git a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi >>>> index 72901772fcad..d76f3678dcab 100644 >>>> --- a/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi >>>> +++ b/arch/arm/boot/dts/exynos4412-galaxy-s3.dtsi >>>> @@ -7,6 +7,7 @@ >>>> */ >>>> >>>> /dts-v1/; >>>> +#include >>>> #include "exynos4412-midas.dtsi" >>>> >>>> / { >>>> @@ -27,6 +28,8 @@ led-controller { >>>> >>>> led { >>>> label = "flash"; >>>> + function = LED_FUNCTION_FLASH; >>>> + color = ; >>> >>> Addition of these two properties will not change anything because >>> the label has precedence. It is deprecated, but if you introduce >>> function and color to the binding instead of the label, the resulting >>> LED class device name will change. >> >> Which is not necessarily what we want, right? Adding these properties is >> a proper description of hardware, regardless whether current Linux >> implementation uses them or not. > > Actually I'd just drop label in addition to your change. > I don't think it would break anybody seriously - not expecting it has > any larger group of users and having uniformly constructed DTS files > in the mainline has greater value. > What about some PostmarketOSos, LineageOS and other OSes? Let me Cc here some folks - Simon, Martin, is the label in flash LED node anyhow important for you? Can it be dropped and replaced with function+color? https://lore.kernel.org/all/20220607085343.72414-3-krzysztof.kozlowski@linaro.org/ Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel