[PATCHv2] PCI: hotplug: compaq: fix X86 dependency

[PATCHv2] PCI: hotplug: compaq: fix X86 dependency

[Rosen Penev]

This driver needs X86_32, not X86. It does not compile on x86_64 because
of invalid assembly there.

Authored-by: Claude:Sonnet-4.6
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 v2: use X86_32
 drivers/pci/hotplug/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/hotplug/Kconfig b/drivers/pci/hotplug/Kconfig
index 891a81bac5e9..392cf75aeac9 100644
--- a/drivers/pci/hotplug/Kconfig
+++ b/drivers/pci/hotplug/Kconfig
@@ -20,7 +20,7 @@ if HOTPLUG_PCI
 
 config HOTPLUG_PCI_COMPAQ
 	tristate "Compaq PCI Hotplug driver"
-	depends on X86 && PCI_BIOS
+	depends on X86_32 && PCI_BIOS
 	help
 	  Say Y here if you have a motherboard with a Compaq PCI Hotplug
 	  controller.
-- 
2.54.0

Re: [PATCHv2] PCI: hotplug: compaq: fix X86 dependency

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The commit introduces a redundant X86_32 dependency and the commit message falsely claims it fixes an x86_64 compilation failure.
--

commit 2f32c0aa3739414d8be655780c2734e155b45ea8
Author: Rosen Penev <rosenp@gmail.com>
PCI: hotplug: compaq: fix X86 dependency

This commit restricts the HOTPLUG_PCI_COMPAQ Kconfig dependency from X86
to X86_32. The commit message indicates this is to prevent x86_64 build
failures caused by invalid assembly.

> This driver needs X86_32, not X86. It does not compile on x86_64 because
> of invalid assembly there.

[Severity: Medium]
Is this description accurate?

HOTPLUG_PCI_COMPAQ already depends on PCI_BIOS. In arch/x86/Kconfig,
config PCI_BIOS is defined with depends on X86_32.

Because of this dependency, the driver could not be selected or compiled
on x86_64 even before this patch. Could the commit message be updated to
reflect this?

> diff --git a/drivers/pci/hotplug/Kconfig b/drivers/pci/hotplug/Kconfig
> index 3207860b52e42..5074c0ac4ce18 100644
> --- a/drivers/pci/hotplug/Kconfig
> +++ b/drivers/pci/hotplug/Kconfig
> @@ -20,7 +20,7 @@ if HOTPLUG_PCI
>  
>  config HOTPLUG_PCI_COMPAQ
>  	tristate "Compaq PCI Hotplug driver"
> -	depends on X86 && PCI_BIOS
> +	depends on X86_32 && PCI_BIOS

[Severity: Medium]
Does this introduce a redundant check?

Since PCI_BIOS already restricts the build to X86_32, changing X86 to
X86_32 appears to provide no functional difference.

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

Re: [PATCHv2] PCI: hotplug: compaq: fix X86 dependency

[Rosen Penev]

On Mon, May 25, 2026 at 9:41 PM <sashiko-bot@kernel.org> wrote:
>
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Medium] The commit introduces a redundant X86_32 dependency and the commit message falsely claims it fixes an x86_64 compilation failure.
Which is accurate. This should not build on x86_64.
> --
>
> commit 2f32c0aa3739414d8be655780c2734e155b45ea8
> Author: Rosen Penev <rosenp@gmail.com>
> PCI: hotplug: compaq: fix X86 dependency
>
> This commit restricts the HOTPLUG_PCI_COMPAQ Kconfig dependency from X86
> to X86_32. The commit message indicates this is to prevent x86_64 build
> failures caused by invalid assembly.
>
> > This driver needs X86_32, not X86. It does not compile on x86_64 because
> > of invalid assembly there.
>
> [Severity: Medium]
> Is this description accurate?
>
> HOTPLUG_PCI_COMPAQ already depends on PCI_BIOS. In arch/x86/Kconfig,
> config PCI_BIOS is defined with depends on X86_32.
>
> Because of this dependency, the driver could not be selected or compiled
> on x86_64 even before this patch. Could the commit message be updated to
> reflect this?
>
> > diff --git a/drivers/pci/hotplug/Kconfig b/drivers/pci/hotplug/Kconfig
> > index 3207860b52e42..5074c0ac4ce18 100644
> > --- a/drivers/pci/hotplug/Kconfig
> > +++ b/drivers/pci/hotplug/Kconfig
> > @@ -20,7 +20,7 @@ if HOTPLUG_PCI
> >
> >  config HOTPLUG_PCI_COMPAQ
> >       tristate "Compaq PCI Hotplug driver"
> > -     depends on X86 && PCI_BIOS
> > +     depends on X86_32 && PCI_BIOS
>
> [Severity: Medium]
> Does this introduce a redundant check?
>
> Since PCI_BIOS already restricts the build to X86_32, changing X86 to
> X86_32 appears to provide no functional difference.
>
> --
> Sashiko AI review · https://sashiko.dev/#/patchset/20260526042614.1013867-1-rosenp@gmail.com?part=1