Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp12143816rwd; Fri, 23 Jun 2023 01:52:30 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4V3+7R4UUex+FI67AzLcnYC1sY4cnPEwDUocTV0AvCOLrOdK1MpYdP42+NxiPm47BffsgF X-Received: by 2002:a05:6a20:3d81:b0:ef:205f:8184 with SMTP id s1-20020a056a203d8100b000ef205f8184mr20832217pzi.13.1687510350455; Fri, 23 Jun 2023 01:52:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687510350; cv=none; d=google.com; s=arc-20160816; b=n61CQzXA331YKYyRPguid4zQImcycSKBdOPnrKx8jDK7H/WLWLDLLcSjw411E3dvwC o5Mjw+s7fjNNY15GsOgqfwLCLR3dMWGnz5i7NP/vbS/YDeF3NfBswaph2ZnNEtqDRrFe kOGnCp3gtLzAtynW8MaBl57z8vXOjPWF0Shf2jPIxckqca8I8Nd2DyisPXr8Q9m1Q/aF yQpOdzM6PaLJHRgk0RciJ+QPX3x16hg14skRouvSumgP0Cdl5J/4naFC0m+KxQo5upT+ 82oMaTm12qsufM1nCAmLQfqui2IzuD90XMzv85FT7kxDo2QkVWlZoZ7xcdGmtEoufYUd yshQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version; bh=DCdqEn6yhGBstHfKPaYM5xS89igd9SPv5bbRcsy9Efk=; b=lfyOcUcCE7hCsE7kJhZXYXbwcjsfWWV+r7oq6OoGDtmZjk/S93dzIqa6st/uhfLrK2 0FHUTttR7ZdO4FWyEWF4zYcK7eYmcv8talifIs1eMBcG2+yqFpr/5O1hCX35Ok9klxqM imspQqZxgZTJdySUkMcR++kbgaqnW29yh3lw4aZU3uiuhiedxuPm98ygYHeU+bTfq0hh 15dSlWh68ljATbBRE4Sy8wkHMxQsdktQSvcqu/W2DkRne/EciOi5PJ3495F+Rmx8kJhG /5kQ+9/6TFx2v0fGIbFSazamZRUcorCyWfFVAs6lRJX89lbHchrfLenPn8X4YGSA50oK 0ewA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b17-20020a170902d51100b001b20e2ba8e1si9398785plg.23.2023.06.23.01.52.17; Fri, 23 Jun 2023 01:52:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231138AbjFWIlA convert rfc822-to-8bit (ORCPT + 99 others); Fri, 23 Jun 2023 04:41:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37426 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230170AbjFWIk7 (ORCPT ); Fri, 23 Jun 2023 04:40:59 -0400 Received: from mail-qt1-f173.google.com (mail-qt1-f173.google.com [209.85.160.173]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EC6801B4; Fri, 23 Jun 2023 01:40:57 -0700 (PDT) Received: by mail-qt1-f173.google.com with SMTP id d75a77b69052e-3ff274778feso4520591cf.1; Fri, 23 Jun 2023 01:40:57 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687509657; x=1690101657; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=s3YbNGYLQjiL7Y5daSRoiDrXijuvl42Jh6Pko97orUI=; b=FZOT9MzloR5f0rp1HVmPr3Ug1nyKl8xD9dclFAlBq+J/SZXjgAQfVYV5mXiMTdOmIy Lmxn7QAmXzMnkNVhlDVUlF61o9at/3k8WwxvQgEmn/u/8chVt9sUCPK1g9Rf6n8mSggM Fc6Stq97z34jknN+ZwD+6E+dyw6OHhXnNLjEzVc6E5VPvqA2FbyHc6aZaSvC3qScgZ1l r1OsRbvtPlKZpr4Uqc9jJpjslh6YBeXhPneuiu59C6QSJTpKa8GkZ/V5p/vZ3KLO5C6f om5GBnGalJ4eGzP7ulWyWoUW2VBrL+Iv/j42TXWkbD6EibwDjI+hQZoOFKTNfM7Itm0y KVRQ== X-Gm-Message-State: AC+VfDz6pQmVvMQ/wI9VMfb6lCe0PCoHlzpmErhEBkFTK6m8KpzYGR2c QBKGDjCTRQFFNfI7+xkkVrAreAmkrewJQg== X-Received: by 2002:a05:622a:20f:b0:3f9:c539:c9d5 with SMTP id b15-20020a05622a020f00b003f9c539c9d5mr28207811qtx.68.1687509656750; Fri, 23 Jun 2023 01:40:56 -0700 (PDT) Received: from mail-yb1-f169.google.com (mail-yb1-f169.google.com. [209.85.219.169]) by smtp.gmail.com with ESMTPSA id v15-20020a2583cf000000b00bfee65aff13sm1382807ybm.40.2023.06.23.01.40.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 23 Jun 2023 01:40:56 -0700 (PDT) Received: by mail-yb1-f169.google.com with SMTP id 3f1490d57ef6-bd77424c886so448269276.0; Fri, 23 Jun 2023 01:40:56 -0700 (PDT) X-Received: by 2002:a25:c746:0:b0:b9e:889:420f with SMTP id w67-20020a25c746000000b00b9e0889420fmr20362043ybe.12.1687509655921; Fri, 23 Jun 2023 01:40:55 -0700 (PDT) MIME-Version: 1.0 References: <20230623080135.15696-1-fabrizio.castro.jz@renesas.com> In-Reply-To: <20230623080135.15696-1-fabrizio.castro.jz@renesas.com> From: Geert Uytterhoeven Date: Fri, 23 Jun 2023 10:40:43 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] arm64: dts: renesas: rzv2mevk2: Fix eMMC/SDHI pinctrl names To: Fabrizio Castro Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Magnus Damm , linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Chris Paterson , Biju Das , Lad Prabhakar , Linus Walleij , "open list:GPIO SUBSYSTEM" Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT X-Spam-Status: No, score=-1.4 required=5.0 tests=BAYES_00, FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS, LOTS_OF_MONEY,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Fabrizio, CC pinctrl Thanks for your patch! On Fri, Jun 23, 2023 at 10:01 AM Fabrizio Castro wrote: > The original commit uses the same names ("data" and "ctrl") > for the subnodes of pinctrl definitions for both eMMC and > SDHI0 (emmc_pins, sdhi0_pins, and sdhi0_pins_uhs) leading to That should be fine, as the parents of these subnodes do have different names? > the below issue: > > pinctrl-rzv2m b6250000.pinctrl: pin P8_2 already requested by 85000000.mmc; cannot claim for 85020000.mmc > pinctrl-rzv2m b6250000.pinctrl: pin-130 (85020000.mmc) status -22 > renesas_sdhi_internal_dmac 85020000.mmc: Error applying setting, reverse things back To me, that sounds like a bug in the pinctrl core. Or am I missing something? > This commit fixes the problem by making the names for the > pinctrl subnodes of eMMC and SDHI0 unique. > > Fixes: b6c0be722b0c ("arm64: dts: renesas: rzv2mevk2: Add uSD card and eMMC support") > Signed-off-by: Fabrizio Castro > --- > .../arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts > index 39fe3f94991e..11c5cffea5a5 100644 > --- a/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts > +++ b/arch/arm64/boot/dts/renesas/r9a09g011-v2mevk2.dts > @@ -167,7 +167,7 @@ &i2c2 { > > &pinctrl { > emmc_pins: emmc { > - data { > + emmc_data { > pinmux = , /* MMDAT0 */ > , /* MMDAT1 */ > , /* MMDAT2 */ > @@ -179,7 +179,7 @@ data { > power-source = <1800>; > }; > > - ctrl { > + emmc_ctrl { > pinmux = , /* MMCMD */ > ; /* MMCLK */ > power-source = <1800>; > @@ -197,7 +197,7 @@ i2c2_pins: i2c2 { > }; > > sdhi0_pins: sd0 { > - data { > + sd0_data { > pinmux = , /* SD0DAT0 */ > , /* SD0DAT1 */ > , /* SD0DAT2 */ > @@ -205,20 +205,20 @@ data { > power-source = <3300>; > }; > > - ctrl { > + sd0_ctrl { > pinmux = , /* SD0CMD */ > ; /* SD0CLK */ > power-source = <3300>; > }; > > - cd { > + sd0_cd { > pinmux = ; /* SD0CD */ > power-source = <3300>; > }; > }; > > sdhi0_pins_uhs: sd0-uhs { > - data { > + sd0_uhs_data { > pinmux = , /* SD0DAT0 */ > , /* SD0DAT1 */ > , /* SD0DAT2 */ > @@ -226,13 +226,13 @@ data { > power-source = <1800>; > }; > > - ctrl { > + sd0_uhs_ctrl { > pinmux = , /* SD0CMD */ > ; /* SD0CLK */ > power-source = <1800>; > }; > > - cd { > + sd0_uhs_cd { > pinmux = ; /* SD0CD */ > power-source = <1800>; > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds