lkml.org 
[lkml]   [2023]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v3 9/9] i2c: designware: Add support for AMDI0020 ACPI ID
From
On 3/6/23 06:55, Andy Shevchenko wrote:
> On Mon, Mar 06, 2023 at 02:28:05PM +0200, Jarkko Nikula wrote:
>> On 3/6/23 14:04, Andy Shevchenko wrote:
>>> On Fri, Mar 03, 2023 at 10:50:47AM -0600, Mario Limonciello wrote:
>>>> Cezanne and Skyrim have the same PSP hardware but use a different
>>>> protocol to negotiate I2C arbitration. To disambiguate this going
>>>> forward introduce a new ACPI ID to represent the protocol that utilizes
>>>> a doorbell.
>
> ...
>
>>>> - if (boot_cpu_data.x86 == 25 && boot_cpu_data.x86_model == 80)
>>>
>>> Ah, in this form it's getting better than I thought!
>>>
>> These removed lines were added by previous patch. I think a bit too short
>> lived if the same patchset adds and then removes lines?
>
> That what I have missed. Okay, coming to square 1, i.e. dropping CPU ID
> completely from the series.
>
> Note, for testing purposes you may always add a HACK patch at the end of the
> series, marking it respectively. So, people may test it all and maintainer
> apply w/o unneeded tail.
>

If it still works then new ID can be reserved and patches 8 and 9 could
be squashed together either by subsystem maintainer when merging or for
v4. My apologies if this wasn't obvious to reviewers. My goal was to
separate the scalability and functionality for test purposes.

The way I did it was the series could be tested with patches 1-8 on both
Cezanne and Skyrim platforms and no BIOS changes. If it works, BIOS for
Skyrim can be patched and patch 9 could be added to test kernel.

\
 
 \ /
  Last update: 2023-03-27 00:45    [W:0.099 / U:0.996 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site