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 CD4A7C4167B for ; Mon, 27 Nov 2023 23:36:05 +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=WSs01k9L9pm3dPZmTkWB2CdyOtpFU6PDDHguRkLP3Bc=; b=Fyu2psD6WaSl0B g6bX11vc9Lc3U9G5Z3049koXJqIJNJyihFnMLRQe+QRWbnX3v1V5u+FCWb1fuyAxOsJTJm09Xm/8e WrcoHdakZo82F8tQTzUK0V6muhsPPZpmxkC0BGVo5PO4VCB09RQELDiCzVuzErr2VqVok+X2l1zYt rB/yKhJeQPvX0ke7E0zaAHz4TrKYw2a8pJzsw/X7shbbRBk8ezHa5lxiN0xLpVyLzPVV2OHgEqUyi HR+229RKYT1cQM4Sjw1eKTi1yVQDv2V4X+a9RkXA3XRIADptXRco8KFPqgkhHlYGNXUOwJHMFKoQL Zdqk2VIjinCTA6wISRiQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r7l8d-003g4p-20; Mon, 27 Nov 2023 23:35:39 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1r7l8Z-003g3r-1s; Mon, 27 Nov 2023 23:35:37 +0000 Received: from i53875bf8.versanet.de ([83.135.91.248] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r7l8U-0007RK-4f; Tue, 28 Nov 2023 00:35:30 +0100 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Alex Bee Cc: Jagan Teki , devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org, Tim Lunn , Rob Herring , linux-arm-kernel@lists.infradead.org, Krzysztof Kozlowski , Conor Dooley , Krzysztof Kozlowski Subject: Re: [PATCH v2 4/9] ARM: dts: rockchip: rv1126: Add i2c2 nodes Date: Tue, 28 Nov 2023 00:35:29 +0100 Message-ID: <24475706.hxa6pUQ8Du@diego> In-Reply-To: <5e89eb12-3462-422e-aca3-7cc18743310c@gmail.com> References: <20231122122232.952696-1-tim@feathertop.org> <50694679.MN2xkq1pzW@diego> <5e89eb12-3462-422e-aca3-7cc18743310c@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231127_153535_646379_44E1D3CE X-CRM114-Status: GOOD ( 28.45 ) 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="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Alex, Am Dienstag, 28. November 2023, 00:11:37 CET schrieb Alex Bee: > Hi Heiko, > Am 27.11.23 um 10:45 schrieb Heiko St=FCbner: > > Am Montag, 27. November 2023, 10:27:41 CET schrieb Krzysztof Kozlowski: > >> On 27/11/2023 10:23, Tim Lunn wrote: > >>> Hi > >>> > >>> On 11/27/23 17:52, Krzysztof Kozlowski wrote: > >>>> On 27/11/2023 00:06, Heiko St=FCbner wrote: > >>>>> Hi Krzysztof, > >>>>> > >>>>> Am Mittwoch, 22. November 2023, 13:29:47 CET schrieb Krzysztof Kozl= owski: > >>>>>> On 22/11/2023 13:22, Tim Lunn wrote: > >>>>>>> Add i2c2 node and i2c2_xfer pinctrl for Rockchip RV1126 > >>>>>>> > >>>>>>> Signed-off-by: Tim Lunn > >>>>>>> --- > >>>>>>> > >>>>>>> (no changes since v1) > >>>>>>> > >>>>>>> arch/arm/boot/dts/rockchip/rv1126-pinctrl.dtsi | 10 ++++++++++ > >>>>>>> arch/arm/boot/dts/rockchip/rv1126.dtsi | 15 ++++++++++= +++++ > >>>>>>> 2 files changed, 25 insertions(+) > >>>>>>> > >>>>>>> diff --git a/arch/arm/boot/dts/rockchip/rv1126-pinctrl.dtsi b/arc= h/arm/boot/dts/rockchip/rv1126-pinctrl.dtsi > >>>>>>> index 4f85b7b3fc4c..167a48afa3a4 100644 > >>>>>>> --- a/arch/arm/boot/dts/rockchip/rv1126-pinctrl.dtsi > >>>>>>> +++ b/arch/arm/boot/dts/rockchip/rv1126-pinctrl.dtsi > >>>>>>> @@ -87,6 +87,16 @@ i2c0_xfer: i2c0-xfer { > >>>>>>> <0 RK_PB5 1 &pcfg_pull_none_drv_level_0_smt>; > >>>>>>> }; > >>>>>>> }; > >>>>>>> + i2c2 { > >>>>>>> + /omit-if-no-ref/ > >>>>>>> + i2c2_xfer: i2c2-xfer { > >>>>>>> + rockchip,pins =3D > >>>>>>> + /* i2c2_scl */ > >>>>>>> + <0 RK_PC2 1 &pcfg_pull_none_drv_level_0_smt>, > >>>>>>> + /* i2c2_sda */ > >>>>>>> + <0 RK_PC3 1 &pcfg_pull_none_drv_level_0_smt>; > >>>>>>> + }; > >>>>>>> + }; > >>>>>>> pwm2 { > >>>>>>> /omit-if-no-ref/ > >>>>>>> pwm2m0_pins: pwm2m0-pins { > >>>>>>> diff --git a/arch/arm/boot/dts/rockchip/rv1126.dtsi b/arch/arm/bo= ot/dts/rockchip/rv1126.dtsi > >>>>>>> index 6c5c928f06c7..cf1df75df418 100644 > >>>>>>> --- a/arch/arm/boot/dts/rockchip/rv1126.dtsi > >>>>>>> +++ b/arch/arm/boot/dts/rockchip/rv1126.dtsi > >>>>>>> @@ -21,6 +21,7 @@ / { > >>>>>>> = > >>>>>>> aliases { > >>>>>>> i2c0 =3D &i2c0; > >>>>>>> + i2c2 =3D &i2c2; > >>>>>> No, this should be per-board to match board labeling/schematics. > >>>>> At least for i2c, uarts and i.e. spi ... Rockchip manuals, pin nami= ngs > >>>>> and also all board schematics I've seen so far are very consistent = for > >>>>> these ... i2c2 for example is labled i2c2 both in the pins in the s= ocs > >>>>> and also in the board-schematics using them. > >>>>> > >>>>> So while I can agree that things like mmc-aliases might be board-sp= ecific, > >>>>> I do think aliases for the core busses should be able to live in th= e soc dtsi > >>>>> as for all Rockchip SoCs so far? > >>>> If you do not list here all aliases, it is already board-specific, i= sn't it? > >>>> > >>>> https://lore.kernel.org/linux-rockchip/CAK8P3a25iYksubCnQb1-e5yj=3Dc= rEsK37RB9Hn4ZGZMwcVVrG7g@mail.gmail.com/ > >>> I had only added aliases for nodes that are implemented. RV1126 has 6 > >>> i2c busses but so far not all these exist in the soc.dtsi. > >>> > >> > >> OK, that would explain why you have only two. Anyway, it is just gener= ic > >> guideline, so up to Heiko what to do with it. > > = > > People see "uart2-tx" on their pin-header description and then of course > > want to use ttyS2 ... same with reading i2c2-sda on the pin-header and = then > > running i2cdetect on said i2c2 dev node. > > = > If that's "the" rule, why was rk3128.dtsi merged after similar = > complaints came up [0] without having aliases for the core buses = > defined? probably because it slipped through or so. At least in my argument in the mail you linked I'm consistent to here ;-) > It's pretty strange to have to define them on board-level there = > as well. > > I'm happy to submit a patch which changes that. Yep and rk3128 really is the only outlier there, so sure, go ahead with moving the core ones. Heiko > [0] https://lore.kernel.org/all/22076018.EfDdHjke4D@diego/ _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel