[RFC PATCH 2/3] usb: udc: delete the vbus indicator at individual driver

[peter.chen@freescale.com (Peter Chen)]

On Tue, Mar 12, 2013 at 11:46:49AM +0200, Felipe Balbi wrote:
> On Tue, Mar 12, 2013 at 05:03:18PM +0800, Peter Chen wrote:
> > Let the common struct usb_gadget own it.
> > 
> > CC: Alexander Shishkin <alexander.shishkin@linux.intel.com>
> > CC: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> > CC: Li Yang <leoli@freescale.com>
> > CC: Roland Stigge <stigge@antcom.de>
> > CC: Yu Xu <yuxu@marvell.com>
> > CC: Chao Xie <chao.xie@marvell.com>
> > CC: Eric Miao <eric.y.miao@gmail.com>
> > CC: Ben Dooks <ben-linux@fluff.org>
> > Signed-off-by: Peter Chen <peter.chen@freescale.com>
> 
> you need to split this patch, should be something like below:
> 
> for_each_driver() {
> 	initialize gadget->vbus_active (don't remove private method)
> }
> 
> add vbus_active support to udc-core
> 
> for_each_driver() {
> 	remove driver's private vbus_active method
> }
> 
> you need one patch per driver so it's easy to review and ack.

Eg, If I changed 10 udc drivers, I need
1(add vbus_active to struct usb_gadget) + 10 + 1 + 10 = 22 patches?

> 
> -- 
> balbi



-- 

Best Regards,
Peter Chen