[meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

[Darren Hart]

On 10/10/14, 10:46, "Kamble, Nitin A" <nitin.a.kamble at intel.com> wrote:

>
>
>On 10/10/14, 10:31 AM, "Zanussi, Tom" <tom.zanussi at intel.com> wrote:
>
>>On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote:
>>> 
>>> On 10/10/14, 6:37 AM, "Zanussi, Tom" <tom.zanussi at intel.com> wrote:
>>> 
>>> >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kamble at intel.com wrote:
>>> >> From: Nitin A Kamble <nitin.a.kamble at intel.com>
>>> >> 
>>> >> The gmx500 graphics driver does not work on this BSP, but it takes
>>> >> the ownership of the graphics hardware at boot time, blocking other
>>> >> drivers from using the graphics hardware.
>>> >> 
>>> >> Fix the issue by blacklisting the gma500_gfx kernel module in the
>>>kmod
>>> >> configuration, so that it doesn't get loaded at boot time.

Does gma500 work anywhere? If not, perhaps it doesn't belong in the
intel-common BSPs. I really wanted it to be a reasonable fallback to EMGD,
but it just doesn't seem to work anywhere.

--
Darren

>>> >> 
>>> >
>>> >Is this really the right way to do this?  It will only work if
>>> >CONFIG_XXX=m.  If m->y at some point, the whole .bbappend becomes
>>> >useless at best.
>>> 
>>> That is why the gma500 is converted from y to m. And with the common
>>> kernel,
>>> It should be m anyway. If you have better way to solve you can share.
>>> 
>>
>>Why not a config fragment that turns it off for that BSP?
>>
>>The more general question is whether the common kernel mandates
>>everything be 'm' for this reason.
>>
>>Tom
>
>This will be going backwards. We are trying to consolidate kernel and
>BSPs. 
>And this suggested approach will make a separate kernel for this BSP.
>
>The common kernel has lot of drivers enabled. And very few of them will be
>actually used on any particular BSP. Not using kernel modules for these
>many
>drivers will be insane.
>
>Nitin
>
>
>>
>>> Nitin
>>> 
>>> 
>>> >
>>> >Tom
>>> >
>>> >> Fixes Bug:
>>> >> [YOCTO #6807]
>>> >> 
>>> >> Signed-off-by: Nitin A Kamble <nitin.a.kamble at intel.com>
>>> >> ---
>>> >>  meta-crownbay/recipes-kernel/kmod/kmod_git.bbappend | 3 +++
>>> >>  1 file changed, 3 insertions(+)
>>> >>  create mode 100644
>>>meta-crownbay/recipes-kernel/kmod/kmod_git.bbappend
>>> >> 
>>> >> diff --git a/meta-crownbay/recipes-kernel/kmod/kmod_git.bbappend
>>> >>b/meta-crownbay/recipes-kernel/kmod/kmod_git.bbappend
>>> >> new file mode 100644
>>> >> index 0000000..56d8fcb
>>> >> --- /dev/null
>>> >> +++ b/meta-crownbay/recipes-kernel/kmod/kmod_git.bbappend
>>> >> @@ -0,0 +1,3 @@
>>> >> +do_install_append () {
>>> >> +	echo "blacklist gma500_gfx" >
>>> >>${D}${sysconfdir}/modprobe.d/prohibit_gma500_gfx.conf
>>> >> +}
>>> >
>>> >
>>> 
>>
>>
>
>


-- 
Darren Hart
Intel Open Source Technology Center