2021-05-12 14:46:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

The NFC subsystem is orphaned. I am happy to spend some cycles to
review the patches, send pull requests and in general keep the NFC
subsystem running.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

I admit I don't have big experience in NFC part but this will be nice
opportunity to learn something new. I am already maintainer of few
other parts: memory controller drivers, Samsung ARM/ARM64 SoC and some
drviers. I have a kernel.org account and my GPG key is:
https://git.kernel.org/pub/scm/docs/kernel/pgpkeys.git/tree/keys/1B93437D3B41629B.asc

Best regards,
Krzysztof
---
MAINTAINERS | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index cc81667e8bab..adc6cbe29f78 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12899,8 +12899,9 @@ F: include/uapi/linux/nexthop.h
F: net/ipv4/nexthop.c

NFC SUBSYSTEM
+M: Krzysztof Kozlowski <[email protected]>
L: [email protected]
-S: Orphan
+S: Maintained
F: Documentation/devicetree/bindings/net/nfc/
F: drivers/nfc/
F: include/linux/platform_data/nfcmrvl.h
--
2.25.1


2021-05-12 14:48:31

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/2] MAINTAINERS: nfc: include linux-nfc mailing list

Keep all NFC related patches in existing [email protected] mailing
list.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index adc6cbe29f78..83f18e674880 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12900,6 +12900,7 @@ F: net/ipv4/nexthop.c

NFC SUBSYSTEM
M: Krzysztof Kozlowski <[email protected]>
+L: [email protected] (moderated for non-subscribers)
L: [email protected]
S: Maintained
F: Documentation/devicetree/bindings/net/nfc/
--
2.25.1

2021-05-12 15:37:52

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Wed, May 12, 2021 at 10:43:18AM -0400, Krzysztof Kozlowski wrote:
> The NFC subsystem is orphaned. I am happy to spend some cycles to
> review the patches, send pull requests and in general keep the NFC
> subsystem running.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> I admit I don't have big experience in NFC part but this will be nice
> opportunity to learn something new. I am already maintainer of few
> other parts: memory controller drivers, Samsung ARM/ARM64 SoC and some
> drviers. I have a kernel.org account and my GPG key is:
> https://git.kernel.org/pub/scm/docs/kernel/pgpkeys.git/tree/keys/1B93437D3B41629B.asc
>
> Best regards,
> Krzysztof
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cc81667e8bab..adc6cbe29f78 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12899,8 +12899,9 @@ F: include/uapi/linux/nexthop.h
> F: net/ipv4/nexthop.c
>
> NFC SUBSYSTEM
> +M: Krzysztof Kozlowski <[email protected]>
> L: [email protected]
> -S: Orphan
> +S: Maintained
> F: Documentation/devicetree/bindings/net/nfc/
> F: drivers/nfc/
> F: include/linux/platform_data/nfcmrvl.h

FWIW,

Acked-by: Mark Greer <[email protected]>

2021-05-12 15:38:48

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 2/2] MAINTAINERS: nfc: include linux-nfc mailing list

On Wed, May 12, 2021 at 10:43:19AM -0400, Krzysztof Kozlowski wrote:
> Keep all NFC related patches in existing [email protected] mailing
> list.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index adc6cbe29f78..83f18e674880 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12900,6 +12900,7 @@ F: net/ipv4/nexthop.c
>
> NFC SUBSYSTEM
> M: Krzysztof Kozlowski <[email protected]>
> +L: [email protected] (moderated for non-subscribers)
> L: [email protected]
> S: Maintained
> F: Documentation/devicetree/bindings/net/nfc/

FWIW,

Acked-by: Mark Greer <[email protected]>

2021-05-12 16:03:38

by Daniel Lezcano

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
> The NFC subsystem is orphaned. I am happy to spend some cycles to
> review the patches, send pull requests and in general keep the NFC
> subsystem running.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> I admit I don't have big experience in NFC part but this will be nice
> opportunity to learn something new.

NFC has been lost in the limbos since a while. Good to see someone
volunteering to take care of it.

May I suggest to create a simple nfc reading program in the 'tools'
directory (could be a training exercise ;)


> I am already maintainer of few
> other parts: memory controller drivers, Samsung ARM/ARM64 SoC and some
> drviers. I have a kernel.org account and my GPG key is:
> https://git.kernel.org/pub/scm/docs/kernel/pgpkeys.git/tree/keys/1B93437D3B41629B.asc
>
> Best regards,
> Krzysztof
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cc81667e8bab..adc6cbe29f78 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12899,8 +12899,9 @@ F: include/uapi/linux/nexthop.h
> F: net/ipv4/nexthop.c
>
> NFC SUBSYSTEM
> +M: Krzysztof Kozlowski <[email protected]>
> L: [email protected]
> -S: Orphan
> +S: Maintained
> F: Documentation/devicetree/bindings/net/nfc/
> F: drivers/nfc/
> F: include/linux/platform_data/nfcmrvl.h




--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

2021-05-12 16:21:39

by Frieder Schrempf

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12.05.21 16:43, Krzysztof Kozlowski wrote:
> The NFC subsystem is orphaned. I am happy to spend some cycles to
> review the patches, send pull requests and in general keep the NFC
> subsystem running.

That's great, thanks!

Maybe you also want to have a look at the userspace side and talk to Mark Greer (on cc). He recently said, that he is supposed to be taking over maintenance for the neard daemon (see this thread: [1]) which currently looks like it's close to being dead (no release for several years, etc.).

I don't know much about the NFC stack and if/how people use it, but without reliable and maintained userspace tooling, the whole thing seems of little use in the long run. Qt has already dropped their neard support for Qt 6 [2], which basically means the mainline NFC stack won't be supported anymore in one of the most common application frameworks for IoT/embedded.

[1] https://lists.01.org/hyperkitty/list/[email protected]/thread/OHD5IQHYPFUPUFYWDMNSVCBNO24M45VK/
[2] https://bugreports.qt.io/browse/QTBUG-81824

>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> I admit I don't have big experience in NFC part but this will be nice
> opportunity to learn something new. I am already maintainer of few
> other parts: memory controller drivers, Samsung ARM/ARM64 SoC and some
> drviers. I have a kernel.org account and my GPG key is:
> https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Fdocs%2Fkernel%2Fpgpkeys.git%2Ftree%2Fkeys%2F1B93437D3B41629B.asc&amp;data=04%7C01%7Cfrieder.schrempf%40kontron.de%7Cb7a843cdd3a9421a4fa908d9155456fe%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637564274699221204%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=dJy3BljqfnZfwekCSGDdUy2kNcr7XxoDUUXJJwydq9c%3D&amp;reserved=0
>
> Best regards,
> Krzysztof
> ---
> MAINTAINERS | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cc81667e8bab..adc6cbe29f78 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12899,8 +12899,9 @@ F: include/uapi/linux/nexthop.h
> F: net/ipv4/nexthop.c
>
> NFC SUBSYSTEM
> +M: Krzysztof Kozlowski <[email protected]>
> L: [email protected]
> -S: Orphan
> +S: Maintained
> F: Documentation/devicetree/bindings/net/nfc/
> F: drivers/nfc/
> F: include/linux/platform_data/nfcmrvl.h
>

2021-05-12 16:37:16

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 11:11, Daniel Lezcano wrote:
> On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>> review the patches, send pull requests and in general keep the NFC
>> subsystem running.
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>
>> ---
>>
>> I admit I don't have big experience in NFC part but this will be nice
>> opportunity to learn something new.
>
> NFC has been lost in the limbos since a while. Good to see someone
> volunteering to take care of it.
>
> May I suggest to create a simple nfc reading program in the 'tools'
> directory (could be a training exercise ;)
>

Noted, thanks. I also need to get a simple hardware dongle for this....


Best regards,
Krzysztof

2021-05-12 19:37:01

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Wed, May 12, 2021 at 11:43:13AM -0400, Krzysztof Kozlowski wrote:
> On 12/05/2021 11:11, Daniel Lezcano wrote:
> > On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
> >> The NFC subsystem is orphaned. I am happy to spend some cycles to
> >> review the patches, send pull requests and in general keep the NFC
> >> subsystem running.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >>
> >> ---
> >>
> >> I admit I don't have big experience in NFC part but this will be nice
> >> opportunity to learn something new.
> >
> > NFC has been lost in the limbos since a while. Good to see someone
> > volunteering to take care of it.
> >
> > May I suggest to create a simple nfc reading program in the 'tools'
> > directory (could be a training exercise ;)
> >
>
> Noted, thanks. I also need to get a simple hardware dongle for this....

Krzysztof, the NFC portion of the kernel has a counterpart in userspace
called neard. I'm supposed to be maintaining it but I have next to no
time to do so. If you have spare cycles, any help would be appreciated.

Anyway, in neard, there are some simple test scripts (python2 - I/we need
to update to python3). The current home of neard is:

git://git.kernel.org/pub/scm/network/nfc/neard.git

Mark
--

2021-05-12 19:37:02

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Wed, May 12, 2021 at 05:32:35PM +0200, Frieder Schrempf wrote:
> On 12.05.21 16:43, Krzysztof Kozlowski wrote:
> > The NFC subsystem is orphaned. I am happy to spend some cycles to
> > review the patches, send pull requests and in general keep the NFC
> > subsystem running.
>
> That's great, thanks!
>
> Maybe you also want to have a look at the userspace side and talk to Mark Greer (on cc). He recently said, that he is supposed to be taking over maintenance for the neard daemon (see this thread: [1]) which currently looks like it's close to being dead (no release for several years, etc.).
>
> I don't know much about the NFC stack and if/how people use it, but without reliable and maintained userspace tooling, the whole thing seems of little use in the long run. Qt has already dropped their neard support for Qt 6 [2], which basically means the mainline NFC stack won't be supported anymore in one of the most common application frameworks for IoT/embedded.
>
> [1] https://lists.01.org/hyperkitty/list/[email protected]/thread/OHD5IQHYPFUPUFYWDMNSVCBNO24M45VK/
> [2] https://bugreports.qt.io/browse/QTBUG-81824

Re: QT - I've already talked to Alex Blasche from QT (CC'd). With some
work we can get Linux NFC/neard back into their good graces. I/we need
to find time to put in the work, though.

An example of the issues they have seen is:

https://bugreports.qt.io/browse/QTBUG-43802

Another issue I have--and I suspect you, Krzysztof, have as well--is
lack of hardware. If anyone reading this wants to volunteer to be a
tester, please speak up.

To help get everyone on the same page, this is a response I made to
Krzysztof in another email:

"Krzysztof, the NFC portion of the kernel has a counterpart in userspace
called neard. I'm supposed to be maintaining it but I have next to no
time to do so. If you have spare cycles, any help would be appreciated.

Anyway, in neard, there are some simple test scripts (python2 - I/we need
to update to python3). The current home of neard is:

git://git.kernel.org/pub/scm/network/nfc/neard.git"

Thanks,

Mark
--

2021-05-12 20:53:13

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 13:01, Mark Greer wrote:
> On Wed, May 12, 2021 at 05:32:35PM +0200, Frieder Schrempf wrote:
>> On 12.05.21 16:43, Krzysztof Kozlowski wrote:
>>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>>> review the patches, send pull requests and in general keep the NFC
>>> subsystem running.
>>
>> That's great, thanks!
>>
>> Maybe you also want to have a look at the userspace side and talk to Mark Greer (on cc). He recently said, that he is supposed to be taking over maintenance for the neard daemon (see this thread: [1]) which currently looks like it's close to being dead (no release for several years, etc.).
>>
>> I don't know much about the NFC stack and if/how people use it, but without reliable and maintained userspace tooling, the whole thing seems of little use in the long run. Qt has already dropped their neard support for Qt 6 [2], which basically means the mainline NFC stack won't be supported anymore in one of the most common application frameworks for IoT/embedded.
>>
>> [1] https://lists.01.org/hyperkitty/list/[email protected]/thread/OHD5IQHYPFUPUFYWDMNSVCBNO24M45VK/
>> [2] https://bugreports.qt.io/browse/QTBUG-81824
>
> Re: QT - I've already talked to Alex Blasche from QT (CC'd). With some
> work we can get Linux NFC/neard back into their good graces. I/we need
> to find time to put in the work, though.
>
> An example of the issues they have seen is:
>
> https://bugreports.qt.io/browse/QTBUG-43802
>
> Another issue I have--and I suspect you, Krzysztof, have as well--is
> lack of hardware. If anyone reading this wants to volunteer to be a
> tester, please speak up.

