2021-09-02 19:59:42

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH 0/3] dma-buf: Add missing dependencies on DMA_SHARED_BUFFER

Hi Sumit, Christian,

This patch series adds missing dependencies on DMA_SHARED_BUFFER to
various options of DMA-BUF, and drops a bogus select.

As drivers/dma-buf/Kconfig contains interleaved options that select or
depend on DMA_SHARED_BUFFER, they can't be wrapped inside a big
"if DMA_SHARED_BUFFER / endif" block.

Thanks!

Geert Uytterhoeven (3):
dma-buf: DMABUF_MOVE_NOTIFY should depend on DMA_SHARED_BUFFER
dma-buf: DMABUF_DEBUG should depend on DMA_SHARED_BUFFER
dma-buf: DMABUF_SYSFS_STATS should depend on DMA_SHARED_BUFFER

drivers/dma-buf/Kconfig | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)

--
2.25.1

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


2021-09-03 17:11:43

by Sumit Semwal

[permalink] [raw]
Subject: Re: [PATCH 0/3] dma-buf: Add missing dependencies on DMA_SHARED_BUFFER

Hello Geert,

On Thu, 2 Sept 2021 at 18:19, Geert Uytterhoeven <[email protected]> wrote:
>
> Hi Sumit, Christian,
>
> This patch series adds missing dependencies on DMA_SHARED_BUFFER to
> various options of DMA-BUF, and drops a bogus select.
>
> As drivers/dma-buf/Kconfig contains interleaved options that select or
> depend on DMA_SHARED_BUFFER, they can't be wrapped inside a big
> "if DMA_SHARED_BUFFER / endif" block.

Thanks for catching these issues and the patch-set. LGTM!

I'll apply them over.
>
> Thanks!
>
> Geert Uytterhoeven (3):
> dma-buf: DMABUF_MOVE_NOTIFY should depend on DMA_SHARED_BUFFER
> dma-buf: DMABUF_DEBUG should depend on DMA_SHARED_BUFFER
> dma-buf: DMABUF_SYSFS_STATS should depend on DMA_SHARED_BUFFER
>
> drivers/dma-buf/Kconfig | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> --
> 2.25.1
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds

Best,
Sumit.