Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1238468 > unrolled thread

Re: [PATCH] usb-host: Remove fusbh200 driver

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] usb-host: Remove fusbh200 driver Felipe Balbi <balbi@ti.com> - 2015-10-02 19:50 +0200
    Re: [PATCH] usb-host: Remove fusbh200 driver Peter Senna Tschudin <peter.senna@gmail.com> - 2015-10-03 12:00 +0200
      Re: [PATCH] usb-host: Remove fusbh200 driver Felipe Balbi <balbi@ti.com> - 2015-10-03 23:30 +0200
        RE: [PATCH] usb-host: Remove fusbh200 driver John Feng-Hsin Chiang(江峰興)   <john453@faraday-tech.com> - 2015-10-05 02:50 +0200

#1238468 — Re: [PATCH] usb-host: Remove fusbh200 driver

[Multipart message — attachments visible in raw view] — view raw

On Fri, Oct 02, 2015 at 01:18:27PM +0200, Peter Senna Tschudin wrote:
> fusbh200 and fotg210 are very similar. The initial idea was to consolidate
> both drivers but I'm afraid fusbh200 is not being used.
> 
> This patch remove the fusbh200 source code, update Kconfig and two
> Makefiles.
> 
> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>

after all this work on these previous patches, you just remove fusbh200 ?

that's a bit odd. Are you sure there are no users for this driver ? It has been
in tree since 2013.

-- 
balbi

[toc] | [next] | [standalone]

#1238791

On Fri, Oct 2, 2015 at 7:39 PM, Felipe Balbi <balbi@ti.com> wrote:
> On Fri, Oct 02, 2015 at 01:18:27PM +0200, Peter Senna Tschudin wrote:
>> fusbh200 and fotg210 are very similar. The initial idea was to consolidate
>> both drivers but I'm afraid fusbh200 is not being used.
>>
>> This patch remove the fusbh200 source code, update Kconfig and two
>> Makefiles.
>>
>> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
>
> after all this work on these previous patches, you just remove fusbh200 ?
>
> that's a bit odd. Are you sure there are no users for this driver ? It has been
> in tree since 2013.
I don't know about users, but I could not find devices using fusbh200.
The closest I got was:

http://www.ebay.fr/itm/Digital-Video-Langzeit-Recorder-H264-DVR-3G-4-Kanal-/370525106495

But it only says: Main Processor: Faraday. I don't know which usb host
controller it uses.

The idea of deleting fusbh200 came from contacting the driver authors.
I was asking where to find hw for testing, and I was told that the
fusbh200 driver can be deleted. Also at least Fedora and Ubuntu build
modules for these host controllers by default. If fusbh200 and fotg210
are only available integrated into SOCs, maybe building the modules by
default for x86 is not a good idea. But if there are users I'll be
happy to continue the integration work, even better if I find hardware
for testing.

John Feng-Hsin Chiang, can you confirm that from your side the
fusbh200 driver can be deleted?

For the patches I sent, 10 of 14 are for fotg210 which I'll fix and resend.

-- 
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1239004

[Multipart message — attachments visible in raw view] — view raw

Peter Senna Tschudin <peter.senna@gmail.com> writes:

> On Fri, Oct 2, 2015 at 7:39 PM, Felipe Balbi <balbi@ti.com> wrote:
>> On Fri, Oct 02, 2015 at 01:18:27PM +0200, Peter Senna Tschudin wrote:
>>> fusbh200 and fotg210 are very similar. The initial idea was to consolidate
>>> both drivers but I'm afraid fusbh200 is not being used.
>>>
>>> This patch remove the fusbh200 source code, update Kconfig and two
>>> Makefiles.
>>>
>>> Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
>>
>> after all this work on these previous patches, you just remove fusbh200 ?
>>
>> that's a bit odd. Are you sure there are no users for this driver ? It has been
>> in tree since 2013.
> I don't know about users, but I could not find devices using fusbh200.
> The closest I got was:
>
> http://www.ebay.fr/itm/Digital-Video-Langzeit-Recorder-H264-DVR-3G-4-Kanal-/370525106495
>
> But it only says: Main Processor: Faraday. I don't know which usb host
> controller it uses.
>
> The idea of deleting fusbh200 came from contacting the driver authors.
> I was asking where to find hw for testing, and I was told that the
> fusbh200 driver can be deleted. Also at least Fedora and Ubuntu build
> modules for these host controllers by default. If fusbh200 and fotg210
> are only available integrated into SOCs, maybe building the modules by
> default for x86 is not a good idea. But if there are users I'll be
> happy to continue the integration work, even better if I find hardware
> for testing.