Yes, testing would be very appreciated. I don't know how many unit tests
neard has, but maybe some mockups with unit testing would solve some of
problems?



Best regards,
Krzysztof

2021-05-12 20:57:13

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 12:49, Mark Greer wrote:
> On Wed, May 12, 2021 at 11:43:13AM -0400, Krzysztof Kozlowski wrote:
>> On 12/05/2021 11:11, Daniel Lezcano wrote:
>>> On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
>>>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>>>> review the patches, send pull requests and in general keep the NFC
>>>> subsystem running.
>>>>
>>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>>>
>>>> ---
>>>>
>>>> I admit I don't have big experience in NFC part but this will be nice
>>>> opportunity to learn something new.
>>>
>>> NFC has been lost in the limbos since a while. Good to see someone
>>> volunteering to take care of it.
>>>
>>> May I suggest to create a simple nfc reading program in the 'tools'
>>> directory (could be a training exercise ;)
>>>
>>
>> Noted, thanks. I also need to get a simple hardware dongle for this....
>
> Krzysztof, the NFC portion of the kernel has a counterpart in userspace
> called neard. I'm supposed to be maintaining it but I have next to no
> time to do so. If you have spare cycles, any help would be appreciated.
>
> Anyway, in neard, there are some simple test scripts (python2 - I/we need
> to update to python3). The current home of neard is:
>
> git://git.kernel.org/pub/scm/network/nfc/neard.git

Thanks for sharing this. Let me take a look at it first, before
committing to something too big.

Best regards,
Krzysztof

2021-05-12 22:24:30

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

Hello:

This series was applied to netdev/net.git (refs/heads/master):

On Wed, 12 May 2021 10:43:18 -0400 you wrote:
> The NFC subsystem is orphaned. I am happy to spend some cycles to
> review the patches, send pull requests and in general keep the NFC
> subsystem running.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
>
> [...]

Here is the summary with links:
- [1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer
https://git.kernel.org/netdev/net/c/8aa5713d8b2c
- [2/2] MAINTAINERS: nfc: include linux-nfc mailing list
https://git.kernel.org/netdev/net/c/4a64541f2ceb

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html


2021-05-12 22:25:50

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 17:20, [email protected] wrote:
> Hello:
>
> This series was applied to netdev/net.git (refs/heads/master):
>
> On Wed, 12 May 2021 10:43:18 -0400 you wrote:
>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>> review the patches, send pull requests and in general keep the NFC
>> subsystem running.
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>
>>
>> [...]
>
> Here is the summary with links:
> - [1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer
> https://git.kernel.org/netdev/net/c/8aa5713d8b2c
> - [2/2] MAINTAINERS: nfc: include linux-nfc mailing list
> https://git.kernel.org/netdev/net/c/4a64541f2ceb


Hi David and Jakub,

Thanks for taking the patches above. Can you share your view (or point
me to the docs) about maintenance process you would like to have?

For example:
1. Do you expect pull requests or only reviews?
2. Shall I update anything on patchwork (or pwbot will take care about
it entirely)?
https://patchwork.kernel.org/project/netdevbpf/list/

Best regards,
Krzysztof

2021-05-13 01:37:45

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Wed, May 12, 2021 at 04:21:25PM -0400, Krzysztof Kozlowski wrote:
> On 12/05/2021 13:01, Mark Greer wrote:

> > Re: QT - I've already talked to Alex Blasche from QT (CC'd). With some
> > work we can get Linux NFC/neard back into their good graces. I/we need
> > to find time to put in the work, though.
> >
> > An example of the issues they have seen is:
> >
> > https://bugreports.qt.io/browse/QTBUG-43802
> >
> > Another issue I have--and I suspect you, Krzysztof, have as well--is
> > lack of hardware. If anyone reading this wants to volunteer to be a
> > tester, please speak up.
>
> Yes, testing would be very appreciated. I don't know how many unit tests
> neard has, but maybe some mockups with unit testing would solve some of
> problems?

I'm not sure what you mean by this. Do you mean creating some tests that
sit directly on top of the kernel and test the kernel parts (e.g., use
netlink)? If so, that would be useful but you may end up implementing
some of the NFC stack. If you mean more/better tests that use neard to
exercise the whole stack then that would be really good too.

Speaking of NFC stack, the NFC forum has their specs paywalled which is
a real bummer/pain. I can try reaching out to them to see if they will
help us help their mission but I'm not hopeful.

FYI, most of the issues that I ran into and know still exist are
timing/race issues in neard. I'm sure if we tested more, we could find
some in the kernel though.

Mark
--

2021-05-13 19:16:09

by Stephan Gerhold

[permalink] [raw]
Subject: Re: Testing wanted for Linux NFC subsystem

Hi!

On Wed, May 12, 2021 at 10:01:35AM -0700, Mark Greer wrote:
> On Wed, May 12, 2021 at 05:32:35PM +0200, Frieder Schrempf wrote:
> > On 12.05.21 16:43, Krzysztof Kozlowski wrote:
> > > The NFC subsystem is orphaned. I am happy to spend some cycles to
> > > review the patches, send pull requests and in general keep the NFC
> > > subsystem running.
> >
> > That's great, thanks!
> >
> > Maybe you also want to have a look at the userspace side and talk to Mark Greer (on cc). He recently said, that he is supposed to be taking over maintenance for the neard daemon (see this thread: [1]) which currently looks like it's close to being dead (no release for several years, etc.).
> >
> > I don't know much about the NFC stack and if/how people use it, but without reliable and maintained userspace tooling, the whole thing seems of little use in the long run. Qt has already dropped their neard support for Qt 6 [2], which basically means the mainline NFC stack won't be supported anymore in one of the most common application frameworks for IoT/embedded.
> >
> > [1] https://lists.01.org/hyperkitty/list/[email protected]/thread/OHD5IQHYPFUPUFYWDMNSVCBNO24M45VK/
> > [2] https://bugreports.qt.io/browse/QTBUG-81824
>
> Re: QT - I've already talked to Alex Blasche from QT (CC'd). With some
> work we can get Linux NFC/neard back into their good graces. I/we need
> to find time to put in the work, though.
>
> An example of the issues they have seen is:
>
> https://bugreports.qt.io/browse/QTBUG-43802
>
> Another issue I have--and I suspect you, Krzysztof, have as well--is
> lack of hardware. If anyone reading this wants to volunteer to be a
> tester, please speak up.
>

Glad to see that Linux NFC might be somewhat less dead now :)

I have a couple of "recycled" smartphones running mainline Linux
and some of them do have NFC chips. I have two with NXP PN547
(supported by nxp,nxp-nci-i2c), one with Samsung S3FWRN5
(samsung,s3fwrn5-i2c) and even one with Broadcom BCM2079x I think
(this one does not have a driver for the Linux NFC subsystem sadly).

+Cc [email protected], in case other people there are
interested in NFC :)

The NXP/Samsung ones seems to work just fine. However, since there are
barely any userspace tools making use of Linux NFC all my testing so far
was limited to polling for devices with "nfctool" and being happy enough
when it realizes that I hold some NFC tag close to the device. :S

I would be happy to do some more testing if someone has something useful
that can be tested. However, I guess ideally we would have someone who
actually uses Linux NFC actively for some real application. :)

