Re: [PATCH V6 5/5] mmc: host: sdhci-pci: Add Genesys Logic GL975x support

[Adrian Hunter]

On 28/08/19 4:47 PM, Michael K. Johnson wrote:
> On Wed, Aug 28, 2019 at 04:13:03PM +0300, Adrian Hunter wrote:
>> On 27/08/19 3:33 AM, Ben Chuang wrote:
>> Looks good, one minor comment
> ...
>>> +#ifdef CONFIG_MMC_SDHCI_IO_ACCESSORS
>>
>> Arguably CONFIG_MMC_SDHCI_IO_ACCESSORS needs to be removed altogether. i.e.
>> making the accessors always available.  So for now, I'd prefer to select
>> MMC_SDHCI_IO_ACCESSORS:
>>
>> diff --git a/drivers/mmc/host/Kconfig b/drivers/mmc/host/Kconfig
>> index 14d89a108edd..c3bd967d8a1a 100644
>> --- a/drivers/mmc/host/Kconfig
>> +++ b/drivers/mmc/host/Kconfig
>> @@ -94,6 +94,7 @@ config MMC_SDHCI_PCI
>>  	depends on MMC_SDHCI && PCI
>>  	select MMC_CQHCI
>>  	select IOSF_MBI if X86
>> +	select MMC_SDHCI_IO_ACCESSORS
>>  	help
>>  	  This selects the PCI Secure Digital Host Controller Interface.
>>  	  Most controllers found today are PCI devices.
> 
> Unless I'm missing something, this seems like a separate patch;
> are you asking for it first, as a predecessor to the GLI patch?
> 

No, it is fine with this patch.