* [PATCH] of: remove duplicate declaration of of_iomap()
@ 2021-09-28 20:12 trix
2021-09-28 20:33 ` Randy Dunlap
2021-09-29 14:26 ` Rob Herring
0 siblings, 2 replies; 5+ messages in thread
From: trix @ 2021-09-28 20:12 UTC (permalink / raw)
To: robh+dt, frowand.list; +Cc: devicetree, linux-kernel, Tom Rix
From: Tom Rix <trix@redhat.com>
A ranconfig produces this linker error
irq-al-fic.c:252: undefined reference to `of_iomap'
The declaration of of_iomap() is dependent on OF
The definition of of_iomap() is dependent on OF_ADDRESS
These should match. There are duplicate declarations
of of_iomap(), remove of_iomap() and the
of_address_to_resource() duplicate.
Signed-off-by: Tom Rix <trix@redhat.com>
---
include/linux/of_address.h | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/include/linux/of_address.h b/include/linux/of_address.h
index 45598dbec269..a190996b4b0b 100644
--- a/include/linux/of_address.h
+++ b/include/linux/of_address.h
@@ -122,13 +122,7 @@ static inline bool of_dma_is_coherent(struct device_node *np)
{
return false;
}
-#endif /* CONFIG_OF_ADDRESS */
-#ifdef CONFIG_OF
-extern int of_address_to_resource(struct device_node *dev, int index,
- struct resource *r);
-void __iomem *of_iomap(struct device_node *node, int index);
-#else
static inline int of_address_to_resource(struct device_node *dev, int index,
struct resource *r)
{
@@ -139,7 +133,7 @@ static inline void __iomem *of_iomap(struct device_node *device, int index)
{
return NULL;
}
-#endif
+#endif /* CONFIG_OF_ADDRESS */
#define of_range_parser_init of_pci_range_parser_init
static inline const __be32 *of_get_address(struct device_node *dev, int index,
--
2.26.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] of: remove duplicate declaration of of_iomap()
2021-09-28 20:12 [PATCH] of: remove duplicate declaration of of_iomap() trix
@ 2021-09-28 20:33 ` Randy Dunlap
2021-09-28 20:42 ` Tom Rix
2021-09-29 14:26 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2021-09-28 20:33 UTC (permalink / raw)
To: trix, robh+dt, frowand.list; +Cc: devicetree, linux-kernel
On 9/28/21 1:12 PM, trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
>
> A ranconfig produces this linker error
randconfig .. on what arch, please?
> irq-al-fic.c:252: undefined reference to `of_iomap'
>
> The declaration of of_iomap() is dependent on OF
> The definition of of_iomap() is dependent on OF_ADDRESS
> These should match. There are duplicate declarations
> of of_iomap(), remove of_iomap() and the
> of_address_to_resource() duplicate.
>
> Signed-off-by: Tom Rix <trix@redhat.com>
> ---
> include/linux/of_address.h | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
thanks.
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] of: remove duplicate declaration of of_iomap()
2021-09-28 20:33 ` Randy Dunlap
@ 2021-09-28 20:42 ` Tom Rix
0 siblings, 0 replies; 5+ messages in thread
From: Tom Rix @ 2021-09-28 20:42 UTC (permalink / raw)
To: Randy Dunlap, robh+dt, frowand.list; +Cc: devicetree, linux-kernel
On 9/28/21 1:33 PM, Randy Dunlap wrote:
> On 9/28/21 1:12 PM, trix@redhat.com wrote:
>> From: Tom Rix <trix@redhat.com>
>>
>> A ranconfig produces this linker error
>
> randconfig .. on what arch, please?
s390
>
>> irq-al-fic.c:252: undefined reference to `of_iomap'
>>
>> The declaration of of_iomap() is dependent on OF
>> The definition of of_iomap() is dependent on OF_ADDRESS
>> These should match. There are duplicate declarations
>> of of_iomap(), remove of_iomap() and the
>> of_address_to_resource() duplicate.
>>
>> Signed-off-by: Tom Rix <trix@redhat.com>
>> ---
>> include/linux/of_address.h | 8 +-------
>> 1 file changed, 1 insertion(+), 7 deletions(-)
>>
>
>
> thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] of: remove duplicate declaration of of_iomap()
2021-09-28 20:12 [PATCH] of: remove duplicate declaration of of_iomap() trix
2021-09-28 20:33 ` Randy Dunlap
@ 2021-09-29 14:26 ` Rob Herring
2021-09-29 14:58 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Rob Herring @ 2021-09-29 14:26 UTC (permalink / raw)
To: trix; +Cc: Frank Rowand, devicetree, linux-kernel
On Tue, Sep 28, 2021 at 3:12 PM <trix@redhat.com> wrote:
>
> From: Tom Rix <trix@redhat.com>
>
> A ranconfig produces this linker error
> irq-al-fic.c:252: undefined reference to `of_iomap'
>
> The declaration of of_iomap() is dependent on OF
> The definition of of_iomap() is dependent on OF_ADDRESS
> These should match. There are duplicate declarations
> of of_iomap(), remove of_iomap() and the
> of_address_to_resource() duplicate.
>
> Signed-off-by: Tom Rix <trix@redhat.com>
> ---
> include/linux/of_address.h | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/include/linux/of_address.h b/include/linux/of_address.h
> index 45598dbec269..a190996b4b0b 100644
> --- a/include/linux/of_address.h
> +++ b/include/linux/of_address.h
> @@ -122,13 +122,7 @@ static inline bool of_dma_is_coherent(struct device_node *np)
> {
> return false;
> }
> -#endif /* CONFIG_OF_ADDRESS */
>
> -#ifdef CONFIG_OF
> -extern int of_address_to_resource(struct device_node *dev, int index,
> - struct resource *r);
> -void __iomem *of_iomap(struct device_node *node, int index);
This is going to break sparc which has !OF_ADDRESS and its own
of_iomap and of_address_to_resource implementations. I don't want to
add CONFIG_SPARC in here, so I think we should solve this in kconfig.
OF and !OF_ADDRESS is supposed to mean the arch provides these
functions.
I'd really like to do away with HAS_IOMEM. It doesn't serve much
purpose other than disabling a bunch of drivers.
> -#else
> static inline int of_address_to_resource(struct device_node *dev, int index,
> struct resource *r)
> {
> @@ -139,7 +133,7 @@ static inline void __iomem *of_iomap(struct device_node *device, int index)
> {
> return NULL;
> }
> -#endif
> +#endif /* CONFIG_OF_ADDRESS */
> #define of_range_parser_init of_pci_range_parser_init
>
> static inline const __be32 *of_get_address(struct device_node *dev, int index,
> --
> 2.26.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] of: remove duplicate declaration of of_iomap()
2021-09-29 14:26 ` Rob Herring
@ 2021-09-29 14:58 ` Rob Herring
0 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2021-09-29 14:58 UTC (permalink / raw)
To: trix; +Cc: Frank Rowand, devicetree, linux-kernel
On Wed, Sep 29, 2021 at 9:26 AM Rob Herring <robh+dt@kernel.org> wrote:
>
> On Tue, Sep 28, 2021 at 3:12 PM <trix@redhat.com> wrote:
> >
> > From: Tom Rix <trix@redhat.com>
> >
> > A ranconfig produces this linker error
> > irq-al-fic.c:252: undefined reference to `of_iomap'
> >
> > The declaration of of_iomap() is dependent on OF
> > The definition of of_iomap() is dependent on OF_ADDRESS
> > These should match. There are duplicate declarations
> > of of_iomap(), remove of_iomap() and the
> > of_address_to_resource() duplicate.
> >
> > Signed-off-by: Tom Rix <trix@redhat.com>
> > ---
> > include/linux/of_address.h | 8 +-------
> > 1 file changed, 1 insertion(+), 7 deletions(-)
> >
> > diff --git a/include/linux/of_address.h b/include/linux/of_address.h
> > index 45598dbec269..a190996b4b0b 100644
> > --- a/include/linux/of_address.h
> > +++ b/include/linux/of_address.h
> > @@ -122,13 +122,7 @@ static inline bool of_dma_is_coherent(struct device_node *np)
> > {
> > return false;
> > }
> > -#endif /* CONFIG_OF_ADDRESS */
> >
> > -#ifdef CONFIG_OF
> > -extern int of_address_to_resource(struct device_node *dev, int index,
> > - struct resource *r);
> > -void __iomem *of_iomap(struct device_node *node, int index);
>
> This is going to break sparc which has !OF_ADDRESS and its own
> of_iomap and of_address_to_resource implementations. I don't want to
> add CONFIG_SPARC in here, so I think we should solve this in kconfig.
> OF and !OF_ADDRESS is supposed to mean the arch provides these
> functions.
>
> I'd really like to do away with HAS_IOMEM. It doesn't serve much
> purpose other than disabling a bunch of drivers.
Following UML (the only other arch with configurable HAS_IOMEM), I
think dummy ioremap() implementations is the way to go here. UML added
its own dummy functions, but it would be better if we can just do
this:
diff --git a/include/asm-generic/io.h b/include/asm-generic/io.h
index e93375c710b9..a9d02403b2fe 100644
--- a/include/asm-generic/io.h
+++ b/include/asm-generic/io.h
@@ -946,7 +946,7 @@ static inline void *phys_to_virt(unsigned long address)
* can provide stricter non-posted write semantics if the architecture
* implements them.
*/
-#ifndef CONFIG_MMU
+#if !defined(CONFIG_GENERIC_IOREMAP)
#ifndef ioremap
#define ioremap ioremap
static inline void __iomem *ioremap(phys_addr_t offset, size_t size)
@@ -961,7 +961,7 @@ static inline void iounmap(void __iomem *addr)
{
}
#endif
-#elif defined(CONFIG_GENERIC_IOREMAP)
+#else /* CONFIG_GENERIC_IOREMAP */
#include <linux/pgtable.h>
void __iomem *ioremap_prot(phys_addr_t addr, size_t size, unsigned long prot);
@@ -972,7 +972,7 @@ static inline void __iomem *ioremap(phys_addr_t
addr, size_t size)
/* _PAGE_IOREMAP needs to be supplied by the architecture */
return ioremap_prot(addr, size, _PAGE_IOREMAP);
}
-#endif /* !CONFIG_MMU || CONFIG_GENERIC_IOREMAP */
+#endif /* CONFIG_GENERIC_IOREMAP */
#ifndef ioremap_wc
#define ioremap_wc ioremap
Then we can apply the following patch:
diff --git a/drivers/of/Kconfig b/drivers/of/Kconfig
index 3dfeae8912df..4a2453e4d4f3 100644
--- a/drivers/of/Kconfig
+++ b/drivers/of/Kconfig
@@ -64,7 +64,7 @@ config OF_DYNAMIC
config OF_ADDRESS
def_bool y
- depends on !SPARC && (HAS_IOMEM || UML)
+ depends on !SPARC
config OF_IRQ
def_bool y
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-09-29 14:59 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-28 20:12 [PATCH] of: remove duplicate declaration of of_iomap() trix
2021-09-28 20:33 ` Randy Dunlap
2021-09-28 20:42 ` Tom Rix
2021-09-29 14:26 ` Rob Herring
2021-09-29 14:58 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).