From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752380AbeAQIuL (ORCPT + 1 other); Wed, 17 Jan 2018 03:50:11 -0500 Received: from [117.201.178.166] ([117.201.178.166]:43288 "EHLO batcave" rhost-flags-FAIL-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1750801AbeAQIuJ (ORCPT ); Wed, 17 Jan 2018 03:50:09 -0500 X-Greylist: delayed 442 seconds by postgrey-1.27 at vger.kernel.org; Wed, 17 Jan 2018 03:50:08 EST From: Naveen Panwar To: gregkh@linuxfoundation.org Cc: naveen@spics.info, naveen.panwar27@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/2] Staging: netlogic: platform_net.c: introduced a helper function Date: Wed, 17 Jan 2018 14:10:47 +0530 Message-Id: <1516178447-4303-1-git-send-email-naveen.panwar27@gmail.com> X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: In staging/netlogic/platform.c Refactoring the identical code used in several places for calculating the physical address of memory-mapped objects on the device.Put that into an inline helper function and use it. Adjust the types of arguments of xlr_resource_init() - 'offset' and 'irq' are actually unsigned. The values passed to those fit into 0..2G range, so the behaviour is unchanged, but it's more consistent that way. Signed-off-by: Naveen Panwar --- drivers/staging/netlogic/platform_net.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/drivers/staging/netlogic/platform_net.c b/drivers/staging/netlogic/platform_net.c index f45b67a..2196dad 100644 --- a/drivers/staging/netlogic/platform_net.c +++ b/drivers/staging/netlogic/platform_net.c @@ -76,11 +76,16 @@ static struct resource xlr_net1_res[8]; static u32 __iomem *gmac4_addr; static u32 __iomem *gpio_addr; -static void xlr_resource_init(struct resource *res, int offset, int irq) +static inline phys_addr_t physaddr(u32 offset) +{ + return CPHYSADDR(nlm_mmio_base(offset)); +} + +static void xlr_resource_init(struct resource *res, u32 offset, u32 irq) { res->name = "gmac"; - res->start = CPHYSADDR(nlm_mmio_base(offset)); + res->start = physaddr(offset); res->end = res->start + 0xfff; res->flags = IORESOURCE_MEM; @@ -107,8 +112,7 @@ static struct platform_device *gmac_controller2_init(void *gmac0_addr) .dev.platform_data = &ndata1, }; - gmac4_addr = ioremap(CPHYSADDR - (nlm_mmio_base(NETLOGIC_IO_GMAC_4_OFFSET)), 0xfff); + gmac4_addr = ioremap(physaddr(NETLOGIC_IO_GMAC_4_OFFSET), 0xfff); ndata1.serdes_addr = gmac4_addr; ndata1.pcs_addr = gmac4_addr; ndata1.mii_addr = gmac0_addr; @@ -134,8 +138,7 @@ static void xls_gmac_init(void) { int mac; struct platform_device *xlr_net_dev1; - void __iomem *gmac0_addr = ioremap(CPHYSADDR - (nlm_mmio_base(NETLOGIC_IO_GMAC_0_OFFSET)), 0xfff); + void __iomem *gmac0_addr = ioremap(physaddr(NETLOGIC_IO_GMAC_0_OFFSET), 0xfff); static struct xlr_net_data ndata0 = { .rfr_station = FMN_STNID_GMACRFR_0, @@ -153,8 +156,7 @@ static void xls_gmac_init(void) ndata0.mii_addr = gmac0_addr; /* Passing GPIO base for serdes init. Only needed on sgmii ports */ - gpio_addr = ioremap(CPHYSADDR - (nlm_mmio_base(NETLOGIC_IO_GPIO_OFFSET)), 0xfff); + gpio_addr = ioremap(physaddr(NETLOGIC_IO_GPIO_OFFSET), 0xfff); ndata0.gpio_addr = gpio_addr; ndata0.cpu_mask = nlm_current_node()->coremask; @@ -214,8 +216,7 @@ static void xlr_gmac_init(void) .id = 0, .dev.platform_data = &ndata0, }; - ndata0.mii_addr = ioremap(CPHYSADDR - (nlm_mmio_base(NETLOGIC_IO_GMAC_0_OFFSET)), 0xfff); + ndata0.mii_addr = ioremap(physaddr(NETLOGIC_IO_GMAC_0_OFFSET), 0xfff); ndata0.cpu_mask = nlm_current_node()->coremask; -- 2.7.4