The QT integration might be useful for people running postmarketOS [1]
on these smartphones, but so far I haven't really seen any useful NFC
Qt applications either. :/

Thanks!
Stephan

[1]: https://postmarketos.org

2021-07-09 09:28:03

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 12/05/2021 18:49, Mark Greer wrote:
> On Wed, May 12, 2021 at 11:43:13AM -0400, Krzysztof Kozlowski wrote:
>> On 12/05/2021 11:11, Daniel Lezcano wrote:
>>> On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
>>>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>>>> review the patches, send pull requests and in general keep the NFC
>>>> subsystem running.
>>>>
>>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>>>
>>>> ---
>>>>
>>>> I admit I don't have big experience in NFC part but this will be nice
>>>> opportunity to learn something new.
>>>
>>> NFC has been lost in the limbos since a while. Good to see someone
>>> volunteering to take care of it.
>>>
>>> May I suggest to create a simple nfc reading program in the 'tools'
>>> directory (could be a training exercise ;)
>>>
>>
>> Noted, thanks. I also need to get a simple hardware dongle for this....
>
> Krzysztof, the NFC portion of the kernel has a counterpart in userspace
> called neard. I'm supposed to be maintaining it but I have next to no
> time to do so. If you have spare cycles, any help would be appreciated.
>
> Anyway, in neard, there are some simple test scripts (python2 - I/we need
> to update to python3). The current home of neard is:
>
> git://git.kernel.org/pub/scm/network/nfc/neard.git

I guess none of us have problem of too much spare time :), so it took me
a while before I looked at neard.

With newer Gcc, neard did not even compile (which I am fixing now). I
set up a fork:
https://github.com/krzk/neard
However I can give early disclaimer - playing with GLib userspace code
is not something I am in long term interested. If this was written in
Rust, would be different story. :)

I also configured basic CI (or rather continuous building):
https://github.com/krzk/neard/actions/runs/1014641944

However I still do not have proper testing setup. No hardware. Would be
nice if Samsung. ST, NXP or Intel could spare some development board
with the NFC chip supported by kernel. Till then, I will try the NFC
simulator and virtual NCI drivers.

My next plan for neard is to extend the CI. There is no way I (or anyone
else I believe) can keep good quality of releases without automated
checks. I'll add some more distros, clang and later many some linters or
cppcheck.


Best regards,
Krzysztof

2021-07-15 18:36:11

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Fri, Jul 09, 2021 at 11:24:41AM +0200, Krzysztof Kozlowski wrote:
> On 12/05/2021 18:49, Mark Greer wrote:
> > On Wed, May 12, 2021 at 11:43:13AM -0400, Krzysztof Kozlowski wrote:
> >> On 12/05/2021 11:11, Daniel Lezcano wrote:
> >>> On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
> >>>> The NFC subsystem is orphaned. I am happy to spend some cycles to
> >>>> review the patches, send pull requests and in general keep the NFC
> >>>> subsystem running.
> >>>>
> >>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >>>>
> >>>> ---
> >>>>
> >>>> I admit I don't have big experience in NFC part but this will be nice
> >>>> opportunity to learn something new.
> >>>
> >>> NFC has been lost in the limbos since a while. Good to see someone
> >>> volunteering to take care of it.
> >>>
> >>> May I suggest to create a simple nfc reading program in the 'tools'
> >>> directory (could be a training exercise ;)
> >>>
> >>
> >> Noted, thanks. I also need to get a simple hardware dongle for this....
> >
> > Krzysztof, the NFC portion of the kernel has a counterpart in userspace
> > called neard. I'm supposed to be maintaining it but I have next to no
> > time to do so. If you have spare cycles, any help would be appreciated.
> >
> > Anyway, in neard, there are some simple test scripts (python2 - I/we need
> > to update to python3). The current home of neard is:
> >
> > git://git.kernel.org/pub/scm/network/nfc/neard.git
>
> I guess none of us have problem of too much spare time :), so it took me
> a while before I looked at neard.
>
> With newer Gcc, neard did not even compile (which I am fixing now). I
> set up a fork:
> https://github.com/krzk/neard
> However I can give early disclaimer - playing with GLib userspace code
> is not something I am in long term interested. If this was written in
> Rust, would be different story. :)
>
> I also configured basic CI (or rather continuous building):
> https://github.com/krzk/neard/actions/runs/1014641944
>
> However I still do not have proper testing setup. No hardware. Would be
> nice if Samsung. ST, NXP or Intel could spare some development board
> with the NFC chip supported by kernel. Till then, I will try the NFC
> simulator and virtual NCI drivers.
>
> My next plan for neard is to extend the CI. There is no way I (or anyone
> else I believe) can keep good quality of releases without automated
> checks. I'll add some more distros, clang and later many some linters or
> cppcheck.

