[PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support
tharvey at gateworks.com
Thu Apr 22 17:19:21 CEST 2021
On Thu, Apr 22, 2021 at 3:34 AM Ying-Chun Liu (PaulLiu)
<paulliu at debian.org> wrote:
> Hi Marek,
> I have the same issue on my iMX8MM based board.
> Are you continue working on the patch for iMX8MM or should I dig into
> this problem?
> I'm asking this just for not duplicating efforts.
> So if you are not working on this then I'll see if I can properly
> retrieve the necessary patches to make this work.
> I think something is missing from previous patch set
It's not something missing from Peng's previous patchset since that
didn't support DM. Marek added the dm bits but it requires some dt
changes that are still not present to make mx6_parse_dt_addrs work (or
some changes are needed to that func).
The way mx6_parse_dt_addrs is written you need to have a 'usb0' alias
for example that points to the phy node (compatible 'fsl,usbmisc') but
that is not what has typically been done for USB as typically the
alias points to the host controller instead.
I'm still interested in what Marek tested with and also think there's
a dt patch missing. The aliases should be added to imx8mm.dtsi, but I
also think they should point to the controller and not the phy so
there may be a code change needed as well.
Also, I wasn't aware that the power-domain dt bindings were accepted
upstream to Linux as there has been quite a bit of controversy there.
cc'ing some others that are probably interested in mainline USB on
IMX8M and likely know what's going on with IMX8M power domain. I
regret missing this patchset when it was submitted or I would have
asked these questions then... I just can't keep up with Linux and
U-Boot mailing lists with my work-load in other areas.
More information about the U-Boot