Re: Unblock xserver-xorg-video-geode
2010/9/18 Martin-Éric Racine <email@example.com>:
> 2010/9/3 Martin-Éric Racine <firstname.lastname@example.org>:
>> On Fri, Sep 3, 2010 at 11:46 AM, Julien Cristau <email@example.com> wrote:
>>> On Fri, Sep 3, 2010 at 09:33:19 +0800, Huang, FrankR wrote:
>>>> I saw this thread from top to the end.
>>>> Is it because the patch
>>>> block the geode driver to be accepted?
>>>> If that is so, can we agree with the suggestion from Julien?
>>>> if ((pGeode->Output & (OUTPUT_PANEL | OUTPUT_DCON)) ==
>>>> (OUTPUT_PANEL | OUTPUT_DCON))
>>> Well, or you could make it if (pGeode->Output == (OUTPUT_PANEL |
>>> OUTPUT_DCON)). Or alternatively, explain why the existing code is
>> We'll discuss this particular one with Otavio and let him comment on
>> the solution.
> Otavio, we're still waiting for your explanation on this issue.
Andres Salomon submitted a simple fix for this issue. Basically, we no
longer check the panel bit. Instead, we only check if we're running on
DCON-equipped hardware to make an exact match for the OLPC XO-1
configuration which, if found, will set the display to the XO-1's
unique resolution. This has been committed upstream and could be added
to the current 2.11.9 package as a patch and pushed as 2.11.9-3.
Is there any other issue left to address, before this unblock request
can be approved?