Search Linux Wireless

Re: [PATCH v3 1/3] NFC: Update names and structs to NCI spec 1.0 d18

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux