回复: [PATCH 1/2] powerpc:mm: export symbol ioremap_coherent
From: Wenhu Wang
Date: Wed Jun 15 2022 - 00:37:50 EST
>发件人: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
>发送时间: 2022年6月14日 18:45
>收件人: Wang Wenhu <wenhu.wang@xxxxxxxxxxx>; gregkh@xxxxxxxxxxxxxxxxxxx <gregkh@xxxxxxxxxxxxxxxxxxx>; christophe.leroy@xxxxxxxxxx <christophe.leroy@xxxxxxxxxx>
>抄送: linuxppc-dev@xxxxxxxxxxxxxxxx <linuxppc-dev@xxxxxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>; Wang Wenhu <wenhu.wang@xxxxxxxxxxx>
>主题: Re: [PATCH 1/2] powerpc:mm: export symbol ioremap_coherent
>
>Wang Wenhu <wenhu.wang@xxxxxxxxxxx> writes:
>> The function ioremap_coherent may be called by modules such as
>> fsl_85xx_cache_sram. So export it for access in other modules.
>
>ioremap_coherent() is powerpc specific, and only has one other caller,
>I'd like to remove it.
>
>Does ioremap_cache() work for you?
>
Yes, it works. I will update in v2 to use ioremap_cache.
I tested and compared the outcomes of ioremap_cache and ioremap_coherent,
and found they ended same values.
Thanks,
Wenhu
>
>> diff --git a/arch/powerpc/mm/ioremap.c b/arch/powerpc/mm/ioremap.c
>> index 4f12504fb405..08a00dacef0b 100644
>> --- a/arch/powerpc/mm/ioremap.c
>> +++ b/arch/powerpc/mm/ioremap.c
>> @@ -40,6 +40,7 @@ void __iomem *ioremap_coherent(phys_addr_t addr, unsigned long size)
>> return iowa_ioremap(addr, size, prot, caller);
>> return __ioremap_caller(addr, size, prot, caller);
>> }
>> +EXPORT_SYMBOL(ioremap_coherent);
>>
>> void __iomem *ioremap_prot(phys_addr_t addr, unsigned long size, unsigned long flags)
> {
> --
> 2.25.1