On 05/29/2013 08:57 AM, WANG Chao wrote:
> On 05/29/13 at 10:45am, Dave Young wrote:
>> On 05/28/2013 08:09 PM, Harald Hoyer wrote:
>>> On 05/24/2013 10:22 AM, WANG Chao wrote:
>>>> Now only 50plymouth module can pull in 50drm and that's very limited.
>>>> Because KMS capable drm drivers are much needed in initrd to have a
>>>> frame buffer console on the screen.
>>>>
>>>> Particularly, when building kdump initrd, plymouth module is omitted,
>>>> but drm is necessary to setup a working screen for one sitting in front
>>>> of a monitor.
>>>>
>>>> Signed-off-by: WANG Chao <chaowang@xxxxxxxxxx>
>>>> ---
>>>> modules.d/50drm/module-setup.sh | 4 ++++
>>>> 1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/modules.d/50drm/module-setup.sh b/modules.d/50drm/module-setup.sh
>>>> index bf33c5b..648e8f4 100644
>>>> --- a/modules.d/50drm/module-setup.sh
>>>> +++ b/modules.d/50drm/module-setup.sh
>>>> @@ -3,6 +3,10 @@
>>>> # ex: ts=8 sw=4 sts=4 et filetype=sh
>>>>
>>>> check() {
>>>> + if [[ $hostonly ]] && [ -d /sys/module/drm/drivers ]; then
>>>> + return 0
>>>> + fi
>>>> +
>>>> return 255
>>>> }
>>>>
>>>>
>>>
>>> Hmm, why not let the kdump module depend on drm?
>>
>> If you do not like it we can consider add it to kdump module..
>
> So now if I build initrd w/o plymouth, I will not have frame buffer
> console in initrd. That looks more like a general issue to me.
>
> WANG Chao
Do we really need a fb console?
If there is a valid reason, I will change "return 255" to "return 0".
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
