Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753789AbdFSJvZ (ORCPT ); Mon, 19 Jun 2017 05:51:25 -0400 Received: from szxga01-in.huawei.com ([45.249.212.187]:8750 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751865AbdFSJvW (ORCPT ); Mon, 19 Jun 2017 05:51:22 -0400 From: Gabriele Paoloni To: "Rafael J. Wysocki" , Mika Westerberg CC: Lorenzo Pieralisi , "Rafael J. Wysocki" , "catalin.marinas@arm.com" , "will.deacon@arm.com" , "robh+dt@kernel.org" , "frowand.list@gmail.com" , "bhelgaas@google.com" , "arnd@arndb.de" , "linux-arm-kernel@lists.infradead.org" , "mark.rutland@arm.com" , "brian.starkey@arm.com" , "olof@lixom.net" , "benh@kernel.crashing.org" , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" , Linuxarm , "linux-pci@vger.kernel.org" , "minyard@acm.org" , John Garry , "xuwei (O)" Subject: RE: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning Thread-Topic: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO devices before scanning Thread-Index: AQHS1UuGzbU5u381iUa+4J2CuQvNbqIMw+oAgAFaBMCACVtbgIAJ46YAgAEanYCAAILAkP//5+WAgABQ2mCAAAEigIADHYUwgADYkoCAAC/eAIAACiIAgAAGKYCABKntQA== Date: Mon, 19 Jun 2017 09:50:49 +0000 Message-ID: References: <20170606085553.GA20085@red-moon> <20170612155700.GA31930@red-moon> <20170613084831.GP3187@lahna.fi.intel.com> <20170613151013.GT3187@lahna.fi.intel.com> <20170613200339.GX3187@lahna.fi.intel.com> <20170616083313.GY3187@lahna.fi.intel.com> <20170616120048.GC629@lahna.fi.intel.com> In-Reply-To: Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.203.135.63] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020205.59479E88.00CD,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=169.254.1.157, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 96dc3753f728769af6773120f6143f42 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v5J9pYBX028707 Content-Length: 2168 Lines: 53 Hi Rafael, Mika, Lorenzo > -----Original Message----- > From: rjwysocki@gmail.com [mailto:rjwysocki@gmail.com] On Behalf Of > Rafael J. Wysocki > Sent: 16 June 2017 13:23 > To: Mika Westerberg > Cc: Rafael J. Wysocki; Gabriele Paoloni; Lorenzo Pieralisi; Rafael J. > Wysocki; catalin.marinas@arm.com; will.deacon@arm.com; > robh+dt@kernel.org; frowand.list@gmail.com; bhelgaas@google.com; > arnd@arndb.de; linux-arm-kernel@lists.infradead.org; > mark.rutland@arm.com; brian.starkey@arm.com; olof@lixom.net; > benh@kernel.crashing.org; linux-kernel@vger.kernel.org; linux- > acpi@vger.kernel.org; Linuxarm; linux-pci@vger.kernel.org; > minyard@acm.org; John Garry; xuwei (O) > Subject: Re: [PATCH v9 5/7] ACPI: Translate the I/O range of non-MMIO > devices before scanning > > On Fri, Jun 16, 2017 at 2:00 PM, Mika Westerberg > wrote: > > On Fri, Jun 16, 2017 at 01:24:32PM +0200, Rafael J. Wysocki wrote: > >> > In fact it may be that it is not sufficient in this case because > the > >> > ACPI core might enumerate child devices before the LPC driver even > gets > >> > a chance to probe so you would need to add also scan handler to > the > >> > child devices and mark them already enumerated or something like > that. > >> > >> Or extend the special I2C/SPI handling to them. > > > > Sure but those have I2c/SpiSerialBus() resources which we can use to > > identify them but for the ipmi thing there is nothing else than _HID > so > > we would need to keep a list of such devices in ACPI core. > > OK, so adding a scan handler for that would be the way to go IMO. Many thanks for your response and your help here. I guess that as conclusion with respect to the current v9 patchset we can disregard the idea of MFD and modify the current v9 so that it doesn't touch directly ACPI resources. Instead as I proposed before we can have the scan handler to enumerate the children devices and translate its addresses filling dev->resources[] and at the same time we can modify acpi_default_enumeration to check acpi_device_enumerated() before continuing with device enumeration...? Do you think it as a viable solution? Thanks Gab