From:   "D, Lakshmi Sowjanya" <lakshmi.sowjanya.d@intel.com>
To:     Wolfram Sang <wsa@kernel.org>,
        Jarkko Nikula <jarkko.nikula@linux.intel.com>
CC:     "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Senthil, Bala" <bala.senthil@intel.com>,
        "N, Pandith" <pandith.n@intel.com>,
        "Andy Shevchenko" <andriy.shevchenko@linux.intel.com>,
        "Saha, Tamal" <tamal.saha@intel.com>
Subject: RE: [PATCH v1 1/2] i2c: designware-pci: Add support for Fast Mode
 Plus and High Speed Mode
Thread-Topic: [PATCH v1 1/2] i2c: designware-pci: Add support for Fast Mode
 Plus and High Speed Mode
Thread-Index: AQHX1VWadmPN2CWbpkS9Sg6fkPAU8awa2Y+AgAD5/wCAAW+RsIAAHbMAgAARsgCAABCqsA==
Date:   Wed, 1 Dec 2021 09:39:31 +0000
Message-ID: <DM6PR11MB366003BA82D2A65F60D3AF4BC4689@DM6PR11MB3660.namprd11.prod.outlook.com>
References: <20211109103552.18677-1-lakshmi.sowjanya.d@intel.com>
 <YaUGV3lEmW9qtP+3@kunai>
 <042d2c9f-4333-44b1-09c0-b0953a51c176@linux.intel.com>
 <DM6PR11MB36609A77B3F5D7730FA16A52C4689@DM6PR11MB3660.namprd11.prod.outlook.com>
 <4a420cc0-d18a-32db-ed58-f611336c99ec@linux.intel.com>
 <Yac0JgqrEdUJcfsf@ninjato>
