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;
From:   Asmaa Mnebhi <asmaa@nvidia.com>
To:     Wolfram Sang <wsa@kernel.org>
CC:     "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Khalil Blaiech <kblaiech@nvidia.com>
Subject: RE: [PATCH v1 4/7] i2c-mlxbf.c: incorrect base address passed during
 io write
Thread-Topic: [PATCH v1 4/7] i2c-mlxbf.c: incorrect base address passed during
 io write
Thread-Index: AQHYscMo59o17RgFDU6O7GkYAYNjea2020kAgAADfQCAAAo+MIAANHyAgAABldA=
Date:   Thu, 18 Aug 2022 20:30:13 +0000
Message-ID: <CH2PR12MB38958540981FE6BC85E96178D76D9@CH2PR12MB3895.namprd12.prod.outlook.com>
References: <20220816225412.9095-1-asmaa@nvidia.com>
 <20220816225412.9095-5-asmaa@nvidia.com>
 <20220818162728.lnq5y7qzxbdsyexr@the-dreams.de>
 <CH2PR12MB38951132E24E6F1EDCCD18C6D76D9@CH2PR12MB3895.namprd12.prod.outlook.com>
 <CH2PR12MB3895EBE43CB5B23F4E4ECA65D76D9@CH2PR12MB3895.namprd12.prod.outlook.com>
 <Yv6f+zR9qk5vLOmI@shikoro>
In-Reply-To: <Yv6f+zR9qk5vLOmI@shikoro>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?brVfEMrKZjjwEPqR/VPvE4FGkd8Z32fWhQc/Xuz7v9ox9+LW2t+lcsJLxd3S?=
 =?us-ascii?Q?r6fcgnvb2RrrxamEpZQ4qwk6Ce7p+ZE1qwne5Z+aF0iGZg8f+hGYsPn0zBAa?=
 =?us-ascii?Q?aKK235GsLq9bma237GSPzCwKU0+borvNgXMhUVvnXyaw95l+lJOUspqTff6N?=
 =?us-ascii?Q?0b4OnvearR8J2sItS39+yGcI5I5sN5E4pcEJw19p88dr7XkaV1txRrwBHbyb?=
 =?us-ascii?Q?5rNPN6EyB0MfDZJWc8TfeZ+dzGpo4KK8M11mAdPFojGJs1Sv8r5XqUIDyKsL?=
 =?us-ascii?Q?x0lhp50Vwx3vyapM4D+JlH2bOpQGp76c1ikmYG9sVbUTYCHlHHwZoNsds8vC?=
 =?us-ascii?Q?d4o+frjgMArcHEPG6Tdd8V7UH4PtaXYU1H8ygJSEyOQEC1SxUbdPOgn4e+cQ?=
 =?us-ascii?Q?7PlAjmS5ANkhAdn9sxPFnhrSKiqdPBm2m4grhz+o4WJtqC8JMl4FhOfD3h4G?=
 =?us-ascii?Q?Rs9fR55Se8WLWJBMH58G46qq9MjF55Ysrr3yBQHRqwjrBpuWAZZiLdhjvpTL?=
 =?us-ascii?Q?DLruGJKXr6wHYmchs7XzufiI2aad03S2sG6tO+YgV/okmg6umjZIYczj9o/f?=
 =?us-ascii?Q?9NMmzP7Is4MezeLnbHXXQwYX0RrpYpIZWMtOX4JppRzaEh/JRTD86XQhbt+3?=
 =?us-ascii?Q?wnyaMweNgVUb6KyUeg7oz5jizuDda8dba1+fiSgSZ7fbxnlErqaUCMVw710V?=
 =?us-ascii?Q?e9qWo6akgqpMEtTyj3YEgPd7KUQjvG2eSq8RBUuAfzh6FnLO8JNOoLf3gNsO?=
 =?us-ascii?Q?eIcxjkQVg+BSRfigG207FmgoNdQ0xeIUt7f62bupphQdtUukDjkBLK7r4emO?=
 =?us-ascii?Q?M78h++ko9L1VP3zfG5WGBkd4SalK4DMzeQ5R4EJvp0HMzHz0pfhN+RuM5c6W?=
 =?us-ascii?Q?/3Wz+e40vMvLr1q2hGkJ6jbfbqXOMSbKQMS3wPq30BespaUrWgCtDYoOxUn+?=
 =?us-ascii?Q?VdvqkdXNXoGMsskPm7bBLneubyTKNoI9N+v/B/dnZvsvzIOgkdXAn6s6qxGj?=
 =?us-ascii?Q?9zNsrF7wSU0VUzJ+vdT6RObczuw5cyGsLPmG11YCS8HRnTIFARWdiHwIdP9A?=
 =?us-ascii?Q?Jj/xM1woD7MdvxODs9ApqvoSiZx/QATf7T/8wEwwsKtW04ciD99juuce+pmR?=
 =?us-ascii?Q?OUTT2vC4QwgM0/mLrKh0OVBaa7kKITSOasuONEpBZTmIbT0gN5LYochHf4n5?=
 =?us-ascii?Q?HuQpcK7g+woKut18F7f9+l/IDPHFzv7iOseBKUFG7dQ68Q3ifzEuU9VQTc2V?=
 =?us-ascii?Q?QC5le3gvnZgGzUnZn+9L9jHn3ctE2rOOjGMbfUMjbWJGWe5cb968W36hub/e?=
 =?us-ascii?Q?bliOTjlFFWDx1r/yfRJoBZrSp3OB2hbnaaN/C/BNWtnmEhN75sCBHN2Kv/db?=
 =?us-ascii?Q?Ue51mAvdiaqX7fWD/r2y4bU99n+D46unmO/+8Hw6CsjDnGLm+TkQz954AmXY?=
 =?us-ascii?Q?Nkhqq4cBXe5Z5h9dXLg63Dq/y1mcUlN3xW4l9grUuUgwrg0G1kzCLL0hHoKz?=
 =?us-ascii?Q?jxmvfDNCT0eo+d0YagPjHdYd/aoularoSclhRc9mfWQkvTxLCU3i0QyDk0bY?=
 =?us-ascii?Q?sHk5cHMPbwHizw2NbHE=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB3895.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 15c5c105-cdd8-4f04-b2a5-08da81587480
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Aug 2022 20:30:13.7418
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: BYlpkcrixOq4tODU3kFZOL+jEgjkwVeMWLS8EcGUo91RvaNSBXcsIMyYW+AH+fe/cKIyPVnoGk0fazMi8d0MAA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR12MB1485
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Sure!

-----Original Message-----
From: Wolfram Sang <wsa@kernel.org>=20
Sent: Thursday, August 18, 2022 4:24 PM
To: Asmaa Mnebhi <asmaa@nvidia.com>
Cc: linux-i2c@vger.kernel.org; linux-kernel@vger.kernel.org; Khalil Blaiech=
 <kblaiech@nvidia.com>
Subject: Re: [PATCH v1 4/7] i2c-mlxbf.c: incorrect base address passed duri=
ng io write


Thanks for the heads up.

> didn't have any impact on the overall functionality of the read/write tra=
nsactions. Writing to smbus->io + 0x18 (aka  MLXBF_I2C_CAUSE_OR_CLEAR) corr=
esponds to sc_low_timeout register which sets the timeout value before a re=
ad/write transaction aborts.=20

I understand. Could we have the above sentences (or something similar) in t=
he commit message then?