Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Date:   Tue, 28 Nov 2023 02:34:34 +0000
To:     Alexander Koskovich <AKoskovich@pm.me>
From:   Rahul Rameshbabu <sergeantsagara@protonmail.com>
Cc:     roderick.colenbrander@sony.com, jikos@kernel.org,
        benjamin.tissoires@redhat.com, linux-input@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/1] hid-playstation: Fix button maps for the DualSense Edge controller
Message-ID: <8734wqehc9.fsf@protonmail.com>
Feedback-ID: 26003777:user:proton
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Tue, 28 Nov, 2023 02:19:42 +0000 "Alexander Koskovich" <AKoskovich@pm.me=
> wrote:
> To clarify, I did test this patch locally on Fedora Rawhide and confirm i=
t works with games. It does resolve the issue, and is a workaround.
> It's not just Steam/Proton I'm seeing this issue in, I'm seeing it in nat=
ive Linux games like SuperTuxKart.
>

Thanks for confirming. I am not that familiar with stk but I believe it
uses SDL2, which might be your common culprit.

  https://github.com/supertuxkart/stk-code/blob/a57ac415bbaf8e22a1c35f3ac09=
49c4dca322637/src/input/sdl_controller.cpp#L288

>
>
> On Monday, November 27th, 2023 at 9:16 PM, Rahul Rameshbabu <sergeantsaga=
ra@protonmail.com> wrote:
>
>
>>=20
>>=20
>> On Sun, 26 Nov, 2023 00:15:49 +0000 "Alexander Koskovich" AKoskovich@pm.=
me wrote:
>>=20
>> > This brings functionality of the DualSense Edge controller inline
>> > with the stock PS5 controller.
>> >=20
>> > Signed-off-by: Alexander Koskovich akoskovich@pm.me
>> > ---
>>=20
>>=20
>> Will provide a follow-up to the relevant discussion.
>>=20
>> https://lore.kernel.org/linux-input/P8jVfYBAwiM_8MIgshN0osVVfshfBH2-oZCQ=
uqoqh0Hy76_031zuZvYXWl0edtfTUwDOSNlc5priSRXI3G5dboVh5VPbcdxzAcEF7EvUVgo=3D@=
protonmail.com/T/#t
>>=20
>> Since I assume this patch was not actually tested to resolve the issue
>> based on the evtest results, I think we should drop this patch. Will
>> mention some details I might have with regards to the behavior you are
>> seeing with Steam/Proton specifically.
>>=20
>> --
>> Thanks,
>>=20
>> Rahul Rameshbabu