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 A17BBC433EF for ; Mon, 11 Apr 2022 14:23:23 +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:Message-ID:Date:Subject: In-Reply-To:References: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=BOux1ovqfM15NQH/6E+nWOzIwV5oPCDwDRn3wPjN27Q=; b=HsIk1ENm0YmaVc sOPo2detAxbbfQEzRBOBa6qTljLk27oM1UIjg8R8pnIdOlOEuu4TYEeJ29EHZ8cOUpcR03MsHKOQe 0tUZYj0Qag+ezsPCm8YHUo46GSvZE5Dc6fPTrhTOsL6FLuMvoSMudPnRvIzt3UUDgnNd8rO9Rf9Ib x24U1btL5fs2UO5dO4Lsq3Cvm1WV4MoD4XlHZ5S/ZpMwIpasixJZuf3iyJTLlGvLj/8MboVSENZux 98etbcP0UTtTSR4VhDyUm0W5vt93v6XxtQUNCeJni+ZgH6gmg4wH3CO8LCdzINbv7Gofdhhn6bKko Z3X9vLqvbAOGI6TT+wew==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nduvx-009NPu-3V; Mon, 11 Apr 2022 14:22:25 +0000 Received: from mail-qt1-x833.google.com ([2607:f8b0:4864:20::833]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nduvt-009NO6-QW for linux-arm-kernel@lists.infradead.org; Mon, 11 Apr 2022 14:22:23 +0000 Received: by mail-qt1-x833.google.com with SMTP id t7so16257097qta.10 for ; Mon, 11 Apr 2022 07:22:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=prism19-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:references:in-reply-to:subject:date:message-id :mime-version:content-transfer-encoding:content-language :thread-index; bh=v6tXzO9I8TspPgqH1ah6TZclEmyVKH3/QhCMq11pSMw=; b=Je821PhFC5P3FHTw1UR+bMOphA0/XDjPnNyeyfYj3pwOkuSjEm57rj3VxHLZiJfoLS C0S+4/3dGYUOHO6667EokV9VWEpV2bIh2FdPVh1DbDiQh/HwfsT8ep01Qjbkg/Q5tsU/ +UQgPL20kN0X8wPE3AWj563Pk//ENptZpHuB42T8oa0MDIT3WEviHzeta5frg7z/heI1 wUS2dz2AD05v9t1Tb7Nrd8r74BCXV0nnXNsGgctW0hFcXimyY3r3i2tAb+fVS2mOQ/HS ooJDYQ5TqXilfvBI2daqx5OuPsSbrsEP+OJI4vM6+RujDljRGr2lsXB9J0a0toOwoDHG FYpg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:references:in-reply-to:subject:date :message-id:mime-version:content-transfer-encoding:content-language :thread-index; bh=v6tXzO9I8TspPgqH1ah6TZclEmyVKH3/QhCMq11pSMw=; b=YIzTvYoI7NNvcZtFvq7OI/OPaGw03MsbjoMqggtbLhWgAYEAs+xyLtbFE8Zbq7zyp7 IoEv8oXm9oSn+VLjEN80OeVqdlAuqvpeC3SBbjrSSDBj5wGiGHHeE8BtfYsvOyd6RwOB UETd62ER9+Pjvygs4tnyVmmkS9GPG+hrdZ8wsJF5lvbEpRidPvMTI+m0S5EvTVhxUMO0 AZexcsjyKrhDLXl8GtLxpDMSLIyrZNiZQ0dQv50EUX0V2qiPl+Ir8Hnnvf302U+NefxX KDCEwMfvTrIQ9P/blrGx2xVN03DwinVPpirX/Rkvmzx9NwkFWnUl58t5z6fYa8AUquu9 Q4Yg== X-Gm-Message-State: AOAM532J9g2td3MR7HJjVfniO/Ee/89dFH+6GwsejmBxAc80MLA3CWsq FVgfNRbcT1qL5/vFRxB6qJ+q7Q== X-Google-Smtp-Source: ABdhPJwWNDqdxtfx45QKb+u63H6yXodmATetW98BqkU68PPc3b1MNtwXg+Mr1W0ydz4G7Ra4PFMwpw== X-Received: by 2002:ac8:4f12:0:b0:2eb:de47:e30 with SMTP id b18-20020ac84f12000000b002ebde470e30mr13928661qte.193.1649686939452; Mon, 11 Apr 2022 07:22:19 -0700 (PDT) Received: from RADIUM (bras-base-mtrlpq2718w-grc-02-76-67-205-66.dsl.bell.ca. [76.67.205.66]) by smtp.gmail.com with ESMTPSA id u187-20020a3792c4000000b0067e679cfe5asm19314489qkd.59.2022.04.11.07.22.18 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 11 Apr 2022 07:22:18 -0700 (PDT) From: To: "'Neil Armstrong'" , Cc: "'Rob Herring'" , "'Krzysztof Kozlowski'" , "'Kevin Hilman'" , "'Jerome Brunet'" , "'Martin Blumenstingl'" , , , , References: <20220411140401.450-1-ggiraudon@prism19.com> <8bde8ee2-ee3f-bba6-66da-28adb3897178@baylibre.com> In-Reply-To: <8bde8ee2-ee3f-bba6-66da-28adb3897178@baylibre.com> Subject: RE: [PATCH RESEND] arm64: dts: meson-sm1-bananapi-m5: fix wrong GPIO pin labeling for CON1 Date: Mon, 11 Apr 2022 10:22:17 -0400 Message-ID: <001401d84daf$8d001cc0$a7005640$@prism19.com> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Content-Language: en-ca Thread-Index: AQFrGI7YFugmhoni/8+f7eNP0fB8BAOIHqkvrajs4iA= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220411_072221_889033_1F14ABC0 X-CRM114-Status: GOOD ( 15.44 ) 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 Will do. Thanks Neil. G -----Original Message----- From: Neil Armstrong Sent: April 11, 2022 10:21 AM To: Guillaume Giraudon ; kernel-dev@prism19.com Cc: Rob Herring ; Krzysztof Kozlowski ; Kevin Hilman ; Jerome Brunet ; Martin Blumenstingl ; devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-amlogic@lists.infradead.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND] arm64: dts: meson-sm1-bananapi-m5: fix wrong GPIO pin labeling for CON1 Hi, On 11/04/2022 16:04, Guillaume Giraudon wrote: > The labels for lines 61 through 84 on the periphs-banks were offset by 2. > Realigned them to match the Banana Pi M5 schematics. > > Fixes: 976e920183e4 ("arm64: dts: meson-sm1: add Banana PI BPI-M5 > board dts") > > Signed-off-by: Guillaume Giraudon > --- > arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dts > b/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dts > index 5751c48620ed..754c3d43ef0b 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dts > @@ -448,7 +448,7 @@ > "", > /* GPIOA */ > "", "", "", "", "", "", "", "", > - "", "", "", "", "", "", > + "", "", "", "", "", "", "", "", > "CON1-P27", /* GPIOA_14 */ > "CON1-P28", /* GPIOA_15 */ > /* GPIOX */ While reviewing more closely I figured that in fact the issue comes from the : /* BOOT */ "eMMC_D0", /* BOOT_0 */ "eMMC_D1", /* BOOT_1 */ "eMMC_D2", /* BOOT_2 */ "eMMC_D3", /* BOOT_3 */ "eMMC_D4", /* BOOT_4 */ "eMMC_D5", /* BOOT_5 */ "eMMC_D6", /* BOOT_6 */ "eMMC_D7", /* BOOT_7 */ "eMMC_CLK", /* BOOT_8 */ "", "eMMC_CMD", /* BOOT_10 */ "", "eMMC_RST#", /* BOOT_12 */ "eMMC_DS", /* BOOT_13 */ /* GPIOC */ it contains 14 entries, but there is 16 BOOT gpio lines. So instead, send a V2 with 2 entries added in BOOT and leave the GPIOA entries as-is. Thanks, Neil _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel