On 09/29/2016 12:40 AM, Bruce Ashfield wrote:
On 09/28/2016 04:04 PM, Alejandro Hernandez wrote:


On 09/28/2016 01:45 PM, Bruce Ashfield wrote:
On 09/27/2016 12:48 PM, Alejandro Hernandez wrote:
Signed-off-by: Alejandro Hernandez <alejandro.hernan...@linux.intel.com>
---
  bsp/common-pc-64/common-pc-64.scc | 1 +
  1 file changed, 1 insertion(+)

diff --git a/bsp/common-pc-64/common-pc-64.scc
b/bsp/common-pc-64/common-pc-64.scc
index 5737a83..f5a3076 100644
--- a/bsp/common-pc-64/common-pc-64.scc
+++ b/bsp/common-pc-64/common-pc-64.scc
@@ -10,6 +10,7 @@ include cfg/x86_64.scc
  include cfg/amd.scc
  include cfg/intel.scc
  include features/pci/pci.scc
+include features/mmc/mmc-sdhci.scc

Rather than including this here, and inflicting the extra build/modules
on everyone that uses the common-pc baseline, wouldn't it be better if
the BSPs that need this support included it directly ?

On that topic, what sorts of platforms and use cases need this ? If it
is a broad set of platforms, then adding it in common-pc makes sense ..

Bruce

I agree, in this case genericx86 and genericx86-64 should include mmc
support, is there a better place to add this for this scenario?

Hmmm. That is the problem, genericx86* map directly to common-pc, so
they are fundamentally the same thing.

The only way to make it optional for boards like that is to use
KERNEL_FEATURES_genericx86_append .. and then list the features.
We could do this if necessary, I wouldn't mind.

But we don't necessarily have to go that way, if we just cover what
mmc devices are being used, and what use case they are covering
(i.e. are these common parts that are being used for storage ?
boot ? .. something else ?).

In this case, trying to install the system on an SD card using either genericx86
or genericx86-64 fails since the SD card is not detected, on a booted system
the SD card is unavailable to use as storage as well.

While I believe that using it as storage is important, I think that installing the system
on an SD card is a feature that should be provided.

Since as I said, if they are common and the use case is valid, we
can just turn them on in common-pc.

Bruce


Alejandro

  include features/usb/ehci-hcd.scc
  include features/usb/uhci-hcd.scc
  include features/usb/ohci-hcd.scc





--
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to