From mboxrd@z Thu Jan 1 00:00:00 1970 From: hdegoede@redhat.com (Hans de Goede) Date: Thu, 10 Sep 2015 20:23:23 +0200 Subject: [linux-sunxi] [PATCH] musb: sunxi: Ignore VBus errors in host-only mode In-Reply-To: <55E93D8C.9020804@schinagl.nl> References: <1438723553-4136-1-git-send-email-hdegoede@redhat.com> <55C31963.2010701@schinagl.nl> <55C3709C.4080609@redhat.com> <55C47041.3000603@schinagl.nl> <55E93D8C.9020804@schinagl.nl> Message-ID: <55F1CA9B.3070204@redhat.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 04-09-15 08:43, Olliver Schinagl wrote: > Hey Hans, > > On 07-08-15 10:45, Olliver Schinagl wrote: >> >>> If you change the dr_mode to host then you _must_ also remove any id_det and vbus_det >>> gpio settings from the usb_phy node in the dts, as the sun4i phy code detects >>> host vs otg mode by checking for the presence of these. >> Yes, this fixes it and makes it work. Thanks. >> > I've been going back to this and am wondering if this is something I can look into to fix properly? E.g. if the dts sets dr_mode = host, can we simply ignore the pins and treat them as unset? AFAIK you cannot unset something in dts. The only solution I can comeup with is to add a dr_mode argument to the phy like we already have for the otg controller itself. This is something which we likely need to do anyways to add support for peripheral only mode, which we seem to need for some "hdmi sticks". Patches doing this are welcome from my pov. Regards, Hans