fair enough, if can be deleted it's fine...

> John Feng-Hsin Chiang, can you confirm that from your side the
> fusbh200 driver can be deleted?

... but let's get this confirmation.

> For the patches I sent, 10 of 14 are for fotg210 which I'll fix and resend.

cool, thanks

-- 
balbi

[toc] | [prev] | [next] | [standalone]

#1239242

SGkgQmFsYmksDQoNCj4+IEpvaG4gRmVuZy1Ic2luIENoaWFuZywgY2FuIHlvdSBjb25maXJtIHRo
YXQgZnJvbSB5b3VyIHNpZGUgdGhlDQo+PiBmdXNiaDIwMCBkcml2ZXIgY2FuIGJlIGRlbGV0ZWQ/
DQo+Li4uIGJ1dCBsZXQncyBnZXQgdGhpcyBjb25maXJtYXRpb24uDQoNClllcywgeW91IGNhbiBk
ZWxldGUgZnVzYmgyMDAuDQoNCkJlc3QgUmVnYXJkcywNCkpvaG4gQ2hpYW5nDQoNCi0tLS0tT3Jp
Z2luYWwgTWVzc2FnZS0tLS0tDQpGcm9tOiBGZWxpcGUgQmFsYmkgW21haWx0bzpiYWxiaUB0aS5j
b21dIA0KU2VudDogU3VuZGF5LCBPY3RvYmVyIDA0LCAyMDE1IDU6MjEgQU0NClRvOiBQZXRlciBT
ZW5uYSBUc2NodWRpbg0KQ2M6IEpvaG4gRmVuZy1Ic2luIENoaWFuZyjLsNzo/ekpOyBZdWFuLUhz
aW4gQ2hlbjsgR3JlZyBLcm9haC1IYXJ0bWFuOyBBbGFuIFN0ZXJuOyBtYXRoaWFzLm55bWFuQGxp
bnV4LmludGVsLmNvbTsgcmFsZkBsaW51eC1taXBzLm9yZzsgYWxleGFuZHJlLmJlbGxvbmlAZnJl
ZS1lbGVjdHJvbnMuY29tOyByYW1uZWVrLm1laHJlc2hAZnJlZXNjYWxlLmNvbTsgTGF1cmVudCBQ
aW5jaGFydDsgYWJyZXN0aWNAY2hyb21pdW0ub3JnOyBzYm95ZEBjb2RlYXVyb3JhLm9yZzsgUmFm
YampIE1pqallY2tpOyBoYW9rZXhpbkBnbWFpbC5jb207IHBlYm9sbGVAdGlzY2FsaS5ubDsgU2Vy
Z2VpIFNodHlseW92OyBNYXNhbmFyaSBJaWRhOyBSYW5keSBEdW5sYXA7IGNocmlzQHJvcnZpY2su
Y29tOyBsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnOyBsaW51eC11c2JAdmdlci5rZXJuZWwu
b3JnDQpTdWJqZWN0OiBSZTogW1BBVENIXSB1c2ItaG9zdDogUmVtb3ZlIGZ1c2JoMjAwIGRyaXZl
cg0KDQpQZXRlciBTZW5uYSBUc2NodWRpbiA8cGV0ZXIuc2VubmFAZ21haWwuY29tPiB3cml0ZXM6
DQoNCj4gT24gRnJpLCBPY3QgMiwgMjAxNSBhdCA3OjM5IFBNLCBGZWxpcGUgQmFsYmkgPGJhbGJp
QHRpLmNvbT4gd3JvdGU6DQo+PiBPbiBGcmksIE9jdCAwMiwgMjAxNSBhdCAwMToxODoyN1BNICsw
MjAwLCBQZXRlciBTZW5uYSBUc2NodWRpbiB3cm90ZToNCj4+PiBmdXNiaDIwMCBhbmQgZm90ZzIx
MCBhcmUgdmVyeSBzaW1pbGFyLiBUaGUgaW5pdGlhbCBpZGVhIHdhcyB0byANCj4+PiBjb25zb2xp
ZGF0ZSBib3RoIGRyaXZlcnMgYnV0IEknbSBhZnJhaWQgZnVzYmgyMDAgaXMgbm90IGJlaW5nIHVz
ZWQuDQo+Pj4NCj4+PiBUaGlzIHBhdGNoIHJlbW92ZSB0aGUgZnVzYmgyMDAgc291cmNlIGNvZGUs
IHVwZGF0ZSBLY29uZmlnIGFuZCB0d28gDQo+Pj4gTWFrZWZpbGVzLg0KPj4+DQo+Pj4gU2lnbmVk
LW9mZi1ieTogUGV0ZXIgU2VubmEgVHNjaHVkaW4gPHBldGVyLnNlbm5hQGdtYWlsLmNvbT4NCj4+
DQo+PiBhZnRlciBhbGwgdGhpcyB3b3JrIG9uIHRoZXNlIHByZXZpb3VzIHBhdGNoZXMsIHlvdSBq
dXN0IHJlbW92ZSBmdXNiaDIwMCA/DQo+Pg0KPj4gdGhhdCdzIGEgYml0IG9kZC4gQXJlIHlvdSBz
dXJlIHRoZXJlIGFyZSBubyB1c2VycyBmb3IgdGhpcyBkcml2ZXIgPyANCj4+IEl0IGhhcyBiZWVu
IGluIHRyZWUgc2luY2UgMjAxMy4NCj4gSSBkb24ndCBrbm93IGFib3V0IHVzZXJzLCBidXQgSSBj
b3VsZCBub3QgZmluZCBkZXZpY2VzIHVzaW5nIGZ1c2JoMjAwLg0KPiBUaGUgY2xvc2VzdCBJIGdv
dCB3YXM6DQo+DQo+IGh0dHA6Ly93d3cuZWJheS5mci9pdG0vRGlnaXRhbC1WaWRlby1MYW5nemVp
dC1SZWNvcmRlci1IMjY0LURWUi0zRy00LUsNCj4gYW5hbC0vMzcwNTI1MTA2NDk1DQo+DQo+IEJ1
dCBpdCBvbmx5IHNheXM6IE1haW4gUHJvY2Vzc29yOiBGYXJhZGF5LiBJIGRvbid0IGtub3cgd2hp
Y2ggdXNiIGhvc3QgDQo+IGNvbnRyb2xsZXIgaXQgdXNlcy4NCj4NCj4gVGhlIGlkZWEgb2YgZGVs
ZXRpbmcgZnVzYmgyMDAgY2FtZSBmcm9tIGNvbnRhY3RpbmcgdGhlIGRyaXZlciBhdXRob3JzLg0K
PiBJIHdhcyBhc2tpbmcgd2hlcmUgdG8gZmluZCBodyBmb3IgdGVzdGluZywgYW5kIEkgd2FzIHRv
bGQgdGhhdCB0aGUNCj4gZnVzYmgyMDAgZHJpdmVyIGNhbiBiZSBkZWxldGVkLiBBbHNvIGF0IGxl
YXN0IEZlZG9yYSBhbmQgVWJ1bnR1IGJ1aWxkIA0KPiBtb2R1bGVzIGZvciB0aGVzZSBob3N0IGNv
bnRyb2xsZXJzIGJ5IGRlZmF1bHQuIElmIGZ1c2JoMjAwIGFuZCBmb3RnMjEwIA0KPiBhcmUgb25s
eSBhdmFpbGFibGUgaW50ZWdyYXRlZCBpbnRvIFNPQ3MsIG1heWJlIGJ1aWxkaW5nIHRoZSBtb2R1
bGVzIGJ5IA0KPiBkZWZhdWx0IGZvciB4ODYgaXMgbm90IGEgZ29vZCBpZGVhLiBCdXQgaWYgdGhl
cmUgYXJlIHVzZXJzIEknbGwgYmUgDQo+IGhhcHB5IHRvIGNvbnRpbnVlIHRoZSBpbnRlZ3JhdGlv
biB3b3JrLCBldmVuIGJldHRlciBpZiBJIGZpbmQgaGFyZHdhcmUgDQo+IGZvciB0ZXN0aW5nLg0K
DQpmYWlyIGVub3VnaCwgaWYgY2FuIGJlIGRlbGV0ZWQgaXQncyBmaW5lLi4uDQoNCj4gSm9obiBG
ZW5nLUhzaW4gQ2hpYW5nLCBjYW4geW91IGNvbmZpcm0gdGhhdCBmcm9tIHlvdXIgc2lkZSB0aGUN
Cj4gZnVzYmgyMDAgZHJpdmVyIGNhbiBiZSBkZWxldGVkPw0KDQouLi4gYnV0IGxldCdzIGdldCB0
aGlzIGNvbmZpcm1hdGlvbi4NCg0KPiBGb3IgdGhlIHBhdGNoZXMgSSBzZW50LCAxMCBvZiAxNCBh
cmUgZm9yIGZvdGcyMTAgd2hpY2ggSSdsbCBmaXggYW5kIHJlc2VuZC4NCg0KY29vbCwgdGhhbmtz
DQoNCi0tDQpiYWxiaQ0K

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web