| Message ID | 20181223232521.11320-10-okaya@kernel.org (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | Specify CONFIG_PCI dependency explicitly | expand |
On Mon, Dec 24, 2018 at 12:26 AM Sinan Kaya <okaya@kernel.org> wrote: > > Select IOSF_MBI only when PCI is set. > > Signed-off-by: Sinan Kaya <okaya@kernel.org> > --- > drivers/mmc/host/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig > index e26b8145efb3..4c5f037b246d 100644 > --- a/drivers/mmc/host/Kconfig > +++ b/drivers/mmc/host/Kconfig > @@ -117,7 +117,7 @@ config MMC_RICOH_MMC > config MMC_SDHCI_ACPI > tristate "SDHCI support for ACPI enumerated SDHCI controllers" > depends on MMC_SDHCI && ACPI It should depend on PCI as well, shouldn't it? As it stands with the patch applied, randconfig can still select it if PCI is unset AFAICS, can't it? > - select IOSF_MBI if X86 > + select IOSF_MBI if (X86 && PCI) > help > This selects support for ACPI enumerated SDHCI controllers, > identified by ACPI Compatibility ID PNP0D40 or specific > --
On Wed, Dec 26, 2018 at 12:41 PM Rafael J. Wysocki <rafael@kernel.org> wrote: > > On Mon, Dec 24, 2018 at 12:26 AM Sinan Kaya <okaya@kernel.org> wrote: > > > > Select IOSF_MBI only when PCI is set. > > > > Signed-off-by: Sinan Kaya <okaya@kernel.org> > > --- > > drivers/mmc/host/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig > > index e26b8145efb3..4c5f037b246d 100644 > > --- a/drivers/mmc/host/Kconfig > > +++ b/drivers/mmc/host/Kconfig > > @@ -117,7 +117,7 @@ config MMC_RICOH_MMC > > config MMC_SDHCI_ACPI > > tristate "SDHCI support for ACPI enumerated SDHCI controllers" > > depends on MMC_SDHCI && ACPI > > It should depend on PCI as well, shouldn't it? > I don't think so. The PCI dependency is at IOSF_MBI code so code should select IOSF_MBI only when x86 and pci is defined as below. This driver does not depend on PCI. > As it stands with the patch applied, randconfig can still select it if > PCI is unset AFAICS, can't it? True, we also should be careful about separating ACPI from PCI. Someone should be able to define a sdhci acpi object without PCI. What do you think about this? > > > - select IOSF_MBI if X86 > > + select IOSF_MBI if (X86 && PCI) > > help > > This selects support for ACPI enumerated SDHCI controllers, > > identified by ACPI Compatibility ID PNP0D40 or specific > > --
On Sun, Dec 30, 2018 at 7:57 PM Sinan Kaya <Okaya@kernel.org> wrote: > > On Wed, Dec 26, 2018 at 12:41 PM Rafael J. Wysocki <rafael@kernel.org> wrote: > > > > On Mon, Dec 24, 2018 at 12:26 AM Sinan Kaya <okaya@kernel.org> wrote: > > > > > > Select IOSF_MBI only when PCI is set. > > > > > > Signed-off-by: Sinan Kaya <okaya@kernel.org> > > > --- > > > drivers/mmc/host/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig > > > index e26b8145efb3..4c5f037b246d 100644 > > > --- a/drivers/mmc/host/Kconfig > > > +++ b/drivers/mmc/host/Kconfig > > > @@ -117,7 +117,7 @@ config MMC_RICOH_MMC > > > config MMC_SDHCI_ACPI > > > tristate "SDHCI support for ACPI enumerated SDHCI controllers" > > > depends on MMC_SDHCI && ACPI > > > > It should depend on PCI as well, shouldn't it? > > > > I don't think so. The PCI dependency is at IOSF_MBI code so code > should select IOSF_MBI only when x86 and pci is defined as below. > > This driver does not depend on PCI. Are you sure? Do you know any systems in which to use it without PCI?
On 31/12/18 11:40 AM, Rafael J. Wysocki wrote: > On Sun, Dec 30, 2018 at 7:57 PM Sinan Kaya <Okaya@kernel.org> wrote: >> >> On Wed, Dec 26, 2018 at 12:41 PM Rafael J. Wysocki <rafael@kernel.org> wrote: >>> >>> On Mon, Dec 24, 2018 at 12:26 AM Sinan Kaya <okaya@kernel.org> wrote: >>>> >>>> Select IOSF_MBI only when PCI is set. >>>> >>>> Signed-off-by: Sinan Kaya <okaya@kernel.org> >>>> --- >>>> drivers/mmc/host/Kconfig | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig >>>> index e26b8145efb3..4c5f037b246d 100644 >>>> --- a/drivers/mmc/host/Kconfig >>>> +++ b/drivers/mmc/host/Kconfig >>>> @@ -117,7 +117,7 @@ config MMC_RICOH_MMC >>>> config MMC_SDHCI_ACPI >>>> tristate "SDHCI support for ACPI enumerated SDHCI controllers" >>>> depends on MMC_SDHCI && ACPI >>> >>> It should depend on PCI as well, shouldn't it? >>> >> >> I don't think so. The PCI dependency is at IOSF_MBI code so code >> should select IOSF_MBI only when x86 and pci is defined as below. >> >> This driver does not depend on PCI. > > Are you sure? > > Do you know any systems in which to use it without PCI? > Qualcomm use it and I don't know if those systems have PCI. AMD use it but probably also have PCI. And yes Intel systems (just BYT and CHT IIRC) that use this driver will always have PCI.
On Wed, Jan 2, 2019 at 2:04 PM Adrian Hunter <adrian.hunter@intel.com> wrote: > > On 31/12/18 11:40 AM, Rafael J. Wysocki wrote: > > On Sun, Dec 30, 2018 at 7:57 PM Sinan Kaya <Okaya@kernel.org> wrote: > >> > >> On Wed, Dec 26, 2018 at 12:41 PM Rafael J. Wysocki <rafael@kernel.org> wrote: > >>> > >>> On Mon, Dec 24, 2018 at 12:26 AM Sinan Kaya <okaya@kernel.org> wrote: > >>>> > >>>> Select IOSF_MBI only when PCI is set. > >>>> > >>>> Signed-off-by: Sinan Kaya <okaya@kernel.org> > >>>> --- > >>>> drivers/mmc/host/Kconfig | 2 +- > >>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>> > >>>> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig > >>>> index e26b8145efb3..4c5f037b246d 100644 > >>>> --- a/drivers/mmc/host/Kconfig > >>>> +++ b/drivers/mmc/host/Kconfig > >>>> @@ -117,7 +117,7 @@ config MMC_RICOH_MMC > >>>> config MMC_SDHCI_ACPI > >>>> tristate "SDHCI support for ACPI enumerated SDHCI controllers" > >>>> depends on MMC_SDHCI && ACPI > >>> > >>> It should depend on PCI as well, shouldn't it? > >>> > >> > >> I don't think so. The PCI dependency is at IOSF_MBI code so code > >> should select IOSF_MBI only when x86 and pci is defined as below. > >> > >> This driver does not depend on PCI. > > > > Are you sure? > > > > Do you know any systems in which to use it without PCI? > > > > Qualcomm use it and I don't know if those systems have PCI. > AMD use it but probably also have PCI. And yes Intel systems (just BYT and > CHT IIRC) that use this driver will always have PCI. QCOM doesn't use PCI for SDHC.
diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig index e26b8145efb3..4c5f037b246d 100644 --- a/drivers/mmc/host/Kconfig +++ b/drivers/mmc/host/Kconfig @@ -117,7 +117,7 @@ config MMC_RICOH_MMC config MMC_SDHCI_ACPI tristate "SDHCI support for ACPI enumerated SDHCI controllers" depends on MMC_SDHCI && ACPI - select IOSF_MBI if X86 + select IOSF_MBI if (X86 && PCI) help This selects support for ACPI enumerated SDHCI controllers, identified by ACPI Compatibility ID PNP0D40 or specific
Select IOSF_MBI only when PCI is set. Signed-off-by: Sinan Kaya <okaya@kernel.org> --- drivers/mmc/host/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)