Message ID | 20230414181434.2046049-3-u-kumar1@ti.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp578342vqo; Fri, 14 Apr 2023 11:24:28 -0700 (PDT) X-Google-Smtp-Source: AKy350Y0KB9aC1MWxbTZAvFg0Li89hjm5MCsV0NzTZv2Y9fNgNT5Y0BVM7X42s98hOdYgNwDZlZx X-Received: by 2002:a17:90a:883:b0:23b:2ce5:2ddb with SMTP id v3-20020a17090a088300b0023b2ce52ddbmr7368148pjc.8.1681496668134; Fri, 14 Apr 2023 11:24:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1681496668; cv=none; d=google.com; s=arc-20160816; b=STVcEeZcGwrTMVn1DIwr6ojqrhexutUqudDebP+o4eiRdNJVtMNJ+z2vHl0jX3JJpd jLqlLQDMDvs2oiDocXUu/eaSUtZEcaXOCo8toX41PeWQU/A7vtpnfY9OtuE8UzTDtRcY zLSGOo3CkoO0N6Hwpg9uztjoDThkmSVYB4lEezI45yj1vDrXPOjY6Jy3AjmAO19tvM0D s8rCjvi9zMK9i9fCxfW1okhAvo6XsnuoF364VZyd9FarDb0CmCiqLdFN537xXApRCuaG 1CO/4YMTSJnOPKSzDhczQwa6X7ohEuJ9+OTBxd0yhny4mBybwvaXyf7Gk4fSXQJN9OxL P4QQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=opTwY9akY/OuTWmpN9fS9vNfhXu6dCH+OuqiAWaN7cw=; b=kK78u7aePM1th6udUxtfPhMMi+qbSScwZmwFPHAZedo+KI/DC/oVBgI87dYKYq2tdf d0HlFJfRlRxnMiQU8YeQOv3oAL76xB32mJZzCYnFOgZrVk/dHKjnKYDCwXlTFeQTeSph gdTPm0hhXA19bjtsW0wCsLbz7gAgN8oXZda/LACrjawDm6duecwJXx5W+MO10YKrvSgx xpLYq55uUd7FWP7bWizbZIgPG7lS3KA8WBp0SZjE9vQrUZWEep/6p6E6t12MHzfl59EG B/CEUj+3OrT+8zp5NrlTo1wrxIsuoJzm662vL1NlmvF6q5Z/fT/nrKpDKjbIXbf7b8IQ qdDg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=i281RPcq; 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; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id q11-20020a170902b10b00b001a288c5d34bsi4984966plr.85.2023.04.14.11.24.14; Fri, 14 Apr 2023 11:24:28 -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; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=i281RPcq; 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; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230033AbjDNSPA (ORCPT <rfc822;leviz.kernel.dev@gmail.com> + 99 others); Fri, 14 Apr 2023 14:15:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51348 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229992AbjDNSO4 (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 14 Apr 2023 14:14:56 -0400 Received: from fllv0015.ext.ti.com (fllv0015.ext.ti.com [198.47.19.141]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D0DDAA26E; Fri, 14 Apr 2023 11:14:55 -0700 (PDT) Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 33EIEnGU118269; Fri, 14 Apr 2023 13:14:49 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1681496089; bh=opTwY9akY/OuTWmpN9fS9vNfhXu6dCH+OuqiAWaN7cw=; h=From:To:CC:Subject:Date:In-Reply-To:References; b=i281RPcqabaMEw4Saxt+dmxdUO4u8ms+rb/7ec9eh1BaXKIrsYea0iZpvS0dkSIiL xRhPyC/wRNocDu9G0ficUczyGianzXb3sKyVFtiyJ7VTDqutOVT4/mHBgmOsbKYfhC y05ZRsTAGjM3Wt1jzE9c29PbNUYebELji6q7VMwo= Received: from DFLE112.ent.ti.com (dfle112.ent.ti.com [10.64.6.33]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 33EIEnvX055189 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 14 Apr 2023 13:14:49 -0500 Received: from DFLE104.ent.ti.com (10.64.6.25) by DFLE112.ent.ti.com (10.64.6.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16; Fri, 14 Apr 2023 13:14:49 -0500 Received: from lelv0326.itg.ti.com (10.180.67.84) by DFLE104.ent.ti.com (10.64.6.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.16 via Frontend Transport; Fri, 14 Apr 2023 13:14:49 -0500 Received: from udit-HP-Z2-Tower-G9-Workstation-Desktop-PC.dhcp.ti.com (ileaxei01-snat.itg.ti.com [10.180.69.5]) by lelv0326.itg.ti.com (8.15.2/8.15.2) with ESMTP id 33EIEaEx032189; Fri, 14 Apr 2023 13:14:45 -0500 From: Udit Kumar <u-kumar1@ti.com> To: <linux-arm-kernel@lists.infradead.org>, <afd@ti.com>, <bb@ti.com>, <vaishnav.a@ti.com>, <j-choudhary@ti.com>, <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <nm@ti.com>, <vigneshr@ti.com>, <kristo@kernel.org>, <robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>, <a-govindraju@ti.com>, <kishon@ti.com> CC: Udit Kumar <u-kumar1@ti.com> Subject: [PATCH 2/2] arm64: dts: ti: k3-j7200-common-proc-board: Enable I2C Date: Fri, 14 Apr 2023 23:44:34 +0530 Message-ID: <20230414181434.2046049-3-u-kumar1@ti.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230414181434.2046049-1-u-kumar1@ti.com> References: <20230414181434.2046049-1-u-kumar1@ti.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1763177050157070635?= X-GMAIL-MSGID: =?utf-8?q?1763177050157070635?= |
Series |
arm64: dts: ti: k3-j7200-common-proc-board fixes
|
|
Commit Message
Kumar, Udit
April 14, 2023, 6:14 p.m. UTC
All I2C nodes were disabled in SOC dts file and in board
dts needed nodes were enabled.
This patch enables wkup_i2c0 node in board dts file.
Fixes: a9ed915cd55c ("arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level")
Signed-off-by: Udit Kumar <u-kumar1@ti.com>
---
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 4 ++++
1 file changed, 4 insertions(+)
Comments
On 23:44-20230414, Udit Kumar wrote: > All I2C nodes were disabled in SOC dts file and in board > dts needed nodes were enabled. redundant. > This patch enables wkup_i2c0 node in board dts file. > > Fixes: a9ed915cd55c ("arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level") This should'nt be fixing the commit - the commit is correct in the sense that if the board does'nt describe a node with pinmux, the i2c is'nt functional in the first place. > > Signed-off-by: Udit Kumar <u-kumar1@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > index 63633e4f6c59..e66bbfe65ff2 100644 > --- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > @@ -254,6 +254,10 @@ exp3: gpio@20 { > }; > }; > > +&wkup_i2c0 { > + status = "okay"; How about the pinmux? speed? what peripheral is on the wakeup i2c? Do we have an eeprom or something here? > +}; > + > &main_sdhci0 { > /* eMMC */ > non-removable; > -- > 2.34.1 >
Hi Nishanth On 4/14/2023 11:54 PM, Nishanth Menon wrote: > On 23:44-20230414, Udit Kumar wrote: >> All I2C nodes were disabled in SOC dts file and in board >> dts needed nodes were enabled. > redundant. ok > >> This patch enables wkup_i2c0 node in board dts file. >> >> Fixes: a9ed915cd55c ("arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level") > > This should'nt be fixing the commit - the commit is correct in the sense > that if the board does'nt describe a node with pinmux, the i2c is'nt > functional in the first place. well, how i look at before this patch each i2c was functional. this fix is making needed i2c as disabled, however this fix is correct I am ok to remove Fixes >> Signed-off-by: Udit Kumar <u-kumar1@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts >> index 63633e4f6c59..e66bbfe65ff2 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts >> +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts >> @@ -254,6 +254,10 @@ exp3: gpio@20 { >> }; >> }; >> >> +&wkup_i2c0 { >> + status = "okay"; > How about the pinmux? speed? what peripheral is on the wakeup i2c? > > Do we have an eeprom or something here? This got eeprom and PMIC, eeprom read is ok with pin mux and speed. if you want to specify, let me know >> +}; >> + >> &main_sdhci0 { >> /* eMMC */ >> non-removable; >> -- >> 2.34.1 >>
On 09:36-20230415, Kumar, Udit wrote: [...] > > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > > > index 63633e4f6c59..e66bbfe65ff2 100644 > > > --- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > > > +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > > > @@ -254,6 +254,10 @@ exp3: gpio@20 { > > > }; > > > }; > > > +&wkup_i2c0 { > > > + status = "okay"; > > How about the pinmux? speed? what peripheral is on the wakeup i2c? > > > > Do we have an eeprom or something here? > > > This got eeprom and PMIC, > > eeprom read is ok with pin mux and speed. > > if you want to specify, let me know yes, you must specify eeprom at least. I understand pmic is in discussion atm [1], So, that can be resolved seperately. Also if the eeprom is on the SoM, you should add this in the SoM dtsi, not the baseboard dtsi. [1] https://lore.kernel.org/all/714a56b1-438c-7aad-920b-f3f1d72afc5f@linaro.org/
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts index 63633e4f6c59..e66bbfe65ff2 100644 --- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts @@ -254,6 +254,10 @@ exp3: gpio@20 { }; }; +&wkup_i2c0 { + status = "okay"; +}; + &main_sdhci0 { /* eMMC */ non-removable;