From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 70E8A2C21D8 for ; Sat, 20 Sep 2025 20:22:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758399756; cv=none; b=kPHqzuHlSTpKya6Z4O1t9Kzgisdjq4tz65dvnQJpqlEjl6Tc4u0wJpGDQEgky/9cS8sKTIh6OWVHx+rvsx37RVtOwM9DHfglxqSK6QyuLQnrw5mMLrM8vFGmvCALSm88/jycciCfSOAFYX8tChvHrI2LACKrzwouDc9DnBUHMC4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758399756; c=relaxed/simple; bh=Am3FqAor/UTkbjr2Ye8bRp/s+L2cvBen6QV6UQ5vJDE=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=g5tOeNviga8C0JFfNIHPGPJTqHnylotg9Kh5SkfoTnInwLC5ix73q1grBkscQD+ZnllSCijVqvW4wYI6ZMBWh+cE5O2JJWyeBuk5oeTTlqQ3IAza3OWT2bEcAS2woNZV0eQKxpYQrnYOI1HYF05Lu50RmjCAp3Ezd8xEujtH2h0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=CjRtdm5Q; arc=none smtp.client-ip=209.85.216.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="CjRtdm5Q" Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-33067909400so2057757a91.2 for ; Sat, 20 Sep 2025 13:22:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1758399754; x=1759004554; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to; bh=oEDh+Wvs/2MhWrMqDGofeZax2aipiW7qS5Up+Qu8dgY=; b=CjRtdm5QANpR1A7Vu1tlBf24Iv7p5TWXvsy1q8IFnBj1Q/klm0xEVuiK58/6dYrR3y /DsBGbxlEKH9JDE4LnAcPqEnugfH/HWGM0O64whTMiB2cTWN1yimUE2pf4tL0DmWp2FN BN8d4pH+zapGrmJMiT+lVv9SMzDIQ2ZM3tfZjTLuGOWjGIWkE+1vLwNJ3HDnubR3+LxG PPoePo5H0t5m/jeqaWnpfzIr6CDk+iyqWHJeB5Fpwqfvl/+lmRykl8iuBMjU54rLV7IA 8LLZo53VHIQSboJeI6/ED65n0ZIxkRbjt8rE9+EoQ98yxUSqGy98J+8JyPJfOo0UlcTi cwdQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758399754; x=1759004554; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=oEDh+Wvs/2MhWrMqDGofeZax2aipiW7qS5Up+Qu8dgY=; b=rwTykb3MLOD+Z4iKSPnHrqJGCX/T7ZeSVnH1+r6LYzpYASU11/2ZMdTO11ew1PVeo7 jgR62YJpLp5UILqqe5HuRvkrYOfEkjVYHvcuZsAdzcdy9W4QRP/M92+lsZsYp2Fxuyb9 OJpqOBZ/rR2siAVn208ogJvZAIHYXNRHvjg3UxWWORQjiaHvo6tNe+c8TtOCO0Y/U/81 B+OS64hSyVg3quAXbHwzsRQnlMkdONSLJL/x3+45ubdDbQkiWRUREvUeIx/jfIOCe0rd 7XzZzt2ywM4Z3f8qyvDwU3kwTJXhcsbkf8gmpazPmC613pBAt3S5eTJ+HxScIe8P6gN8 HScQ== X-Forwarded-Encrypted: i=1; AJvYcCW4mD1xSps9k+lZR9Ywpaw8yzQ13sk62fPtY3Bkv0oOBW1J2ge/5IisCq3sXAEtccLE1zY+Sg0/bgWR1Wyy@vger.kernel.org X-Gm-Message-State: AOJu0YxYG77oevtTl70j9yG3mm9mjmwymBB2fwKfBD5ftZ+38/ioeTnQ D1dcWwalkhne/ku2bzMMIEESr6LcRTiyto9Xc0uJsS+E+RVf8Mm4z0d0 X-Gm-Gg: ASbGncuJW/OV+k4qID3pBqjWFJf4miNxJUVYQhu5jteJVSFHT125tfjyIAUcGCbctXa JXcVLTxl9KP3F8gFYEojfQ2vo0olF6puouQjpPDt5NGOv/+IfV6xYALQHtHwRUabiJ7Fh7YHjEy n+UtfPEsn1KjLcgHxu5iDifHASw9P2WsXxnBmdG0+X20SFKpqYYV6WH7JrMGhb8u7erzaJK0brA ZAym7OqNpd2WOv2vclW5smFqpURaH3eRwCWvMmeWGlbNQlUjY80u37Eqr048GaF73uMqI6I1Syg h1KAA0XbASMIvTp6ZaoqfFdgJ9sVjvmf/msXPxUjV7Jpxpxkro2GwczcOEia7NhQX6Ffv6zfxop tEHZXYCWwiJ9esMgeaYX0ixeMNoE6ZqCPc4Xlyw56Gy0= X-Google-Smtp-Source: AGHT+IG3D81KvO3jDKAMB7j9mZS/RFnD2xJducLybPfAPsNEHFakSNlx4mse/NO+LH4HnBqVFQfMJg== X-Received: by 2002:a17:90b:2c8f:b0:32e:96b1:fb70 with SMTP id 98e67ed59e1d1-33098004a50mr9769248a91.12.1758399753726; Sat, 20 Sep 2025 13:22:33 -0700 (PDT) Received: from ehlo.thunderbird.net ([191.193.70.152]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-3309c81f50bsm5417043a91.23.2025.09.20.13.22.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 20 Sep 2025 13:22:33 -0700 (PDT) Date: Sat, 20 Sep 2025 17:22:28 -0300 From: =?ISO-8859-1?Q?Eric_Gon=E7alves?= To: Dmitry Baryshkov CC: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/5] arm64: dts: qcom: r0q: add gpio keys User-Agent: Thunderbird for Android In-Reply-To: References: <20250920014637.38175-1-ghatto404@gmail.com> <20250920014637.38175-2-ghatto404@gmail.com> <5436E420-E459-4E47-9752-EF80F550FA6F@gmail.com> Message-ID: Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On September 20, 2025 5:15:41 PM GMT-03:00, Dmitry Baryshkov wrote: >On Sat, Sep 20, 2025 at 03:02:42PM -0300, Eric Gon=C3=A7alves wrote: >>=20 >>=20 >> On September 20, 2025 12:45:27 PM GMT-03:00, Dmitry Baryshkov wrote: >> >On Sat, Sep 20, 2025 at 01:46:33AM +0000, Eric Gon=C3=A7alves wrote: >> >> Add GPIO keys support to Samsung Galaxy S22 (r0q)=2E >> >>=20 >> >> Signed-off-by: Eric Gon=C3=A7alves >> >> --- >> >> =2E=2E=2E/boot/dts/qcom/sm8450-samsung-r0q=2Edts | 53 ++++++++= +++++++---- >> >> 1 file changed, 44 insertions(+), 9 deletions(-) >> >>=20 >> >> diff --git a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q=2Edts b/arc= h/arm64/boot/dts/qcom/sm8450-samsung-r0q=2Edts >> >> index 880d74ae6032=2E=2E70e953824996 100644 >> >> --- a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q=2Edts >> >> +++ b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q=2Edts >> >> @@ -2,11 +2,12 @@ >> >> =20 >> >> /dts-v1/; >> >> =20 >> >> +#include >> >> #include >> >> - >> >> #include "sm8450=2Edtsi" >> >> #include "pm8350=2Edtsi" >> >> #include "pm8350c=2Edtsi" >> >> +#include "pmk8350=2Edtsi" >> >> =20 >> >> / { >> >> model =3D "Samsung Galaxy S22 5G"; >> >> @@ -28,13 +29,19 @@ framebuffer: framebuffer@b8000000 { >> >> }; >> >> }; >> >> =20 >> >> - vph_pwr: regulator-vph-pwr { >> >> - compatible =3D "regulator-fixed"; >> >> - regulator-name =3D "vph_pwr"; >> >> - regulator-min-microvolt =3D <3700000>; >> >> - regulator-max-microvolt =3D <3700000>; >> >> - regulator-always-on; >> >> - regulator-boot-on; >> > >> >Please don't mix up refactorings and new features=2E Split this patch = into >> >several=2E >> The patch only added gpio-keys node and pon_resin, >> pon_pwrkey=2E Do you mean I have to split each button >> into separate patches? > >No=2E The patch also moves regulator-vph-pwr and changes the comment in >the TLMM node=2E > I thought they would be too small to be patches on their own=2E Is it okay if I split them into 2: arm64: dts: qcom: r0q: small refactor and arm64: dts: qcom: r0q: add gpio keys? >> > >> >> + gpio-keys { >> >> + compatible =3D "gpio-keys"; >> >> + autorepeat; >> >> + >> >> + pinctrl-0 =3D <&vol_up_n>; >> >> + pinctrl-names =3D "default"; >> >> + >> >> + key-vol-up { >> >> + label =3D "Volume Up"; >> >> + linux,code =3D ; >> >> + gpios =3D <&pm8350_gpios 6 GPIO_ACTIVE_LOW>; >> >> + debounce-interval =3D <15>; >> >> + }; >> >> }; >> >> =20 >> >> reserved-memory { >> >> @@ -47,6 +54,15 @@ splash-region@b8000000 { >> >> no-map; >> >> }; >> >> }; >> >> + >> >> + vph_pwr: regulator-vph-pwr { >> >> + compatible =3D "regulator-fixed"; >> >> + regulator-name =3D "vph_pwr"; >> >> + regulator-min-microvolt =3D <3700000>; >> >> + regulator-max-microvolt =3D <3700000>; >> >> + regulator-always-on; >> >> + regulator-boot-on; >> >> + }; >> >> }; >> >> =20 >> >> &apps_rsc { >> >> @@ -119,8 +135,27 @@ vreg_l1c_1p8: ldo1 { >> >> }; >> >> }; >> >> =20 >> >> +&pm8350_gpios { >> >> + vol_up_n: vol-up-n-state { >> >> + pins =3D "gpio6"; >> >> + function =3D "normal"; >> >> + power-source =3D <1>; >> >> + input-enable; >> >> + }; >> >> +}; >> >> + >> >> +&pon_pwrkey { >> >> + status =3D "okay"; >> >> +}; >> >> + >> >> +&pon_resin { >> >> + linux,code =3D ; >> >> + >> >> + status =3D "okay"; >> >> +}; >> >> + >> >> &tlmm { >> >> - gpio-reserved-ranges =3D <36 4>; /* SPI (not linked to anything) *= / >> >> + gpio-reserved-ranges =3D <36 4>; /* SPI (Unused) */ >> >> }; >> >> =20 >> >> &usb_1 { >> >> --=20 >> >> 2=2E51=2E0 >> >>=20 >> > >> Resend because I forgot to CC the mailing lists >