| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
This should fix pn544 lock-ups seen during wired-mode stress tests.
Change-Id: I606b5909338dc889a475ad880df8646546b9ba40
|
|\
| |
| |
| | |
c1f33130d9d2100e6232f8f3c782949c75205be6" into gingerbread
|
| |
| |
| |
| |
| |
| | |
c1f33130d9d2100e6232f8f3c782949c75205be6
Change-Id: I8c2bd923d9dedf89c784199f03fba0e2cfc12009
|
| |
| |
| |
| |
| |
| |
| |
| | |
o Fix DEFAULT_NFCIP_INITIATOR_MODE_SUPPORT, DEFAULT_NFCIP_TARGET_MODE_SUPPORT
to allow all possible modes (actual modes selected by Nfc.apk).
o Fix phNfc_eP2PMode_t enum to list initiator modes.
Change-Id: I841b6f1387ac536f47d357a0430eb9362b419810
|
|/
|
|
|
|
| |
NXP have confirmed that 3ms delay is only needed for writes destined for SWP.
Change-Id: I90a5229dbb738e1f00d37a052c7ac55e0e9d215c
|
|
|
|
|
|
|
| |
This patch permit to display the NFC frames exchanged through the I2C line.
To enable that, you just need to enable LOW_LEVEL_TRACES macro.
Change-Id: Ica2ff2e6d111b76c056161ec557346f23d21ada0
|
|
|
|
|
|
|
|
| |
Valid LLCP Service Access Point values are 0x00 to 0x3F, not 0x2F.
Before this patch, a socket with a local SAP between 0x30 and 0x3F
was unable to communicate (operation callbacks were never called).
Change-Id: Ic3f9b9f353a01e2682e28575152f171d7ec421e4
|
|
|
|
| |
Change-Id: I9fd0291bfd8ba9d26a0c93bc14d7184a9a188b90
|
|
|
|
|
|
|
|
| |
This patch permit to perform a recovery when a transaction failure is detected.
To detect this exception, we are filtering the Ox57 bytes because this means that the IRQ
line is high but no data is available from PN544.
Change-Id: Ibb55ea3db2bc6cdff2c5b28bbe2127250c5637a8
|
|\ |
|
| |
| |
| |
| |
| |
| | |
Disabling P2P 106 passive caused a large P2P discovery regression.
Change-Id: I9856ba1a29cd717da2f578abd073b341c54fce36
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If a NFCIP frame is received (NXP_EVT_NFC_RCV_DATA event) before a call to
phHal4Nfc_Receive() function, the frame was lost. This could happen if the local
host acts as a Target and the remote peer sends its first frame too fast.
The data is actually saved in a temporary buffer but, when the phHal4Nfc_Receive()
is eventualy called, it triggers a callback with a status code
NFCSTATUS_MORE_INFORMATION, which is handled as an error.
This patch fixes this behaviour and makes sure that the correct status
code is provided with the frame itself.
Change-Id: I5311762ca70c71f98dea0287b0844eb1c8dddb6b
|
|
|
|
|
|
|
|
|
|
|
| |
The LLCP specification defines that if a TLV cannot be interpreted,
it shall be ignored rather than generating an error.
This patch removed all error reporting on TLV interpretation. The
only error that can still happen is length inconsistency in the
whole TLV array buffer.
Change-Id: Ibf289fceb2283bfddcdc0ddee9e687ece3e90887
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The parsing algorithm for AGF frames has been fixed. It was previously
only working if aggregated frames were 4-bytes long.
The current stack only support mandatory de-aggregation. It does not
send aggregated frames, so there won't be interop issue with previous
versions.
This fix prevents reading erroneous de-aggregated frames and rejecting
them using FRMR when communicating with an LLCP implementation
using aggregation mechanism.
Change-Id: Ia5728496a2917b6a57f478cc51dc0aaa0fcc165a
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
P2P target 106 passive appears as an ISO 14443-3A target to reader/writers
in the field. This can confuse some terminals, especially when we also have
card emulation turned on, since it looks like we have two -3A targets in
the field.
P2P target 106 passive is just one of many P2P sub-modes. We do not know of any
P2P devices that only initiate using only 106 passive, so this should not
present inter-op issues.
Change-Id: I1cb84ed791a86a7f225a51806e99bab031dcb6ca
|
| |
|
|
|
|
| |
Change-Id: I6f3a96b9b063b66f735ff97c1aa63aaaad0a03b9
|
|
|
|
| |
Change-Id: I2a7a185ead5de8f2b165c81dcc8ab8fb46c1ddc0
|
|
|
|
|
|
| |
This patch allows reading and writing of NDEF messages on ICODE tags.
Change-Id: I43b4845c898c2bf268f609a5a81e991460059ff6
|
|
|
|
|
|
|
|
| |
Time-out value of 0 is used to indicate minimum value, see e.g.
PH_HAL4NFC_RECV_CB_TIMEOUT in phHal4Nfc_P2P.c where it is used with P2P target
receive in case data has already been received by the PN544
Change-Id: I9f15f2135ca003b848590fc87265edb39d85bf4c
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
During extensive testing we found that some ISO15693 tags do not react
to some commands. However, they do send an error code to the PN544 indicating
they do not support the command. The PN544 in turn sends one specific error code
over the HCI interface. However, libNFC maps that error code always
to NFCSTATUS_TARGET_LOST, which in the end maps to the application API reporting
a TargetLostException.
This fix maps only the timeout return values from the PN544 to NFCSTATUS_TARGET_LOST;
all other values are mapped to NFCSTATUS_FAILED, which in turn maps to an
IOException at API level.
In effect, a timeout is still not the same as a target lost; libNFC does not do any
additional internal presence check to make sure the tag is really gone, although
it usually is. I'll submit a separate fix on packages/apps/Nfc to take care of this.
Change-Id: If64857658d683aff29134f74b42efd99b695e634
|
|
|
|
|
|
|
|
| |
Wired Mode.
This fix permit to detect the Secure Element in Wired Mode, even we are in front of a RF Reader
Change-Id: I8f87e043ff24724addd179ce025cf3d038a82a5e
|
|
|
|
| |
Change-Id: I57eaa3ccd65b89510e198f7ebf24f7266c4dad2e
|
|
|
|
| |
Change-Id: If6b490d906fcc37960a1120e9344470d68c825f6
|
|
|
|
|
|
|
|
|
|
| |
This allows for reading/writing more than 255 bytes of NDEF data on an
NFC Forum Type 2 tag.
This fix overrides https://partner.source.android.com/g/#change,4907 ,
since that fix only worked for reading.
Change-Id: I7fb2ea1bc04f35ba6e6be56e0c89abab567e3ac7
|
|
|
|
| |
Change-Id: Idcbf0ab9b982b66c39b68762129766e16017558c
|
|
|
|
|
|
| |
This should avoid calling a irrelevant callback at abort.
Change-Id: Ied8d54268613f248fd505d4b25d0ab8a0905a336
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The phLibNfc_Llcp_Send() function is expected to trigger a callback
when the packet is actually sent but, in some situations, the callback
was triggered by another frame sending (such as RR or DM frames).
This patch fixes the following socket internal flags handling:
- pfSocketSend_Cb: set to non-NULL value when a send operation is
pending,
- bSocketSendPending: set to TRUE if the LLCP link layer is not
available for sending and reset to FALSE once the frame is
actually forwarded to the LLCP link layer.
The send callback triggering has been updated to ensure it is only
done for the expected data frame (i.e.: send pending and already
forwarded to link layer).
Change-Id: Id7410ee4075fbbbc9de382abf54e89a97f6a8b37
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moved data allocation from stack memory to heap in order
to avoid invalid memory access.
Previously, When trying to send a DM (acknowledgment to
socket disconnect request), the stack was allocating some
frame data on the stack but these data were used after the
function returned.
Other calls to the same function are already using variables
allocated on heap.
Change-Id: Id7356a876fdecdd979ed3ddc6dbe100d6e92d43d
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before sending a packet, the LLCP stack have to check if the remote
peer is ready to send. To do so, a Receive Window mechanism based on
packet numbering and acknowledgment is used.
Basically, the sender have to make sure that he must have not sent
more than RW-1 unaknowledged frames before sending a new one. This
patch is correcting this test which was failing in some situations.
Change-Id: I525f6b472a909ce48feb938aa02858b9456edc11
|
|
|
|
|
|
|
|
|
|
|
|
| |
The correct behavior of asynchronous functions should be:
Immediate operation => function returns with success/error status,
callback never invoked
Delayed operation => function returns with pending status,
callback is invoked at the end of operation
Some minor fixes are also included in this commit (code style).
Change-Id: Ieb801ab759045c5c5d9fc59b4715f3c142d5b33e
|
|
|
|
|
|
|
|
|
| |
The listen callback is a notification callback, not an
end-of-operation callback so, when the socket is closed,
it should not be triggered since there is no connection
event.
Change-Id: I6e6994f6bcb09f9fadda04800d22528d76aff121
|
|
|
|
|
|
|
| |
The sequence value may not be provided in some situations, when
rejecting a connection oriented frame.
Change-Id: I116179191f9a23072442cf9da7a4c4207128bd32
|
|
|
|
|
|
| |
Avoid NULL pointer dereferencing in some situations.
Change-Id: I020f0a5342acdc49f7f3804e9341ed0fc0f307ee
|
|
|
|
| |
Change-Id: I08e8dda5ae23ec251a5b53f71b0e87fb3ba7d02c
|
|
|
|
|
|
|
| |
Check that internal allocated buffer is not NULL before deallocating
it and make sure the pointer is resetted to NULL.
Change-Id: I4212489c8661c7a8dff8ef352029100ec5ab157e
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This enables to register a server socket prior to LLCP link
establishement and avoid any race condition between the local
server and the remote client.
The modification implies a LLCP socket API change (added the
device handle parameter in all LLCP API that can lead to
active data exchange). The LLCP socket internal structures are
now initialized during LLCP link parameter configuration step
instead of LLCP compliancy check to save socket state
independently from peer detection.
Also removed unused socket states and added some traces.
Change-Id: I961c48af4ca9ace68d41b9569dc1038bb2bbdc71
|
|
|
|
|
|
|
| |
This avoids an application to be stuck in a send request while the
LLCP link has been lost (i.e. remote peer lost).
Change-Id: I74982d497c410ae4bdb466afac013efc9eb17b03
|
|
|
|
| |
Change-Id: I1bd1da26d0c27297e9db8dd41cb6c2f4b2ec806e
|
|
|
|
|
|
|
|
| |
The NFC Forum Type 1 Tag and Type 2 Tag specification allows for tags
with a dynamic memory layout. Support for making tags with the dynamic
layout readonly has been added in this patch.
Change-Id: I5bc8912d80f448fdea95e1ee21631c0f186ad79a
|
|
|
|
|
|
|
|
|
|
|
| |
Made the ISO XCHG timer (used for IsoDep transceive's) a configurable
setting. When this timer triggers in the hardware, the transaction will be
aborted and the connection to the tag is broken.
Also made the HCI timer configurable. This timer serves as a software watchdog
for any HCI transaction.
Change-Id: I927c7755378f793899c6d262a2f23ab5dca3817b
|
|
|
|
| |
Change-Id: Ieafb6ab9094d9dc4b7edfb1e3676fd9794d8af28
|
|
|
|
|
|
| |
Source: Patrice @ NXP
Change-Id: I9437a2c2798ffe7ea34f3f18df097d0036310568
|
|
|
|
| |
Change-Id: Id2991644e9a45a66172377d66b5dc3e99bbd6a39
|
|
|
|
| |
Change-Id: I594e588fb7f0f8151e54ac872640bb8d3e19bfce
|
|
|
|
|
|
|
|
| |
Implemented for T1T and T2T.
There's also added code for formatting Desfire EV1, but it will not be used
by the current implementation (DesFIRE doesn't have NdefFormatable tech).
Change-Id: Iec1b85b560fbf800291fd307b56ab84328737635
|
|
|
|
|
|
|
| |
This is needed for proper multi-protocol tag support.
(Before this required a restart of the polling loop).
Change-Id: If61da437cda40d82bdbacf5a01ea4c585c7a5be9
|
|
|
|
|
|
|
| |
The receive function was failing when trying to read the data present in
the linear buffer after receiving a remote disconnection request (DISC).
Change-Id: I48a2aa058e85e265ca53e61dcde281e47a6e187f
|
|
|
|
| |
Change-Id: I4784722900b9da58719b7b38a1cd31cbac3ff147
|
|
|
|
|
|
| |
This would prohibit detection of multiple incoming socket in a row.
Change-Id: I9e3907a92b706de2da7642ccd92668b9868d1996
|