[PATCH] dm: button: support remapping phone keys

Caleb Connolly caleb.connolly at linaro.org
Mon Jul 15 10:51:07 CEST 2024


Hi Quentin,
>> +static int button_remap_phone_keys(int code)
>> +{
>> +    switch (code) {
>> +    case KEY_VOLUMEUP:
>> +        return KEY_UP;
>> +    case KEY_VOLUMEDOWN:
>> +        return KEY_DOWN;
>> +    case KEY_POWER:
>> +        return KEY_ENTER;
>> +    default:
>> +        return code;
>> +    }
>> +}
>> +
> 
> ....... I suggest to make this a weak function that can be overridden by 
> boards (should it maybe be only defined in boards C file?) so that it's 
> easy for people to come up with their own mapping without having to deal 
> with two people/the maintainer disagreeing with what should be the one 
> and true mapping for that key.

This is intentionally not a board specific feature. It is a generic 
option that does precisely one thing: remap the keys on a phone to be 
useful for navigating boot menus.

If some folks have a strong disagreement about e.g. what KEY_CAMERA 
should be (for the devices that have it) then I would rather propose 
making this all configurable via an environment variable, or better yet 
introducing a new bootloader,code property in devicetree to define what 
a key should do in a bootloader.
> 
> Cheers,
> Quentin

-- 
// Caleb (they/them)


More information about the U-Boot mailing list