Re: [PATCH v2] coco/tdx-host: add Kconfig description

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] kbuild: commit message claims to add Kconfig description but patch does not
--

commit 424217cd6059405518820db0188355121477942b
Author: Dmitry Voytik <voytikd@gmail.com>

coco/tdx-host: add Kconfig description

This commit updates the default value of the TDX_HOST_SERVICES Kconfig option
from 'm' to 'KVM_INTEL'. It is intended to fix an issue where the driver
always compiles as a module despite the user configuration.

> coco/tdx-host: add Kconfig description

> diff --git a/drivers/virt/coco/tdx-host/Kconfig b/drivers/virt/coco/tdx-host/Kconfig
> index 57d0c01a43577..fe018e00822b3 100644
> --- a/drivers/virt/coco/tdx-host/Kconfig
> +++ b/drivers/virt/coco/tdx-host/Kconfig
> @@ -3,4 +3,4 @@ config TDX_HOST_SERVICES
>  	depends on INTEL_TDX_HOST
>  	select FW_LOADER
>  	select FW_UPLOAD
> -	default m
> +	default KVM_INTEL

[Severity: Medium]
The commit subject claims to add a Kconfig description, which typically implies
adding a prompt string (such as tristate "Intel TDX host services") to make
the symbol visible in menuconfig. However, the patch only modifies the default
value and leaves the symbol hidden.

Was the hunk adding the description accidentally dropped from this version of
the patch, or should the commit subject be updated to reflect only the default
value change?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261012175348.1256363-1-voytikd@gmail.com?part=1