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 05CA9C43334 for ; Tue, 26 Jul 2022 12:35:31 +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:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=13lKoPXuZ16Udw6S5Mm25i5Hj25D+t0S7NxF2R2MVAM=; b=3z3fiZQJ0GvnId MednL6yvgCeKuu3gXM0kAZ1A5pGkIPK79BWdXIA7oFzCkC5mZ+f0AofX9R/kMtCDfzNFxf7H8Ysyl 4TyBJO/+fu9bPgNbuYT8DkFF8XaqwwonwX1jvRxBZlN4Ga1zd7/CjeSmQc9tqqZwiQMjT56+XSEzU yd3vOQxtkdGEZ1fHD6fqoz4462AG4DCqBV7UcL0uL8USj9eI7IdpIY/en2k2qflB6svg339Lo3G5P B5B29744H/4vTxWDmSQCemF9TRLtbYO7Ol2CDP68DVB2xwFZl+MEXHP8qg8jNHcVGeQ/7JhpEabMi RUbiKtJBK+9tfDfLuQKg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oGJlf-00GyPz-Ez; Tue, 26 Jul 2022 12:34:31 +0000 Received: from smtp16.bhosted.nl ([2a02:9e0:8000::27]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oGJlc-00GyNx-HT for linux-arm-kernel@lists.infradead.org; Tue, 26 Jul 2022 12:34:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonic.nl; s=202111; h=content-transfer-encoding:content-type:mime-version:references:in-reply-to: message-id:subject:cc:to:from:date:from; bh=opcQav5T4MbREVpWWZz38bgIRyaBxYgmV1agTFpwly8=; b=oE297L+EUvSzWrbNHHzYfm/xzBWWfnxJLe9FNr5KBCREvrJzRqghey9Ykyi/4iRibuHdutOCBn3uN Z/a9X+Ognfx4w2yroWe/rh22vsMiKPm1b+9Ht5SUnw5FycAxX4/QYYsL0lXw4KKjFnlQSIjUFdlZcS s+CekPQ7XozkZX1Hc6QDxz8tooQh9coJJtr0LP4OM8jy/BBFp9IeRlM1LbHgO5Szn6yrV6uLdOOVmI X5sWKdwssIZVF+1OcST3JRkK3QXzYIXKtm0AX9YXVCLdQBwdcsRGREz7N/6EMrSB6TNJ505/pRbI7D rC/PCQFvDlDNEEUq03KE03LJ1YaKOjg== X-MSG-ID: 495cfb04-0cdf-11ed-9051-0050569d2c73 Date: Tue, 26 Jul 2022 14:34:26 +0200 From: David Jander To: Krzysztof Kozlowski Cc: Rob Herring , Krzysztof Kozlowski , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Oleksij Rempel , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm: dts: imx6qdl-vicut1.dtsi: Add backlight-isb node Message-ID: <20220726143426.1e8a186f@erd992> In-Reply-To: <340a4a81-d4ec-2f42-28ae-49124830157a@linaro.org> References: <20220725124813.1001032-1-david@protonic.nl> <20220726083006.5c65b597@erd992> <340a4a81-d4ec-2f42-28ae-49124830157a@linaro.org> Organization: Protonic Holland X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220726_053428_752725_00C412FB X-CRM114-Status: GOOD ( 28.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 Tue, 26 Jul 2022 09:01:47 +0200 Krzysztof Kozlowski wrote: > On 26/07/2022 08:30, David Jander wrote: > > On Mon, 25 Jul 2022 22:33:12 +0200 > > Krzysztof Kozlowski wrote: > > > >> On 25/07/2022 14:48, David Jander wrote: > >>> On some older hardware models this is needed. It will do no harm if used > >>> with newer models though. > >>> > >>> Signed-off-by: David Jander > >>> --- > >>> arch/arm/boot/dts/imx6qdl-vicut1.dtsi | 22 ++++++++++++++++++++++ > >>> 1 file changed, 22 insertions(+) > >>> > >>> diff --git a/arch/arm/boot/dts/imx6qdl-vicut1.dtsi b/arch/arm/boot/dts/imx6qdl-vicut1.dtsi > >>> index a1676b5d2980..08d425020088 100644 > >>> --- a/arch/arm/boot/dts/imx6qdl-vicut1.dtsi > >>> +++ b/arch/arm/boot/dts/imx6qdl-vicut1.dtsi > >>> @@ -37,6 +37,16 @@ backlight_led: backlight_led { > >>> power-supply = <®_3v3>; > >>> }; > >>> > >>> + /* only for backwards compatibility with old HW */ > >>> + backlight_isb: backlight_isb { > >> > >> Node name: backlight-isb > >> > >> I know that other node had underscore, but these are not correct for names. > > > > I know the names are not correct, but they are in the style of existing node > > names in the same file. Do you want me to change them all, or just this one? > > This change would break existing user-space software though, which is something > > I prefer not to do. Unfortunately both names are relied upon by user-space > > software for almost 10 years now, and changing them will force me to maintain > > off-tree patches that revert these changes. > > I understand the reason for the change though, and if you insist I will do it. > > Node names are not ABI, thus any user-space depending on them does it > wrong. Could you link/name which user-space depends on it? > > I don't understand why new node is also already a dependency. It's a new > node, there cannot be anything which depends on it before it is being > applied in upstream. Never mind. I already sent out v2 which fixes both node names. Thanks. Best regards, -- David Jander _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel