From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755598AbdABMCr (ORCPT ); Mon, 2 Jan 2017 07:02:47 -0500 Received: from mail-pg0-f48.google.com ([74.125.83.48]:36759 "EHLO mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755356AbdABMCo (ORCPT ); Mon, 2 Jan 2017 07:02:44 -0500 Message-ID: <586A4155.3090905@linaro.org> Date: Mon, 02 Jan 2017 20:02:29 +0800 From: Hanjun Guo User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: "Rafael J. Wysocki" , Hanjun Guo CC: Marc Zyngier , "Rafael J. Wysocki" , Lorenzo Pieralisi , ACPI Devel Maling List , "linux-arm-kernel@lists.infradead.org" , Linux Kernel Mailing List , Thomas Gleixner , Greg KH , Tomasz Nowicki , Ma Jun , Kefeng Wang , Agustin Vega-Frias , Sinan Kaya , Charles Garcia-Tobin , huxinwei@huawei.com, yimin@huawei.com, linuxarm@huawei.com, Jon Masters Subject: Re: [PATCH v5 09/14] ACPI: platform: setup MSI domain for ACPI based platform device References: <1482384922-21507-1-git-send-email-guohanjun@huawei.com> <1482384922-21507-10-git-send-email-guohanjun@huawei.com> <585E24FB.9050805@huawei.com> <586072E9.3060609@huawei.com> <58663C04.9000603@huawei.com> In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/01/2017 04:45 AM, Rafael J. Wysocki wrote: > On Fri, Dec 30, 2016 at 11:50 AM, Hanjun Guo wrote: [...] >> >> So how about just add the code as below? > > Works for me. OK, will send out the updated patch set soon. > >> diff --git a/drivers/acpi/glue.c b/drivers/acpi/glue.c >> index 11e63dd..37a8dfe 100644 >> --- a/drivers/acpi/glue.c >> +++ b/drivers/acpi/glue.c >> @@ -316,7 +316,8 @@ static int acpi_platform_notify(struct device *dev) >> if (!adev) >> goto out; >> >> + if (dev->bus == &platform_bus_type) >> + acpi_configure_pmsi_domain(dev); >> >> if (type && type->setup) >> type->setup(dev); Thanks for your comments. Hanjun