Hello.
On 09/11/2012 08:51 AM, Felipe Balbi wrote:
On Mon, Sep 10, 2012 at 06:46:50PM -0700, Kuninori Morimoto wrote:
Hi Felipe
Thank you for checking this patch
+/* USBH common register */
+#define USBPCTRL0 0x0800
+#define USBPCTRL1 0x0804
+#define USBST 0x0808
+#define USBEH0 0x080C
+#define USBOH0 0x081C
+#define USBCTL0 0x0858
+#define EIIBC1 0x0094
+#define EIIBC2 0x009C
(snip)
+ reg0 = devm_ioremap_nocache(dev, res0->start, resource_size(res0));
+ reg1 = devm_ioremap_nocache(dev, res1->start, resource_size(res1));
please use devm_request_and_ioremap().
Sorry about this, but I would like to keep devm_ioremap_nocache() here,
because this Renesas R-Car PHY address has strange mapping.
Some registers of this PHY address are mapped to crevice of USB EHCI/OHCI register area.
like this
0xAAAA0000 ===========
EHCI
----------
0xAAAA0094 PHY
0xAAAA009C PHY
----------
EHCI
0xAAAA0400 ==========
...
0xAAAA0800 ----------
PHY
0xAAAA0900 ----------
This driver couldn't probe if it used "request_mem_region()"
Fair enough. I must say, though. That's a peculiar register mapping ;-)
Not peculiar enough to not be able to overcome it. Those registers
at 0xAAAA009[4C]
are not really PHY related, they control EHCI internal buffering. They
probably shouldn't
have been a part of this driver even...
WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
