Re: [PATCH] atyfb: Fix bootup hangs on sparc64.

["Fabio M. Di NItto" <fabbione@fabbione.net>]

On 01/10/2011 11:03 PM, Sam Ravnborg wrote:
> On Mon, Jan 10, 2011 at 01:17:05PM -0800, David Miller wrote:
>> From: Sam Ravnborg <sam@ravnborg.org>
>> Date: Mon, 10 Jan 2011 20:32:51 +0100
>>
>>> If I comment out the call to resume_pll() then I no longer see garbled output.
>>
>> Yeah, that just makes it simply not program the PLL at all.
>>
>> I just booted Linus's current tree on my Samsung 216BW and it works
>> just fine.  I'll try a few other monitors I have.
>>
>> If you have the time can you try these two tests?
>>
>> 1) In atyfb_setup_sparc(), comment out the whole block of code protected
>>    by:
>>
>> 	if (IS_XL(pdev->device)) {
>> 		/*
>> 		 * Fix PROMs idea of MEM_CNTL settings...
>> 		 */
>>
>> 2) Force the:
>>
>> 	dp = pci_device_to_OF_node(pdev);
>> 	if (dp = of_console_device) {
>>
>>    test to fail.
>>
>> I've always been suspect of all of the special mode and MEM_CNTRL handling
>> we have here for Sparc.  If disabling it fixes stuff, I'd happily see this
>> code get tossed out completely.
> 
> I have tried 1) and 2)
> Independent and together.
> 
> The result was the same in all cases.
> 
> I have also tried with a LG Flatron W2425T Monotor - the same result.
> 
> On this box I have never had any graphical application running - so
> we should also consider that this is a HW bug on my video card.
> I found another SB100 on ebay wich I have placed a bid on - maybe I
> am lucky to get it.

No panic. I'll try on my SB100 too. I have one original SUN monitor to
test with and a bunch of others. It will just take me a bit more time
due to the headless setup I have at home (need to shuffle stuff around
to make space) :) but we will get there.

Fabio