Message ID | 000801cf0829$f3a48300$daed8900$%han@samsung.com |
---|---|
State | Accepted |
Headers | show |
On Fri, Jan 03, 2014 at 11:17:29AM +0900, Jingoo Han wrote: > The site-specific OOM messages are unnecessary, because they > duplicate the MM subsystem generic OOM message. > > Signed-off-by: Jingoo Han <jg1.han@samsung.com> > --- > Change since v1 > - Re-based on the newer version of the patch "mtd: plat_nand: > Use devm_*() functions". Pushed to l2-mtd.git. Thanks! Brian
diff --git a/drivers/mtd/nand/plat_nand.c b/drivers/mtd/nand/plat_nand.c index d00e3a7..4ebed72 100644 --- a/drivers/mtd/nand/plat_nand.c +++ b/drivers/mtd/nand/plat_nand.c @@ -55,10 +55,8 @@ static int plat_nand_probe(struct platform_device *pdev) /* Allocate memory for the device structure (and zero it) */ data = devm_kzalloc(&pdev->dev, sizeof(struct plat_nand_data), GFP_KERNEL); - if (!data) { - dev_err(&pdev->dev, "failed to allocate device structure.\n"); + if (!data) return -ENOMEM; - } data->io_base = devm_ioremap_resource(&pdev->dev, res); if (IS_ERR(data->io_base))
The site-specific OOM messages are unnecessary, because they duplicate the MM subsystem generic OOM message. Signed-off-by: Jingoo Han <jg1.han@samsung.com> --- Change since v1 - Re-based on the newer version of the patch "mtd: plat_nand: Use devm_*() functions". drivers/mtd/nand/plat_nand.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)