* [PATCH] lib: devres: Export devm_ioremap_resource_wc
@ 2020-01-23 15:47 Daniel Baluta (OSS)
2020-01-23 17:56 ` gregkh
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Baluta (OSS) @ 2020-01-23 15:47 UTC (permalink / raw)
To: bgolaszewski
Cc: arnd, akpm, gregkh, info, ztuowen, sergei.shtylyov, linux-kernel,
Daniel Baluta
From: Daniel Baluta <daniel.baluta@nxp.com>
So that modules can also use it.
Fixes: b873af620e58863b ("lib: devres: provide devm_ioremap_resource_wc()")
Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
---
lib/devres.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/devres.c b/lib/devres.c
index 6ef51f159c54..7fe2bd75dfa3 100644
--- a/lib/devres.c
+++ b/lib/devres.c
@@ -182,6 +182,7 @@ void __iomem *devm_ioremap_resource_wc(struct device *dev,
{
return __devm_ioremap_resource(dev, res, DEVM_IOREMAP_WC);
}
+EXPORT_SYMBOL(devm_ioremap_resource_wc);
/*
* devm_of_iomap - Requests a resource and maps the memory mapped IO
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] lib: devres: Export devm_ioremap_resource_wc
2020-01-23 15:47 [PATCH] lib: devres: Export devm_ioremap_resource_wc Daniel Baluta (OSS)
@ 2020-01-23 17:56 ` gregkh
2020-01-23 18:16 ` Daniel Baluta
0 siblings, 1 reply; 4+ messages in thread
From: gregkh @ 2020-01-23 17:56 UTC (permalink / raw)
To: Daniel Baluta (OSS)
Cc: bgolaszewski, arnd, akpm, info, ztuowen, sergei.shtylyov,
linux-kernel, Daniel Baluta
On Thu, Jan 23, 2020 at 03:47:21PM +0000, Daniel Baluta (OSS) wrote:
> From: Daniel Baluta <daniel.baluta@nxp.com>
>
> So that modules can also use it.
>
> Fixes: b873af620e58863b ("lib: devres: provide devm_ioremap_resource_wc()")
> Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
> ---
> lib/devres.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/devres.c b/lib/devres.c
> index 6ef51f159c54..7fe2bd75dfa3 100644
> --- a/lib/devres.c
> +++ b/lib/devres.c
> @@ -182,6 +182,7 @@ void __iomem *devm_ioremap_resource_wc(struct device *dev,
> {
> return __devm_ioremap_resource(dev, res, DEVM_IOREMAP_WC);
> }
> +EXPORT_SYMBOL(devm_ioremap_resource_wc);
EXPORT_SYMBOL_GPL() perhaps?
What in-tree driver needs this?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] lib: devres: Export devm_ioremap_resource_wc
2020-01-23 17:56 ` gregkh
@ 2020-01-23 18:16 ` Daniel Baluta
2020-01-23 18:22 ` gregkh
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Baluta @ 2020-01-23 18:16 UTC (permalink / raw)
To: gregkh
Cc: Daniel Baluta (OSS),
bgolaszewski, arnd, akpm, info, ztuowen, sergei.shtylyov,
linux-kernel, Daniel Baluta
On Thu, Jan 23, 2020 at 7:57 PM gregkh@linuxfoundation.org
<gregkh@linuxfoundation.org> wrote:
>
> On Thu, Jan 23, 2020 at 03:47:21PM +0000, Daniel Baluta (OSS) wrote:
> > From: Daniel Baluta <daniel.baluta@nxp.com>
> >
> > So that modules can also use it.
> >
> > Fixes: b873af620e58863b ("lib: devres: provide devm_ioremap_resource_wc()")
> > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
> > ---
> > lib/devres.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/lib/devres.c b/lib/devres.c
> > index 6ef51f159c54..7fe2bd75dfa3 100644
> > --- a/lib/devres.c
> > +++ b/lib/devres.c
> > @@ -182,6 +182,7 @@ void __iomem *devm_ioremap_resource_wc(struct device *dev,
> > {
> > return __devm_ioremap_resource(dev, res, DEVM_IOREMAP_WC);
> > }
> > +EXPORT_SYMBOL(devm_ioremap_resource_wc);
>
> EXPORT_SYMBOL_GPL() perhaps?
>
> What in-tree driver needs this?
I was experimenting with an out of tree driver and I also was using it wrong :D.
Indeed looks like there is no real potential user so far in the kernel tree.
Perhaps: drivers/net/ethernet/amazon/ena/ena_netdev.c
Also, I chose EXPORT_SYMBOL because the same way its cousin
devm_ioremap_resource.
Daniel.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] lib: devres: Export devm_ioremap_resource_wc
2020-01-23 18:16 ` Daniel Baluta
@ 2020-01-23 18:22 ` gregkh
0 siblings, 0 replies; 4+ messages in thread
From: gregkh @ 2020-01-23 18:22 UTC (permalink / raw)
To: Daniel Baluta
Cc: Daniel Baluta (OSS),
bgolaszewski, arnd, akpm, info, ztuowen, sergei.shtylyov,
linux-kernel, Daniel Baluta
On Thu, Jan 23, 2020 at 08:16:35PM +0200, Daniel Baluta wrote:
> On Thu, Jan 23, 2020 at 7:57 PM gregkh@linuxfoundation.org
> <gregkh@linuxfoundation.org> wrote:
> >
> > On Thu, Jan 23, 2020 at 03:47:21PM +0000, Daniel Baluta (OSS) wrote:
> > > From: Daniel Baluta <daniel.baluta@nxp.com>
> > >
> > > So that modules can also use it.
> > >
> > > Fixes: b873af620e58863b ("lib: devres: provide devm_ioremap_resource_wc()")
> > > Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
> > > ---
> > > lib/devres.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/lib/devres.c b/lib/devres.c
> > > index 6ef51f159c54..7fe2bd75dfa3 100644
> > > --- a/lib/devres.c
> > > +++ b/lib/devres.c
> > > @@ -182,6 +182,7 @@ void __iomem *devm_ioremap_resource_wc(struct device *dev,
> > > {
> > > return __devm_ioremap_resource(dev, res, DEVM_IOREMAP_WC);
> > > }
> > > +EXPORT_SYMBOL(devm_ioremap_resource_wc);
> >
> > EXPORT_SYMBOL_GPL() perhaps?
> >
> > What in-tree driver needs this?
>
> I was experimenting with an out of tree driver and I also was using it wrong :D.
> Indeed looks like there is no real potential user so far in the kernel tree.
Great, we can drop it then :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-23 18:22 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-23 15:47 [PATCH] lib: devres: Export devm_ioremap_resource_wc Daniel Baluta (OSS)
2020-01-23 17:56 ` gregkh
2020-01-23 18:16 ` Daniel Baluta
2020-01-23 18:22 ` gregkh
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).