[PATCH 09/17] staging: xgifb: SetFlag: delete EnableChB

[Aaro Koskinen]

On Mon, Apr 01, 2013 at 01:30:47PM +0300, Dan Carpenter wrote:
> On Sat, Mar 30, 2013 at 11:18:05PM +0200, Aaro Koskinen wrote:
> > @@ -4582,12 +4582,6 @@ static unsigned char XGI_EnableChISLCD(struct vb_device_info *pVBInfo,
> >  			return 0;
> >  	}
> >  
> > -	if (!(tempbx & EnableChB))
> > -		return 0;
> > -
> > -	if (tempah & 0x01) /* Chk LCDB Mode */
> > -		return 1;
> 
> I don't understand why this return is deleted.

Because EnableChB was never set in tempbx, the code took always the
"return 0" path and the remaing code was unreachable. So we have to
delete it too, otherwise there would be a change in functionality.

A.