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 Ionly 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&data=02%7C01%7Cpeter.chen%40nxp. com%7C3c73fdf4ec98477da49308d634d78b39%7C686ea1d3bc2b4c6fa92cd99c5c 301635%7C0%7C0%7C636754497687793991&sdata=aHhMhaj%2FbdTFu0B qXi0loiBh%2BHzl%2F%2BTSw3txyjBSNLQ%3D&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&data=02%7C01%7Cpeter.chen%40nxp.com%7C3c73fdf4ec9 8477da49308d634d78b39%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0% 7C636754497687793991&sdata=7cVhAMQpWWYjGHjY8Xh2g0CG3jPaeG8X 9K368AooQcc%3D&reserved=0