On Fri, Oct 11, 2019 at 12:38:49PM +0200, Rafael J. Wysocki wrote:
> On Wed, Oct 9, 2019 at 3:04 PM Samuel Ortiz wrote:
> >
> > The Generic Event Device (GED) is a hardware-reduced platform device.
>
> No, it is not AFAICS.
It's a little confusing, I was not sure what&
is
hardware-reduced. For that, the driver must be unconditionally built in.
Signed-off-by: Arjan van de Ven
Signed-off-by: Samuel Ortiz
---
drivers/acpi/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile
index 5d361e4e3405
The Generic Event Device (GED) is a hardware-reduced platform device.
Probing this driver on fixed platforms should fail.
Signed-off-by: Samuel Ortiz
---
drivers/acpi/evged.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/acpi/evged.c b/drivers/acpi/evged.c
index aba0d0027586
platforms.
- The second one disable the conditional evged build.
Arjan van de Ven (1):
acpi: Always build evged in
Samuel Ortiz (1):
acpi: Fail GED probe when not on hardware-reduced
drivers/acpi/Makefile | 2 +-
drivers/acpi/evged.c | 3 +++
2 files changed, 4 insertions(+), 1 deletion
Hi Colin,
On Fri, Nov 24, 2017 at 01:37:41PM +, Colin King wrote:
> From: Colin Ian King
>
> Use setup_timer function instead of initializing timer with the
> function and data fields.
>
> Signed-off-by: Colin Ian King
> ---
> drivers/nfc/st21nfca/se.c | 10 --
> 1 file changed, 4
On Thu, Oct 05, 2017 at 10:47:12AM +0100, Colin King wrote:
> From: Colin Ian King
>
> The structure nci_ops is local to the source and does not need to
> be in global scope, so make it static.
>
> Cleans up sparse warning:
> symbol 'nci_ops' was not declared. Should it be static?
>
> Signed-of
Hi Colin,
On Tue, Sep 19, 2017 at 03:25:15PM +0100, Colin King wrote:
> From: Colin Ian King
>
> Don't populate the read-only array match on the stack, instead make
> it static const. Makes the object code smaller by over 310 bytes:
>
> Before:
>text data bss dec hex f
Hi Arvind,
On Mon, Aug 21, 2017 at 10:33:52PM +0530, Arvind Yadav wrote:
> i2c_device_id are not supposed to change at runtime. All functions
> working with i2c_device_id provided by work with
> const i2c_device_id. So mark the non-const structs as const.
>
> Arvind Yadav (8):
> [PATCH 1/8] nf
Hi Geoff,
On Thu, Apr 27, 2017 at 05:28:46PM -0400, Geoff Lansberry wrote:
> 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 +
On Tue, Jun 13, 2017 at 06:44:28PM +0200, Mateusz Jurczyk wrote:
> Verify that the caller-provided sockaddr structure is large enough to
> contain the sa_family field, before accessing it in bind() handlers of the
> AF_NFC socket. Since the syscall doesn't enforce a minimum size of the
> correspond
On Tue, May 30, 2017 at 03:43:07PM -0500, Gustavo A. R. Silva wrote:
> NULL checks at line 457: if (!link0 || !link1) {, implies that both
> pointers link0 and link1 might be NULL.
> Function nfcsim_link_free() dereference pointers link0 and link1.
> Add NULL checks before calling nfcsim_link_free(
Hi Gustavo,
On Tue, Jun 13, 2017 at 11:37:18AM -0500, Gustavo A. R. Silva wrote:
> Remove unnecessary NULL check for pointer conn_info.
> conn_info is set in list_for_each_entry() using container_of(),
> which is never NULL.
>
> Addresses-Coverity-ID: 1362349
> Cc: Guenter Roeck
> Signed-off-by:
Hi Mateusz,
On Wed, May 24, 2017 at 12:42:26PM +0200, Mateusz Jurczyk wrote:
> Check that the NFC_ATTR_TARGET_INDEX and NFC_ATTR_PROTOCOLS attributes (in
> addition to NFC_ATTR_DEVICE_INDEX) are provided by the netlink client
> prior to accessing them. This prevents potential unhandled NULL pointe
Hi Mateusz,
On Wed, May 24, 2017 at 12:26:20PM +0200, Mateusz Jurczyk wrote:
> Fix the sockaddr length verification in the connect() handler of NFC/LLCP
> sockets, to compare against the size of the actual structure expected on
> input (sockaddr_nfc_llcp) instead of its shorter version (sockaddr_n
Hi Markus,
On Mon, May 22, 2017 at 02:57:42PM +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 22 May 2017 14:50:05 +0200
>
> Two update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (2):
> Improve a size determination in four fu
Hi Colin,
On Mon, Apr 24, 2017 at 02:36:02PM +0100, Colin King wrote:
> From: Colin Ian King
>
> The "or" condition (clk_freq != TRF7970A_27MHZ_CLOCK_FREQUENCY) ||
> (clk_freq != TRF7970A_13MHZ_CLOCK_FREQUE) will always be true because
> clk_freq cannot be equal to two different values at the sa
Hi Johan,
On Thu, Mar 30, 2017 at 12:15:34PM +0200, Johan Hovold wrote:
> This started out with the observation that the nfcmrvl_uart driver
> unconditionally dereferenced the tty class device despite the fact that
> not every tty has an associated struct device (Unix98 ptys). Some
> further chang
Hi Johan,
On Tue, Apr 18, 2017 at 12:09:16PM +0200, Johan Hovold wrote:
> On Thu, Mar 30, 2017 at 12:15:34PM +0200, Johan Hovold wrote:
> > This started out with the observation that the nfcmrvl_uart driver
> > unconditionally dereferenced the tty class device despite the fact that
> > not every t
On Thu, Apr 06, 2017 at 05:58:59PM +0100, Al Viro wrote:
> On Thu, Apr 06, 2017 at 05:48:47PM +0100, Al Viro wrote:
> > * use unaligned.h, not unaligned/access_ok.h
>
> ... which got misspelled in that patch, sorry... Fixed variant follows:
>
> commit b3e79ba1708c9b74781079c9f8617448fce36b51
> A
Hi Geoff,
On Wed, Dec 21, 2016 at 11:18:32PM -0500, Geoff Lansberry wrote:
> 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
Hi Sudip,
On Tue, Dec 20, 2016 at 09:09:04PM +, Sudip Mukherjee wrote:
> We are checking phy after dereferencing it. We can print the debug
> information after checking it. If phy is NULL then we will get a good
> stack trace to tell us that we are in this irq handler.
>
> Signed-off-by: Sudi
Hi Heiko,
On Tue, Feb 07, 2017 at 06:22:04AM +0100, Heiko Schocher wrote:
> From: Guan Ben
>
> Make the EN2 pin optional. This is useful for boards,
> which have this pin fix wired, for example to ground.
>
> Signed-off-by: Guan Ben
> Signed-off-by: Mark Jonas
> Signed-off-by: Heiko Schocher
Hi Nicholas,
On Sun, Jan 22, 2017 at 01:28:39PM +0100, Nicholas Mc Guire wrote:
> ulseep_range() uses hrtimers and provides no advantage over msleep()
> for larger delays. For this large delay msleep() is preferable.
>
> Fixes: commit 6be88670fc59 ("NFC: nxp-nci_i2c: Add I2C support to NXP NCI
>
riant for unaligned access to use needs to be
> left to the arch level and not used at the driver level. Since not
> all arch will have sourced asm/unaligned.h already, we need to call
> it out and then the arch can give us just the one definition that
> is needed.
>
> See commit 06410
Hi Christophe,
On Sun, Feb 19, 2017 at 10:58:47AM +0100, Christophe JAILLET wrote:
> If all bits of 'dev_mask' are already set, there is a memory leak because
> 'info' should be freed before returning.
>
> While fixing it, 'return -ENOMEM' directly if the first kzalloc fails.
> This makes the cod
Hi Corentin,
On Thu, Dec 15, 2016 at 03:22:44PM +0100, Corentin Labbe wrote:
> drivers/nfc/nxp-nci/i2c.c does not use any miscdevice, so this patch
> remove this unnecessary inclusion.
>
> Signed-off-by: Corentin Labbe
> ---
> drivers/nfc/nxp-nci/i2c.c | 1 -
> 1 file changed, 1 deletion(-)
All
Hi Tobias,
On Wed, Oct 26, 2016 at 11:00:12AM +0200, Tobias Klauser wrote:
> Including linux/unaligned/access_ok.h causes the allmodconfig build on
> ia64 (and maybe others) to fail with the following warnings:
>
> include/linux/unaligned/access_ok.h:7:19: error: redefinition of
> 'get_unaligned
Hi Geliang,
On Thu, Nov 24, 2016 at 09:58:34PM +0800, Geliang Tang wrote:
> Drop duplicate header gpio.h from nfcmrvl/spi.c.
>
> Signed-off-by: Geliang Tang
> ---
> drivers/nfc/nfcmrvl/spi.c | 1 -
> 1 file changed, 1 deletion(-)
Patch applied, thanks.
Cheers,
Samuel.
Hi Paul,
On Tue, Mar 28, 2017 at 06:55:26PM -0400, Paul Gortmaker wrote:
> On Mon, Jan 9, 2017 at 12:52 PM, Paul Gortmaker
> wrote:
> > Currently ia64 fails building allmodconfig with variations of:
> >
> >In file included from drivers/nfc/nxp-nci/i2c.c:39:0:
> >./include/linux/unaligned/
Hi Hirofumi,
On Sat, Feb 04, 2017 at 10:15:22AM +0900, OGAWA Hirofumi wrote:
>
>
> Signed-off-by: OGAWA Hirofumi
> ---
>
> drivers/nfc/port100.c |8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
All 4 patches applied, thanks.
Cheers,
Samuel.
t; Cc: Ilan Elias
> Cc: Marcel Holtmann
> Cc: Samuel Ortiz
> Cc: Lauro Ramos Venancio
> Cc: Aloisio Almeida Jr
> Cc: linux-wirel...@vger.kernel.org
> Signed-off-by: Rob Herring
> ---
> drivers/nfc/Kconfig | 11 -
>
Hi Rob,
On Fri, Feb 24, 2017 at 02:56:48PM -0600, Rob Herring wrote:
> On Wed, Jan 25, 2017 at 11:54 PM, Marcel Holtmann wrote:
> > Hi Rob,
> >
> >> It appears that TI WiLink devices including NFC (WL185x/WL189x) never
> >> shipped. The only information I found were announcements in Feb
> >> 2012
Hi Julia,
On Tue, Aug 09, 2016 at 07:03:50PM +0200, Julia Lawall wrote:
> The nfcmrvl_if_ops structures are never modified, so declare them as const.
>
> Done with the help of Coccinelle.
>
> Signed-off-by: Julia Lawall
>
> ---
> drivers/nfc/nfcmrvl/i2c.c |2 +-
> drivers/nfc/nfcmrvl/
Hi Julia,
On Tue, Aug 09, 2016 at 06:11:02PM +0200, Julia Lawall wrote:
> The pn533_phy_ops are never modified, so declare them as const.
>
> Done with the help of Coccinelle.
>
> Signed-off-by: Julia Lawall
>
> ---
> drivers/nfc/pn533/i2c.c |2 +-
> drivers/nfc/pn533/pn533.c |2 +-
Hi Markus,
On Mon, Aug 15, 2016 at 09:12:29AM +0200, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 15 Aug 2016 09:00:26 +0200
>
> The field "owner" is set by core. Thus delete an extra initialisation.
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
> Signed-o
Hi Colin,
On Fri, Jul 15, 2016 at 02:00:33PM +0100, Colin King wrote:
> From: Colin Ian King
>
> When the firmware is released for info->ram_patch currently
> info->otp_patch is being nullified and not info->ram_patch.
> This looks like a cut-n-paste coding error. Fix this by
> nullifing the cor
Hi Marcel,
On Wed, Jul 13, 2016 at 11:56:02AM +0100, Marcel Holtmann wrote:
> Hi Mauro,
>
> >> On mips and parisc:
> >>
> >>drivers/bluetooth/btwilink.c: In function 'ti_st_open':
> >>drivers/bluetooth/btwilink.c:174:21: warning: overflow in implicit
> >> constant conversion [-Woverflow
Hi Michael,
On Thu, Apr 21, 2016 at 04:43:56PM +0200, Michael Thalmeier wrote:
> @@ -1259,7 +1259,8 @@ static int pn533_rf_complete(struct pn533 *dev, void
> *arg,
> if (IS_ERR(resp)) {
> rc = PTR_ERR(resp);
>
> - nfc_err(dev->dev, "RF setting error %d\n", rc);
>
Hi Michael,
On Thu, Apr 21, 2016 at 04:43:55PM +0200, Michael Thalmeier wrote:
> Make sure cmd is set before a frame is passed to the transport layer for
> sending. In addition pn533_send_async_complete checks if cmd is set before
> accessing its members.
>
> Signed-off-by: Michael Thalmeier
> -
Hi Michael,
On Thu, Apr 21, 2016 at 04:43:54PM +0200, Michael Thalmeier wrote:
> When a poll ist stopped we need to kill the out_urb request too before
> starting a new request.
>
> Additionally check if cmd is set in pn533_recv_ack befor accessing its struct
> members.
I understand those 2 (stop
Hi Michael,
On Thu, Apr 21, 2016 at 04:43:57PM +0200, Michael Thalmeier wrote:
> When multiple receive frames need to be put together in pn533_build_response
> we need to use nfc_alloc_recv_skb instead of the normal alloc_skb. Otherwise
> the nfc core causes an skb error when it tries to push the
Hi Krzysztof,
On Wed, Mar 30, 2016 at 09:51:04AM +0900, Krzysztof Kozlowski wrote:
> i2c_driver does not need to set an owner because i2c_register_driver()
> will set it.
>
> Signed-off-by: Krzysztof Kozlowski
>
> ---
>
> The coccinelle script which generated the patch was sent here:
> http://
Hi Michael,
On Mon, Apr 11, 2016 at 04:36:02PM +0200, Michael Thalmeier wrote:
> Add pn533-i2c phy devicetree documentation
>
> Signed-off-by: Michael Thalmeier
> ---
> .../devicetree/bindings/net/nfc/pn533-i2c.txt | 31
> ++
> 1 file changed, 31 insertions(+)
> creat
Hi Michael,
On Fri, Mar 25, 2016 at 03:46:50PM +0100, Michael Thalmeier wrote:
> Michael Thalmeier (4):
> NFC: pn533: Send ATR_REQ only if NFC_PROTO_NFC_DEP bit is set in
> poll_protocols
> NFC: pn533: fix deadlock when socket is closed while processing
> command
> NFC: pn533: Separa
Hi Michael,
On Fri, Mar 25, 2016 at 03:46:54PM +0100, Michael Thalmeier wrote:
> This adds the I2C phy interface for the pn533 driver. This way the driver can
> be used to interact with I2C connected pn532.
>
> Signed-off-by: Michael Thalmeier
> ---
> drivers/nfc/pn533/Kconfig | 11 ++
> driv
Hi Robert,
On Thu, Mar 10, 2016 at 03:22:43PM +0100, Robert Baldyga wrote:
> Add Krzysztof Opasiak as maintainer of S3FWRN5 driver.
>
> Signed-off-by: Robert Baldyga
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
Applied to nfc-next.
Cheers,
Samuel.
Hi Tang,
On Wed, Dec 23, 2015 at 12:18:42AM +0800, Geliang Tang wrote:
> Use to_spi_device() instead of open-coding it.
>
> Signed-off-by: Geliang Tang
> ---
> drivers/nfc/trf7970a.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
Applied and pushed, thanks.
Cheers,
Samuel.
--
Hi Saurabh,
On Mon, Dec 21, 2015 at 12:29:30AM +0530, Saurabh Sengar wrote:
> added the rx delay parameter as a device tunable parameter.
>
> Signed-off-by: Saurabh Sengar
> ---
> This is a follow up patch after the review comments by Samuel Ortiz
> on the initial patch.
>
Hi Julia,
On Fri, Nov 13, 2015 at 01:04:41PM +0100, Julia Lawall wrote:
> The s3fwrn5_phy_ops structure is never modified, so declare it as const.
>
> Done with the help of Coccinelle.
Applied to nfc-next, thanks.
Cheers,
Samuel.
--
To unsubscribe from this list: send the line "unsubscribe linux
Hi Saurabh,
On Sun, Dec 13, 2015 at 01:34:35PM +0530, Saurabh Sengar wrote:
> added the sysfs entry for nfcsim workqueue delay, as tx_delay
>
> Signed-off-by: Saurabh Sengar
> ---
> In case this TODO is not expected to be done, please let me know.
> I wonder after my repeated attempts since last
Hi Arnd,
On Tue, Nov 03, 2015 at 03:03:33PM +0100, Arnd Bergmann wrote:
> The newly added nfcmrvl_spi driver uses the spi_nci
> infrastructure, but does not have a Kconfig dependency on
> that, so we can get a link-time error:
>
> drivers/built-in.o: In function `nfcmrvl_spi_nci_send':
> (.text+0
Hi Julia,
On Sat, Oct 17, 2015 at 11:32:19AM +0200, Julia Lawall wrote:
> The exit label performs device_unlock(&dev->dev);, which will fail when dev
> is NULL, and nfc_put_device(dev);, which is not useful when dev is NULL, so
> just exit the function immediately.
>
> Problem found using scripts
Hi Christophe,
On Tue, Oct 13, 2015 at 08:31:04AM +0200, Christophe JAILLET wrote:
> There is no need to have the 'struct nfcwilink *drv' variable static in the
> probe function.
> It only wastes a few bytes of memory.
>
> Signed-off-by: Christophe JAILLET
> ---
> drivers/nfc/nfcwilink.c | 2 +-
gt;
> ./net/nfc/netlink.c:1175:21-24: ERROR: dev is NULL but dereferenced.
>
> Signed-off-by: Vincent Stehlé
> Cc: Thierry Escande
> Cc: Samuel Ortiz
> ---
> net/nfc/netlink.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/net/n
Hi Julia,
On Sun, Oct 11, 2015 at 01:24:13PM +0200, Julia Lawall wrote:
> The only instance of a nxp_nci_phy_ops structure is never modified. Thus
> the declaration of the structure and all references to the structure type
> can be made const.
>
> Done with the help of Coccinelle.
>
> Signed-of
Hi Javier,
On Wed, Sep 16, 2015 at 11:08:42AM +0200, Javier Martinez Canillas wrote:
> The Documentation/devicetree/bindings/net/nfc/trf7970a.txt DT binding doc
> lists "ti,trf7970a" as a compatible string but the corresponding driver
> does not have an OF match table. Add the table to the driver
Hi Robert,
On Mon, Aug 31, 2015 at 10:02:40AM +0200, Robert Baldyga wrote:
> [ +cc Samuel Ortiz and ]
>
> Hi Samuel,
>
> Could you please apply this patch?
Applied, thanks.
Cheers,
Samuel.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
ose_setup()
> callback.
>
> Best regards,
> Robert Baldyga
>
> Changelog:
>
> v3:
> - Addressed comments from Samuel Ortiz:
> - Used nci_prop_cmd and nci_prop_ops to handle proprietary requests
> - Refactorized s3fwrn5_i2c_nci_read and s3fwrn5_i2c_fw_read
>
Hi Robert,
On Thu, Jul 30, 2015 at 04:26:16PM +0200, Robert Baldyga wrote:
> +static int s3fwrn5_firmware_update(struct s3fwrn5_info *info)
> +{
> + u32 version;
> + bool need_update;
> + int ret;
> +
> + ENTER();
We have many tracing tools and frameworks in the kernel, I don't thi
Hi Joe,
On Sun, May 31, 2015 at 05:44:45PM -0700, Joe Perches wrote:
> Several of these goto exit; uses should be direct returns
> as skb is not yet initialized by nci_hci_get_param().
>
> Miscellanea:
>
> o Use !memcmp instead of memcmp() == 0
> o Remove unnecessary goto from if () {... goto ex
Hi Tomas,
On Thu, May 07, 2015 at 04:38:30PM +0300, Tomas Winkler wrote:
> In microread_i2c_irq_thread_fn 'client' set but not used
>
> Cc: Lauro Ramos Venancio
> Cc: Aloisio Almeida Jr
> Signed-off-by: Tomas Winkler
> ---
> drivers/nfc/microread/i2c.c | 3 ---
> 1 file changed, 3 deletions(-
Hi Valentin,
On Tue, Apr 28, 2015 at 11:08:47AM +0200, Valentin Rothberg wrote:
> The CPP identifier 'DEBUG' is not used in the source code of nfc at all,
> so we can safely remove setting it in both Makefiles.
>
> Signed-off-by: Valentin Rothberg
> ---
> I detected this issue with ./scripts/che
Hi Msanari,
On Fri, Jun 05, 2015 at 09:38:19PM +0900, Masanari Iida wrote:
> This patch fix a spelling typo in nfc-hci.txt
>
> Signed-off-by: Masanari Iida
> ---
> Documentation/nfc/nfc-hci.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
Cheers,
Samuel.
--
To unsub
Hi Robert,
On Tue, Mar 31, 2015 at 05:03:42PM +0300, Robert Dolca wrote:
> On Thu, Mar 26, 2015 at 2:29 AM, Samuel Ortiz wrote:
> > Hi Robert,
> >
> > On Tue, Feb 24, 2015 at 12:01:45PM +0200, Robert Dolca wrote:
> >> By calling __nci_request instead of nci_re
Hi Robert,
On Wed, Apr 01, 2015 at 06:35:31PM +0300, Robert Dolca wrote:
> On Thu, Mar 26, 2015 at 2:30 AM, Samuel Ortiz wrote:
> >> + /* If a patch was applied the new version is checked */
> >> + if (patched) {
> >> + r = nci_init(n
Hi Robert,
On Tue, Mar 31, 2015 at 05:05:53PM +0300, Robert Dolca wrote:
> On Thu, Mar 26, 2015 at 2:29 AM, Samuel Ortiz wrote:
> > Hi Robert,
> >
> > On Tue, Feb 24, 2015 at 12:01:47PM +0200, Robert Dolca wrote:
> >> In order to communicate with the device du
Hi Joe,
On Tue, Apr 07, 2015 at 12:17:00AM -0700, Joe Perches wrote:
> Add missing terminating newlines to nfc_info and nfc_err
> to avoid possible interleaving from other messages.
>
> Miscellanea:
>
> o typo fix of "unknonwn" in message
> o remove unnecessary OOM messages as there's a generic
Hi Julia,
On Sun, Apr 05, 2015 at 02:06:36PM +0200, Julia Lawall wrote:
> Return a negative error code on failure.
>
> A simplified version of the semantic match that finds this problem is as
> follows: (http://coccinelle.lip6.fr/)
>
> //
> @@
> identifier ret; expression e1,e2;
> @@
> (
> if (
Robert,
Another comment:
On Tue, Feb 24, 2015 at 12:01:52PM +0200, Robert Dolca wrote:
> +static struct i2c_device_id fdp_nci_i2c_id_table[] = {
> + {"INT339A", 0},
> + {}
> +};
> +
> +MODULE_DEVICE_TABLE(i2c, fdp_nci_i2c_id_table);
> +
> +
> +static const struct acpi_device_id fdp_nci_i2
Hi Clément,
On Mon, Mar 09, 2015 at 11:12:02AM +0100, clement.perroch...@effinnov.com wrote:
> From: Clément Perrochaud
>
> This patch brings support for the NXP-NCI NFC controllers family.
>
> It has been successfully tested on the following SoC boards:
> - BeagleBone
> - BeagleBone Black
>
Hi Robert,
On Tue, Feb 24, 2015 at 12:01:52PM +0200, Robert Dolca wrote:
> The device can be enumerated using ACPI using the id INT339A.
Please give us some more details about the device. NCI ? HCI ?
Features ? What does the initial patchset support ?
> +config NFC_FDP
> + tristate "Intel FDP
Hi Robert,
On Tue, Feb 24, 2015 at 12:01:49PM +0200, Robert Dolca wrote:
> If the previous nci_request (NCI reset) failed the setup function
> was being called anyway. It shouldn't be called if the reset failed.
>
> The result of the setup function is taken into consideration. If it
> fails the i
Hi Robert,
On Tue, Feb 24, 2015 at 12:01:48PM +0200, Robert Dolca wrote:
> This patch adds nci_request_driver and nci_req_complete_driver
> as a wrapper for __nci_request. When nci_req_complete_driver is
> called it also sets cmd_cnt to 1. This is done because the response is not
> sent to the NFC
Hi Robert,
On Tue, Feb 24, 2015 at 12:01:47PM +0200, Robert Dolca wrote:
> In order to communicate with the device during the setup
> phase, the driver may need to initialize the device. After
> the setup is done the driver should reset the device to leave
> it in the same state that it was before
Hi Robert,
On Tue, Feb 24, 2015 at 12:01:45PM +0200, Robert Dolca wrote:
> By calling __nci_request instead of nci_request allows the driver to use
> the function while initializing the device (setup stage)
>
> Signed-off-by: Robert Dolca
> ---
> net/nfc/nci/core.c | 2 +-
> 1 file changed, 1 i
Hi Clement,
On Thu, Jan 22, 2015 at 04:27:38PM +0100, clement.perroch...@effinnov.com wrote:
> @@ -686,7 +686,7 @@ L:alsa-de...@alsa-project.org (moderated for
> non-subscribers)
> W: http://blackfin.uclinux.org/
> S: Supported
> F: sound/soc/blackfin/*
> -
> +
Unneeded change,
Hi Clement,
On Thu, Jan 22, 2015 at 04:27:40PM +0100, clement.perroch...@effinnov.com wrote:
> From: Clément Perrochaud
>
> Signed-off-by: Clément Perrochaud
> Signed-off-by: Clément Perrochaud
> ---
> drivers/nfc/nxp-nci/core.c | 7 +++
> drivers/nfc/nxp-nci/firmware.c | 9 ++---
Hi Clement,
On Thu, Jan 22, 2015 at 04:27:39PM +0100, clement.perroch...@effinnov.com wrote:
> From: Clément Perrochaud
>
> Signed-off-by: Clément Perrochaud
> Signed-off-by: Clément Perrochaud
Are you sure you want both S-O-B lines ?
> +static int nxp_nci_i2c_fw_read(struct nxp_nci_i2c_phy
Hi Clement,
On Thu, Jan 22, 2015 at 04:27:37PM +0100, clement.perroch...@effinnov.com wrote:
> From: Clément Perrochaud
- It really is not obvious that FWDL actually means firmware download
- A small commit message explaining why you need this (Mostly for the
NXP chipset for now) would be nice
Hi Robert,
On Mon, Jan 26, 2015 at 01:13:35PM +0200, Robert Dolca wrote:
> This patch set introduces ACPI support for PN544.
>
> gpio_set_value was replaced with gpio_set_value_cansleep in order
> to allow GPIO access that may sleep. This is particularelly useful
> when GPIO is accessed using bus
Hi Robert,
On Mon, Jan 26, 2015 at 01:13:37PM +0200, Robert Dolca wrote:
> @@ -1022,9 +1127,12 @@ static int pn544_hci_i2c_probe(struct i2c_client
> *client,
> PN544_I2C_FRAME_HEADROOM, PN544_I2C_FRAME_TAILROOM,
> PN544_HCI_I2C_LLC_MAX_PAYLOAD,
Hi Mika,
On Thu, Sep 18, 2014 at 10:49:43AM +0300, Mika Westerberg wrote:
> On Wed, Sep 17, 2014 at 03:47:01PM +0200, Loic Poulain wrote:
> > Direct Irq En bit can be initialized to a bad value.
> > This bit has to be cleared for io access mode.
>
> +Eric
>
> I would like to have a bit better ex
Hi Micky,
On Mon, Feb 17, 2014 at 10:00:54AM +0800, micky_ch...@realsil.com.cn wrote:
> From: Micky Ching
>
> First we fix the card poweroff bug: the card power is not shutdown when sd/mmc
> card removed, this will make UHS-card failed to running in high speed mode if
> we
> insert the card aga
Hi Francis,
On Fri, Jan 10, 2014 at 08:26:13AM +0100, Francis Moreau wrote:
> Hi.
>
> On 12/10/2013 09:29 AM, Samuel Ortiz wrote:
> > Hi Micky,
> >
> > On Tue, Dec 10, 2013 at 09:56:48AM +0800, micky wrote:
> >> Hi Francis:
> >> On 12/10/2013 0
Hi Linus,
This is the 2nd MFD pull request for 3.13
It only contains one fix for the rtsx_pcr driver. Without it we see a
kernel panic on some machines, when resuming from suspend to RAM.
Thanks in advance for pulling it in.
The following changes since commit 78fd82238d0e5716578c326404184a27ba6
Hi Alexey,
On Sun, Jan 05, 2014 at 12:08:05AM +0400, Alexey Khoroshilov wrote:
> port100_probe() calls usb_get_dev(), but there is no usb_put_dev()
> in port100_disconnect(). The patch adds one.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Alexey Khoroshi
Hi Micky,
On Tue, Dec 10, 2013 at 09:56:48AM +0800, micky wrote:
> Hi Francis:
> On 12/10/2013 09:39 AM, wwang wrote:
> >which is based on Thomas' patch.
>
> Can you help us test this patch, we disable irq while suspend here.
I already pushed a patch from Thomas to mfd-fixes that seems to fix the
Hi Francis,
On Mon, Dec 09, 2013 at 08:33:32PM +0100, Francis Moreau wrote:
> On 12/03/2013 09:14 AM, Francis Moreau wrote:
> > Hello Thomas,
> >
> > On 12/02/2013 12:20 PM, Thomas Gleixner wrote:
> >> On Mon, 2 Dec 2013, Thomas Gleixner wrote:
> >>> On Sat, 30 Nov 2013, Francis Moreau wrote:
> >
Hi Francis,
Adding Lee to the Cc list.
On Tue, Dec 03, 2013 at 09:14:14AM +0100, Francis Moreau wrote:
> Now that you did the hard work, I hope driver's maintainer/developper
> will care about this issue.
I applied Thomas' patch to mfd-fixes.
Thanks a lot to you and Thomas for that.
Cheers,
Samu
d on I2C=y
Samuel Ortiz (1):
Merge tag 'mfd-lee-3.13-fixes-1' of git://git.linaro.org/people/ljones/mfd
drivers/mfd/Kconfig | 2 +-
drivers/mfd/lpc_ich.c | 2 +-
drivers/mfd/ti-ssp.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
--
Intel Open Source Technology Centre
Hi Geert,
On Sun, Nov 17, 2013 at 07:53:55PM +0100, Geert Uytterhoeven wrote:
> drivers/mfd/ti-ssp.c: In function 'ti_ssp_run':
> drivers/mfd/ti-ssp.c:286:2: error: implicit declaration of function
> 'set_current_state' [-Werror=implicit-function-declaration]
> drivers/mfd/ti-ssp.c:28
mfd: max77686: Include linux/of.h header
mfd: max77693: Include linux/of.h header
Documentation: mfd: Update s2mps11.txt
Samuel Ortiz (3):
Merge tag 'range-macro' of git://git.kernel.org/.../broonie/regmap
Merge tag 'mfd-lee-3.13-1' of git://git.lina
Hi Lee,
On Mon, Nov 11, 2013 at 11:33:08AM +, Lee Jones wrote:
> Hi Sam,
>
> The following changes since commit 61e6cfa80de5760bbe406f4e815b7739205754d2:
>
> Linux 3.12-rc5 (2013-10-13 15:41:28 -0700)
>
> are available in the git repository at:
>
> git://git.linaro.org/people/ljones/mf
Hi James,
On Mon, Nov 04, 2013 at 09:31:20AM -0800, James Ralston wrote:
> This patch adds the TCO Watchdog Device IDs for the Intel Wildcat Point-LP
> PCH.
>
> Signed-off-by: James Ralston
> ---
> drivers/mfd/lpc_ich.c | 13 +
> 1 file changed, 13 insertions(+)
Applied, right on t
Hi Chanwoo,
On Mon, Nov 04, 2013 at 10:58:20AM +0900, Chanwoo Choi wrote:
> Ping!
>
> Thanks,
> Chanwoo Choi
>
> On 10/10/2013 10:05 AM, Chanwoo Choi wrote:
> > The max77693 MFD device use irq domain method which has hardware interrupt
> > number
> > and virtual interrupt number getting through
Hi Salil,
On Thu, Oct 31, 2013 at 11:17:46PM +0530, Salil Kapur wrote:
> using kfree_skb() instead of kfree() for struct sk_buff
>
> Signed-off-by: Salil Kapur
> ---
> drivers/nfc/mei_phy.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to nfc-next, thanks.
Cheers,
Samuel.
--
Hi Mark,
On Fri, Oct 25, 2013 at 09:10:00PM +0100, Mark Brown wrote:
> From: Mark Brown
>
> It's not used outside this file so can be static.
>
> Signed-off-by: Mark Brown
> ---
> drivers/mfd/as3722.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied, thanks.
Cheers,
Samuel.
-
Hi Lee,
On Wed, Oct 23, 2013 at 04:35:10PM +0100, Lee Jones wrote:
> The following changes since commit 61e6cfa80de5760bbe406f4e815b7739205754d2:
>
> Linux 3.12-rc5 (2013-10-13 15:41:28 -0700)
>
> are available in the git repository at:
>
> git://git.linaro.org/people/ljones/mfd.git tags/mf
Hi Laxman,
On Wed, Sep 26, 2012 at 06:18:04PM +0530, Laxman Dewangan wrote:
> Driver throw the warning message if dt node does not
> have the info for VMBCH-Threshold and VMBCH2-Threshold.
> These properties are optional property and hence it
> is not mandatory to have these on DT node and in this
Hi Olof,
On Wed, Oct 09, 2013 at 05:51:14PM -0700, Olof Johansson wrote:
> Hi,
>
> On Wed, Sep 26, 2012 at 5:48 AM, Laxman Dewangan wrote:
> > Driver throw the warning message if dt node does not
> > have the info for VMBCH-Threshold and VMBCH2-Threshold.
> > These properties are optional proper
1 - 100 of 685 matches
Mail list logo