On 04/23/2012 11:31 AM, Bodke, Kishore K wrote: > >>> KERNEL_FEATURES_append_cedartrail += " cfg/smp.scc" >>> +KERNEL_FEATURES_append_cedartrail += " cfg/drm-cdvpvr.scc" >>> +KERNEL_FEATURES_append_cedartrail += " bsp/cedartrail/cedartrail-pvr- >> merge.scc" >>> +KERNEL_FEATURES_append_cedartrail += " cfg/efi-ext.scc" >> >> >> I suggest moving all of this into the cedartrail.scc file in the >> linux-yocto meta branch. These are all core bits required for the BSP, >> they don't need to be easily changed from recipe-space. >> >> The exception would be the pvr bits. You can make a single >> cedartrail.scc that supports both configs (with and without pvr) and >> only modify the pvr one here in recipe space - the rest belongs in >> linux-yocto/meta > > Hi Darren, > > All the above are specific to pvr only.
You can't use EFI to boot the system with the pvr driver? -- Darren > (cfg/drm-cdvpvr.scc , bsp/cedartrail/cedartrail-pvr-merge.scc, and > cfg/efi-ext.scc) > These are not needed for without pvr versions of the BSP. > I think if I move them to Cedartrail.scc, these will get pulled in for no-pvr > version of the build and may cause the build to fail. > > Thanks > Kishore. > > >> >> >>> +COMPATIBLE_MACHINE_cedartrail-nopvr = "cedartrail" >>> +KMACHINE_cedartrail-nopvr = "cedartrail" >>> +KERNEL_FEATURES_append_cedartrail-nopvr += " cfg/smp.scc" >> >> Same here. >> >> -- >> Darren >> >> -- >> Darren Hart >> Intel Open Source Technology Center >> Yocto Project - Linux Kernel -- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto