Hi Michael,
>>>Which was merged in May 2006. So the powerpc version has almost 10 years
>>>precedence.
Agreed but keeping the licensing consistent will help drivers making seamless
use of it on all architectures.
>>> But I guess it's a pretty boring API. So I'll merge this unless anyone else
>>> objects.
Thanks. Appreciate your quick response and help.
Thanks,
Shailendra
-Original Message-
From: Michael Ellerman [mailto:m...@ellerman.id.au]
Sent: Thursday, February 09, 2017 4:19 AM
To: Shailendra Singh <shailend...@nvidia.com>; linuxppc-dev@lists.ozlabs.org
Cc: j...@ozlabs.org; apop...@au1.ibm.com; balb...@au1.ibm.com; Andy Ritger
<arit...@nvidia.com>; John Hubbard <jhubb...@nvidia.com>; Sherry Cheung
<sche...@nvidia.com>; Aruna Manjunatha <amanjuna...@nvidia.com>; John McKenna
<jmcke...@nvidia.com>; david.da...@cavium.com; r...@kernel.org;
will.dea...@arm.com; Arnd Bergmann <a...@arndb.de>
Subject: Re: [PATCH] powerpc: Fix inconsistent of_node_to_nid EXPORT_SYMBOL
handling
Shailendra Singh <shailend...@nvidia.com> writes:
> The generic implementation of of_node_to_nid is EXPORT_SYMBOL.
True. Added in 298535c00a2c, in April 2016.
> The powerpc implementation added by following commit is EXPORT_SYMBOL_GPL.
> commit 953039c8df7b ("[PATCH] powerpc: Allow devices to register with
> numa
> topology")
Which was merged in May 2006.
So the powerpc version has almost 10 years precedence.
[Shailendra] - Agreed. Keeping the licensing consistent will help drivers
making use of it on all architectures.
> This creates an inconsistency for of_node_to_nid callers across
> architectures.
>
> Update the powerpc implementation to be exported consistently with the
> generic implementation.
But I guess it's a pretty boring API. So I'll merge this unless anyone else
objects.
[Shailendra] - Thanks. Appreciate your quick response.
cheers
> Signed-off-by: Shailendra Singh <shailend...@nvidia.com>
> Reviewed-by: Andy Ritger <arit...@nvidia.com>
> ---
> arch/powerpc/mm/numa.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c index
> b1099cb2f393..8aa4ca3c84c9 100644
> --- a/arch/powerpc/mm/numa.c
> +++ b/arch/powerpc/mm/numa.c
> @@ -290,7 +290,7 @@ int of_node_to_nid(struct device_node *device)
>
> return nid;
> }
> -EXPORT_SYMBOL_GPL(of_node_to_nid);
> +EXPORT_SYMBOL(of_node_to_nid);
>
> static int __init find_min_common_depth(void) {
> --
> 2.4.11
---
This email message is for the sole use of the intended recipient(s) and may
contain
confidential information. Any unauthorized review, use, disclosure or
distribution
is prohibited. If you are not the intended recipient, please contact the
sender by
reply email and destroy all copies of the original message.
---
