On Wed, Mar 09, 2011 at 10:23:01AM +0200, Aaro Koskinen wrote: > Hi, > > On Wed, 9 Mar 2011, Dan Carpenter wrote: > >On Tue, Mar 08, 2011 at 10:16:11PM +0200, Aaro Koskinen wrote: > >>bIntegratedMMEnabled is always true, so the field and checks can be > >>eliminated. > >> > >What about for the chips that came before the XG40? Are we dropping > >support for them? > > They were never supported. See the earlier switch (xgi_video_info.chip_id) > statement in the probe routine - it will return -ENODEV for other chips > than XG40..XG27. > Ah... That's true. This patch is fine by me then. Reviewed-by: Dan Carpenter <error27@xxxxxxxxx> regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
