On Sat, Mar 30, 2013 at 11:18:08PM +0200, Aaro Koskinen wrote: > The function returns always false, delete it. > > Signed-off-by: Aaro Koskinen <[email protected]> > --- > drivers/staging/xgifb/vb_setmode.c | 33 +++++---------------------------- > 1 file changed, 5 insertions(+), 28 deletions(-) > > diff --git a/drivers/staging/xgifb/vb_setmode.c > b/drivers/staging/xgifb/vb_setmode.c > index d9e1545..aa1c65a 100644 > --- a/drivers/staging/xgifb/vb_setmode.c > +++ b/drivers/staging/xgifb/vb_setmode.c > @@ -4559,23 +4559,6 @@ static unsigned char XGI_IsLCDON(struct vb_device_info > *pVBInfo) > return 0; > } > > -/* --------------------------------------------------------------------- */ > -/* Function : XGI_EnableChISLCD */ > -/* Input : */ > -/* Output : 0 -> Not LCD mode */ > -/* Description : if bool enable = true -> enable, else disable */ > -/* --------------------------------------------------------------------- */ > -static unsigned char XGI_EnableChISLCD(struct vb_device_info *pVBInfo, > - bool enable) > -{ > - unsigned short tempbx, tempah; > - > - tempbx = 0; > - tempah = ~((unsigned short) xgifb_reg_get(pVBInfo->Part1Port, 0x2E)); > -
It used to not be false until we stopped checking "tempah" without explaining why... *grumble* *grumble*. :P > - return 0; > -} regards, dan carpenter _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