In-Reply-To: <Yac0JgqrEdUJcfsf@ninjato>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-version: 11.6.200.16
dlp-reaction: no-action
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?grmx+julM3egW8WXeS45AB7yaacQHWCAq/JDrvkWfHR4u/ZhxcarCPkdG2FU?=
 =?us-ascii?Q?mX0xORh/BYganjH/On63Q2wydpYyl8RsAG6om4ZEMafn0ydX2b05AUOV7Dee?=
 =?us-ascii?Q?kY0eyWX631XMw7M9SiGeZDSlDOk4GPtXVPZuLZvP5EuZ3uxFUaIebiBkgBaS?=
 =?us-ascii?Q?UY3mFTYKwJ7F1cgN783oIYMUtodXhnKOM8eOK+sj+ByShQrnGxvRpTvYDNIY?=
 =?us-ascii?Q?j2CxH7J3RCLsVONP1t4rYK23EMto/NwjROsxe6oyek21AczVVcKl057PK9Bn?=
 =?us-ascii?Q?g4dO2dRLEsNetKOB84HOtmd55WPWRibB/DrIeVfoZf5lvpJZv6tIO7/1DV+Y?=
 =?us-ascii?Q?4Smz5eEZqSZUUH5KdPrZvMEMZIutfo4f1am4FtWf1Lcs9vNP9iKV+EuJLXFN?=
 =?us-ascii?Q?bDw5DrM7qa6Cimx4LVg5sjxhmwDsCJh/GP5/Munhm4N64tSdNJ3U4egeJvtN?=
 =?us-ascii?Q?d46z/Vlh541lkTuUHqX+Tma/i9/pZgFIXnwlOd5xI//W1c7r7H5dXBqxm7QZ?=
 =?us-ascii?Q?ZpL9HZfasKlZ2mkGhI6cMdMdiAiSGH9tPJ6lEuVYqP2/zTPW/f/PGDNh22Zr?=
 =?us-ascii?Q?UmMFUPeG+pyBaKlFnjuq+TgVx3kaD1N0nkcf9FibHcaQHK+/5WuYbsQ9VSph?=
 =?us-ascii?Q?hOTejPSTXO4unQXUfJODp69tmuN/IdBP+56JFCwAEhFbZAlntJgwnT0h+OXa?=
 =?us-ascii?Q?/1xez/yE/oDu5GvOIbGEZL/dIvulEamXLrhcX2+q9bN4KU9UrjY2d3scqbZU?=
 =?us-ascii?Q?zu6ZJN1wV64C3jiaj52zgvHWQdb4Me1Y0PdpOeHdgDBWNLq1l9+tY3Hzbw1D?=
 =?us-ascii?Q?xl2TGLd6UpqiDiO0IZ7VScRSMo0uxr/mnrrTrOQ72mSN2g1X0Vf4ppIfl0Xb?=
 =?us-ascii?Q?+wwPPPkLPk+Bsnd9foCAXkDPcSYwMUg/fZMRagyHIZzdyq/4qZcVUA1lGYUk?=
 =?us-ascii?Q?1zfkY5O/g9dyF4YNZJOdu+XtQIdscEUKz6LFpIHF6fx9POhzWRfRrVKF5flu?=
 =?us-ascii?Q?cJVj+qcZNQA+kv7iMqgSnKxoU2B0BWNKv14gen2O46Y7F2iAcVqtrgF85ukH?=
 =?us-ascii?Q?QDfXMrmH7gv2nFaAnFEfGs6CGXa4JeWb8V1KDweYQb1prvEJDHHeKTag2v7r?=
 =?us-ascii?Q?69FaT3vXg8LJ1T9HU1cuoJpkw3/AiGfCPyliS/jPwRiCy2A2GgvdxTOff1/L?=
 =?us-ascii?Q?GiXiBu/Q9oMJsv1M9B09+5E9tYu0+CtrAOGx6Blc6JTIzOMzfhKUbmmoBA+1?=
 =?us-ascii?Q?wJkAu0GVwtPbe4XIMBkFpbeQRdKLAh4losTa3wsT+i1Q1Q53uk8N8w0mDCuQ?=
 =?us-ascii?Q?uByHTmwPu5azUgfPb9uD+smY0/j8k8q+WF3zce4O4o8BDaWbj6Lq4hPTIg1k?=
 =?us-ascii?Q?lHaiN7ptY8MS29KGNrQV/2qXqCX4TP4KiVoVaKSjkokbr3gi6/xvodqOG5q2?=
 =?us-ascii?Q?q/ThXOQvHdoaE0UOZhAGj6P7+Rhly1GY2uc0dqNb4X4yLFe5udQ2vRLv4eeL?=
 =?us-ascii?Q?ZBMUFmgGHvok1dcOJXGxLiDz9igOP/hP4yAvKOjlXnlwWMpsDx33TouP1ber?=
 =?us-ascii?Q?J7GYhOzUvzVC6qsMbkNH8d2shFjIA6oPMH9J6QWOnpk7wxFMBpneMIWIGZIf?=
 =?us-ascii?Q?mgx1fYiEYskBoc0umJW9Hel25jFNdIVMXRubZUR5fCQBPSTGPfINJqGSPLVy?=
 =?us-ascii?Q?Q6pqrQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3660.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 336b3f5b-4115-45c1-85fd-08d9b4ae79f8
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Dec 2021 09:39:31.2545
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7EQbCxtph2uZpJrMqmoXhwn7GAvnFAJ6XPBYA06HQ7+kphNh2zfNBkfqzjTcOCec+Au8zXxsVAkk4L9p3EbzFS7ikQnK9ifQo8ctSLozpCU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1849
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


>-----Original Message-----
>From: Wolfram Sang <wsa@kernel.org>
>Sent: Wednesday, December 1, 2021 2:07 PM
>To: Jarkko Nikula <jarkko.nikula@linux.intel.com>
>Cc: D, Lakshmi Sowjanya <lakshmi.sowjanya.d@intel.com>; linux-
>i2c@vger.kernel.org; linux-kernel@vger.kernel.org; Senthil, Bala
><bala.senthil@intel.com>; N, Pandith <pandith.n@intel.com>; Andy
>Shevchenko <andriy.shevchenko@linux.intel.com>; Saha, Tamal
><tamal.saha@intel.com>
>Subject: Re: [PATCH v1 1/2] i2c: designware-pci: Add support for Fast Mode
>Plus and High Speed Mode
>
>
>> See the comments from Andy. Recent enough Elkhart Lake BIOS and his
>> patches from last year already provide the timing parameters. Feeling
>> embarrassed I forgot them :-(
>
>But the u16 conversion of the existing parameters is still useful?

Jarkko: I've seen the changes by Andy. These patches are no more required a=
s suggested. I'm following up on the same, regarding ACPI table entries.

I have the same query...is the u16 conversion of existing parameters still =
useful?

Regards
Lakshmi Sowjanya D