From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hanjun Guo Subject: Re: [PATCH 01/19] ARM64 / ACPI: Get RSDP and ACPI boot-time tables Date: Tue, 29 Jul 2014 22:07:40 +0800 Message-ID: <53D7AAAC.6070606@linaro.org> References: <1406206825-15590-1-git-send-email-hanjun.guo@linaro.org> <1406206825-15590-2-git-send-email-hanjun.guo@linaro.org> <53D69678.20002@arm.com> <53D79CD8.9050709@linaro.org> <53D7A6A2.9020005@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <53D7A6A2.9020005@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sudeep Holla Cc: Mark Rutland , Mark Brown , Catalin Marinas , Will Deacon , Lv Zheng , Lorenzo Pieralisi , Al Stone , Daniel Lezcano , Robert Moore , "linux-acpi@vger.kernel.org" , "grant.likely@linaro.org" , Charles Garcia-Tobin , Robert Richter , Jason Cooper , Arnd Bergmann , Marc Zyngier , Liviu Dudau , Tomasz Nowicki , Bjorn Helgaas , "linux-arm-kernel@lists.infradead.org" , "graeme.gregory@linaro.org" List-Id: linux-acpi@vger.kernel.org T24gMjAxNOW5tDA35pyIMjnml6UgMjE6NTAsIFN1ZGVlcCBIb2xsYSB3cm90ZToKPgo+IE9uIDI5 LzA3LzE0IDE0OjA4LCBIYW5qdW4gR3VvIHdyb3RlOgo+PiBPbiAyMDE0LTctMjkgMjoyOSwgU3Vk ZWVwIEhvbGxhIHdyb3RlOgo+PiBbLi4uXQo+Pj4+ICsKPj4+PiArLyoKPj4+PiArICogV2UgbmV2 ZXIgcGxhbiB0byB1c2UgUlNEVCBvbiBhcm0vYXJtNjQgYXMgaXRzIGRlcHJlY2F0ZWQgaW4gc3Bl YyBidXQgdGhpcwo+Pj4+ICsgKiB2YXJpYWJsZSBpcyBzdGlsbCByZXF1aXJlZCBieSB0aGUgQUNQ SSBjb3JlCj4+Pj4gKyAqLwo+Pj4+ICt1MzIgYWNwaV9yc2R0X2ZvcmNlZDsKPj4+Cj4+PiBUaGlz IGxvb2tzIGxpa2UgeWV0IGFub3RoZXIgc3RyYXkgdmFyaWFibGUgbm90IHVzZWQgYW55d2hlcmUg aW5jbHVkaW5nCj4+PiBjb3JlIEFDUEkKPj4KPj4gSXQgd2lsbCBiZSB1c2VkIGJ5IEFDUElDQSB0 byBnZXQgdGhlIFJTRFAgaW4gYWNwaV90Yl9wYXJzZV9yb290X3RhYmxlKCkKPj4gaW4gdGJ1dGls cy5jLgo+Pgo+Cj4gTm8gSSBjb3VsZG4ndCBmaW5kIGl0IGluIG1haW5saW5lL2FjcGljYS9saW51 eC1uZXh0KHBtIHRyZWUpLgo+Cj4gT24gZnVydGhlciBkaWdnaW5nIGZvdW5kIHRoYXQgY29tbWl0 IGZhYjQ2MTA1ODM4NSgiQUNQSUNBOiBDbGVhbnVwIHRoZQo+IG9wdGlvbiBvZiBmb3JjaW5nIHRo ZSB1c2Ugb2YgdGhlIFJTRFQiKSBoYXMgcmVuYW1lZCBpdCB0bwo+IGFjcGlfZ2JsX2RvX25vdF91 c2VfeHNkdCBhbmQgbW92ZWQgaXQgaW50byBhY3BpY2Egd2l0aCBkZWZhdWx0IGZhbHNlLgo+IFNv IHlvdSBjYW4gcmVtb3ZlIHRoaXMuCgpBaCwgbXkgYmFkLiBJIGdyZXBwZWQgaXQgaW4gYW5vdGhl ciB2ZXJzaW9uIG9mIGtlcm5lbCBhbmQgZm91bmQgdGhlIG91dGRhdGVkCnJlc3VsdCwgSSB3aWxs IHVwZGF0ZSB0aGlzIHBhdGNoLgoKVGhhbmtzCkhhbmp1bgoKCl9fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0 CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFk ZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: hanjun.guo@linaro.org (Hanjun Guo) Date: Tue, 29 Jul 2014 22:07:40 +0800 Subject: [PATCH 01/19] ARM64 / ACPI: Get RSDP and ACPI boot-time tables In-Reply-To: <53D7A6A2.9020005@arm.com> References: <1406206825-15590-1-git-send-email-hanjun.guo@linaro.org> <1406206825-15590-2-git-send-email-hanjun.guo@linaro.org> <53D69678.20002@arm.com> <53D79CD8.9050709@linaro.org> <53D7A6A2.9020005@arm.com> Message-ID: <53D7AAAC.6070606@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 2014?07?29? 21:50, Sudeep Holla wrote: > > On 29/07/14 14:08, Hanjun Guo wrote: >> On 2014-7-29 2:29, Sudeep Holla wrote: >> [...] >>>> + >>>> +/* >>>> + * We never plan to use RSDT on arm/arm64 as its deprecated in spec but this >>>> + * variable is still required by the ACPI core >>>> + */ >>>> +u32 acpi_rsdt_forced; >>> >>> This looks like yet another stray variable not used anywhere including >>> core ACPI >> >> It will be used by ACPICA to get the RSDP in acpi_tb_parse_root_table() >> in tbutils.c. >> > > No I couldn't find it in mainline/acpica/linux-next(pm tree). > > On further digging found that commit fab461058385("ACPICA: Cleanup the > option of forcing the use of the RSDT") has renamed it to > acpi_gbl_do_not_use_xsdt and moved it into acpica with default false. > So you can remove this. Ah, my bad. I grepped it in another version of kernel and found the outdated result, I will update this patch. Thanks Hanjun