* dmaengine: ppc4xx: fix off-by-one build failure
@ 2018-10-16 20:28 Christian Lamparter
0 siblings, 0 replies; 3+ messages in thread
From: Christian Lamparter @ 2018-10-16 20:28 UTC (permalink / raw)
To: Vinod; +Cc: dmaengine, Dan Williams, Greg Kroah-Hartman
On Tuesday, October 16, 2018 4:40:27 PM CEST Vinod wrote:
> On 14-10-18, 23:28, Christian Lamparter wrote:
> > There are two poly_store, but one should have been poly_show.
> >
> > |adma.c:4382:16: error: conflicting types for 'poly_store'
> > | static ssize_t poly_store(struct device_driver *dev, const char *buf,
> > | ^~~~~~~~~~
> > |adma.c:4363:16: note: previous definition of 'poly_store' was here
> > | static ssize_t poly_store(struct device_driver *dev, char *buf)
> > | ^~~~~~~~~~
>
> How come no one noticed till now :(
Well, I don't know if this driver works or not. I tried to evaluate
if this driver could be modified to fit the PLB4DMA of the APM82181.
But sadly this was not the case because the APM82181 and PPC460EX
have a new design. So, I just happend to find this little bug, no
idea if there's more bit-rot.
Regards,
Christian
^ permalink raw reply [flat|nested] 3+ messages in thread
* dmaengine: ppc4xx: fix off-by-one build failure
@ 2018-10-16 14:40 Vinod Koul
0 siblings, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2018-10-16 14:40 UTC (permalink / raw)
To: Christian Lamparter; +Cc: dmaengine, Dan Williams, Greg Kroah-Hartman
On 14-10-18, 23:28, Christian Lamparter wrote:
> There are two poly_store, but one should have been poly_show.
>
> |adma.c:4382:16: error: conflicting types for 'poly_store'
> | static ssize_t poly_store(struct device_driver *dev, const char *buf,
> | ^~~~~~~~~~
> |adma.c:4363:16: note: previous definition of 'poly_store' was here
> | static ssize_t poly_store(struct device_driver *dev, char *buf)
> | ^~~~~~~~~~
How come no one noticed till now :(
Applied and cced stable
Thanks
^ permalink raw reply [flat|nested] 3+ messages in thread
* dmaengine: ppc4xx: fix off-by-one build failure
@ 2018-10-14 21:28 Christian Lamparter
0 siblings, 0 replies; 3+ messages in thread
From: Christian Lamparter @ 2018-10-14 21:28 UTC (permalink / raw)
To: dmaengine; +Cc: Dan Williams, Vinod Koul, Greg Kroah-Hartman
There are two poly_store, but one should have been poly_show.
|adma.c:4382:16: error: conflicting types for 'poly_store'
| static ssize_t poly_store(struct device_driver *dev, const char *buf,
| ^~~~~~~~~~
|adma.c:4363:16: note: previous definition of 'poly_store' was here
| static ssize_t poly_store(struct device_driver *dev, char *buf)
| ^~~~~~~~~~
Fixes: 13efe1a05384 ("dmaengine: ppc4xx: remove DRIVER_ATTR() usage")
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
---
drivers/dma/ppc4xx/adma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/ppc4xx/adma.c b/drivers/dma/ppc4xx/adma.c
index 4cf0d4d0cecf..25610286979f 100644
--- a/drivers/dma/ppc4xx/adma.c
+++ b/drivers/dma/ppc4xx/adma.c
@@ -4360,7 +4360,7 @@ static ssize_t enable_store(struct device_driver *dev, const char *buf,
}
static DRIVER_ATTR_RW(enable);
-static ssize_t poly_store(struct device_driver *dev, char *buf)
+static ssize_t poly_show(struct device_driver *dev, char *buf)
{
ssize_t size = 0;
u32 reg;
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-10-16 20:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-16 20:28 dmaengine: ppc4xx: fix off-by-one build failure Christian Lamparter
-- strict thread matches above, loose matches on Subject: below --
2018-10-16 14:40 Vinod Koul
2018-10-14 21:28 Christian Lamparter
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.