* [U-Boot] [PATCH v1] mmc: pci: only compile on platforms that need it
@ 2017-02-13 14:56 ` Andy Shevchenko
2017-02-15 4:26 ` Jaehoon Chung
0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2017-02-13 14:56 UTC (permalink / raw)
To: u-boot
From: Felipe Balbi <felipe.balbi@linux.intel.com>
We should only compile pci_mmc.c on platforms that actually need
this. Some platforms are using generic sdhci through driver model.
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
drivers/mmc/Makefile | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
index 49413dfaf1..26db92e582 100644
--- a/drivers/mmc/Makefile
+++ b/drivers/mmc/Makefile
@@ -36,7 +36,9 @@ obj-$(CONFIG_MVEBU_MMC) += mvebu_mmc.o
obj-$(CONFIG_MMC_OMAP_HS) += omap_hsmmc.o
obj-$(CONFIG_MMC_MXC) += mxcmmc.o
obj-$(CONFIG_MMC_MXS) += mxsmmc.o
-obj-$(CONFIG_X86) += pci_mmc.o
+obj-$(CONFIG_INTEL_BAYTRAIL) += pci_mmc.o
+obj-$(CONFIG_INTEL_QUARK) += pci_mmc.o
+obj-$(CONFIG_INTEL_QUEENSBAY) += pci_mmc.o
obj-$(CONFIG_PXA_MMC_GENERIC) += pxa_mmc_gen.o
obj-$(CONFIG_SUPPORT_EMMC_RPMB) += rpmb.o
obj-$(CONFIG_S3C_SDI) += s3c_sdi.o
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH v1] mmc: pci: only compile on platforms that need it
2017-02-13 14:56 ` [U-Boot] [PATCH v1] mmc: pci: only compile on platforms that need it Andy Shevchenko
@ 2017-02-15 4:26 ` Jaehoon Chung
2017-02-15 7:13 ` Felipe Balbi
0 siblings, 1 reply; 3+ messages in thread
From: Jaehoon Chung @ 2017-02-15 4:26 UTC (permalink / raw)
To: u-boot
Hi,
On 02/13/2017 11:56 PM, Andy Shevchenko wrote:
> From: Felipe Balbi <felipe.balbi@linux.intel.com>
>
> We should only compile pci_mmc.c on platforms that actually need
> this. Some platforms are using generic sdhci through driver model.
>
> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> drivers/mmc/Makefile | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
> index 49413dfaf1..26db92e582 100644
> --- a/drivers/mmc/Makefile
> +++ b/drivers/mmc/Makefile
> @@ -36,7 +36,9 @@ obj-$(CONFIG_MVEBU_MMC) += mvebu_mmc.o
> obj-$(CONFIG_MMC_OMAP_HS) += omap_hsmmc.o
> obj-$(CONFIG_MMC_MXC) += mxcmmc.o
> obj-$(CONFIG_MMC_MXS) += mxsmmc.o
> -obj-$(CONFIG_X86) += pci_mmc.o
> +obj-$(CONFIG_INTEL_BAYTRAIL) += pci_mmc.o
> +obj-$(CONFIG_INTEL_QUARK) += pci_mmc.o
> +obj-$(CONFIG_INTEL_QUEENSBAY) += pci_mmc.o
How about adding new CONFIG_MMC_X86_PCI for this.
obj-$(CONFIG_MMC_X86_PCI) += pci_mmc.o
Of course, you need to modify Kconfig file.
Best Regards,
Jaehoon Chung
> obj-$(CONFIG_PXA_MMC_GENERIC) += pxa_mmc_gen.o
> obj-$(CONFIG_SUPPORT_EMMC_RPMB) += rpmb.o
> obj-$(CONFIG_S3C_SDI) += s3c_sdi.o
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH v1] mmc: pci: only compile on platforms that need it
2017-02-15 4:26 ` Jaehoon Chung
@ 2017-02-15 7:13 ` Felipe Balbi
0 siblings, 0 replies; 3+ messages in thread
From: Felipe Balbi @ 2017-02-15 7:13 UTC (permalink / raw)
To: u-boot
Hi,
Jaehoon Chung <jh80.chung@samsung.com> writes:
> On 02/13/2017 11:56 PM, Andy Shevchenko wrote:
>> From: Felipe Balbi <felipe.balbi@linux.intel.com>
>>
>> We should only compile pci_mmc.c on platforms that actually need
>> this. Some platforms are using generic sdhci through driver model.
>>
>> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
>> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>> ---
>> drivers/mmc/Makefile | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/mmc/Makefile b/drivers/mmc/Makefile
>> index 49413dfaf1..26db92e582 100644
>> --- a/drivers/mmc/Makefile
>> +++ b/drivers/mmc/Makefile
>> @@ -36,7 +36,9 @@ obj-$(CONFIG_MVEBU_MMC) += mvebu_mmc.o
>> obj-$(CONFIG_MMC_OMAP_HS) += omap_hsmmc.o
>> obj-$(CONFIG_MMC_MXC) += mxcmmc.o
>> obj-$(CONFIG_MMC_MXS) += mxsmmc.o
>> -obj-$(CONFIG_X86) += pci_mmc.o
>> +obj-$(CONFIG_INTEL_BAYTRAIL) += pci_mmc.o
>> +obj-$(CONFIG_INTEL_QUARK) += pci_mmc.o
>> +obj-$(CONFIG_INTEL_QUEENSBAY) += pci_mmc.o
>
> How about adding new CONFIG_MMC_X86_PCI for this.
>
> obj-$(CONFIG_MMC_X86_PCI) += pci_mmc.o
>
> Of course, you need to modify Kconfig file.
doable, no problems for me.
--
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170215/2dff4c44/attachment.sig>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-02-15 7:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20170213145642epcas5p203104bcafa169c40915ef98f691d0863@epcas5p2.samsung.com>
2017-02-13 14:56 ` [U-Boot] [PATCH v1] mmc: pci: only compile on platforms that need it Andy Shevchenko
2017-02-15 4:26 ` Jaehoon Chung
2017-02-15 7:13 ` Felipe Balbi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox