Re: [PATCH 1/2] Input: add `SW_MACHINE_COVER`

From: Merlijn Wajer
Date: Mon Jun 29 2020 - 17:49:15 EST


Hi,

Looks like we're blocking on this input patch.

On 16/06/2020 12:50, Pavel Machek wrote:
> On Fri 2020-06-12 14:53:58, Merlijn Wajer wrote:
>> This event code represents the state of a removable cover of a device.
>> Value 0 means that the cover is open or removed, value 1 means that the
>> cover is closed.
>>
>> Reviewed-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx> Acked-by: Tony Lindgren
>> <tony@xxxxxxxxxxx> Signed-off-by: Merlijn Wajer <merlijn@xxxxxxxxxx> ---
>
> Dmitry, can we get some kind of comment here, or better yet can we get you to apply this?

This is part of a patch series to resolve problems with the Nokia N900
not booting when the cover is removed (making the cover be the card
detect was also just weird IMHO). Just removing the card-detect from the
DTS is fine, but it was suggested that we expose the data instead as
input event. And that's gotten no response for about four months.

Should we just drop the feature and only remove the cd-gpios line from
the DTS, assuming upstream doesn't want this SW_MACHINE_COVER code?

Merlijn

Attachment: signature.asc
Description: OpenPGP digital signature