[PATCH 06/15] staging: xgifb: delete bIntegratedMMEnabled

Dan Carpenter error27 at gmail.com
Wed Mar 9 11:08:05 UTC 2011


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 at gmail.com>

regards,
dan carpenter



More information about the devel mailing list