On Wed, 2013-10-09 at 12:52 +0200, Michal Simek wrote: > Simplify driver probe and release function. [] > diff --git a/drivers/video/xilinxfb.c b/drivers/video/xilinxfb.c [] > @@ -423,7 +419,7 @@ static int xilinxfb_of_probe(struct platform_device *pdev) > pdata = xilinx_fb_default_pdata; > > /* Allocate the driver data region */ > - drvdata = kzalloc(sizeof(*drvdata), GFP_KERNEL); > + drvdata = devm_kzalloc(&pdev->dev, sizeof(*drvdata), GFP_KERNEL); > if (!drvdata) { > dev_err(&pdev->dev, "Couldn't allocate device private record\n"); Be nice to remove the unnecessary OOM message. There's already a generic dump_stack on OOM. -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html