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 X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8092AC47080 for ; Mon, 24 May 2021 01:53:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5440B6109E for ; Mon, 24 May 2021 01:53:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232172AbhEXBzV (ORCPT ); Sun, 23 May 2021 21:55:21 -0400 Received: from twspam01.aspeedtech.com ([211.20.114.71]:13328 "EHLO twspam01.aspeedtech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231765AbhEXBzU (ORCPT ); Sun, 23 May 2021 21:55:20 -0400 Received: from mail.aspeedtech.com ([192.168.0.24]) by twspam01.aspeedtech.com with ESMTP id 14O1eLhp073824; Mon, 24 May 2021 09:40:21 +0800 (GMT-8) (envelope-from jamin_lin@aspeedtech.com) Received: from aspeedtech.com (192.168.100.253) by TWMBX02.aspeed.com (192.168.0.24) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 24 May 2021 09:53:14 +0800 Date: Mon, 24 May 2021 09:53:11 +0800 From: Jamin Lin To: Tao Ren CC: Joel Stanley , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/ASPEED MACHINE SUPPORT" , "moderated list:ARM/ASPEED I2C DRIVER" , Brendan Higgins , open list , "Rob Herring" , Rayn Chen , "open list:I2C SUBSYSTEM HOST DRIVERS" , Steven Lee , "moderated list:ARM/ASPEED MACHINE SUPPORT" Subject: Re: [PATCH 1/3] i2c: aspeed: avoid new registers definition of AST2600 Message-ID: <20210524015310.GA2591@aspeedtech.com> References: <20210519080436.18975-1-jamin_lin@aspeedtech.com> <20210519080436.18975-2-jamin_lin@aspeedtech.com> <20210520033140.GA3656@aspeedtech.com> <20210521020033.GB19153@taoren-ubuntu-R90MNF91> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <20210521020033.GB19153@taoren-ubuntu-R90MNF91> User-Agent: Mutt/1.9.4 (2018-02-28) X-Originating-IP: [192.168.100.253] X-ClientProxiedBy: TWMBX02.aspeed.com (192.168.0.24) To TWMBX02.aspeed.com (192.168.0.24) X-DNSRBL: X-MAIL: twspam01.aspeedtech.com 14O1eLhp073824 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The 05/21/2021 02:00, Tao Ren wrote: > Hi Jamin, > > On Thu, May 20, 2021 at 11:31:41AM +0800, Jamin Lin wrote: > > The 05/19/2021 22:59, Joel Stanley wrote: > > > On Wed, 19 May 2021 at 08:05, Jamin Lin wrote: > > > > > > > > The register definition between AST2600 A2 and A3 is different. > > > > This patch avoid new registers definition of AST2600 to use > > > > this driver. We will submit the path for the new registers > > > > definition of AST2600. > > > > > > The AST2600 v9 datasheet says that bit 2 selects between old and new > > > register sets, and that the old register set is the default. > > > > > > Has the default changed for the A3?, and the datasheet is incorrect? > > > > > > Does the A3 still support the old register set? > > > > > We suggest user to use the new i2c driver for AST2600 and we will sumbit > > it. This driver is used to AST2500 and AST2400 SOCs. Change this > > driver to check global register of i2c to avoid user build the wrong driver. > > If I understand correctly, the answer implies old register set is still > supported in A3 although aspeed suggest people using the new driver/mode? > > Can you please share more context behind the suggestion? Such as new > register mode has better performance? Or some known issues that were > deteted in old mode are fixed in new register mode? > Yes, AST2600 A1, A2 and A3 support both old and new register set. The difference between old and new register set are the register address and supported registers. User can choose to use both old and new register set. However, ASPEED would like to change new register set by default for AST2600. Thanks-Jamin > > Cheers, > > Tao 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 X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4CDF7C47080 for ; Mon, 24 May 2021 01:54:15 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9E8E96109E for ; Mon, 24 May 2021 01:54:14 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9E8E96109E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=aspeedtech.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=openbmc-bounces+openbmc=archiver.kernel.org@lists.ozlabs.org Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4FpKyF2Gfkz2ymN for ; Mon, 24 May 2021 11:54:13 +1000 (AEST) Authentication-Results: lists.ozlabs.org; spf=fail (SPF fail - not authorized) smtp.mailfrom=aspeedtech.com (client-ip=211.20.114.71; helo=twspam01.aspeedtech.com; envelope-from=jamin_lin@aspeedtech.com; receiver=) Received: from twspam01.aspeedtech.com (twspam01.aspeedtech.com [211.20.114.71]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4FpKxq4QjVz2yXl; Mon, 24 May 2021 11:53:48 +1000 (AEST) Received: from mail.aspeedtech.com ([192.168.0.24]) by twspam01.aspeedtech.com with ESMTP id 14O1eLhp073824; Mon, 24 May 2021 09:40:21 +0800 (GMT-8) (envelope-from jamin_lin@aspeedtech.com) Received: from aspeedtech.com (192.168.100.253) by TWMBX02.aspeed.com (192.168.0.24) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 24 May 2021 09:53:14 +0800 Date: Mon, 24 May 2021 09:53:11 +0800 From: Jamin Lin To: Tao Ren Subject: Re: [PATCH 1/3] i2c: aspeed: avoid new registers definition of AST2600 Message-ID: <20210524015310.GA2591@aspeedtech.com> References: <20210519080436.18975-1-jamin_lin@aspeedtech.com> <20210519080436.18975-2-jamin_lin@aspeedtech.com> <20210520033140.GA3656@aspeedtech.com> <20210521020033.GB19153@taoren-ubuntu-R90MNF91> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <20210521020033.GB19153@taoren-ubuntu-R90MNF91> User-Agent: Mutt/1.9.4 (2018-02-28) X-Originating-IP: [192.168.100.253] X-ClientProxiedBy: TWMBX02.aspeed.com (192.168.0.24) To TWMBX02.aspeed.com (192.168.0.24) X-DNSRBL: X-MAIL: twspam01.aspeedtech.com 14O1eLhp073824 X-BeenThere: openbmc@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development list for OpenBMC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/ASPEED MACHINE SUPPORT" , Steven Lee , "moderated list:ARM/ASPEED I2C DRIVER" , Brendan Higgins , open list , Rob Herring , Rayn Chen , "moderated list:ARM/ASPEED MACHINE SUPPORT" , "open list:I2C SUBSYSTEM HOST DRIVERS" Errors-To: openbmc-bounces+openbmc=archiver.kernel.org@lists.ozlabs.org Sender: "openbmc" The 05/21/2021 02:00, Tao Ren wrote: > Hi Jamin, > > On Thu, May 20, 2021 at 11:31:41AM +0800, Jamin Lin wrote: > > The 05/19/2021 22:59, Joel Stanley wrote: > > > On Wed, 19 May 2021 at 08:05, Jamin Lin wrote: > > > > > > > > The register definition between AST2600 A2 and A3 is different. > > > > This patch avoid new registers definition of AST2600 to use > > > > this driver. We will submit the path for the new registers > > > > definition of AST2600. > > > > > > The AST2600 v9 datasheet says that bit 2 selects between old and new > > > register sets, and that the old register set is the default. > > > > > > Has the default changed for the A3?, and the datasheet is incorrect? > > > > > > Does the A3 still support the old register set? > > > > > We suggest user to use the new i2c driver for AST2600 and we will sumbit > > it. This driver is used to AST2500 and AST2400 SOCs. Change this > > driver to check global register of i2c to avoid user build the wrong driver. > > If I understand correctly, the answer implies old register set is still > supported in A3 although aspeed suggest people using the new driver/mode? > > Can you please share more context behind the suggestion? Such as new > register mode has better performance? Or some known issues that were > deteted in old mode are fixed in new register mode? > Yes, AST2600 A1, A2 and A3 support both old and new register set. The difference between old and new register set are the register address and supported registers. User can choose to use both old and new register set. However, ASPEED would like to change new register set by default for AST2600. Thanks-Jamin > > Cheers, > > Tao 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 X-Spam-Level: X-Spam-Status: No, score=-5.6 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 368A4C04FF3 for ; Mon, 24 May 2021 19:45:49 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id F19806109F for ; Mon, 24 May 2021 19:45:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org F19806109F Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=aspeedtech.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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:In-Reply-To:MIME-Version:References: 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=zJKBByvPboeq/tMW6lNYaHPGVxw7sT1PS6FNIhjERxM=; b=inxVE0oEXJPUvL j8j4YhK9goDkveqUJ+tr/gyt5LcNcdc2SghxK2KiVl+OaGq0xFIZYBSMpi99hRIUAYuBSP1oeQs1C 9HCiyABZKvDvl4i0ZYZG08oHctHJHTm24u6ZR+QfiH93IiDNafnffZ6ZEDddMfepGgdG8Qt1AkABE pdDC8kHr1enwna2O4E24kED4BQqv5e/DKeCeXbwTfWIxQNzxl3HQLxrbDJ1VhTmFLGTwdzkKK+ZKs H9funIEu33BxN8hNvON1W3B21VDOUWKfHBHi5KRFuJQwl4a4R4HaaJuLl6XpbMhYdviyeL7FYAxga FQYMJnLwzZTGjGBrfFyA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1llGTc-001imv-6l; Mon, 24 May 2021 19:43:01 +0000 Received: from twspam01.aspeedtech.com ([211.20.114.71]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lkzmm-000e39-39 for linux-arm-kernel@lists.infradead.org; Mon, 24 May 2021 01:53:42 +0000 Received: from mail.aspeedtech.com ([192.168.0.24]) by twspam01.aspeedtech.com with ESMTP id 14O1eLhp073824; Mon, 24 May 2021 09:40:21 +0800 (GMT-8) (envelope-from jamin_lin@aspeedtech.com) Received: from aspeedtech.com (192.168.100.253) by TWMBX02.aspeed.com (192.168.0.24) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 24 May 2021 09:53:14 +0800 Date: Mon, 24 May 2021 09:53:11 +0800 From: Jamin Lin To: Tao Ren CC: Joel Stanley , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/ASPEED MACHINE SUPPORT" , "moderated list:ARM/ASPEED I2C DRIVER" , Brendan Higgins , open list , "Rob Herring" , Rayn Chen , "open list:I2C SUBSYSTEM HOST DRIVERS" , Steven Lee , "moderated list:ARM/ASPEED MACHINE SUPPORT" Subject: Re: [PATCH 1/3] i2c: aspeed: avoid new registers definition of AST2600 Message-ID: <20210524015310.GA2591@aspeedtech.com> References: <20210519080436.18975-1-jamin_lin@aspeedtech.com> <20210519080436.18975-2-jamin_lin@aspeedtech.com> <20210520033140.GA3656@aspeedtech.com> <20210521020033.GB19153@taoren-ubuntu-R90MNF91> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210521020033.GB19153@taoren-ubuntu-R90MNF91> User-Agent: Mutt/1.9.4 (2018-02-28) X-Originating-IP: [192.168.100.253] X-ClientProxiedBy: TWMBX02.aspeed.com (192.168.0.24) To TWMBX02.aspeed.com (192.168.0.24) X-DNSRBL: X-MAIL: twspam01.aspeedtech.com 14O1eLhp073824 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210523_185340_433512_94130526 X-CRM114-Status: GOOD ( 24.25 ) 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 The 05/21/2021 02:00, Tao Ren wrote: > Hi Jamin, > > On Thu, May 20, 2021 at 11:31:41AM +0800, Jamin Lin wrote: > > The 05/19/2021 22:59, Joel Stanley wrote: > > > On Wed, 19 May 2021 at 08:05, Jamin Lin wrote: > > > > > > > > The register definition between AST2600 A2 and A3 is different. > > > > This patch avoid new registers definition of AST2600 to use > > > > this driver. We will submit the path for the new registers > > > > definition of AST2600. > > > > > > The AST2600 v9 datasheet says that bit 2 selects between old and new > > > register sets, and that the old register set is the default. > > > > > > Has the default changed for the A3?, and the datasheet is incorrect? > > > > > > Does the A3 still support the old register set? > > > > > We suggest user to use the new i2c driver for AST2600 and we will sumbit > > it. This driver is used to AST2500 and AST2400 SOCs. Change this > > driver to check global register of i2c to avoid user build the wrong driver. > > If I understand correctly, the answer implies old register set is still > supported in A3 although aspeed suggest people using the new driver/mode? > > Can you please share more context behind the suggestion? Such as new > register mode has better performance? Or some known issues that were > deteted in old mode are fixed in new register mode? > Yes, AST2600 A1, A2 and A3 support both old and new register set. The difference between old and new register set are the register address and supported registers. User can choose to use both old and new register set. However, ASPEED would like to change new register set by default for AST2600. Thanks-Jamin > > Cheers, > > Tao _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel