* [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible
@ 2022-11-28 17:37 Luca Weiss
2022-11-28 17:37 ` [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC Luca Weiss
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Luca Weiss @ 2022-11-28 17:37 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss,
Krzysztof Kozlowski, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni, Rob Herring, netdev, devicetree,
linux-kernel
The NQ310 is another NFC chip from NXP, document the compatible in the
bindings.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
RESEND to fix Cc
Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
index b2558421268a..6924aff0b2c5 100644
--- a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
+++ b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
@@ -14,7 +14,9 @@ properties:
oneOf:
- const: nxp,nxp-nci-i2c
- items:
- - const: nxp,pn547
+ - enum:
+ - nxp,nq310
+ - nxp,pn547
- const: nxp,nxp-nci-i2c
enable-gpios:
--
2.38.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC
2022-11-28 17:37 [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Luca Weiss
@ 2022-11-28 17:37 ` Luca Weiss
2022-11-29 13:29 ` Konrad Dybcio
2022-11-28 17:46 ` [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Krzysztof Kozlowski
` (2 subsequent siblings)
3 siblings, 1 reply; 8+ messages in thread
From: Luca Weiss @ 2022-11-28 17:37 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss, Andy Gross,
Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
devicetree, linux-kernel
Configure the node for the NQ310 chip found on this device, which is
compatible with generic nxp-nci-i2c driver.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
RESEND to fix Cc
arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
index 2920504461d3..fde93cbcd180 100644
--- a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
+++ b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
@@ -63,6 +63,21 @@ touchscreen@48 {
};
};
+&i2c_5 {
+ status = "okay";
+
+ nfc@28 {
+ compatible = "nxp,nq310", "nxp,nxp-nci-i2c";
+ reg = <0x28>;
+
+ interrupt-parent = <&tlmm>;
+ interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
+
+ enable-gpios = <&tlmm 16 GPIO_ACTIVE_HIGH>;
+ firmware-gpios = <&tlmm 62 GPIO_ACTIVE_HIGH>;
+ };
+};
+
&pm8953_resin {
status = "okay";
linux,code = <KEY_VOLUMEDOWN>;
--
2.38.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible
2022-11-28 17:37 [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Luca Weiss
2022-11-28 17:37 ` [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC Luca Weiss
@ 2022-11-28 17:46 ` Krzysztof Kozlowski
2022-12-01 5:50 ` patchwork-bot+netdevbpf
2022-12-28 4:36 ` (subset) " Bjorn Andersson
3 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-28 17:46 UTC (permalink / raw)
To: Luca Weiss, linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Rob Herring, netdev,
devicetree, linux-kernel
On 28/11/2022 18:37, Luca Weiss wrote:
> The NQ310 is another NFC chip from NXP, document the compatible in the
> bindings.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
> RESEND to fix Cc
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC
2022-11-28 17:37 ` [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC Luca Weiss
@ 2022-11-29 13:29 ` Konrad Dybcio
2022-11-29 18:31 ` Luca Weiss
0 siblings, 1 reply; 8+ messages in thread
From: Konrad Dybcio @ 2022-11-29 13:29 UTC (permalink / raw)
To: Luca Weiss, linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Andy Gross,
Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, devicetree,
linux-kernel
On 28.11.2022 18:37, Luca Weiss wrote:
> Configure the node for the NQ310 chip found on this device, which is
> compatible with generic nxp-nci-i2c driver.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
> RESEND to fix Cc
>
> arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> index 2920504461d3..fde93cbcd180 100644
> --- a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> @@ -63,6 +63,21 @@ touchscreen@48 {
> };
> };
>
> +&i2c_5 {
> + status = "okay";
> +
> + nfc@28 {
> + compatible = "nxp,nq310", "nxp,nxp-nci-i2c";
Unless there was some binding change (that was not emailed to me with
this email, I only got 2/2), only "nxp,nxp-nci-i2c" and
"nxp,pn547", "nxp,nxp-nci-i2c" are allowed.
The node looks good though.
Konrad
> + reg = <0x28>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
> +
> + enable-gpios = <&tlmm 16 GPIO_ACTIVE_HIGH>;
> + firmware-gpios = <&tlmm 62 GPIO_ACTIVE_HIGH>;
> + };
> +};
> +
> &pm8953_resin {
> status = "okay";
> linux,code = <KEY_VOLUMEDOWN>;
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC
2022-11-29 13:29 ` Konrad Dybcio
@ 2022-11-29 18:31 ` Luca Weiss
2022-11-30 14:19 ` Konrad Dybcio
0 siblings, 1 reply; 8+ messages in thread
From: Luca Weiss @ 2022-11-29 18:31 UTC (permalink / raw)
To: linux-arm-msm, Konrad Dybcio
Cc: ~postmarketos/upstreaming, phone-devel, Andy Gross,
Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, devicetree,
linux-kernel
On Dienstag, 29. November 2022 14:29:36 CET Konrad Dybcio wrote:
> On 28.11.2022 18:37, Luca Weiss wrote:
> > Configure the node for the NQ310 chip found on this device, which is
> > compatible with generic nxp-nci-i2c driver.
> >
> > Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> > ---
> > RESEND to fix Cc
> >
> > arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts | 15 +++++++++++++++
> > 1 file changed, 15 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> > b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts index
> > 2920504461d3..fde93cbcd180 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> > +++ b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
> > @@ -63,6 +63,21 @@ touchscreen@48 {
> >
> > };
> >
> > };
> >
> > +&i2c_5 {
> > + status = "okay";
> > +
> > + nfc@28 {
> > + compatible = "nxp,nq310", "nxp,nxp-nci-i2c";
>
> Unless there was some binding change (that was not emailed to me with
> this email, I only got 2/2), only "nxp,nxp-nci-i2c" and
> "nxp,pn547", "nxp,nxp-nci-i2c" are allowed.
Yes, I've added the double compatible to the yaml.
https://lore.kernel.org/lkml/20221128173744.833018-1-luca@z3ntu.xyz/
I'll try to work on my Cc-script setup, currently it adds the output of
get_maintainers.pl for each patch (and adds all for the cover letter), which
is based on some script I found a while ago online.
Is there like a recommended way to put all people in Cc for a series, I'm not
aware of anything at least...
Perhaps I'll look into b4 for sending patches, that seems to have some goodies
there.
Regards
Luca
>
> The node looks good though.
>
> Konrad
>
> > + reg = <0x28>;
> > +
> > + interrupt-parent = <&tlmm>;
> > + interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
> > +
> > + enable-gpios = <&tlmm 16 GPIO_ACTIVE_HIGH>;
> > + firmware-gpios = <&tlmm 62 GPIO_ACTIVE_HIGH>;
> > + };
> > +};
> > +
> >
> > &pm8953_resin {
> >
> > status = "okay";
> > linux,code = <KEY_VOLUMEDOWN>;
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC
2022-11-29 18:31 ` Luca Weiss
@ 2022-11-30 14:19 ` Konrad Dybcio
0 siblings, 0 replies; 8+ messages in thread
From: Konrad Dybcio @ 2022-11-30 14:19 UTC (permalink / raw)
To: Luca Weiss, linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Andy Gross,
Bjorn Andersson, Rob Herring, Krzysztof Kozlowski, devicetree,
linux-kernel
On 29.11.2022 19:31, Luca Weiss wrote:
> On Dienstag, 29. November 2022 14:29:36 CET Konrad Dybcio wrote:
>> On 28.11.2022 18:37, Luca Weiss wrote:
>>> Configure the node for the NQ310 chip found on this device, which is
>>> compatible with generic nxp-nci-i2c driver.
>>>
>>> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
>>> ---
>>> RESEND to fix Cc
>>>
>>> arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts | 15 +++++++++++++++
>>> 1 file changed, 15 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
>>> b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts index
>>> 2920504461d3..fde93cbcd180 100644
>>> --- a/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
>>> +++ b/arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts
>>> @@ -63,6 +63,21 @@ touchscreen@48 {
>>>
>>> };
>>>
>>> };
>>>
>>> +&i2c_5 {
>>> + status = "okay";
>>> +
>>> + nfc@28 {
>>> + compatible = "nxp,nq310", "nxp,nxp-nci-i2c";
>>
>> Unless there was some binding change (that was not emailed to me with
>> this email, I only got 2/2), only "nxp,nxp-nci-i2c" and
>> "nxp,pn547", "nxp,nxp-nci-i2c" are allowed.
>
> Yes, I've added the double compatible to the yaml.
> https://lore.kernel.org/lkml/20221128173744.833018-1-luca@z3ntu.xyz/
Ok, this looks good then.
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>
> I'll try to work on my Cc-script setup, currently it adds the output of
> get_maintainers.pl for each patch (and adds all for the cover letter), which
> is based on some script I found a while ago online.
>
> Is there like a recommended way to put all people in Cc for a series, I'm not
> aware of anything at least...
>
> Perhaps I'll look into b4 for sending patches, that seems to have some goodies
> there.
Check out this reply from Rob Herring here https://lore.kernel.org/lkml/CAL_JsqLubWBr2W3xZPsuPLOGav7CFgBdH=aCfT22F_m0_cx3cQ@mail.gmail.com/
Konrad
>
> Regards
> Luca
>
>>
>> The node looks good though.
>>
>> Konrad
>>
>>> + reg = <0x28>;
>>> +
>>> + interrupt-parent = <&tlmm>;
>>> + interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
>>> +
>>> + enable-gpios = <&tlmm 16 GPIO_ACTIVE_HIGH>;
>>> + firmware-gpios = <&tlmm 62 GPIO_ACTIVE_HIGH>;
>>> + };
>>> +};
>>> +
>>>
>>> &pm8953_resin {
>>>
>>> status = "okay";
>>> linux,code = <KEY_VOLUMEDOWN>;
>
>
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible
2022-11-28 17:37 [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Luca Weiss
2022-11-28 17:37 ` [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC Luca Weiss
2022-11-28 17:46 ` [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Krzysztof Kozlowski
@ 2022-12-01 5:50 ` patchwork-bot+netdevbpf
2022-12-28 4:36 ` (subset) " Bjorn Andersson
3 siblings, 0 replies; 8+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-12-01 5:50 UTC (permalink / raw)
To: Luca Weiss
Cc: linux-arm-msm, ~postmarketos/upstreaming, phone-devel,
krzysztof.kozlowski, davem, edumazet, kuba, pabeni, robh+dt,
netdev, devicetree, linux-kernel
Hello:
This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:
On Mon, 28 Nov 2022 18:37:43 +0100 you wrote:
> The NQ310 is another NFC chip from NXP, document the compatible in the
> bindings.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
> RESEND to fix Cc
>
> [...]
Here is the summary with links:
- [RESEND,1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible
https://git.kernel.org/netdev/net-next/c/a933e7f05bd4
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: (subset) [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible
2022-11-28 17:37 [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Luca Weiss
` (2 preceding siblings ...)
2022-12-01 5:50 ` patchwork-bot+netdevbpf
@ 2022-12-28 4:36 ` Bjorn Andersson
3 siblings, 0 replies; 8+ messages in thread
From: Bjorn Andersson @ 2022-12-28 4:36 UTC (permalink / raw)
To: luca, linux-arm-msm
Cc: ~postmarketos/upstreaming, netdev, edumazet, krzysztof.kozlowski,
linux-kernel, davem, robh+dt, devicetree, kuba, phone-devel,
pabeni
On Mon, 28 Nov 2022 18:37:43 +0100, Luca Weiss wrote:
> The NQ310 is another NFC chip from NXP, document the compatible in the
> bindings.
>
>
Applied, thanks!
[2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC
commit: 29dcf3c1a8159acdf56905c377a214381eda5a24
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-12-28 4:39 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-28 17:37 [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Luca Weiss
2022-11-28 17:37 ` [RESEND PATCH 2/2] arm64: dts: qcom: sdm632-fairphone-fp3: Add NFC Luca Weiss
2022-11-29 13:29 ` Konrad Dybcio
2022-11-29 18:31 ` Luca Weiss
2022-11-30 14:19 ` Konrad Dybcio
2022-11-28 17:46 ` [RESEND PATCH 1/2] dt-bindings: nfc: nxp,nci: Document NQ310 compatible Krzysztof Kozlowski
2022-12-01 5:50 ` patchwork-bot+netdevbpf
2022-12-28 4:36 ` (subset) " Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).