<br><br><div class="gmail_quote">On Thu, Jul 26, 2012 at 8:24 PM, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Saturday 21 July 2012, Vivek Gautam wrote:<br>
> From: Ajay Kumar <<a href="mailto:ajaykumar.rs@samsung.com">ajaykumar.rs@samsung.com</a>><br>
><br>
> This patch adds address mapping of USB 2.0 PHY for exynos5<br>
><br>
> Signed-off-by: Ajay Kumar <<a href="mailto:ajaykumar.rs@samsung.com">ajaykumar.rs@samsung.com</a>><br>
> Signed-off-by: Vivek Gautam <<a href="mailto:gautam.vivek@samsung.com">gautam.vivek@samsung.com</a>><br>
> ---<br>
>  arch/arm/mach-exynos/common.c           |    5 +++++<br>
>  arch/arm/mach-exynos/include/mach/map.h |    2 ++<br>
>  2 files changed, 7 insertions(+), 0 deletions(-)<br>
><br>
> diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c<br>
> index 4eb39cd..92d5e61 100644<br>
> --- a/arch/arm/mach-exynos/common.c<br>
> +++ b/arch/arm/mach-exynos/common.c<br>
> @@ -275,6 +275,11 @@ static struct map_desc exynos5_iodesc[] __initdata = {<br>
>                 .pfn            = __phys_to_pfn(EXYNOS5_PA_GIC_DIST),<br>
>                 .length         = SZ_4K,<br>
>                 .type           = MT_DEVICE,<br>
> +       }, {<br>
> +               .virtual        = (unsigned long)S3C_VA_USB_HSPHY,<br>
> +               .pfn            = __phys_to_pfn(EXYNOS5_PA_USB_PHY),<br>
> +               .length         = SZ_4K,<br>
> +               .type           = MT_DEVICE,<br>
>         },<br>
>  };<br>
<br>
</div>Can you pleae explain why this is done in the changelog?<br>
<br>
We try hard to do such mappings from the device driver instead,<br>
so I'm surprised that this is necessary fo rthe USB phy.<br></blockquote><div><br></div><div>We are doing the mapping for device address in the driver, but this memory</div><div>mapping for USB PHY registers that need to be programmed by the software</div>
<div>is done here. This is similar to what we see for exynos4 also. Is it something</div><div>that i can still change? Please suggest.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
        Arnd<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
devicetree-discuss mailing list<br>
<a href="mailto:devicetree-discuss@lists.ozlabs.org">devicetree-discuss@lists.ozlabs.org</a><br>
<a href="https://lists.ozlabs.org/listinfo/devicetree-discuss" target="_blank">https://lists.ozlabs.org/listinfo/devicetree-discuss</a></div></div></blockquote><div><br></div><div>Best regards</div><div>Vivek </div></div>
<br>