Re: [PATCH 2/6] crownbay: customize the xorg.conf for the flat panel on the corwnbay kit

[Darren Hart <dvhart@linux.intel.com>]

On 07/26/2012 09:00 PM, Kamble, Nitin A wrote:
> 
> 
>> -----Original Message-----
>> From: Darren Hart [mailto:dvhart@linux.intel.com]
>> Sent: Friday, July 27, 2012 1:41 AM
>> To: Kamble, Nitin A
>> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com; Zanussi, Tom
>> Subject: Re: [yocto] [PATCH 2/6] crownbay: customize the xorg.conf for the
>> flat panel on the corwnbay kit
>>
>> On 07/23/2012 03:06 AM, nitin.a.kamble@intel.com wrote:
>>> From: Nitin A Kamble <nitin.a.kamble@intel.com>
>>>
>>> The kit has Auo 800x600 LCD screen. Configuring Xorg for it.
>>
>> I presume the crownbay has additional display options? how does this impact
>> those?
>>
> This change sets the resolution of the screen to 800x600. And this is applicable to 
> LCD screen on the kit as well as external monitor.
>   I will add a note about it in the log.

I presume that without this change it was able to detect the appropriate
resolution of the connected monitor, but not of the LCD?

If so, this change effectively breaks that autodetection and forces
everything to the 800x600 display which is arguably very low-resolution
by today's standards. Why should this be the default? When you refer to
"the kit", what exactly are you referring to?

Also, is this a discussion you have already had with Tom? I don't want
to contradict what he has said regarding this BSP.

--
Darren

> 
> Thanks,
> Nitin
> 
> 
>> --
>> Darren
>>
>>>
>>> Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
>>> ---
>>>  .../xserver-xf86-config/crownbay/xorg.conf         |   14 ++++++++++++++
>>>  1 files changed, 14 insertions(+), 0 deletions(-)
>>>
>>> diff --git
>>> a/meta-crownbay/recipes-graphics/xorg-xserver/xserver-xf86-
>> config/crow
>>> nbay/xorg.conf
>>> b/meta-crownbay/recipes-graphics/xorg-xserver/xserver-xf86-
>> config/crow
>>> nbay/xorg.conf
>>> index 662f60f..d71173c 100644
>>> ---
>>> a/meta-crownbay/recipes-graphics/xorg-xserver/xserver-xf86-
>> config/crow
>>> nbay/xorg.conf
>>> +++ b/meta-crownbay/recipes-graphics/xorg-xserver/xserver-xf86-config/
>>> +++ crownbay/xorg.conf
>>> @@ -4,12 +4,26 @@
>>>  ## DriverVer=
>>>  ##
>>>
>>> +# AUO G084SN05 V8 lcd flat panel
>>> +Section "Monitor"
>>> +    Identifier     "Monitor0"
>>> +    VendorName     "Auo"
>>> +#    HorizSync       33.6 - 48.3
>>> +#    VertRefresh     60.0 - 60.0
>>> +    Option         "DPMS"
>>> +EndSection
>>> +
>>>  Section "Screen"
>>>      Identifier    "Screen0"
>>>      Device        "IntelEMGD-0"
>>>      Monitor       "Monitor0"
>>> +
>>> +    # for the AUO flat screen
>>>      SubSection    "Display"
>>> +        Depth      24
>>> +        Modes      "800x600"
>>>      EndSubSection
>>> +
>>>  EndSection
>>>
>>>  # Primary (First/only) display
>>>
>>
>> --
>> Darren Hart
>> Intel Open Source Technology Center
>> Yocto Project - Technical Lead - Linux Kernel

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Technical Lead - Linux Kernel