Re: drm/radeon/kms: fix up LVDS handling on macs (v2)

[Alex Deucher]

2010/1/14 Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>:
> On Thu, 2010-01-14 at 11:09 +0100, Michel Dänzer wrote:
>> I have looked at radeonfb for a patch I posted a while ago to expose a
>> backlight class device with KMS, and I can only see what you describe
>> above affecting the LVDS_GEN_CNTL backlight level, not BLON or
>> BL_MOD_EN. Am I missing something?
>
> No, you aren't, it's radeonfb who is :-) I never quite got that part
> right. I was somewhat hoping several times that Hui and then Alex would
> manage to dig some info from ATI about what the right method is to drive
> those panels I think :-)
>

I've tried, but we are coming up on 10 years ago for some of these
chips so they aren't exactly fresh in people's minds and most of the
code is in long term storage at this point.  The problem is the
sequence is oem specific, even on the PC side there are cases were we
probably aren't doing it quite right.  The current non-mac code in kms
is the generic code that the PC bios team gave me.

Alex

> But my observation on those HW was definitely that when inverted,
> disabling the modulation was also turning the panel to full bright on,
> rather than turning it off.
>
>> Without my patch, KMS doesn't change the backlight level at all, so
>> assuming the firmware manages to enable the backlight, it should stay
>> like that.
>
> Your patch might make things better than they are today, I'm not arguing
> there :-) I'm just saying that now might be a good time to try to sort
> out exactly what should be done in there :-)
>
> Cheers,
> Ben.
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/