On Wednesday, November 29, 2017 5:45 PM, Lukasz Majewski wrote:
> This commit cleans up the code by using dedicated macros instead of full definitions.
>
> Signed-off-by: Lukasz Majewski <[email protected]>
> Acked-by: Alexander Sverdlin <[email protected]>
<snip>
> @@ -187,16 +187,8 @@ static struct platform_device ts72xx_rtc_device = { };
>
> static struct resource ts72xx_wdt_resources[] = {
> - {
> - .start = TS72XX_WDT_CONTROL_PHYS_BASE,
> - .end = TS72XX_WDT_CONTROL_PHYS_BASE + SZ_4K - 1,
> - .flags = IORESOURCE_MEM,
> - },
> - {
> - .start = TS72XX_WDT_FEED_PHYS_BASE,
> - .end = TS72XX_WDT_FEED_PHYS_BASE + SZ_4K - 1,
> - .flags = IORESOURCE_MEM,
> - },
> + DEFINE_RES_MEM(TS72XX_WDT_CONTROL_PHYS_BASE, SZ_4K),
> + DEFINE_RES_MEM(TS72XX_WDT_FEED_PHYS_BASE, SZ_4K),
> };
Lukasz,
The chunk above is already cleaned up in linux-next by:
commit 97c3bca922468996c864917c9a8ea6a410233985
ARM: ep93xx: tidy up TS-72xx Watchdog resources
Can you please rebase this series against linux-next?
I'm reviewing the rest of the series now and will reply with any other issues.
Thanks,
Hartley
From 1585449877672301854@xxx Thu Nov 30 00:47:08 +0000 2017
X-GM-THRID: 1584302672956416146
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread