[PATCH v2] build: make gpudev optional
Bruce Richardson
bruce.richardson at intel.com
Wed Nov 17 15:10:53 CET 2021
On Wed, Nov 17, 2021 at 09:39:01PM +0000, eagostini at nvidia.com wrote:
> From: Elena Agostini <eagostini at nvidia.com>
>
> This library can be made optional.
> drivers/gpu and app/test-gpudev depend on this library,
> so there is a check for dependencies.
>
> Signed-off-by: Elena Agostini <eagostini at nvidia.com>
> ---
> app/test-gpudev/meson.build | 1 +
> doc/guides/prog_guide/gpudev.rst | 1 +
> drivers/gpu/meson.build | 1 +
> lib/meson.build | 1 +
> 4 files changed, 4 insertions(+)
>
> diff --git a/app/test-gpudev/meson.build b/app/test-gpudev/meson.build
> index 17bdef3646..02286d3a6a 100644
> --- a/app/test-gpudev/meson.build
> +++ b/app/test-gpudev/meson.build
> @@ -3,3 +3,4 @@
>
> sources = files('main.c')
> deps = ['gpudev', 'ethdev']
> +
This file and "drivers/gpu/meson.build" file can be removed from the patch,
since adding a blank line is unnecessary and is not related to the actual
patch changes. When preparing a patch, I'd suggest always double checking
that all changes introduced in the patch are intentional, and relevant to
the patch itself. If these whitespace fixups are intentional needed,
they should be sent as a separate patch just to make reviewing easier and
ensure each patch has only one logical change in it.
Thanks,
/Bruce
More information about the dev
mailing list