Re: [PATCH] matroxfb_base.c - a little fix

Petr Vandrovec (VANDROVE@vc.cvut.cz)
Tue, 26 Feb 2002 20:43:45 +0100


On 27 Feb 02 at 0:26, Denis Zaitsev wrote:
> It was two identical if-else branches in matroxfb_decode_var - one for
> var->bits_per_pixel == 4
> and the other for
> var->bits_per_pixel <= 8
> . So, I've removed 4's one. It should be ok, if this branch was ok -
> i.e. was intentionally made the same as the 8bpp's. So, Petr, please,
> apply this fix, if all ok.

Somebody changed == 8 to <= 8 when I was not looking... But as
bits_per_pixel is already validated, it looks correct. Applied,
will go to Linus next round.
Best regards,
Petr Vandrovec
vandrove@vc.cvut.cz

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/