Messages in this thread | | | From | "Petr Vandrovec" <> | Date | Tue, 26 Feb 2002 20:43:45 +0100 | Subject | Re: [PATCH] matroxfb_base.c - a little fix |
| |
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/
| |