[PATCH v4] NFC: trf7970a: Correct register settings for 27MHz clock

2017-04-27 Thread Geoff Lansberry
In prior commits the selected clock frequency does not propagate correctly to what is written to the TRF7970A_MODULATOR_SYS_CLK_CTRL register. Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/nfc/trf7970a.c b/drivers

[PATCH v3] NFC: trf7970a: Correct register settings for 27MHz clock

2017-04-27 Thread Geoff Lansberry
In prior commits the selected clock frequency does not propagate correctly to what is written the the TRF7970A_MODULATOR_SYS_CLK_CTRL register. Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/nfc/trf7970a.c b/drivers

[PATCH v2] NFC: trf7970a: Correct register settings for 27MHz clock

2017-04-26 Thread Geoff Lansberry
In prior commits the selected clock frequency does not propagate correctly to what is written the the TRF7970A_MODULATOR_SYS_CLK_CTRL register. Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/nfc/trf7970a.c b/drivers

[PATCH] NFC: trf7970a: Correct register settings for 27MHz clock

2017-04-26 Thread Geoff Lansberry
In prior commits the selected clock frequency does not propagate correctly to what is written the the TRF7970A_MODULATOR_SYS_CLK_CTRL register. Also fixes a bug that causes the device tree property check to always pass. Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 9 - 1

Re: [PATCH] NFC: trf7970a: fix check of clock frequencies, use && instead of ||

2017-04-25 Thread Geoff Lansberry
CY)) { > + if ((clk_freq != TRF7970A_27MHZ_CLOCK_FREQUENCY) && > + (clk_freq != TRF7970A_13MHZ_CLOCK_FREQUENCY)) { > dev_err(trf->dev, > "clock-frequency (%u Hz) unsupported\n", > clk_freq); > -- > 2.11.0 > Nice catch. Acked-by: Geoff Lansberry

Re: [Patch] NFC: trf7970a:

2017-02-09 Thread Geoff Lansberry
On Thu, Feb 9, 2017 at 4:27 PM, Mark Greer wrote: > > On Thu, Feb 09, 2017 at 10:54:08AM -0500, Geoff Lansberry wrote: > > On Wed, Feb 8, 2017 at 5:56 PM, Mark Greer wrote: > > > > > > I just realized that the linux-nfc is not CC'd so adding it. > > &

Re: [Patch] NFC: trf7970a:

2017-02-09 Thread Geoff Lansberry
On Wed, Feb 8, 2017 at 5:56 PM, Mark Greer wrote: > > I just realized that the linux-nfc is not CC'd so adding it. > > On Wed, Feb 08, 2017 at 03:53:39PM -0700, Mark Greer wrote: > > On Sun, Dec 18, 2016 at 08:07:33PM -0700, Mark Greer wrote: > > > On Sat, Dec 17,

Re: [PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2017-01-08 Thread Geoff Lansberry
On Tue, Jan 3, 2017 at 4:21 PM, Mark Greer wrote: > On Tue, Jan 03, 2017 at 01:35:18PM -0500, Geoff Lansberry wrote: >> On Tue, Jan 3, 2017 at 11:33 AM, Mark Greer wrote: >> > On Tue, Dec 27, 2016 at 09:18:32AM -0500, Geoff Lansberry wrote: > >> >> In the meanti

Re: [PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2017-01-03 Thread Geoff Lansberry
On Tue, Jan 3, 2017 at 11:33 AM, Mark Greer wrote: > [Please stop top-posting. Bottom-post only to these lists.] Sorry; gmail keeps baiting me to do it... > > Hi Geoff & happy new year. > > On Tue, Dec 27, 2016 at 09:18:32AM -0500, Geoff Lansberry wrote: >> Mark - I

Re: [PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-27 Thread Geoff Lansberry
e problem was happening due to cycling through other tag types. It didn't seem to make any difference, but I have not gone back to default. Geoff Geoff Lansberry Engineering Guy Kuvée, Inc 125 Kingston St., 3rd Floor Boston, MA 02111 1-617-290-1118 (m) geoff.lansberry (skype) http://www.kuvee.c

Re: [PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-24 Thread Geoff Lansberry
that, let me know. If you don't have the email from Jaret, also please let me know and I will forward it to you. Geoff Geoff Lansberry Engineering Guy Kuvée, Inc 125 Kingston St., 3rd Floor Boston, MA 02111 1-617-290-1118 (m) geoff.lansberry (skype) http://www.kuvee.com On Sat, Dec 24,

[PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-21 Thread Geoff Lansberry
failing on the send. Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c index e3c72c6..ba5f9b8 100644 --- a/drivers/nfc/trf7970a.c +++ b/drivers/nfc/trf7970a.c @@ -1496,6 +1496,10

[PATCH v3 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-21 Thread Geoff Lansberry
configuration. Signed-off-by: Geoff Lansberry --- .../devicetree/bindings/net/nfc/trf7970a.txt | 2 ++ drivers/nfc/trf7970a.c | 26 +- 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt

[PATCH v3 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-21 Thread Geoff Lansberry
The TRF7970A has configuration options to support hardware designs which use a 27.12MHz clock. This commit adds a device tree option 'clock-frequency' to support configuring the this chip for default 13.56MHz clock or the optional 27.12MHz clock. Signed-off-by: Geoff

Re: [PATCH 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-21 Thread Geoff Lansberry
Thanks Mark. Should I resubmit patches with the requested edits today, or wait a bit for more comments? What is the desired etiquette? > On Dec 20, 2016, at 9:23 PM, Mark Greer wrote: > >> On Tue, Dec 20, 2016 at 11:16:31AM -0500, Geoff Lansberry wrote: >> From: Geoff Lan

Re: [PATCH 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-20 Thread Geoff Lansberry
On Tue, Dec 20, 2016 at 1:11 PM, Mark Greer wrote: > Hi Geoff. > > Please put the version in your subjects when submitting anything but the > initial version of a patch (e.g., [PATCH v2 1/3]). > > Which series do you want reviewed? > > Mark > -- Sorry about the double posting, I had forgotten to e

[PATCH 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-20 Thread Geoff Lansberry
From: Geoff Lansberry The TRF7970A has configuration options for supporting hardware designs with 1.8 Volt or 3.3 Volt IO. This commit adds a device tree option, using a fixed regulator binding, for setting the io voltage to match the hardware configuration. If no option is supplied it

[PATCH 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-20 Thread Geoff Lansberry
trf->state == TRF7970A_ST_WAIT_FOR_RX_DATA_CONT) + trf->ignore_timeout = + !cancel_delayed_work(&trf->timeout_work); ret = -EIO; goto out_err; } -- Signed-off-by: Geoff Lansberry

[PATCH 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-20 Thread Geoff Lansberry
From: Geoff Lansberry The TRF7970A has configuration options to support hardware designs which use a 27.12MHz clock. This commit adds a device tree option 'clock-frequency' to support configuring the this chip for default 13.56MHz clock or the optional 27.12MHz clock. --- .../

Re: [PATCH 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-20 Thread Geoff Lansberry
On Mon, Dec 19, 2016 at 5:35 PM, Rob Herring wrote: > On Thu, Dec 15, 2016 at 05:30:43PM -0500, Geoff Lansberry wrote: >> From: Geoff Lansberry >> >> --- >> Documentation/devicetree/bindings/net/nfc/trf7970a.txt | 2 ++ >> drivers/nfc/trf7970a.c

[PATCH 2/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-20 Thread Geoff Lansberry
trf->state == TRF7970A_ST_WAIT_FOR_RX_DATA_CONT) + trf->ignore_timeout = + !cancel_delayed_work(&trf->timeout_work); ret = -EIO; goto out_err; } -- Signed-off-by: Geoff Lansberry

[PATCH 4/4] mod of frequency

2016-12-20 Thread Geoff Lansberry
(of_property_read_bool(np, "en2-rf-quirk")) -- Signed-off-by: Geoff Lansberry

[PATCH 3/3] mod of frequency

2016-12-20 Thread Geoff Lansberry
(of_property_read_bool(np, "en2-rf-quirk")) -- Signed-off-by: Geoff Lansberry

[PATCH 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-20 Thread Geoff Lansberry
From: Geoff Lansberry The TRF7970A has configuration options to support hardware designs which use a 27.12MHz clock. This commit adds a device tree option 'clock-frequency' to support configuring the this chip for default 13.56MHz clock or the optional 27.12MHz clock. --- .../

[PATCH 1/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-20 Thread Geoff Lansberry
From: Geoff Lansberry The TRF7970A has configuration options for supporting hardware designs with 1.8 Volt or 3.3 Volt IO. This commit adds a device tree option, using a fixed regulator binding, for setting the io voltage to match the hardware configuration. If no option is supplied it

[PATCH 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-20 Thread Geoff Lansberry
trf->state == TRF7970A_ST_WAIT_FOR_RX_DATA_CONT) + trf->ignore_timeout = + !cancel_delayed_work(&trf->timeout_work); ret = -EIO; goto out_err; } -- Signed-off-by: Geoff Lansberry

[PATCH 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-20 Thread Geoff Lansberry
From: Geoff Lansberry The TRF7970A has configuration options for supporting hardware designs with 1.8 Volt or 3.3 Volt IO. This commit adds a device tree option, using a fixed regulator binding, for setting the io voltage to match the hardware configuration. If no option is supplied it

Re: [PATCH 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-19 Thread Geoff Lansberry
'd like to hear your thoughts on my concern. Thanks Geoff Geoff Lansberry Engineering Guy Kuvée, Inc 125 Kingston St., 3rd Floor Boston, MA 02111 1-617-290-1118 (m) geoff.lansberry (skype) http://www.kuvee.com On Mon, Dec 19, 2016 at 5:31 PM, Rob Herring wrote: > On Thu, Dec 15, 2016 a

Re: [Patch] NFC: trf7970a:

2016-12-17 Thread Geoff Lansberry
Mark, from our consultant: It isn't important whether the flood script is successful in writing or not. The point of it is to force a segfault by making many requests. It needs to run for several hundred iterations (successful or not) in order to generate the segfault. Geoff Geoff Lans

[PATCH 1/3] NFC: trf7970a: add device tree option for 27MHz clock

2016-12-15 Thread Geoff Lansberry
From: Geoff Lansberry --- .../devicetree/bindings/net/nfc/trf7970a.txt | 3 ++ drivers/nfc/trf7970a.c | 42 -- 2 files changed, 34 insertions(+), 11 deletions(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt b

[PATCH 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel

2016-12-15 Thread Geoff Lansberry
trf->state == TRF7970A_ST_WAIT_FOR_RX_DATA_CONT) + trf->ignore_timeout = + !cancel_delayed_work(&trf->timeout_work); ret = -EIO; goto out_err; } -- Signed-off-by: Geoff Lansberry

[PATCH 2/3] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-12-15 Thread Geoff Lansberry
From: Geoff Lansberry --- Documentation/devicetree/bindings/net/nfc/trf7970a.txt | 2 ++ drivers/nfc/trf7970a.c | 13 - 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt b

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Geoff Lansberry
On Wed, Dec 14, 2016 at 12:10 PM, Mark Greer wrote: > On Wed, Dec 14, 2016 at 11:17:33AM -0500, Geoff Lansberry wrote: >> On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer wrote: >> > >> > On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote: >> > > H

Re: [Patch] NFC: trf7970a:

2016-12-14 Thread Geoff Lansberry
On Wed, Dec 14, 2016 at 10:57 AM, Mark Greer wrote: > > On Tue, Dec 13, 2016 at 08:50:04PM -0500, Geoff Lansberry wrote: > > Hi Mark - Thanks for getting back to me. It's funny that you ask, > > because we are currently chasing a segfault that is happening in neard, but

[PATCH 4/4] NFC: trf7970a: Add device tree option of 1.8 Volt IO voltage

2016-04-18 Thread Geoff Lansberry
Signed-off-by: Geoff Lansberry --- Documentation/devicetree/bindings/net/nfc/trf7970a.txt | 8 drivers/nfc/trf7970a.c | 11 ++- 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt

[PATCH 3/4] NFC: trf7970a: add device tree option for 27MHz clock

2016-04-18 Thread Geoff Lansberry
Signed-off-by: Geoff Lansberry --- .../devicetree/bindings/net/nfc/trf7970a.txt | 2 ++ drivers/nfc/trf7970a.c | 28 +- 2 files changed, 19 insertions(+), 11 deletions(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt

[PATCH 1/4] NFC: trf7970a: Add support for gpio as SS

2016-04-18 Thread Geoff Lansberry
Signed-off-by: Geoff Lansberry --- .../devicetree/bindings/net/nfc/trf7970a.txt | 2 ++ drivers/nfc/trf7970a.c | 33 -- 2 files changed, 33 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/net/nfc/trf7970a.txt b

[PATCH 2/4] NFC: trf7970a: add TI recommended write of zero to Register 0x18

2016-04-18 Thread Geoff Lansberry
Signed-off-by: Geoff Lansberry --- drivers/nfc/trf7970a.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c index 2c3530a..447b6c9 100644 --- a/drivers/nfc/trf7970a.c +++ b/drivers/nfc/trf7970a.c @@ -1064,6 +1064,10 @@ static int

[Patch] NFC: trf7970a:

2016-04-18 Thread Geoff Lansberry
The current version of the trf7970a driver is missing support for several features that we needed to operate a custom board. We feel that these features will be useful to others as well, and we want to share them. 1: Support for using a gpio as Slave-Select. Our processor has several devices o