On Fri, Feb 14, 2014 at 04:53:36PM +0100, Rolf Evers-Fischer wrote:
> Dear Peter,
> I hope that I didn't make any mistake, because this is my first linux kernel
> patch.
> If you don't mind, I'll send it directly to you as an attachment (and not to 
> the
> kernel
> mailing list).

Please always feel free to send patch and questions to mail list,
we have worked at open kernel source. For how to submit a patch:

cumentation/SubmittingPatches

If you are using git, it is easy to create a patch (git add, git submit)
and format a patch (git format-patch), then you can use git send-email
to send your patch.

> 
> After upgrade to kernel 3.13 my colleague discovered that the HSIC ports of 
> our
> IMX6 board (usbh2 and usbh3) were no longer available.
> 
> I discovered that the 'devm_usb_get_phy_by_phandle()' returns an -ENODEV 
> error,
> which
> causes that the 'ci_hdrc_imx_probe()' function will not be completely 
> executed.
> 
> This patch helps to probe the HSIC ports. Until now I didn't
> see any negative impact after applying this patch on my IMX6 board.
> 

Yes, your finding is correct, but your fix more like a temp workaround
for current kernel version. In fact, we also need a **PHY** for usb
driver no matter it exists physically or not. For HSIC controller, you
can use "usb_phy_gen_xceiv" as virtual PHY, the driver is at: drivers/
usb/phy/phy-generic.c. At dts file, the hsic node has phandle to points
to "usb_phy_gen_xceiv".

I have a work version for hsic at internal branch, I will keep you posted
when I submit hsic enable patchset, of course, you can also send that
with above changes.

-- 

Best Regards,
Peter Chen

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to