[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <96adf8b2-da6e-491e-a20f-acccd988d585@lunn.ch>
Date: Tue, 3 Feb 2026 20:58:38 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Abdurrahman Hussain <abdurrahman@...thop.ai>
Cc: Andy Shevchenko <andriy.shevchenko@...el.com>,
Michal Simek <michal.simek@....com>,
Andi Shyti <andi.shyti@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
linux-arm-kernel@...ts.infradead.org, linux-i2c@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v9 6/7] i2c: xiic: use numbered adapter registration
> Before, the driver was always doing the dynamic allocation due to
> i2c_add_adapter(). So there could not have been a system that relied on
> consistent i2c bus numbering. Even between reboots bus numbering could change.
>
> The systems that did set pdev->id and were expecting a consistent i2c bus
> numbering were broken to begin with.
>
> This patch won’t break any existing systems or cause regressions.
Great, now please add that to the commit message. These are the sort
of things reviewers want to know, is a change going to cause
regressions. If you have done your homework, mentioning it in the
commit message will help avoid the questions being asked.
Andrew
Powered by blists - more mailing lists