Re: [PATCH 1/5] fbdevhw: iterate over all modes that match a mode. (v2)

2011-10-19 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston Looks good, but style-wise, I liked the mode = pScrn->monitor->Modes where it was before ... meh On Oct 19, 2011, at 9:01 AM, Dave Airlie wrote: > From: Adam Jackson > > So on RHEL5 anaconda sets an xorg.conf with a fixed 800x600 mode in it, > we run radeonfb

Re: [PATCH 1/5] fbdevhw: iterate over all modes that match a mode. (v2)

2011-10-19 Thread Jamey Sharp
Reviewed-by: Jamey Sharp I haven't tested it, nor do I have mind-reading powers, but I think this logic means what you think it means. Thanks for the revisions. Jamey On 10/19/11, Dave Airlie wrote: > From: Adam Jackson > > So on RHEL5 anaconda sets an xorg.conf with a fixed 800x600 mode in i

[PATCH 1/5] fbdevhw: iterate over all modes that match a mode. (v2)

2011-10-19 Thread Dave Airlie
From: Adam Jackson So on RHEL5 anaconda sets an xorg.conf with a fixed 800x600 mode in it, we run radeonfb and fbdev since ati won't work in userspace due to domain issues in the older codebase. On certain pseries blades the built-in KVM can't accept an 800x600-43 mode, it requires the 800x600-6