Re: [PATCH v6 00/28] usb: ci13xxx: rework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Fri, May 04, 2012 at 04:47:13PM +0300, Alexander Shishkin wrote:
> This is the 6th version of the patchset aiming to rework ci13xxx
> driver. With this patchset, the driver also supports an older version
> of chipidea found in Marvell's Feroceon SoC, that is, it works with
> sheevaplug (provided you can find or make a USB A-to-A cable).
> 
> This patchset fixes several bugs in the udc driver and addresses
> some of the style issues. This patchset also includes conversion to
> platform_device, which in case if msm is only compile-tested. While
> at it, I have also converted "choice" of UDC controllers into a menu,
> since we can probably have more than one UDC in the system, at least
> theoretically.
> 
> It also converts ci13xxx_udc to the "new style", so that we can finally
> get rid of the global _udc pointer and theoretically can now have as
> many chipidea gadgets in the system as we please.
> 
> Then, it proceeds to move the driver to drivers/usb/chipidea and turn
> it into a dual-role controller driver, complete with role switching
> (manual and ID-pin based) and both device and host roles. Tested on
> both sheevaplug and penwell. The former requires modifications to the
> platform code in arch/arm, which are not part of these series. The
> latter just works.
> 
> Finally, it removes langwell_udc driver, as it is now covered by the
> reworked chipidea driver too and adds a new entry in MAINTAINERS.
> 
> New since the previous version:
>   * moved platform data of the pci driver to driver_data in pci_id
>     tables as Felipe suggested,
>   * split "new style" conversion into two patches, first of which moves
>     endpoint initialization from start() to probe(), per Felipe's request,
>   * fixed deferred address setting per Felipe's comment,
>   * minor fixes here and there per Felipe's comments,
>   * removed langwell_udc,
>   * added a MAINTAINERS entry

unfortunately it doesn't apply cleanly, so I have dropped the series
from my v3.5 pull request. It will have to wait until v3.6. I'm sorry.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux