Re: [PATCH 6/7] USB: OHCI: remove old SSB OHCI driver

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

 



On 03/12/2012 03:28 PM, Alan Stern wrote:
> On Sun, 11 Mar 2012, Hauke Mehrtens wrote:
> 
>> This is now replaced by the new ssb USB driver, which also supports
>> devices with an EHCI controller.
>>
>> Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx>
>> ---
>>  drivers/usb/host/Kconfig    |   13 --
>>  drivers/usb/host/ohci-hcd.c |   21 +----
>>  drivers/usb/host/ohci-ssb.c |  260 -------------------------------------------
>>  3 files changed, 1 insertions(+), 293 deletions(-)
>>  delete mode 100644 drivers/usb/host/ohci-ssb.c
>>
>> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
>> index eab27d5..665fb89 100644
>> --- a/drivers/usb/host/Kconfig
>> +++ b/drivers/usb/host/Kconfig
>> @@ -360,19 +360,6 @@ config USB_OHCI_HCD_PCI
>>  	  Enables support for PCI-bus plug-in USB controller cards.
>>  	  If unsure, say Y.
>>  
>> -config USB_OHCI_HCD_SSB
>> -	bool "OHCI support for Broadcom SSB OHCI core"
>> -	depends on USB_OHCI_HCD && (SSB = y || SSB = USB_OHCI_HCD) && EXPERIMENTAL
>> -	default n
>> -	---help---
>> -	  Support for the Sonics Silicon Backplane (SSB) attached
>> -	  Broadcom USB OHCI core.
>> -
>> -	  This device is present in some embedded devices with
>> -	  Broadcom based SSB bus.
>> -
>> -	  If unsure, say N.
>> -
> 
> I don't like the idea of removing this section entirely.  It will leave 
> people wondering what happened to their driver.
> 
> Instead, for the next few kernel releases, how about leaving this
> section in place and causing it to select USB_OHCI_HCD_PLATFORM?  Maybe 
> also add a line to the help section explaining that from now on, this 
> driver has been replaced by the generic OHCI platform driver.
> 
> And likewise for the SSB EHCI driver, of course.

That sounds good, I will change that.

Hauke



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux