commit 1398dc3b03b10f8c2895afddd1ad6afb12e32370
author Nick Pelly <npelly@google.com> Thu Jul 14 16:06:52 2011 -0700
committer The Android Automerger <android-build@android.com> Thu Jul 14 22:17:27 2011 -0700
tree fe6000c273b677854d1393855bb18cb6459a56d8
parent 23a22e9a09b67902c1183c95e6d93e418437c71b

Revert "Patch to decrease the maximum packet size for NFC firmware download"

This reverts commit 294b456a3692cd69f45713bc8e9700132a0e5aec.

@#$% the pn544.

This patch to execute FW download mode with 64 byte frames has some kind of
side-affect. It causes the pn544 to send

> NFC I2C Received buffer:  0xB2 0x83 0x53 0xC9 0x6B

every 2 seconds. We do not yet know what this command means, it is undocumented.
This side affect appear permanent - even after reverting to a 512 byte FW
download, the NFC controller continues to send these messages.

Reverting ASAP to minimize damage.

Change-Id: I139a4a76c045f580df9760dc11d1c6df7b317a50

src/phDnldNfc.c

diff --git a/src/phDnldNfc.c b/src/phDnldNfc.c
index 54d5b87..ba56513 100644
--- a/src/phDnldNfc.c
+++ b/src/phDnldNfc.c
@@ -95,8 +95,8 @@
 #define PHDNLD_ADDR_SIZE         0x03U
 #define PHDNLD_DATA_LEN_SIZE     0x02U
 #define PHDNLD_MIN_PACKET        0x03U    /* Minimum Packet Size is 3*/
-#define PHDNLD_MAX_PACKET        0x40U    /* Max Total Packet Size is 64 bytes */
-#define PHDNLD_DATA_SIZE         0x38U    /* Max Data Size is 56 bytes */
+#define PHDNLD_MAX_PACKET        0x0200U  /* Max Total Packet Size is 512 */
+#define PHDNLD_DATA_SIZE         0x01F8U  /* Max Data Size is 504 */
 
 #define FW_MAX_SECTION           0x15U    /* Max Number of Sections */
 
@@ -351,7 +351,7 @@
     union param
     {
         phDnldNfc_sParam_t data_param;
-        uint8_t            response_data[PHDNLD_MAX_PACKET<<1];
+        uint8_t            response_data[PHDNLD_MAX_PACKET];
         uint8_t            config_verify_param;
     }param_info;
 }phDnldNfc_sData_t;