mbox series

[0/3] mt6779-keypad: fix compatibles and add to mt8516

Message ID 20241225192631.25017-1-val@packett.cool
Headers show
Series mt6779-keypad: fix compatibles and add to mt8516 | expand

Message

Val Packett Dec. 25, 2024, 7:26 p.m. UTC 
The mt8516 patch was previously sent in another series but rejected due
to not using an mt8516 specific compatible - which the other usages
of this driver (mt8183 and mt8365) weren't doing either.

It's only fair to add the SoC-specific compatibles to all of them then :)

Val Packett (3):
  dt-bindings: mediatek,mt6779-keypad: add more compatibles
  arm64: dts: mediatek: add per-SoC compatibles for keypad nodes
  arm64: dts: mediatek: mt8516: add keypad node

 .../bindings/input/mediatek,mt6779-keypad.yaml        |  3 +++
 arch/arm64/boot/dts/mediatek/mt8183.dtsi              |  3 ++-
 arch/arm64/boot/dts/mediatek/mt8365.dtsi              |  3 ++-
 arch/arm64/boot/dts/mediatek/mt8516.dtsi              | 11 +++++++++++
 4 files changed, 18 insertions(+), 2 deletions(-)

Comments

AngeloGioacchino Del Regno Jan. 2, 2025, 10:29 a.m. UTC | #1 
Il 25/12/24 20:26, Val Packett ha scritto:
> The mt8516 patch was previously sent in another series but rejected due
> to not using an mt8516 specific compatible - which the other usages
> of this driver (mt8183 and mt8365) weren't doing either.
> 
> It's only fair to add the SoC-specific compatibles to all of them then :)
> 

Nice!!

Whole series is

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

> Val Packett (3):
>    dt-bindings: mediatek,mt6779-keypad: add more compatibles
>    arm64: dts: mediatek: add per-SoC compatibles for keypad nodes
>    arm64: dts: mediatek: mt8516: add keypad node
> 
>   .../bindings/input/mediatek,mt6779-keypad.yaml        |  3 +++
>   arch/arm64/boot/dts/mediatek/mt8183.dtsi              |  3 ++-
>   arch/arm64/boot/dts/mediatek/mt8365.dtsi              |  3 ++-
>   arch/arm64/boot/dts/mediatek/mt8516.dtsi              | 11 +++++++++++
>   4 files changed, 18 insertions(+), 2 deletions(-)
>
Mattijs Korpershoek Jan. 3, 2025, 10:58 a.m. UTC | #2 
Hi Val,

Thank you for the patches.

On mer., déc. 25, 2024 at 16:26, Val Packett <val@packett.cool> wrote:

> The mt8516 patch was previously sent in another series but rejected due
> to not using an mt8516 specific compatible - which the other usages
> of this driver (mt8183 and mt8365) weren't doing either.
>
> It's only fair to add the SoC-specific compatibles to all of them then :)
>
> Val Packett (3):
>   dt-bindings: mediatek,mt6779-keypad: add more compatibles
>   arm64: dts: mediatek: add per-SoC compatibles for keypad nodes
>   arm64: dts: mediatek: mt8516: add keypad node

For the series,

Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>

>
>  .../bindings/input/mediatek,mt6779-keypad.yaml        |  3 +++
>  arch/arm64/boot/dts/mediatek/mt8183.dtsi              |  3 ++-
>  arch/arm64/boot/dts/mediatek/mt8365.dtsi              |  3 ++-
>  arch/arm64/boot/dts/mediatek/mt8516.dtsi              | 11 +++++++++++
>  4 files changed, 18 insertions(+), 2 deletions(-)
>
> -- 
> 2.47.1