Hi Ilan, 2011/11/1 <ilanelias78@xxxxxxxxx>: > From: Ilan Elias <ilane@xxxxxx> > > Addition, deletion and modification of NCI constants. > Changes in NCI commands, responses and notifications structures. > > Signed-off-by: Ilan Elias <ilane@xxxxxx> > --- > include/net/nfc/nci.h | 183 ++++++++++++++++++++++++++------------------ > include/net/nfc/nci_core.h | 11 ++- > net/nfc/nci/core.c | 7 +- > net/nfc/nci/data.c | 5 +- > net/nfc/nci/lib.c | 8 +-- > net/nfc/nci/ntf.c | 137 +++++++++++++++++++++------------ > net/nfc/nci/rsp.c | 58 +++++++++------ > 7 files changed, 244 insertions(+), 165 deletions(-) > > diff --git a/include/net/nfc/nci.h b/include/net/nfc/nci.h > index 39b85bc..37fe8b3 100644 > --- a/include/net/nfc/nci.h > +++ b/include/net/nfc/nci.h > @@ -33,47 +33,49 @@ > #define NCI_MAX_NUM_RF_CONFIGS 10 > #define NCI_MAX_NUM_CONN 10 > > -/* NCI Status Codes */ > -#define NCI_STATUS_OK 0x00 > -#define NCI_STATUS_REJECTED 0x01 > -#define NCI_STATUS_MESSAGE_CORRUPTED 0x02 > -#define NCI_STATUS_BUFFER_FULL 0x03 > -#define NCI_STATUS_FAILED 0x04 > -#define NCI_STATUS_NOT_INITIALIZED 0x05 > -#define NCI_STATUS_SYNTAX_ERROR 0x06 > -#define NCI_STATUS_SEMANTIC_ERROR 0x07 > -#define NCI_STATUS_UNKNOWN_GID 0x08 > -#define NCI_STATUS_UNKNOWN_OID 0x09 > -#define NCI_STATUS_INVALID_PARAM 0x0a > -#define NCI_STATUS_MESSAGE_SIZE_EXCEEDED 0x0b > -/* Discovery Specific Status Codes */ > -#define NCI_STATUS_DISCOVERY_ALREADY_STARTED 0xa0 > -#define NCI_STATUS_DISCOVERY_TARGET_ACTIVATION_FAILED 0xa1 > +/* Generic Status Codes */ > +#define NCI_STATUS_OK 0x00 > +#define NCI_STATUS_REJECTED 0x01 > +#define NCI_STATUS_RF_FRAME_CORRUPTED 0x02 > +#define NCI_STATUS_FAILED 0x03 > +#define NCI_STATUS_NOT_INITIALIZED 0x04 > +#define NCI_STATUS_SYNTAX_ERROR 0x05 > +#define NCI_STATUS_SEMANTIC_ERROR 0x06 > +#define NCI_STATUS_UNKNOWN_GID 0x07 > +#define NCI_STATUS_UNKNOWN_OID 0x08 > +#define NCI_STATUS_INVALID_PARAM 0x09 > +#define NCI_STATUS_MESSAGE_SIZE_EXCEEDED 0x0a > +/* RF Discovery Specific Status Codes */ > +#define NCI_STATUS_DISCOVERY_ALREADY_STARTED 0xa0 > +#define NCI_STATUS_DISCOVERY_TARGET_ACTIVATION_FAILED 0xa1 > +#define NCI_STATUS_DISCOVERY_TEAR_DOWN 0xa2 > /* RF Interface Specific Status Codes */ > -#define NCI_STATUS_RF_TRANSMISSION_ERROR 0xb0 > -#define NCI_STATUS_RF_PROTOCOL_ERROR 0xb1 > -#define NCI_STATUS_RF_TIMEOUT_ERROR 0xb2 > -#define NCI_STATUS_RF_LINK_LOSS_ERROR 0xb3 > +#define NCI_STATUS_RF_TRANSMISSION_ERROR 0xb0 > +#define NCI_STATUS_RF_PROTOCOL_ERROR 0xb1 > +#define NCI_STATUS_RF_TIMEOUT_ERROR 0xb2 > /* NFCEE Interface Specific Status Codes */ > -#define NCI_STATUS_MAX_ACTIVE_NFCEE_INTERFACES_REACHED 0xc0 > -#define NCI_STATUS_NFCEE_INTERFACE_ACTIVATION_FAILED 0xc1 > -#define NCI_STATUS_NFCEE_TRANSMISSION_ERROR 0xc2 > -#define NCI_STATUS_NFCEE_PROTOCOL_ERROR 0xc3 > +#define NCI_STATUS_MAX_ACTIVE_NFCEE_INTERFACES_REACHED 0xc0 > +#define NCI_STATUS_NFCEE_INTERFACE_ACTIVATION_FAILED 0xc1 > +#define NCI_STATUS_NFCEE_TRANSMISSION_ERROR 0xc2 > +#define NCI_STATUS_NFCEE_PROTOCOL_ERROR 0xc3 > #define NCI_STATUS_NFCEE_TIMEOUT_ERROR 0xc4 You are mixing indentation changes with changes in constants. It would be better to put the indentation changes in a another patch. Lauro ��.n��������+%������w��{.n�����{���zW����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f