Re: [RFC PATCH] ARM: OMAP2+: omap-device: Do not overwrite resourcesallocated by OF layer

From: Benoit Cousson
Date: Fri Sep 07 2012 - 13:48:01 EST


Hi Vaibhav,

The following patch is hacing some checkpatch issues.

CHECK: Alignment should match open parenthesis
#169: FILE: arch/arm/plat-omap/omap_device.c:591:
+ dev_dbg(&pdev->dev, "%s(): resources already allocated %d\n",
+ __func__, res_count);

CHECK: Alignment should match open parenthesis
#171: FILE: arch/arm/plat-omap/omap_device.c:593:
+ memcpy(res, pdev->resource,
+ sizeof(struct resource) * pdev->num_resources);

CHECK: Alignment should match open parenthesis
#173: FILE: arch/arm/plat-omap/omap_device.c:595:
+ omap_device_fill_dma_resources(od,
+ &res[pdev->num_resources]);

CHECK: Alignment should match open parenthesis
#176: FILE: arch/arm/plat-omap/omap_device.c:598:
+ dev_dbg(&pdev->dev, "%s(): using resources from hwmod %d\n",
+ __func__, res_count);

total: 0 errors, 0 warnings, 4 checks, 130 lines checked


Since I was in a nice mood, because the week-end is almost there, I fixed them myself.

Please note that I had to rename the API becasue it was way too long to do a proper alignement.

omap_device_fill_dma_resources -> _od_fill_dma_resources

In fact I realized that some private APIs should probably be renamed as well like that for
consistency.

Just let me know if you have any issue with that version.

Regards,
Benoit

---