Thread (19 messages) 19 messages, 6 authors, 2016-11-05

Re: [PATCH 1/4] mfd: ti_am335x_tscadc: store physical address

From: Lee Jones <hidden>
Date: 2016-10-26 12:23:40
Also in: linux-arm-kernel, linux-iio, linux-omap, lkml

On Fri, 30 Sep 2016, Mugunthan V N wrote:
On Wednesday 28 September 2016 01:10 AM, Lee Jones wrote:
quoted
On Wed, 21 Sep 2016, Mugunthan V N wrote:
quoted
store the physical address of the device in its priv to use it
for DMA addressing in the client drivers.

Signed-off-by: Mugunthan V N <redacted>
---
 drivers/mfd/ti_am335x_tscadc.c       | 1 +
 include/linux/mfd/ti_am335x_tscadc.h | 1 +
 2 files changed, 2 insertions(+)
diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd/ti_am335x_tscadc.c
index c8f027b..0f3fab4 100644
--- a/drivers/mfd/ti_am335x_tscadc.c
+++ b/drivers/mfd/ti_am335x_tscadc.c
@@ -183,6 +183,7 @@ static	int ti_tscadc_probe(struct platform_device *pdev)
 		tscadc->irq = err;
 
 	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+	tscadc->tscadc_phys_base = res->start;
This is unusual.  Can't you use a virt_to_phys() variant instead?
I tried using virt_to_phys(), but its not working for me.
Also saw many drivers uses like this to get physical address
("git grep -n " res->start;" drivers/*").
Very well:

For my own reference:
  Acked-for-MFD-by: Lee Jones [off-list ref]

Let me know how you wish this set to be handled.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help