Re: [PATCH] radeonfb: cleanup and little fixes

From: Andrew Morton
Date: Mon Jul 12 2004 - 01:35:12 EST


David Eger <eger@xxxxxxxxxxxxx> wrote:
>
> Dear Andrew, Please apply.
>

Sure..

> if (rinfo->mon1_EDID)
> kfree(rinfo->mon1_EDID);
> if (rinfo->mon2_EDID)
> kfree(rinfo->mon2_EDID);

kfree(NULL) is legal, and this is a slow-path. Those tests can be taken
out next time, unless such a thing offends you.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/