Aguirre Rodriguez, Sergio Alberto wrote:
> From: [email protected] [[email protected]] On 
> Behalf Of Angelo Arrifano [[email protected]]
> Sent: Tuesday, September 22, 2009 12:21 PM
>> To: Tony Lindgren
>> Cc: Alistair Buxton; [email protected]
>> Subject: Re: [APPLIED] [PATCH] omapfb: Reorder Register_framebuffer call
>>
>> Tony Lindgren wrote:
>>> * Jan Blunck <[email protected]> [090922 07:59]:
>>>> On Tue, Sep 22, 2009 at 3:31 AM, Tony Lindgren <[email protected]> wrote:
>>>>> This patch has been applied to the linux-omap
>>>>> by youw fwiendly patch wobot.
>>>>>
>>>>> Branch in linux-omap: omap-fixes
>>>>>
>>>>> Initial commit ID (Likely to change): 
>>>>> 9aef1066fb5ca8506068eaab1c552ecca4c85475
>>>>>
>>>>> PatchWorks
>>>>> http://patchwork.kernel.org/patch/47089/
>>>>>
>>> Added back the original Cc's that were dropped from the linux-omap
>>> commit message.
>>>
>>>> Is it actually safe to do this? The framebuffer can be used directly
>>>> after it is registered. In this case it would mean it is used before
>>>> it is even fully initialized (set_fb_var(), set_fb_fix(), ... are
>>>> being called).
>>> Good point, dropping the patch.
>>>
>>> Also, let's let Tomi Valkeinen deal with queueing up the omap fb code.
>>> I can then merge Tomi's branck into linux-omap master branch as needed.
>>>
>>> Regards,
>>>
>>> Tony
>> Hello,
>>
>> This patch doesn't work for us at linwizard project (omap850). The
>> commit 537a1bf059fa312355696fa6db80726e655e7f17 makes the HTC Wizard
>> crash at boot due to usage of uninitialized mutex. Applying the referred
>> patch prevents the crash but the LCD screen goes black and doesn't show
>> any data.
> 
> Angelo,
> 
> 
> Some people gave his ACK, because it was booting. and also i kept it since 
> some framebuffer tests in my 3430SDP passed (i.e. it was showing the intended 
> data.)

Yeah, sorry for the late reply. I was going to add a test-by reply to
the patch but I didn't have much info to offer other than it doesn't
work for me. At the time USB was also not working so I really didn't
have a way to debug in case it was asked.

> 
> However, didn't show the Tux during bootup. Isn't the same case as you?
No tux, no text, just a black screen.

> 
> Anyways, Thanks for sharing your results... I'll rework this patch and resend 
> once i'm done.

Thanks,
- Angelo
> 
> Regards,
> Sergio
>> - Angelo
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to