From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7C4181CDFA7 for ; Sat, 14 Sep 2024 07:12:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=114.242.206.163 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726297972; cv=none; b=QfBvpUcq8/aurtVv85TBcD1LHJ0kFuQegtv+KryQ9NqJjdg4Y0iK9MhshY4gwUevNj+wxNLfIFkiEe6Lbtg2/UwWmQXgklLh0G9n54fOANRtJBdpUeVM0yiUKIytVhD4rF0QemQtthfG9L9TP4jlIgJOBtbtohcs9+sFwnajfh4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726297972; c=relaxed/simple; bh=ZceQdqQJqEGlJLdBWTH43nSRYHAKB33VcqzfQ+YQbvE=; h=Subject:To:Cc:References:From:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=P61fbA3OKsFZpPBYNesYNHwWA2T6RPDCggI3VKFejC6SV0dPZ8YgJxnzGhytsHArb937OTTHFVwzs8iVWxmNEqhR+PcQxAzBwWw4LBrr+jkGi3VWW8jT8n2wx2O31eTK+K3AAWEb0QTJz1ckeSP+Dnxn+s6Ua3PgiZbq/aD+K3w= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn; spf=pass smtp.mailfrom=loongson.cn; arc=none smtp.client-ip=114.242.206.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=loongson.cn Received: from loongson.cn (unknown [10.20.42.62]) by gateway (Coremail) with SMTP id _____8DxG+pvN+VmPcsHAA--.18105S3; Sat, 14 Sep 2024 15:12:48 +0800 (CST) Received: from [10.20.42.62] (unknown [10.20.42.62]) by front1 (Coremail) with SMTP id qMiowMDx_9dsN+VmQ6IGAA--.38150S3; Sat, 14 Sep 2024 15:12:47 +0800 (CST) Subject: Re: [PATCH v2] LoongArch: Enable ACPI BGRT handling To: Huacai Chen Cc: "Rafael J . Wysocki" , Len Brown , WANG Xuerui , loongarch@lists.linux.dev, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, kernel test robot References: <20240914065318.2099448-1-maobibo@loongson.cn> From: maobibo Message-ID: <76b79099-ea95-eef5-e02b-6879f0998f18@loongson.cn> Date: Sat, 14 Sep 2024 15:12:44 +0800 User-Agent: Mozilla/5.0 (X11; Linux loongarch64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 Precedence: bulk X-Mailing-List: loongarch@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-CM-TRANSID:qMiowMDx_9dsN+VmQ6IGAA--.38150S3 X-CM-SenderInfo: xpdruxter6z05rqj20fqof0/ X-Coremail-Antispam: 1Uk129KBj93XoWxJF4DCrWDGFWDXw17Gw48uFX_yoWruF1xpF yvkF4kJFs8CF18Grnrt343WF98tr4kKrWIqFy7Ka43ZFnFvr17Ar4kur9IvFyjv3yDKF4F 9FWSqa13uF4UXagCm3ZEXasCq-sJn29KB7ZKAUJUUUU7529EdanIXcx71UUUUU7KY7ZEXa sCq-sGcSsGvfJ3Ic02F40EFcxC0VAKzVAqx4xG6I80ebIjqfuFe4nvWSU5nxnvy29KBjDU 0xBIdaVrnRJUUUPab4IE77IF4wAFF20E14v26r1j6r4UM7CY07I20VC2zVCF04k26cxKx2 IYs7xG6rWj6s0DM7CIcVAFz4kK6r1j6r18M28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48v e4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Gr0_Xr1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI 0_Gr0_Cr1l84ACjcxK6I8E87Iv67AKxVW8Jr0_Cr1UM28EF7xvwVC2z280aVCY1x0267AK xVW8Jr0_Cr1UM2kKe7AKxVWUXVWUAwAS0I0E0xvYzxvE52x082IY62kv0487Mc804VCY07 AIYIkI8VC2zVCFFI0UMc02F40EFcxC0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWU AVWUtwAv7VC2z280aVAFwI0_Gr0_Cr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcVAKI4 8JMxk0xIA0c2IEe2xFo4CEbIxvr21lc7CjxVAaw2AFwI0_JF0_Jw1l42xK82IYc2Ij64vI r41l4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1l4IxYO2xFxVAFwI0_Jrv_JF1lx2IqxVAqx4xG67 AKxVWUJVWUGwC20s026x8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17CE14v26r1q6r43MIIY rxkI7VAKI48JMIIF0xvE2Ix0cI8IcVAFwI0_Jr0_JF4lIxAIcVC0I7IYx2IY6xkF7I0E14 v26r1j6r4UMIIF0xvE42xK8VAvwI8IcIk0rVWUJVWUCwCI42IY6I8E87Iv67AKxVW8JVWx JwCI42IY6I8E87Iv6xkF7I0E14v26r4j6r4UJbIYCTnIWIevJa73UjIFyTuYvjxUcqXdUU UUU On 2024/9/14 下午3:01, Huacai Chen wrote: > Hi, Bibo, > > On Sat, Sep 14, 2024 at 2:53 PM Bibo Mao wrote: >> >> Add ACPI BGRT support on LoongArch so it can display image provied by >> acpi table at boot stage and switch to graphical UI smoothly. >> >> Reported-by: kernel test robot >> Closes: https://lore.kernel.org/oe-kbuild-all/202409102056.DNqh6zzA-lkp@intel.com/ >> Signed-off-by: Bibo Mao >> --- >> v1 ... v2: >> 1. Solve compile warning issue reported from lkp, return type of >> function early_memunmap() is void *, that of function early_ioremap() >> is void __iomem *, force type conversion is added. > I've applied V1, build warnings seems another problem which has no > relationship with this patch itself. Good, thanks for applying it. Regards Bibo Mao > > Huacai > >> --- >> arch/loongarch/include/asm/io.h | 4 +--- >> arch/loongarch/kernel/acpi.c | 8 ++++++-- >> arch/loongarch/mm/ioremap.c | 9 +++++++++ >> drivers/acpi/Kconfig | 2 +- >> 4 files changed, 17 insertions(+), 6 deletions(-) >> >> diff --git a/arch/loongarch/include/asm/io.h b/arch/loongarch/include/asm/io.h >> index 5e95a60df180..3049bccec693 100644 >> --- a/arch/loongarch/include/asm/io.h >> +++ b/arch/loongarch/include/asm/io.h >> @@ -10,6 +10,7 @@ >> >> #include >> #include >> +#include >> #include >> #include >> #include >> @@ -17,9 +18,6 @@ >> extern void __init __iomem *early_ioremap(u64 phys_addr, unsigned long size); >> extern void __init early_iounmap(void __iomem *addr, unsigned long size); >> >> -#define early_memremap early_ioremap >> -#define early_memunmap early_iounmap >> - >> #ifdef CONFIG_ARCH_IOREMAP >> >> static inline void __iomem *ioremap_prot(phys_addr_t offset, unsigned long size, >> diff --git a/arch/loongarch/kernel/acpi.c b/arch/loongarch/kernel/acpi.c >> index 929a497c987e..2993d7921198 100644 >> --- a/arch/loongarch/kernel/acpi.c >> +++ b/arch/loongarch/kernel/acpi.c >> @@ -9,6 +9,7 @@ >> >> #include >> #include >> +#include >> #include >> #include >> #include >> @@ -39,14 +40,14 @@ void __init __iomem * __acpi_map_table(unsigned long phys, unsigned long size) >> if (!phys || !size) >> return NULL; >> >> - return early_memremap(phys, size); >> + return (void __iomem *)early_memremap(phys, size); >> } >> void __init __acpi_unmap_table(void __iomem *map, unsigned long size) >> { >> if (!map || !size) >> return; >> >> - early_memunmap(map, size); >> + early_memunmap((void *)map, size); >> } >> >> void __iomem *acpi_os_ioremap(acpi_physical_address phys, acpi_size size) >> @@ -212,6 +213,9 @@ void __init acpi_boot_table_init(void) >> /* Do not enable ACPI SPCR console by default */ >> acpi_parse_spcr(earlycon_acpi_spcr_enable, false); >> >> + if (IS_ENABLED(CONFIG_ACPI_BGRT)) >> + acpi_table_parse(ACPI_SIG_BGRT, acpi_parse_bgrt); >> + >> return; >> >> fdt_earlycon: >> diff --git a/arch/loongarch/mm/ioremap.c b/arch/loongarch/mm/ioremap.c >> index 70ca73019811..28562ac510c8 100644 >> --- a/arch/loongarch/mm/ioremap.c >> +++ b/arch/loongarch/mm/ioremap.c >> @@ -16,6 +16,15 @@ void __init early_iounmap(void __iomem *addr, unsigned long size) >> >> } >> >> +void __init *early_memremap(resource_size_t phys_addr, unsigned long size) >> +{ >> + return (__force void *)early_ioremap(phys_addr, size); >> +} >> + >> +void __init early_memunmap(void *addr, unsigned long size) >> +{ >> +} >> + >> void *early_memremap_ro(resource_size_t phys_addr, unsigned long size) >> { >> return early_memremap(phys_addr, size); >> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig >> index e3a7c2aedd5f..d67f63d93b2a 100644 >> --- a/drivers/acpi/Kconfig >> +++ b/drivers/acpi/Kconfig >> @@ -451,7 +451,7 @@ config ACPI_HED >> >> config ACPI_BGRT >> bool "Boottime Graphics Resource Table support" >> - depends on EFI && (X86 || ARM64) >> + depends on EFI && (X86 || ARM64 || LOONGARCH) >> help >> This driver adds support for exposing the ACPI Boottime Graphics >> Resource Table, which allows the operating system to obtain >> >> base-commit: 196145c606d0f816fd3926483cb1ff87e09c2c0b >> -- >> 2.39.3 >> >>