Hi Krzysztof, I see you've been busy. Thanks for that.

FYI, I made a repo on github some time back but never announced it. The
only reason I mention it is because the name/link looks more official:

https://github.com/linux-nfc/neard

Let see what happens with permssion on kernel.org and go from there.

Re: hardware - I don't have much reader hardware either. I almost
exclusively use BeagleBone[Black] + RF Cape + trf7970atb. I also have
a USB dongle with a pn533, FWIW. I do have a decent collection of NFC tags,
though. I'll contact you privately to arrange to send some to you.

For peer-to-peer testing, your smartphone probably has an NFC reader but
you'll have to play around to find the sweet spot where they put the
antenna (older phones were notoriously bad for NFC antenna design; newer
ones are generally better).

I will review your patch sets but the earliest I will get to them will
be Sunday.

Mark
--

2021-07-16 10:21:23

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On 15/07/2021 20:34, Mark Greer wrote:
> On Fri, Jul 09, 2021 at 11:24:41AM +0200, Krzysztof Kozlowski wrote:
>> On 12/05/2021 18:49, Mark Greer wrote:
>>> On Wed, May 12, 2021 at 11:43:13AM -0400, Krzysztof Kozlowski wrote:
>>>> On 12/05/2021 11:11, Daniel Lezcano wrote:
>>>>> On 12/05/2021 16:43, Krzysztof Kozlowski wrote:
>>>>>> The NFC subsystem is orphaned. I am happy to spend some cycles to
>>>>>> review the patches, send pull requests and in general keep the NFC
>>>>>> subsystem running.
>>>>>>
>>>>>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>>>>>
>>>>>> ---
>>>>>>
>>>>>> I admit I don't have big experience in NFC part but this will be nice
>>>>>> opportunity to learn something new.
>>>>>
>>>>> NFC has been lost in the limbos since a while. Good to see someone
>>>>> volunteering to take care of it.
>>>>>
>>>>> May I suggest to create a simple nfc reading program in the 'tools'
>>>>> directory (could be a training exercise ;)
>>>>>
>>>>
>>>> Noted, thanks. I also need to get a simple hardware dongle for this....
>>>
>>> Krzysztof, the NFC portion of the kernel has a counterpart in userspace
>>> called neard. I'm supposed to be maintaining it but I have next to no
>>> time to do so. If you have spare cycles, any help would be appreciated.
>>>
>>> Anyway, in neard, there are some simple test scripts (python2 - I/we need
>>> to update to python3). The current home of neard is:
>>>
>>> git://git.kernel.org/pub/scm/network/nfc/neard.git
>>
>> I guess none of us have problem of too much spare time :), so it took me
>> a while before I looked at neard.
>>
>> With newer Gcc, neard did not even compile (which I am fixing now). I
>> set up a fork:
>> https://github.com/krzk/neard
>> However I can give early disclaimer - playing with GLib userspace code
>> is not something I am in long term interested. If this was written in
>> Rust, would be different story. :)
>>
>> I also configured basic CI (or rather continuous building):
>> https://github.com/krzk/neard/actions/runs/1014641944
>>
>> However I still do not have proper testing setup. No hardware. Would be
>> nice if Samsung. ST, NXP or Intel could spare some development board
>> with the NFC chip supported by kernel. Till then, I will try the NFC
>> simulator and virtual NCI drivers.
>>
>> My next plan for neard is to extend the CI. There is no way I (or anyone
>> else I believe) can keep good quality of releases without automated
>> checks. I'll add some more distros, clang and later many some linters or
>> cppcheck.
>
> Hi Krzysztof, I see you've been busy. Thanks for that.
>
> FYI, I made a repo on github some time back but never announced it. The
> only reason I mention it is because the name/link looks more official:
>
> https://github.com/linux-nfc/neard
>
> Let see what happens with permssion on kernel.org and go from there.

