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:   Michael Brunner <michael.brunner@kontron.com>
To:     "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>,
        "andi.shyti@kernel.org" <andi.shyti@kernel.org>
CC:     "wsa@kernel.org" <wsa@kernel.org>,
        "rafael@kernel.org" <rafael@kernel.org>,
        "acpica-devel@lists.linuxfoundation.org" 
        <acpica-devel@lists.linuxfoundation.org>,
        "lenb@kernel.org" <lenb@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "rafael.j.wysocki@intel.com" <rafael.j.wysocki@intel.com>,
        "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
        "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
        "robert.moore@intel.com" <robert.moore@intel.com>
Subject: Re: [PATCH v1 1/2] ACPI: platform: Ignore SMB0001 only when it has
 resources
Thread-Topic: [PATCH v1 1/2] ACPI: platform: Ignore SMB0001 only when it has
 resources
Thread-Index: AQHZo5VI7IrE3xvC7kWrbR8c4raVtK+U4d6AgAAdIACAAD/9gIAAAneA
Date:   Wed, 21 Jun 2023 13:28:48 +0000
Message-ID: <fcee65cded553de501a4b22380e49c1914f63799.camel@kontron.com>
References: <20230620163534.1042-1-andriy.shevchenko@linux.intel.com>
         <b9af5a068cc0b5e785c8e2ddfc70d811fd5929cd.camel@kontron.com>
         <20230621093056.5qfa3kn7ldgwsyeg@intel.intel>
         <ZJL4/X1Z9gCdESxp@smile.fi.intel.com>
In-Reply-To: <ZJL4/X1Z9gCdESxp@smile.fi.intel.com>
Accept-Language: de-DE, en-US
Content-Language: en-US
user-agent: Evolution 3.36.5-0ubuntu1 
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-15?Q?9ltTrb6GMAAMvBB0ofYKPGn7uLug6acFVzQPO/u0o6Mke+W/36qKlHPym?=
 =?iso-8859-15?Q?bOAqENpswPxFiWNNW7N+fsNsx8y8aA6F2Tw6RCo9wgiUQqAo1gL0j6Umm?=
 =?iso-8859-15?Q?fstyMcKj9Lno9o52M4JmkQ/MGUy13vxAFM6mmvPjDXwnhy5VKC5Xdp1lA?=
 =?iso-8859-15?Q?fLrIelr7HP61B27lfbr4uJcHLMlik9/N9r2piUJlphkIpqrOLeM8AHPog?=
 =?iso-8859-15?Q?PJowUZdU68d2Td+In5GGCMBlX6za20CeaOwzVxde0PxPlWw/Knk8Xz8+N?=
 =?iso-8859-15?Q?/eHXBDL+vnFPY/Bk/tZRLFUGwAJtooDQuE1uV7ecuDERGFP667lUx/g6L?=
 =?iso-8859-15?Q?mJqvXIUaii1YgUNZc7BUdI1/CHIiwkniSK0QfwPj9iWzmqRfHQABBV6+s?=
 =?iso-8859-15?Q?saXHPAf0vGH6KEndr8JJGugm35mz9f2iMHSORFyGDEDudaVwLhD3JfMWu?=
 =?iso-8859-15?Q?KfoMG7GRnsHIwtizWYZW4lFvaMfb8q6wVbacFNfjHlmTC6daN2KGBpSGx?=
 =?iso-8859-15?Q?6RHp1mAYF8pY7Y7E/bWy+rgREs0r4dvyHQ/sc9OfTBjcCeDIuETOW4p+t?=
 =?iso-8859-15?Q?7WF9uinGfaXghg8lt2YT50r6hQI9nZ6GeEifOk9t/HUCfmPbwWgJgCaZO?=
 =?iso-8859-15?Q?HnU3twf0tS0xYO1n2S7+T4pi10M8VLYECorOxdz+OoE4126UDoJlINIPa?=
 =?iso-8859-15?Q?jINMDbmdr/8kP9IA1W3Gd4QkyxfUS4pq4yjaicyxgq/epBQF+p8ti272L?=
 =?iso-8859-15?Q?oxtPDCKDFZZ4GOOwJzLJnupezsJ/XCwthzcEbJakHHzub6rlOdVFFSjmI?=
 =?iso-8859-15?Q?QM8MYod3Jtk5IpF7FTA5ZfLGh2324yY1JKsos+9XvaMfb/HWMW9H+otpg?=
 =?iso-8859-15?Q?q5jnhpKIWF4qc4xjfvkKw/idkdo4dhz+cbs0gQ64qtDyXI35Sy4NNWNkS?=
 =?iso-8859-15?Q?9GdqFQOt/gaZrDWVgCSyYDcR92TnDI/PLDrJmDD+3dqY2xjAnnJyq62o5?=
 =?iso-8859-15?Q?WSF538t8KDbwZ6QITbyHiQWb7Cc4aIlMyW3LWGgluHMr63OPIoI0IAKU5?=
 =?iso-8859-15?Q?qJyj50rjCEmAHmW11XYwtbxSs+dHfpGMqqjiNHDw8siYS4oZVycJMz2Oy?=
 =?iso-8859-15?Q?R1TEwIfTRIhH6RUEvMXoH2T+B/wGzJW0UtP9OumvzRlsBvshLIpM91Cry?=
 =?iso-8859-15?Q?f+n3Bvr7+b4pBGCAC7cwmHrvIRTrdJYhrHj40nXDrE4L+9GjLE0UQZrzS?=
 =?iso-8859-15?Q?b2/QbP6eJqUJE1ZK3Gwy6Scgj1ccYMwNxldbxlTVI13iIfFpiyw3XtTqH?=
 =?iso-8859-15?Q?dAfY2tGIaNyAMc6R3J13Gmz76QZ96ssw6R0ruDONY2B1ela/5Uzxl2rI1?=
 =?iso-8859-15?Q?wqyjvbueneJlIy/23Z59wtbQWTqb4W4U4EDOWZp0NN/tR1+MnMWd+vgsa?=
 =?iso-8859-15?Q?6j1f/l8TEc39EUHFOKGZC+zoeX8b4NSLgaugTtcpyC7GMBzX/tUc4Qmwn?=
 =?iso-8859-15?Q?2HFpa5DcawDntpqEoypNKd1BHbbOD0isZ2JGWNhLY+5ikf/IG9KJjeZ/M?=
 =?iso-8859-15?Q?k4rnFG4LXVo0R7PAAySZZSh2W1YsvHBiUR9f335zrQlc7jdB+lYFboXKu?=
 =?iso-8859-15?Q?ZvKgUpNkHwA4g3Q13hEE2iA3H/nMpBbbvj4CTtiwSdCMUXTWYgW6VnaqC?=
 =?iso-8859-15?Q?E9C1g8G01Kl+1yMnOQzBUDFMcPnIp8FXjBtbgxa1gDfPvjA=3D?=
