Re: [PATCH v2] renesas_usbhs: fix platform init error message

[Felipe Balbi <balbi@ti.com>]

[-- Attachment #1: Type: text/plain, Size: 1537 bytes --]

On Fri, Dec 19, 2014 at 03:46:41PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 12/19/2014 4:33 AM, yoshihiro shimoda wrote:
> 
> [...]
> 
> >>>>>>>There is a typo ("prove" instead of "probe") in the error message printed when
> >>>>>>>the platform initialization fails. Replace that word with more fitting "init".
> 
> >>>>>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
> >>>>>>this actually goes through me, I'll take it in a bit.
> 
> >>>>>     Er, OK. Could you update MAINTAINERS?
> 
> >>>>there is no entry for renesas driver in MAINTAINERS.
> 
> >>>>Shimoda-san, care to send a patch adding yourself or Morimoto-san as
> >>>>maintainers for Renesas driver and pointing to my tree in kernel.org ?
> 
> >>>I would like to move the renesas_usbhs driver to drivers/usb/gadget/udc somehow.
> >>>Because the driver is almost used for a gadget driver.
> >>>The driver has a host driver support now. But, it is not used recently.
> 
> >>>After that, this MAINTAINERS issue becomes clear, I think.
> >>>Felipe-san and Sergei-san, what do you think?
> 
> >>     I'm against such move.
> 
> >Thank you for the reply. But, I would like to know why you are against such move.
> 
>    Because we still need the host mode; RZ/A1H (R7S72100) SoC should need it
> soon), and bi-modal USBHS hardware is better placed in its own directory.

yeah, I'll agree with Sergei here. All other dual role IPs have their
own directories (musb, dwc3, dwc2, isp1760, chipidea...).

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]