Re: intel-hid: fix volume buttons on Microsoft Surface Go 4 tablet

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dmitry,

On 19-Feb-25 11:08 PM, Dmitry Panchenko | d-Systems wrote:
> Hello,
> 
> Volume buttons on Microsoft Surface Go 4 tablet didn't send any events. Adding Surface Go 4 to button_array_table fix this.
> 
> The patch is attached.
> 
> Signed-off-by: Dmitry Panchenko <dmitry@xxxxxxxxxxxx>

Thank you for your patch. Unfortunately this is not in the correct format
for submitting patches. It is close though, good job for a first try.

To turn this into a proper patch, make the changes from your attached
patch to a git clone of a recent kernel tree (e.g, Linus Torvald's tree)
and then after making the changes do:

git commit -as

and for the commit message use:

"""
platform/x86: intel-hid: fix volume buttons on Microsoft Surface Go 4 tablet

Volume buttons on Microsoft Surface Go 4 tablet didn't send any events.
Add Surface Go 4 DMI match to button_array_table to fix this.

Signed-off-by: Dmitry Panchenko <dmitry@xxxxxxxxxxxx>
"""

and then do:

git format-patch HEAD~

to get a 0001-xxxxxx.patch file, followed by:

git send-email 0001-xxxxxx.patch

to send an email.

Note this also requires that you first setup a ~/.gitconfig for your
name + email, as well as send-email config, e.g. I have:

"""
[user]
        name = Hans de Goede
        email = hdegoede@xxxxxxxxxx

[sendemail]
        from = Hans de Goede <hdegoede@xxxxxxxxxx>
        smtpserver = smtp.corp.redhat.com
"""

Or alternatively I can send a v2 patch for your putting things in
the right format.

Regards,

Hans






[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux