[PATCH net-next v4 1/2] nfc: Add a virtual nci device driver

2021-01-27 Thread Bongsu Jeon
From: Bongsu Jeon NCI virtual device simulates a NCI device to the user. It can be used to validate the NCI module and applications. This driver supports communication between the virtual NCI device and NCI module. Signed-off-by: Bongsu Jeon --- drivers/nfc/Kconfig | 11 ++ drivers

[PATCH net-next v4 2/2] selftests: Add nci suite

2021-01-27 Thread Bongsu Jeon
From: Bongsu Jeon This is the NCI test suite. It tests the NFC/NCI module using virtual NCI device. Test cases consist of making the virtual NCI device on/off and controlling the device's polling for NCI1.0 and NCI2.0 version. Signed-off-by: Bongsu Jeon --- MAINTAINERS

[PATCH net-next v4 0/2] Add nci suit and virtual nci device driver

2021-01-27 Thread Bongsu Jeon
From: Bongsu Jeon 1/2 is the Virtual NCI device driver. 2/2 is the NCI selftest suite v4: 1/2 - flip the condition for the ioctl. - refactor some code. - remove the unused function after refactoring. v3: 1/2 - change the Kconfig help comment. - remove the mutex init code. - remove

Re: [PATCH net] net: nfc: nci: fix the wrong NCI_CORE_INIT parameters

2021-01-20 Thread Bongsu Jeon
On Wed, Jan 20, 2021 at 10:00 AM wrote: > > Hello: > > This patch was applied to netdev/net.git (refs/heads/master): > > On Tue, 19 Jan 2021 05:55:22 +0900 you wrote: > > From: Bongsu Jeon > > > > Fix the code because NCI_CORE_INIT_C

Re: [PATCH net] net: nfc: nci: fix the wrong NCI_CORE_INIT parameters

2021-01-18 Thread Bongsu Jeon
On Tue, Jan 19, 2021 at 6:01 AM Jakub Kicinski wrote: > > On Tue, 19 Jan 2021 05:55:22 +0900 Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > Fix the code because NCI_CORE_INIT_CMD includes two parameters in NCI2.0 > > but there is no parameters in NCI1.x. >

[PATCH net] net: nfc: nci: fix the wrong NCI_CORE_INIT parameters

2021-01-18 Thread Bongsu Jeon
From: Bongsu Jeon Fix the code because NCI_CORE_INIT_CMD includes two parameters in NCI2.0 but there is no parameters in NCI1.x. Signed-off-by: Bongsu Jeon --- net/nfc/nci/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/nfc/nci/core.c b/net/nfc/nci/core.c index

Re: [PATCH net-next] nfc: Add a virtual nci device driver

2021-01-07 Thread Bongsu Jeon
On Thu, Jan 7, 2021 at 2:01 AM Jakub Kicinski wrote: > > On Wed, 6 Jan 2021 08:16:47 +0900 Bongsu Jeon wrote: > > On Tue, Jan 5, 2021 at 4:48 AM Jakub Kicinski wrote: > > > > thank you for your answer. > > > > I think that neard(NFC deamon) is

Re: [PATCH net-next] nfc: Add a virtual nci device driver

2021-01-05 Thread Bongsu Jeon
On Tue, Jan 5, 2021 at 4:48 AM Jakub Kicinski wrote: > > On Thu, 31 Dec 2020 14:22:45 +0900 Bongsu Jeon wrote: > > On Tue, Dec 29, 2020 at 6:16 AM Jakub Kicinski wrote: > > > > > > On Mon, 28 Dec 2020 18:45:07 +0900 Bongsu Jeon wrote: > > > > From:

Re: [PATCH net-next] nfc: Add a virtual nci device driver

2020-12-30 Thread Bongsu Jeon
On Tue, Dec 29, 2020 at 6:16 AM Jakub Kicinski wrote: > > On Mon, 28 Dec 2020 18:45:07 +0900 Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > A NCI virtual device can be made to simulate a NCI device in user space. > > Using the virtual NCI device, The

[PATCH net-next v2] net: nfc: nci: Change the NCI close sequence

2020-12-30 Thread Bongsu Jeon
From: Bongsu Jeon If there is a NCI command in work queue after closing the NCI device at nci_unregister_device, The NCI command timer starts at flush_workqueue function and then NCI command timeout handler would be called 5 second after flushing the NCI command work queue and destroying

[PATCH net-next] nfc: Add a virtual nci device driver

2020-12-28 Thread Bongsu Jeon
From: Bongsu Jeon A NCI virtual device can be made to simulate a NCI device in user space. Using the virtual NCI device, The NCI module and application can be validated. This driver supports to communicate between the virtual NCI device and NCI module. Signed-off-by: Bongsu Jeon

[PATCH net-next] net: nfc: nci: Change the NCI close sequence

2020-12-27 Thread Bongsu Jeon
From: Bongsu Jeon Change the NCI close sequence because the NCI Command timer should be deleted after flushing the NCI command work queue. Signed-off-by: Bongsu Jeon --- net/nfc/nci/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/nfc/nci/core.c b/net/nfc/nci

[PATCH v2 net-next 2/2] nfc: s3fwrn5: Remove unused NCI prop commands

2020-12-14 Thread Bongsu Jeon
From: Bongsu Jeon Remove the unused NCI prop commands that s3fwrn5 driver doesn't use. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/nci.c | 25 - drivers/nfc/s3fwrn5/nci.h | 22 -- 2 files changed, 47 deletions(-) diff --git a/drivers/nfc

[PATCH v2 net-next 1/2] nfc: s3fwrn5: Remove the delay for NFC sleep

2020-12-14 Thread Bongsu Jeon
From: Bongsu Jeon Remove the delay for NFC sleep because the delay is only needed to guarantee that the NFC is awake. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/phy_common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/phy_common.c b

[PATCH v2 net-next 0/2] nfc: s3fwrn5: Refactor the s3fwrn5 module

2020-12-14 Thread Bongsu Jeon
From: Bongsu Jeon Refactor the s3fwrn5 module. 1/2 is to remove the unneeded delay for NFC sleep. 2/2 is to remove the unused NCI prop commands. ChangeLog: v2: - Update the commit messages. Bongsu Jeon (2): nfc: s3fwrn5: Remove the delay for NFC sleep nfc: s3fwrn5: Remove unused NCI

Re: [PATCH net-next] nfc: s3fwrn5: Remove unused nci prop commands

2020-12-14 Thread Bongsu Jeon
On Tue, Dec 15, 2020 at 1:02 AM Krzysztof Kozlowski wrote: > > On Mon, Dec 14, 2020 at 08:46:58PM +0900, Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > remove the unused nci prop commands that samsung driver doesn't use. > > Don't send patches one-by-one

Re: [linux-nfc] [PATCH net-next] MAINTAINERS: Update maintainer for SAMSUNG S3FWRN5 NFC

2020-12-14 Thread Bongsu Jeon
On Tue, Dec 15, 2020 at 12:44 AM Krzysztof Kozlowski wrote: > > On Mon, Dec 14, 2020 at 09:28:23PM +0900, Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > add an email to look after the SAMSUNG NFC driver. > > Hi Bongsu, > > Review and testing is alway

[PATCH net-next] MAINTAINERS: Update maintainer for SAMSUNG S3FWRN5 NFC

2020-12-14 Thread Bongsu Jeon
From: Bongsu Jeon add an email to look after the SAMSUNG NFC driver. Signed-off-by: Bongsu Jeon --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 5c1a6ba5ef26..cb1634eb010d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15425,6 +15425,7 @@ F

[PATCH net-next] nfc: s3fwrn5: Remove unused nci prop commands

2020-12-14 Thread Bongsu Jeon
From: Bongsu Jeon remove the unused nci prop commands that samsung driver doesn't use. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/nci.c | 25 - drivers/nfc/s3fwrn5/nci.h | 22 -- 2 files changed, 47 deletions(-) diff --git a/drivers/nfc

[PATCH net-next] nfc: s3fwrn5: Remove the delay for nfc sleep

2020-12-13 Thread Bongsu Jeon
From: Bongsu Jeon remove the delay for nfc sleep because nfc doesn't need the sleep delay. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/phy_common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/phy_common.c b/drivers/nfc/s3fwrn5/phy_common.c

[PATCH net-next] nfc: s3fwrn5: Release the nfc firmware

2020-12-13 Thread Bongsu Jeon
From: Bongsu Jeon add the code to release the nfc firmware when the firmware image size is wrong. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/firmware.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/firmware.c b/drivers/nfc/s3fwrn5

[PATCH v2 net-next 2/2] nfc: s3fwrn5: Remove hard coded interrupt trigger type from the i2c module

2020-12-08 Thread Bongsu Jeon
From: Bongsu Jeon For the flexible control of interrupt trigger type, remove the hard coded interrupt trigger type in the i2c module. The trigger type will be loaded from a dts. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/i2c.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion

[PATCH v2 net-next 1/2] dt-bindings: net: nfc: s3fwrn5: Change I2C interrupt trigger type

2020-12-08 Thread Bongsu Jeon
From: Bongsu Jeon Change interrupt trigger from IRQ_TYPE_LEVEL_HIGH to IRQ_TYPE_EDGE_RISING for stable NFC I2C interrupt handling. Samsung's NFC Firmware sends an i2c frame as below. 1. NFC Firmware sets the GPIO(interrupt pin) high when there is an i2c frame to send. 2. If the CPU's I2C

[PATCH v2 net-next 0/2] nfc: s3fwrn5: Change I2C interrupt trigger to EDGE_RISING

2020-12-08 Thread Bongsu Jeon
From: Bongsu Jeon For stable Samsung's I2C interrupt handling, I changed the interrupt trigger from IRQ_TYPE_LEVEL_HIGH to IRQ_TYPE_EDGE_RISING and removed the hard coded interrupt trigger type in the i2c module for the flexible control. 1/2 is the changed dt binding for the edge rising

Re: [PATCH net-next] nfc: s3fwrn5: Change irqflags

2020-12-07 Thread Bongsu Jeon
On Mon, Dec 7, 2020 at 11:13 PM Krzysztof Kozlowski wrote: > > On Mon, Dec 07, 2020 at 10:39:01PM +0900, Bongsu Jeon wrote: > > On Mon, Dec 7, 2020 at 8:51 PM Krzysztof Kozlowski wrote: > > > > > > On Mon, Dec 07, 2020 at 08:38:27PM +0900, Bongsu Jeon wr

Re: [PATCH net-next] nfc: s3fwrn5: Change irqflags

2020-12-07 Thread Bongsu Jeon
On Mon, Dec 7, 2020 at 8:51 PM Krzysztof Kozlowski wrote: > > On Mon, Dec 07, 2020 at 08:38:27PM +0900, Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > change irqflags from IRQF_TRIGGER_HIGH to IRQF_TRIGGER_RISING for stable > > Samsung's nfc interrupt handling. &

[PATCH net-next] nfc: s3fwrn5: Change irqflags

2020-12-07 Thread Bongsu Jeon
From: Bongsu Jeon change irqflags from IRQF_TRIGGER_HIGH to IRQF_TRIGGER_RISING for stable Samsung's nfc interrupt handling. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/i2c.c b/drivers/nfc

[PATCH v2 net-next] nfc: s3fwrn5: skip the NFC bootloader mode

2020-12-03 Thread Bongsu Jeon
From: Bongsu Jeon If there isn't a proper NFC firmware image, Bootloader mode will be skipped. Signed-off-by: Bongsu Jeon --- ChangeLog: v2: - change the commit message. - change the skip handling code. drivers/nfc/s3fwrn5/core.c | 23 +-- drivers/nfc

[PATCH net-next] nfc: s3fwrn5: skip the NFC bootloader mode

2020-12-03 Thread Bongsu Jeon
From: Bongsu Jeon If there isn't proper NFC firmware image, Bootloader mode will be skipped. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/core.c | 44 -- drivers/nfc/s3fwrn5/firmware.c | 11 + drivers/nfc/s3fwrn5/firmware.h | 1 + 3 files

[PATCH net-next v4] net/nfc/nci: Support NCI 2.x initial sequence

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon implement the NCI 2.x initial sequence to support NCI 2.x NFCC. Since NCI 2.0, CORE_RESET and CORE_INIT sequence have been changed. If NFCEE supports NCI 2.x, then NCI 2.x initial sequence will work. In NCI 1.0, Initial sequence and payloads are as below: (DH

Re: [PATCH net-next v3] net/nfc/nci: Support NCI 2.x initial sequence

2020-12-02 Thread Bongsu Jeon
On Tue, Dec 1, 2020 at 11:48 AM Jakub Kicinski wrote: > > On Fri, 27 Nov 2020 22:36:31 +0900 bongsu.je...@gmail.com wrote: > > From: Bongsu Jeon > > > > implement the NCI 2.x initial sequence to support NCI 2.x NFCC. > > Since NCI 2.0, CORE_RESET and CORE_I

Re: [PATCH v5 net-next 1/4] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-12-02 Thread Bongsu Jeon
On Thu, Dec 3, 2020 at 2:16 AM Krzysztof Kozlowski wrote: > > On Wed, Dec 02, 2020 at 08:47:38PM +0900, Bongsu Jeon wrote: > > From: Bongsu Jeon > > > > Since S3FWRN82 NFC Chip, The UART interface can be used. > > S3FWRN82 supports I2C and UART interface. >

[PATCH v5 net-next 4/4] nfc: s3fwrn5: Support a UART interface

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 uses NCI protocol and supports I2C and UART interface. Reviewed-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/Kconfig | 12 +++ drivers/nfc/s3fwrn5/Makefile | 2

[PATCH v5 net-next 3/4] nfc: s3fwrn5: extract the common phy blocks

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon Extract the common phy blocks to reuse it. The UART module will use the common blocks. Reviewed-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/Makefile | 2 +- drivers/nfc/s3fwrn5/i2c.c| 117

[PATCH v5 net-next 2/4] nfc: s3fwrn5: reduce the EN_WAIT_TIME

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon The delay of 20ms is enough to enable and wake up the Samsung's nfc chip. Acked-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/i2c.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/nfc/s3fwrn5/i2c.c b/drivers/nfc

[PATCH v5 net-next 1/4] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- .../bindings/net/nfc/samsung,s3fwrn5.yaml | 31 +++--- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git

[PATCH v5 net-next 0/4] nfc: s3fwrn5: Support a UART interface

2020-12-02 Thread Bongsu Jeon
From: Bongsu Jeon S3FWRN82 is the Samsung's NFC chip that supports the UART communication. Before adding the UART driver module, I did refactoring the s3fwrn5_i2c module to reuse the common blocks. 1/4 is the dt bindings for the RN82 UART interface. 2/4..3/4 are refactoring the s3fwrn5_i2c

[PATCH v4 net-next 3/4] nfc: s3fwrn5: extract the common phy blocks

2020-12-01 Thread Bongsu Jeon
From: Bongsu Jeon Extract the common phy blocks to reuse it. The UART module will use the common blocks. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/Makefile | 2 +- drivers/nfc/s3fwrn5/i2c.c| 117 +-- drivers/nfc/s3fwrn5/phy_common.c

[PATCH v4 net-next 4/4] nfc: s3fwrn5: Support a UART interface

2020-12-01 Thread Bongsu Jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 uses NCI protocol and supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/Kconfig | 12 +++ drivers/nfc/s3fwrn5/Makefile | 2 + drivers/nfc/s3fwrn5/phy_common.c

[PATCH v4 net-next 2/4] nfc: s3fwrn5: reduce the EN_WAIT_TIME

2020-12-01 Thread Bongsu Jeon
From: Bongsu Jeon The delay of 20ms is enough to enable and wake up the Samsung's nfc chip. Acked-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/i2c.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/nfc/s3fwrn5/i2c.c b/drivers/nfc

[PATCH v4 net-next 1/4] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-12-01 Thread Bongsu Jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- .../bindings/net/nfc/samsung,s3fwrn5.yaml | 32 -- 1 file changed, 29 insertions(+), 3 deletions(-) diff --git

[PATCH v4 net-next 0/4] nfc: s3fwrn5: Support a UART interface

2020-12-01 Thread Bongsu Jeon
From: Bongsu Jeon S3FWRN82 is the Samsung's NFC chip that supports the UART communication. Before adding the UART driver module, I did refactoring the s3fwrn5_i2c module to reuse the common blocks. 1/4 is the dt bindings for the RN82 UART interface. 2/4..3/4 are refactoring the s3fwrn5_i2c

[PATCH v2 net-next 4/4] net: nfc: s3fwrn5: Support a UART interface

2020-11-30 Thread Bongsu Jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 uses NCI protocol and supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- Changes in v2: - remove the kfree(phy) because of duplicated free. - use the phy_common blocks. - wrap lines

[PATCH v3 net-next 3/4] nfc: s3fwrn5: extract the common phy blocks

2020-11-30 Thread Bongsu Jeon
From: Bongsu Jeon Extract the common phy blocks to reuse it. The UART module will use the common blocks. Signed-off-by: Bongsu Jeon --- Changes in v3: - move the phy_common object to s3fwrn.ko to avoid duplication. - include the header files to include everything which is used inside

[PATCH net-next 2/4] nfc: s3fwrn5: reduce the EN_WAIT_TIME

2020-11-30 Thread Bongsu Jeon
From: Bongsu Jeon The delay of 20ms is enough to enable and wake up the Samsung's nfc chip. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/i2c.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/nfc/s3fwrn5/i2c.c b/drivers/nfc/s3fwrn5/i2c.c index ae26594

[PATCH v2 net-next 1/4] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-11-30 Thread Bongsu jeon
From: Bongsu Jeon Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- Changes in v2: -change the compatible name. -change the const to enum for compatible. -change the node name to nfc. .../bindings/net/nfc

Re: [PATCH v2 net-next 3/3] nfc: s3fwrn5: extract the common phy blocks

2020-11-29 Thread Bongsu Jeon
On Sat, Nov 28, 2020 at 9:49 PM Krzysztof Kozlowski wrote: > > On Fri, Nov 27, 2020 at 08:22:18PM +0900, bongsu.je...@gmail.com wrote: > > From: Bongsu Jeon > > > > Extract the common phy blocks to reuse it. > > The UART module will use the common blocks. >

Re: [PATCH net-next 1/3] nfc: s3fwrn5: use signed integer for parsing GPIO numbers

2020-11-26 Thread Bongsu Jeon
On 11/27/20, Bongsu Jeon wrote: > On Fri, Nov 27, 2020 at 2:06 AM Krzysztof Kozlowski > wrote: >> >> On Fri, Nov 27, 2020 at 12:33:37AM +0900, bongsu.je...@gmail.com wrote: >> > From: Krzysztof Kozlowski >> > >> > GPIOs - as returned b

Re: [PATCH net-next 3/3] nfc: s3fwrn5: extract the common phy blocks

2020-11-26 Thread Bongsu Jeon
On Fri, Nov 27, 2020 at 2:13 AM Krzysztof Kozlowski wrote: > > On Fri, Nov 27, 2020 at 12:33:39AM +0900, bongsu.je...@gmail.com wrote: > > From: Bongsu Jeon > > > > Extract the common phy blocks to reuse it. > > The UART module will use the common blocks. > >

Re: [PATCH net-next 1/3] nfc: s3fwrn5: use signed integer for parsing GPIO numbers

2020-11-26 Thread Bongsu Jeon
On Fri, Nov 27, 2020 at 2:06 AM Krzysztof Kozlowski wrote: > > On Fri, Nov 27, 2020 at 12:33:37AM +0900, bongsu.je...@gmail.com wrote: > > From: Krzysztof Kozlowski > > > > GPIOs - as returned by of_get_named_gpio() and used by the gpiolib - are > > signed integers, where negative number

Re: [PATCH net-next 2/2] net: nfc: s3fwrn5: Support a UART interface

2020-11-24 Thread Bongsu Jeon
On 11/25/20, Bongsu Jeon wrote: > On 11/24/20, k...@kernel.org wrote: >> On Tue, Nov 24, 2020 at 09:05:52PM +0900, Bongsu Jeon wrote: >>> On Mon, Nov 23, 2020 at 5:55 PM k...@kernel.org wrote: >>> > > +static enum s3fwrn5_mode s3fwrn82_uart_get_mode(void *phy_id

Re: [PATCH net-next 2/2] net: nfc: s3fwrn5: Support a UART interface

2020-11-24 Thread Bongsu Jeon
On 11/24/20, k...@kernel.org wrote: > On Tue, Nov 24, 2020 at 09:05:52PM +0900, Bongsu Jeon wrote: >> On Mon, Nov 23, 2020 at 5:55 PM k...@kernel.org wrote: >> > > +static enum s3fwrn5_mode s3fwrn82_uart_get_mode(void *phy_id) >> > > +{ >> > >

Re: [PATCH net-next 1/2] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-11-24 Thread Bongsu Jeon
On 11/24/20, k...@kernel.org wrote: > On Tue, Nov 24, 2020 at 08:39:40PM +0900, Bongsu Jeon wrote: >> On Mon, Nov 23, 2020 at 5:02 PM k...@kernel.org wrote: >> > >> > On Mon, Nov 23, 2020 at 04:55:26PM +0900, Bongsu Jeon wrote: > > > examples

Re: [PATCH net-next 2/2] net: nfc: s3fwrn5: Support a UART interface

2020-11-24 Thread Bongsu Jeon
On Mon, Nov 23, 2020 at 5:55 PM k...@kernel.org wrote: > > On Mon, Nov 23, 2020 at 04:56:58PM +0900, Bongsu Jeon wrote: > > Since S3FWRN82 NFC Chip, The UART interface can be used. > > S3FWRN82 uses NCI protocol and supports I2C and UART interface. > > > > Signed-o

Re: [PATCH net-next 1/2] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-11-24 Thread Bongsu Jeon
On Mon, Nov 23, 2020 at 5:02 PM k...@kernel.org wrote: > > On Mon, Nov 23, 2020 at 04:55:26PM +0900, Bongsu Jeon wrote: > > Since S3FWRN82 NFC Chip, The UART interface can be used. > > S3FWRN82 supports I2C and UART interface. > > > > Signed-off-by: Bongsu Jeon >

[PATCH net-next v2] net/nfc/nci: Support NCI 2.x initial sequence

2020-11-23 Thread Bongsu Jeon
load Size of the Static HCI Connection, Number of Credits of the Static HCI Connection, Max NFC-V RF Frame Size, Number of Supported RF Interfaces, Supported RF Interfaces. Signed-off-by: Bongsu Jeon --- Changes in v2: - fix the warning of type casting. - changed the __u8 type to unsig

[PATCH net-next 1/2] dt-bindings: net: nfc: s3fwrn5: Support a UART interface

2020-11-22 Thread Bongsu Jeon
Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- .../bindings/net/nfc/samsung,s3fwrn5.yaml | 28 +-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree

[PATCH net-next 2/2] net: nfc: s3fwrn5: Support a UART interface

2020-11-22 Thread Bongsu Jeon
Since S3FWRN82 NFC Chip, The UART interface can be used. S3FWRN82 uses NCI protocol and supports I2C and UART interface. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/Kconfig | 12 ++ drivers/nfc/s3fwrn5/Makefile | 2 + drivers/nfc/s3fwrn5/uart.c | 250

Re: [PATCH net-next] net/nfc/nci: Support NCI 2.x initial sequence

2020-11-17 Thread Bongsu Jeon
On 11/18/20, Jakub Kicinski wrote: > On Tue, 17 Nov 2020 14:37:59 +0900 Bongsu Jeon wrote: >> implement the NCI 2.x initial sequence to support NCI 2.x NFCC. >> Since NCI 2.0, CORE_RESET and CORE_INIT sequence have been changed. >> If NFCEE supports NCI 2.x, then NCI

Re: [PATCH net-next v2 1/3] nfc: s3fwrn5: Remove the max_payload

2020-11-17 Thread Bongsu Jeon
On Tue, Nov 17, 2020 at 5:39 PM Krzysztof Kozlowski wrote: > > On Tue, 17 Nov 2020 at 09:14, Bongsu Jeon wrote: > > > > 2020-11-17 16:42 GMT+09:00, k...@kernel.org : > > > On Tue, Nov 17, 2020 at 10:16:11AM +0900, Bongsu Jeon wrote: > > >> max_payload is

Re: [PATCH net-next v2 1/3] nfc: s3fwrn5: Remove the max_payload

2020-11-17 Thread Bongsu Jeon
2020-11-17 16:42 GMT+09:00, k...@kernel.org : > On Tue, Nov 17, 2020 at 10:16:11AM +0900, Bongsu Jeon wrote: >> max_payload is unused. > > Why did you resend the patch ignoring my review? I already provided you > with a tag, so you should include it. > > https://www.ke

[PATCH net-next v2 3/3] nfc: s3fwrn5: Change the error code

2020-11-17 Thread Bongsu Jeon
ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP. Reviewed-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/s3fwrn5.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/nfc/s3fwrn5/s3fwrn5.h b/drivers/nfc/s3fwrn5/s3fwrn5.h index

[PATCH net-next v2 2/3] nfc: s3fwrn5: Fix the misspelling in a comment

2020-11-17 Thread Bongsu Jeon
stucture should be replaced by structure. Reviewed-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/firmware.c b/drivers/nfc/s3fwrn5/firmware.c index ec930ee2c847

[PATCH net-next v2 1/3] nfc: s3fwrn5: Remove the max_payload

2020-11-17 Thread Bongsu Jeon
max_payload is unused. Reviewed-by: Krzysztof Kozlowski Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/core.c| 3 +-- drivers/nfc/s3fwrn5/i2c.c | 4 +--- drivers/nfc/s3fwrn5/s3fwrn5.h | 3 +-- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/nfc/s3fwrn5/core.c

[PATCH net-next] net/nfc/nci: Support NCI 2.x initial sequence

2020-11-16 Thread Bongsu Jeon
load Size of the Static HCI Connection, Number of Credits of the Static HCI Connection, Max NFC-V RF Frame Size, Number of Supported RF Interfaces, Supported RF Interfaces. Signed-off-by: Bongsu Jeon --- include/net/nfc/nci.h | 39 ++ net/nfc/nci/core.c| 23 +

[PATCH net-next v2 3/3] nfc: s3fwrn5: Change the error code

2020-11-16 Thread Bongsu Jeon
ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/s3fwrn5.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/nfc/s3fwrn5/s3fwrn5.h b/drivers/nfc/s3fwrn5/s3fwrn5.h index 9d5f34759225..bb8f936d13a2 100644

[PATCH net-next v2 2/3] nfc: s3fwrn5: Fix the misspelling in a comment

2020-11-16 Thread Bongsu Jeon
stucture should be replaced by structure. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/firmware.c b/drivers/nfc/s3fwrn5/firmware.c index ec930ee2c847..4cde6dd5c019 100644 --- a/drivers/nfc

[PATCH net-next v2 1/3] nfc: s3fwrn5: Remove the max_payload

2020-11-16 Thread Bongsu Jeon
max_payload is unused. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/core.c| 3 +-- drivers/nfc/s3fwrn5/i2c.c | 4 +--- drivers/nfc/s3fwrn5/s3fwrn5.h | 3 +-- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/nfc/s3fwrn5/core.c b/drivers/nfc/s3fwrn5/core.c

[PATCH net-next v2 0/3] nfc: s3fwrn5: Refactor the s3fwrn5 driver

2020-11-16 Thread Bongsu Jeon
Changes from v1: - Remove the trailing dot from subject. - Remove an empty line at beginning of commit message. - Add a commit message. Bongsu Jeon (3): nfc: s3fwrn5: Remove the max_payload nfc: s3fwrn5: Fix the misspelling in a comment nfc: s3fwrn5: Change the error code drivers/nfc

[PATCH net-next 3/3] nfc: s3fwrn5: Change the error code

2020-11-15 Thread Bongsu Jeon
ENOTSUPP is not a SUSV4 error code, prefer EOPNOTSUPP. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/s3fwrn5.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/nfc/s3fwrn5/s3fwrn5.h b/drivers/nfc/s3fwrn5/s3fwrn5.h index 9d5f34759225..bb8f936d13a2 100644

[PATCH net-next 2/3] nfc: s3fwrn5: Fix the misspelling in a comment

2020-11-15 Thread Bongsu Jeon
stucture should be replaced by structure. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nfc/s3fwrn5/firmware.c b/drivers/nfc/s3fwrn5/firmware.c index ec930ee2c847..4cde6dd5c019 100644 --- a/drivers/nfc

[PATCH net-next 1/3] nfc: s3fwrn5: Remove the max_payload

2020-11-15 Thread Bongsu Jeon
max_payload is unused. Signed-off-by: Bongsu Jeon --- drivers/nfc/s3fwrn5/core.c| 3 +-- drivers/nfc/s3fwrn5/i2c.c | 4 +--- drivers/nfc/s3fwrn5/s3fwrn5.h | 3 +-- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/nfc/s3fwrn5/core.c b/drivers/nfc/s3fwrn5/core.c