* Re: [PATCH 4/4] HID: remove ThingM blink(1) driver
[not found] <f8c59082-d76c-73aa-70b6-7454b731a055@gmail.com>
@ 2016-06-22 13:25 ` Jiri Kosina
2016-06-22 15:42 ` Vivien Didelot
2016-06-22 19:52 ` Heiner Kallweit
0 siblings, 2 replies; 5+ messages in thread
From: Jiri Kosina @ 2016-06-22 13:25 UTC (permalink / raw)
To: Heiner Kallweit
Cc: Benjamin Tissoires, linux-input, linux-kernel, Vivien Didelot
On Tue, 21 Jun 2016, Heiner Kallweit wrote:
> Now that support for ThingM blink(1) was merged into the hid-led driver
> the dedicated driver for this device can be removed.
>
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
> ---
> drivers/hid/Kconfig | 10 --
> drivers/hid/Makefile | 1 -
> drivers/hid/hid-thingm.c | 259 -----------------------------------------------
> 3 files changed, 270 deletions(-)
> delete mode 100644 drivers/hid/hid-thingm.c
>
> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
> index ab22090..6b08a3b 100644
> --- a/drivers/hid/Kconfig
> +++ b/drivers/hid/Kconfig
> @@ -828,16 +828,6 @@ config HID_TOPSEED
> Say Y if you have a TopSeed Cyberlink or BTC Emprex or Conceptronic
> CLLRCMCE remote control.
>
> -config HID_THINGM
Overall the patchset looks good to me. I'd suggest we keep
CONFIG_HID_THINGM as a stub though, that'd just select CONFIG_HID_LED, so
that we don't cause kernel config regressions for users who had
CONFIG_HID_THINGM already turned on.
Also adding Vivien Didelot to CC, as the author of the original driver.
Vivien, the full set can be found in linux-input archives (
http://marc.info/?l=linux-input&r=1&b=201606&w=2 )
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 4/4] HID: remove ThingM blink(1) driver
2016-06-22 13:25 ` [PATCH 4/4] HID: remove ThingM blink(1) driver Jiri Kosina
@ 2016-06-22 15:42 ` Vivien Didelot
2016-06-22 19:54 ` Heiner Kallweit
2016-06-22 19:52 ` Heiner Kallweit
1 sibling, 1 reply; 5+ messages in thread
From: Vivien Didelot @ 2016-06-22 15:42 UTC (permalink / raw)
To: Jiri Kosina, Heiner Kallweit
Cc: Benjamin Tissoires, linux-input, linux-kernel
Hi,
Jiri Kosina <jikos@kernel.org> writes:
> On Tue, 21 Jun 2016, Heiner Kallweit wrote:
>
>> Now that support for ThingM blink(1) was merged into the hid-led driver
>> the dedicated driver for this device can be removed.
>>
>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>> ---
>> drivers/hid/Kconfig | 10 --
>> drivers/hid/Makefile | 1 -
>> drivers/hid/hid-thingm.c | 259 -----------------------------------------------
>> 3 files changed, 270 deletions(-)
>> delete mode 100644 drivers/hid/hid-thingm.c
>>
>> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
>> index ab22090..6b08a3b 100644
>> --- a/drivers/hid/Kconfig
>> +++ b/drivers/hid/Kconfig
>> @@ -828,16 +828,6 @@ config HID_TOPSEED
>> Say Y if you have a TopSeed Cyberlink or BTC Emprex or Conceptronic
>> CLLRCMCE remote control.
>>
>> -config HID_THINGM
>
> Overall the patchset looks good to me. I'd suggest we keep
> CONFIG_HID_THINGM as a stub though, that'd just select CONFIG_HID_LED, so
> that we don't cause kernel config regressions for users who had
> CONFIG_HID_THINGM already turned on.
>
> Also adding Vivien Didelot to CC, as the author of the original driver.
> Vivien, the full set can be found in linux-input archives (
> http://marc.info/?l=linux-input&r=1&b=201606&w=2 )
Thanks a lot for the follow-up Jiri.
If you guys decide to remove hid-thingm.c, you need to update the
MAINTAINERS entry, checkpatch.pl should have warned about that.
I can hardly test the patchset now. Heiner, can you confirm me that with
your patchset, we will still see 3 led_classdev in sysfs for the
blink(1) mk1 (which has one RGB chip) and 6 for mk2 (which has 2 chips)?
Thanks,
Vivien
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 4/4] HID: remove ThingM blink(1) driver
2016-06-22 13:25 ` [PATCH 4/4] HID: remove ThingM blink(1) driver Jiri Kosina
2016-06-22 15:42 ` Vivien Didelot
@ 2016-06-22 19:52 ` Heiner Kallweit
1 sibling, 0 replies; 5+ messages in thread
From: Heiner Kallweit @ 2016-06-22 19:52 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Benjamin Tissoires, linux-input, linux-kernel, Vivien Didelot
Am 22.06.2016 um 15:25 schrieb Jiri Kosina:
> On Tue, 21 Jun 2016, Heiner Kallweit wrote:
>
>> Now that support for ThingM blink(1) was merged into the hid-led driver
>> the dedicated driver for this device can be removed.
>>
>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>> ---
>> drivers/hid/Kconfig | 10 --
>> drivers/hid/Makefile | 1 -
>> drivers/hid/hid-thingm.c | 259 -----------------------------------------------
>> 3 files changed, 270 deletions(-)
>> delete mode 100644 drivers/hid/hid-thingm.c
>>
>> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
>> index ab22090..6b08a3b 100644
>> --- a/drivers/hid/Kconfig
>> +++ b/drivers/hid/Kconfig
>> @@ -828,16 +828,6 @@ config HID_TOPSEED
>> Say Y if you have a TopSeed Cyberlink or BTC Emprex or Conceptronic
>> CLLRCMCE remote control.
>>
>> -config HID_THINGM
>
> Overall the patchset looks good to me. I'd suggest we keep
> CONFIG_HID_THINGM as a stub though, that'd just select CONFIG_HID_LED, so
> that we don't cause kernel config regressions for users who had
> CONFIG_HID_THINGM already turned on.
>
OK
I'll add this and provide a v2 of the patch removing the old thingm driver.
> Also adding Vivien Didelot to CC, as the author of the original driver.
> Vivien, the full set can be found in linux-input archives (
> http://marc.info/?l=linux-input&r=1&b=201606&w=2 )
>
Sorry, forgot that.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 4/4] HID: remove ThingM blink(1) driver
2016-06-22 15:42 ` Vivien Didelot
@ 2016-06-22 19:54 ` Heiner Kallweit
2016-06-22 20:03 ` Vivien Didelot
0 siblings, 1 reply; 5+ messages in thread
From: Heiner Kallweit @ 2016-06-22 19:54 UTC (permalink / raw)
To: Vivien Didelot, Jiri Kosina; +Cc: Benjamin Tissoires, linux-input, linux-kernel
Am 22.06.2016 um 17:42 schrieb Vivien Didelot:
> Hi,
>
> Jiri Kosina <jikos@kernel.org> writes:
>
>> On Tue, 21 Jun 2016, Heiner Kallweit wrote:
>>
>>> Now that support for ThingM blink(1) was merged into the hid-led driver
>>> the dedicated driver for this device can be removed.
>>>
>>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>>> ---
>>> drivers/hid/Kconfig | 10 --
>>> drivers/hid/Makefile | 1 -
>>> drivers/hid/hid-thingm.c | 259 -----------------------------------------------
>>> 3 files changed, 270 deletions(-)
>>> delete mode 100644 drivers/hid/hid-thingm.c
>>>
>>> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
>>> index ab22090..6b08a3b 100644
>>> --- a/drivers/hid/Kconfig
>>> +++ b/drivers/hid/Kconfig
>>> @@ -828,16 +828,6 @@ config HID_TOPSEED
>>> Say Y if you have a TopSeed Cyberlink or BTC Emprex or Conceptronic
>>> CLLRCMCE remote control.
>>>
>>> -config HID_THINGM
>>
>> Overall the patchset looks good to me. I'd suggest we keep
>> CONFIG_HID_THINGM as a stub though, that'd just select CONFIG_HID_LED, so
>> that we don't cause kernel config regressions for users who had
>> CONFIG_HID_THINGM already turned on.
>>
>> Also adding Vivien Didelot to CC, as the author of the original driver.
>> Vivien, the full set can be found in linux-input archives (
>> http://marc.info/?l=linux-input&r=1&b=201606&w=2 )
>
> Thanks a lot for the follow-up Jiri.
>
> If you guys decide to remove hid-thingm.c, you need to update the
> MAINTAINERS entry, checkpatch.pl should have warned about that.
>
Right, thanks for the hint.
> I can hardly test the patchset now. Heiner, can you confirm me that with
> your patchset, we will still see 3 led_classdev in sysfs for the
> blink(1) mk1 (which has one RGB chip) and 6 for mk2 (which has 2 chips)?
>
Yes, this will still be the case.
Regards, Heiner
> Thanks,
>
> Vivien
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 4/4] HID: remove ThingM blink(1) driver
2016-06-22 19:54 ` Heiner Kallweit
@ 2016-06-22 20:03 ` Vivien Didelot
0 siblings, 0 replies; 5+ messages in thread
From: Vivien Didelot @ 2016-06-22 20:03 UTC (permalink / raw)
To: Heiner Kallweit, Jiri Kosina
Cc: Benjamin Tissoires, linux-input, linux-kernel
Hi Heiner,
Heiner Kallweit <hkallweit1@gmail.com> writes:
>>> Also adding Vivien Didelot to CC, as the author of the original driver.
>>> Vivien, the full set can be found in linux-input archives (
>>> http://marc.info/?l=linux-input&r=1&b=201606&w=2 )
>>
>> Thanks a lot for the follow-up Jiri.
>>
>> If you guys decide to remove hid-thingm.c, you need to update the
>> MAINTAINERS entry, checkpatch.pl should have warned about that.
>>
> Right, thanks for the hint.
>
>> I can hardly test the patchset now. Heiner, can you confirm me that with
>> your patchset, we will still see 3 led_classdev in sysfs for the
>> blink(1) mk1 (which has one RGB chip) and 6 for mk2 (which has 2 chips)?
>>
> Yes, this will still be the case.
Looks good to me then! Please Cc' me in the v2 so I can hack the change.
Thanks,
Vivien
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-22 20:04 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <f8c59082-d76c-73aa-70b6-7454b731a055@gmail.com>
2016-06-22 13:25 ` [PATCH 4/4] HID: remove ThingM blink(1) driver Jiri Kosina
2016-06-22 15:42 ` Vivien Didelot
2016-06-22 19:54 ` Heiner Kallweit
2016-06-22 20:03 ` Vivien Didelot
2016-06-22 19:52 ` Heiner Kallweit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).