From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v5 18/18] Documentation: ACPI for ARM64 Date: Tue, 06 Jan 2015 15:05:12 +0100 Message-ID: <2647669.mQf6pODdjV@wuerfel> References: <1413553034-20956-1-git-send-email-hanjun.guo@linaro.org> <54ABC2CB.6@linaro.org> <20150106112929.GB8829@e104818-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from mout.kundenserver.de ([212.227.17.24]:54387 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753929AbbAFOGK (ORCPT ); Tue, 6 Jan 2015 09:06:10 -0500 In-Reply-To: <20150106112929.GB8829@e104818-lin.cambridge.arm.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: linux-arm-kernel@lists.infradead.org Cc: Catalin Marinas , Hanjun Guo , Mark Rutland , "linaro-acpi@lists.linaro.org" , Will Deacon , Lv Zheng , Rob Herring , Lorenzo Pieralisi , Al Stone , Daniel Lezcano , Robert Moore , "linux-acpi@vger.kernel.org" , "jcm@redhat.com" , "grant.likely@linaro.org" , Charles Garcia-Tobin , Robert Richter , Jason Cooper , Marc Zyngier , Liviu Dudau , Mark Brown , Bjorn Helgaas , graeme.gregory@linaro.or On Tuesday 06 January 2015 11:29:29 Catalin Marinas wrote: > > >> We will work on this both on ASWG and linux ACPI driver side, as Dong > > >> and Charles pointed out, _OSI things can be solved in ACPI spec, when > > >> that is done, we can modify the kernel driver to fix the problems above. > > > > > > Which driver? > > > > the ACPICA core driver as you suggested, sorry for the confusion. > > > > > What about ACPI_OS_NAME? Would you suggest it is fine to report > > > "Microsoft Windows NT" on an ARM system? That _OS_ not _OSI. > > > > No, not at all. I prefer "Linux" > > In include/acpi/acconfig.h, when ACPI_OS_NAME defined, it says: > > "OS name, used for the _OS object. The _OS object is essentially > > obsolete,..." > > for some legacy reasons, we needed "Microsoft Windows NT", but ACPI > > for ARM64 on linux is totally new, I think we can change it to > > "Linux" when CONFIG_ARM64 as you suggested. > > We could ignore this change for now if we don't expect the _OS object to > be used at all. But do we have any other way to check the AML code for > this? Would FWTS catch such obsolete cases? How about just leaving it out? It's clearly not used for anything good, so I don't see the point in passing either Linux or "Microsoft Windows NT" here. Arnd