Thread (21 messages) 21 messages, 4 authors, 2018-10-18

RE: [PATCH 0/4] usb: chipidea: imx: add HSIC support

From: Peter Chen <hidden>
Date: 2018-10-18 17:05:04

 
quoted
quoted
My board is currently off-tree so I can't send any patch for the
pinmux settings in devicetree. But I will send a patch with the
changes that should go to imx6qdl.dtsi, imx6sl.dtsi and imx6sx.dtsi. Though I
only tested on i.MX6S.
quoted
quoted
No, the changes are board level specific, you could not add it into SoC file.
Not all changes are board specific. The HSIC-only host controllers can only be used
to interface with on-board chips via HSIC. So they need a "usb-nop-xceiv" dummy
PHY in all use cases.

So if I understand this correctly this should be done in the SoCs dtsi files like in this
patch I just sent: [1].

Please also note that for i.MX7S this is already added in imx7s.dtsi: [2]
Oh, I have misunderstood your meaning, the [1] is needed since it is for SoC.
I thought your board level changes (like pinctrl) would go into SoC dtsi file.
If there are no more comments, I will send v2 next week, feel free to add your
reviewed-by and tested-by tag, thanks.

Peter
[1]:
https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flkml.org%2
Flkml%2F2018%2F10%2F18%2F414&amp;data=02%7C01%7Cpeter.chen%40nxp.
com%7C3c73fdf4ec98477da49308d634d78b39%7C686ea1d3bc2b4c6fa92cd99c5c
301635%7C0%7C0%7C636754497687793991&amp;sdata=aHhMhaj%2FbdTFu0B
qXi0loiBh%2BHzl%2F%2BTSw3txyjBSNLQ%3D&amp;reserved=0
[2]:
https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com
%2Ftorvalds%2Flinux%2Fblob%2Fmaster%2Farch%2Farm%2Fboot%2Fdts%2Fim
x7s.dtsi%23L88&amp;data=02%7C01%7Cpeter.chen%40nxp.com%7C3c73fdf4ec9
8477da49308d634d78b39%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%
7C636754497687793991&amp;sdata=7cVhAMQpWWYjGHjY8Xh2g0CG3jPaeG8X
9K368AooQcc%3D&amp;reserved=0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help