Content-Type: text/plain; charset="iso-8859-15"
Content-ID: <FA38881922A2C74B9499C211422F61AB@EURPRD10.PROD.OUTLOOK.COM>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: kontron.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AS2PR10MB7551.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: 61b7b68f-88e2-4417-cd5d-08db725b71df
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jun 2023 13:28:48.0915
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 8c9d3c97-3fd9-41c8-a2b1-646f3942daf1
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: mVRUkKcjuF2VvXJ20suLitRpdz5EYB0+LB5HV/0bB2YFKWPwaOvUj+2vAHG3+1jqq6EjvWjypB/FBLRjmLnLvPubBFeJ53+JOSmj9oMlADg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4PR10MB5394
X-Spam-Status: No, score=-0.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,FORGED_SPF_HELO,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE 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

On Wed, 2023-06-21 at 16:19 +0300, andriy.shevchenko@linux.intel.com
wrote:
> On Wed, Jun 21, 2023 at 11:30:56AM +0200, Andi Shyti wrote:
> > On Wed, Jun 21, 2023 at 07:46:42AM +0000, Michael Brunner wrote:
> > > On Tue, 2023-06-20 at 19:35 +0300, Andy Shevchenko wrote:
> > > > After switchind i2c-scmi driver to be a plaform one it stopped
> > > > being enumerated on number of Kontron platformsm, because it's
> > > > listed in the forbidden_id_list.
> > > >
> > > > To resolve the situation, split the list to generic one and
> > > > another that holds devices that has to be skiped if and only if
> > > > they have bogus resources attached (_CRS method returns some).
> > > >
> > > > Fixes: 03d4287add6e ("i2c: scmi: Convert to be a platform
> > > > driver")
> > > > Closes:
> > > > https://lore.kernel.org/r/60c1756765b9a3f1eab0dcbd84f59f00fe1caf48.=
camel@kontron.com
> > > > Reported-by: Michael Brunner <michael.brunner@kontron.com>
> > > > Signed-off-by: Andy Shevchenko <
> > > > andriy.shevchenko@linux.intel.com>
> > >
> > > Works as expected - the SMB0001 device shows up as platform
> > > device and
> > > the i2c-scmi driver is enumerated again on the affected boards.
> > > Thanks a lot!
> >
> > is this a "Tested-by: Michael Brunner <michael.brunner@kontron.com>
> > " :)
>
> Michael, indeed, it would be nice to have a formal tag.
> After that I will send a v2 with tags and fixed typos
> as Andi noticed (thank you, Andi!).

No problem:

Tested-by: Michael Brunner <michael.brunner@kontron.com>

Best regards,
  Michael