For the kernel.org I think you need an account @kernel.org (which itself
requires your key to be signed by someone), but I am not sure.

I am happy to move entire development to github and keep kernel.org only
for releases till some distro packages notice the change. If Github,
then your linux-nfc looks indeed nicer.

> Re: hardware - I don't have much reader hardware either. I almost
> exclusively use BeagleBone[Black] + RF Cape + trf7970atb. I also have
> a USB dongle with a pn533, FWIW. I do have a decent collection of NFC tags,
> though. I'll contact you privately to arrange to send some to you.

Thanks! I managed to do some testing with nfc-sim modules, although I am
not sure how much is supported.

> For peer-to-peer testing, your smartphone probably has an NFC reader but
> you'll have to play around to find the sweet spot where they put the
> antenna (older phones were notoriously bad for NFC antenna design; newer
> ones are generally better).
>
> I will review your patch sets but the earliest I will get to them will
> be Sunday.

I just sent one more set :)


Best regards,
Krzysztof

2021-07-16 17:17:49

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Fri, Jul 16, 2021 at 12:17:43PM +0200, Krzysztof Kozlowski wrote:
> On 15/07/2021 20:34, Mark Greer wrote:
> > On Fri, Jul 09, 2021 at 11:24:41AM +0200, Krzysztof Kozlowski wrote:
> >> On 12/05/2021 18:49, Mark Greer wrote:

> For the kernel.org I think you need an account @kernel.org (which itself
> requires your key to be signed by someone), but I am not sure.

Yeah, I just ran into this yesterday. You need a key signed by at least
two people who already have k.o accounts. With that, and your name
in the MAINTAINERS file, you can get an acct pretty quickly. I need to
set that up and get some signatures. That will take a while for me.

> I am happy to move entire development to github and keep kernel.org only
> for releases till some distro packages notice the change. If Github,
> then your linux-nfc looks indeed nicer.

Okay, lets do that. I'm the owner so I can give permissions to whoever
needs them (e.g., you :).

> > I will review your patch sets but the earliest I will get to them will
> > be Sunday.
>
> I just sent one more set :)

Awesome, thanks.

Mark
--

2021-07-16 18:20:24

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Fri, 16 Jul 2021 at 19:15, Mark Greer <[email protected]> wrote:
> > I am happy to move entire development to github and keep kernel.org only
> > for releases till some distro packages notice the change. If Github,
> > then your linux-nfc looks indeed nicer.
>
> Okay, lets do that. I'm the owner so I can give permissions to whoever
> needs them (e.g., you :).

Then please assign some to the account "krzk" on Github. Thanks!

Best regards,
Krzysztof

2021-07-16 19:34:17

by Mark Greer

[permalink] [raw]
Subject: Re: [linux-nfc] Re: [PATCH 1/2] MAINTAINERS: nfc: add Krzysztof Kozlowski as maintainer

On Fri, Jul 16, 2021 at 08:17:31PM +0200, Krzysztof Kozlowski wrote:
> On Fri, 16 Jul 2021 at 19:15, Mark Greer <[email protected]> wrote:
> > > I am happy to move entire development to github and keep kernel.org only
> > > for releases till some distro packages notice the change. If Github,
> > > then your linux-nfc looks indeed nicer.
> >
> > Okay, lets do that. I'm the owner so I can give permissions to whoever
> > needs them (e.g., you :).
>
> Then please assign some to the account "krzk" on Github. Thanks!

Done.