On Fri, Oct 31, 2014 at 11:31 AM, Felipe Balbi <[email protected]> wrote:
> On Fri, Oct 31, 2014 at 11:21:29AM -0700, Ashwini Pahuja wrote:
>> - This patch adds a UDC driver for Broadcom's USB3.0 device controller
>> IP(BDC).
>
> why all these extra spaces here ? why isn't this wrapped at 72
> characters (or so) ?
>
>> - The driver was written using the Felipe's USB testing/next branch as a
>> baseline from:
>> git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git
>
> you don't need to mention this in commit log.
>
OK
>> - The driver is tested on FPGA-PCIe based platform using various gadget
>> drivers such as zero, mass storage, ether(rndis), hid and also with tools
>> such as USB3CV, USB2CV and Link layer compliance.
>
> Use commit log to talk abou this new driver, its features and so on.
> Look at kernel's git log if you need inspiration but this isn't supposed
> to be a bullet-point list of information.
>
OK
> BTW, is this controller peripheral-only or can it also run as host ?
>
peripheral-only.
> --
Let me resend v2 with further improved commit log, thanks for your
patience. I also need to edit the subject to have RESEND in the
subject line.
> balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
