* [PATCH] iio: buffer-dmaengine: make use of the 'struct device *' argument
@ 2024-01-09 14:03 Nuno Sa
2024-01-09 17:56 ` David Lechner
0 siblings, 1 reply; 3+ messages in thread
From: Nuno Sa @ 2024-01-09 14:03 UTC (permalink / raw)
To: linux-iio; +Cc: Jonathan Cameron, Lars-Peter Clausen
Respect the @dev argument in devm_iio_dmaengine_buffer_setup() and bind the
IIO DMA buffer lifetime to that device.
For the only user of this function, the IIO parent device is the
struct device being passed to the API so no real fix in here (just
consistency with other IIO APIs).
Signed-off-by: Nuno Sa <nuno.sa@analog.com>
---
drivers/iio/buffer/industrialio-buffer-dmaengine.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/iio/buffer/industrialio-buffer-dmaengine.c b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
index 5f85ba38e6f6..45fe7d0d42ee 100644
--- a/drivers/iio/buffer/industrialio-buffer-dmaengine.c
+++ b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
@@ -279,8 +279,7 @@ int devm_iio_dmaengine_buffer_setup(struct device *dev,
{
struct iio_buffer *buffer;
- buffer = devm_iio_dmaengine_buffer_alloc(indio_dev->dev.parent,
- channel);
+ buffer = devm_iio_dmaengine_buffer_alloc(dev, channel);
if (IS_ERR(buffer))
return PTR_ERR(buffer);
---
base-commit: 3f4525f924e21d4f532517b17a20ffa5df7c0db7
change-id: 20240109-dmaengine_use_device-853f196ec78f
--
Thanks!
- Nuno Sá
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] iio: buffer-dmaengine: make use of the 'struct device *' argument
2024-01-09 14:03 [PATCH] iio: buffer-dmaengine: make use of the 'struct device *' argument Nuno Sa
@ 2024-01-09 17:56 ` David Lechner
2024-01-13 15:28 ` Jonathan Cameron
0 siblings, 1 reply; 3+ messages in thread
From: David Lechner @ 2024-01-09 17:56 UTC (permalink / raw)
To: Nuno Sa; +Cc: linux-iio, Jonathan Cameron, Lars-Peter Clausen
On Tue, Jan 9, 2024 at 9:47 AM Nuno Sa <nuno.sa@analog.com> wrote:
>
> Respect the @dev argument in devm_iio_dmaengine_buffer_setup() and bind the
> IIO DMA buffer lifetime to that device.
>
> For the only user of this function, the IIO parent device is the
> struct device being passed to the API so no real fix in here (just
> consistency with other IIO APIs).
>
> Signed-off-by: Nuno Sa <nuno.sa@analog.com>
> ---
> drivers/iio/buffer/industrialio-buffer-dmaengine.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/iio/buffer/industrialio-buffer-dmaengine.c b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> index 5f85ba38e6f6..45fe7d0d42ee 100644
> --- a/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> +++ b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> @@ -279,8 +279,7 @@ int devm_iio_dmaengine_buffer_setup(struct device *dev,
> {
> struct iio_buffer *buffer;
>
> - buffer = devm_iio_dmaengine_buffer_alloc(indio_dev->dev.parent,
> - channel);
> + buffer = devm_iio_dmaengine_buffer_alloc(dev, channel);
> if (IS_ERR(buffer))
> return PTR_ERR(buffer);
>
>
> ---
> base-commit: 3f4525f924e21d4f532517b17a20ffa5df7c0db7
> change-id: 20240109-dmaengine_use_device-853f196ec78f
> --
>
> Thanks!
> - Nuno Sá
>
>
I noticed this too but didn't get around to fixing it yet. ;-)
Reviewed-by: David Lechner <dlechner@baylibre.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] iio: buffer-dmaengine: make use of the 'struct device *' argument
2024-01-09 17:56 ` David Lechner
@ 2024-01-13 15:28 ` Jonathan Cameron
0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Cameron @ 2024-01-13 15:28 UTC (permalink / raw)
To: David Lechner; +Cc: Nuno Sa, linux-iio, Lars-Peter Clausen
On Tue, 9 Jan 2024 11:56:47 -0600
David Lechner <dlechner@baylibre.com> wrote:
> On Tue, Jan 9, 2024 at 9:47 AM Nuno Sa <nuno.sa@analog.com> wrote:
> >
> > Respect the @dev argument in devm_iio_dmaengine_buffer_setup() and bind the
> > IIO DMA buffer lifetime to that device.
> >
> > For the only user of this function, the IIO parent device is the
> > struct device being passed to the API so no real fix in here (just
> > consistency with other IIO APIs).
> >
> > Signed-off-by: Nuno Sa <nuno.sa@analog.com>
> > ---
> > drivers/iio/buffer/industrialio-buffer-dmaengine.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/drivers/iio/buffer/industrialio-buffer-dmaengine.c b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> > index 5f85ba38e6f6..45fe7d0d42ee 100644
> > --- a/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> > +++ b/drivers/iio/buffer/industrialio-buffer-dmaengine.c
> > @@ -279,8 +279,7 @@ int devm_iio_dmaengine_buffer_setup(struct device *dev,
> > {
> > struct iio_buffer *buffer;
> >
> > - buffer = devm_iio_dmaengine_buffer_alloc(indio_dev->dev.parent,
> > - channel);
> > + buffer = devm_iio_dmaengine_buffer_alloc(dev, channel);
> > if (IS_ERR(buffer))
> > return PTR_ERR(buffer);
> >
> >
> > ---
> > base-commit: 3f4525f924e21d4f532517b17a20ffa5df7c0db7
> > change-id: 20240109-dmaengine_use_device-853f196ec78f
> > --
> >
> > Thanks!
> > - Nuno Sá
> >
> >
>
> I noticed this too but didn't get around to fixing it yet. ;-)
>
> Reviewed-by: David Lechner <dlechner@baylibre.com>
Applied
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-01-13 15:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-09 14:03 [PATCH] iio: buffer-dmaengine: make use of the 'struct device *' argument Nuno Sa
2024-01-09 17:56 ` David Lechner
2024-01-13 15:28 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox