mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Guido Günther" <agx@sigxcpu.org>
To: david@ixit.cz
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Jonathan Corbet <corbet@lwn.net>, Jiri Kosina <jikos@kernel.org>,
	Benjamin Tissoires <bentiss@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konradybcio@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Casey Connolly <casey.connolly@linaro.org>,
	linux-input@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	devicetree@vger.kernel.org, phone-devel@vger.kernel.org,
	Gergo Koteles <soyer@irl.hu>,
	Casey Connolly <casey@connolly.tech>
Subject: Re: [PATCH v7 0/3] Add support for sound profile switching and leverage for OnePlus slider
Date: Mon, 27 Oct 2025 10:57:05 +0100	[thread overview]
Message-ID: <aP9B8fPs7y2-dGJi@quark2.heme.sigxcpu.org> (raw)
In-Reply-To: <20251014-op6-tri-state-v7-0-938a6367197b@ixit.cz>

Hi,
On Tue, Oct 14, 2025 at 11:20:32AM +0200, David Heidelberg via B4 Relay wrote:
> This series add initial support for OnePlus 6 and 6T, but other OnePlus
> phones contains same mechanism to switch sound profiles.
> 
> This code was tested for two years within the downstream Snapdragon 845 tree.
> It is now perfectly integrated with feedbackd in the Phosh environment.
> 
> The series is also available (until merged) at
>   git@gitlab.com:dhxx/linux.git b4/op6-tri-state
> 
> Changes in v7:
> - Separated GPIO number fix from the original commit
>   "arm64: dts: qcom: sdm845-oneplus: Add alert-slider"
> - Rebased again next-20251008
> - Link to v6: https://lore.kernel.org/r/20250731-op6-tri-state-v6-0-569c25cbc8c2@ixit.cz
> 
> Changes in v6:
> - Rebased again next-20250731, otherwise just a resent.
> - Link to v5: https://lore.kernel.org/r/20250419-op6-tri-state-v5-0-443127078517@ixit.cz
> 
> Changes in v5:
> - Dropped merged
>   "Input: gpio-keys - add support for linux,input-value DTS property"
> - Link to v4: https://lore.kernel.org/all/cover.1677022414.git.soyer@irl.hu/
> 
> Changes in v4:
> - DTS: use default debounce-interval, order alphabetically
> - Link to v3: https://lore.kernel.org/lkml/cover.1676850819.git.soyer@irl.hu/
> 
> Changes in v3:
> - rename tri-state-key to alert-slider, fix DTS warnings,
> 
> Changes in v2:
> - rebase to qcom/for-next
> add SND_PROFILE_* identifiers to input-event-codes.h
> 
> ---
> Gergo Koteles (3):
>       Input: add ABS_SND_PROFILE
>       arm64: dts: qcom: sdm845-oneplus: Correct gpio used for slider
>       arm64: dts: qcom: sdm845-oneplus: Add alert-slider

feedbackd has support for this since 0.5.0 so it would be nice to see
that mainlined.

Tested-by: Guido Günther <agx@sigxcpu.org> # oneplus,fajita & oneplus,enchilada
Reviewed-by: Guido Günther <agx@sigxcpu.org> 

Cheers,
 -- Guido

> 
>  Documentation/input/event-codes.rst                |  6 ++++
>  .../arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi | 39 ++++++++++++++++++++--
>  drivers/hid/hid-debug.c                            |  1 +
>  include/uapi/linux/input-event-codes.h             |  9 +++++
>  4 files changed, 53 insertions(+), 2 deletions(-)
> ---
> base-commit: 52ba76324a9d7c39830c850999210a36ef023cde
> change-id: 20250419-op6-tri-state-ed1a05a11125
> 
> Best regards,
> -- 
> David Heidelberg <david@ixit.cz>
> 
> 

      parent reply	other threads:[~2025-10-27  9:57 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-14  9:20 David Heidelberg via B4 Relay
2025-10-14  9:20 ` [PATCH v7 1/3] Input: add ABS_SND_PROFILE David Heidelberg via B4 Relay
2025-10-27 17:08   ` Bjorn Andersson
2025-10-14  9:20 ` [PATCH v7 2/3] arm64: dts: qcom: sdm845-oneplus: Correct gpio used for slider David Heidelberg via B4 Relay
2025-10-14  9:22   ` Konrad Dybcio
2025-10-14  9:20 ` [PATCH v7 3/3] arm64: dts: qcom: sdm845-oneplus: Add alert-slider David Heidelberg via B4 Relay
2025-10-27  9:57 ` Guido Günther [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aP9B8fPs7y2-dGJi@quark2.heme.sigxcpu.org \
    --to=agx@sigxcpu.org \
    --cc=andersson@kernel.org \
    --cc=bentiss@kernel.org \
    --cc=casey.connolly@linaro.org \
    --cc=casey@connolly.tech \
    --cc=conor+dt@kernel.org \
    --cc=corbet@lwn.net \
    --cc=david@ixit.cz \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jikos@kernel.org \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=phone-devel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=soyer@irl.hu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®