This patch removes the warnings reported by checkpatch.pl
for line over 80 characters.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Ack_no variable to ack_no
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Data_offset variable to data_offset
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Total_Length variable to total_length
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch removes do-nothing else condition case.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Ack Session_index  variable to session_index
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Ack member variable to ack
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the add_TCP_Pending_Ack function to add_tcp_pending_ack
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Update_TCP_track_session function to update_tcp_session
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the add_TCP_track_session function to add_tcp_session
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Init_TCP_tracking function to init_tcp_tracking
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Pending_Acks variable to pending_acks
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Opened_TCP_session variable to tcp_session
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the PendingAcks_arrBase variable to pending_base
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Pending_Acks_info variable to pending_acks_info
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename the Acks_keep_track_info variable to ack_session_info
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Session_index of struct pending_acks_info to session_index
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch remove typedef of struct Pending_Acks_info_t.
And, rename the Pending_Acks_info_t to pending_acks_info.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Bigger_Ack_num of struct ack_session_info to bigger_ack_num
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Ack_seq_num of struct ack_session_info to seq_num
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Ack_session_info struct variable to ack_session_info
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Statisitcs_DroppedAcks variable to dropped_acks
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Statisitcs_totalAcks variable to total_acks
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch replace explicit NULL comparison with ! operator to simplify code.
Reported by checkpatch.pl for Comparison to NULL could be written "!XXX" or "XXX".
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename genuChipPSstate variable to chip_ps_state
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the warnings reported by checkpatch.pl
for Missing a blank line after declarations.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the warnings reported by checkpatch.pl
for please use a blank line after function/struct/union/enum declarations
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the checks reported by checkpatch.pl
for Blank lines aren't necessary after an open brace '{' and
Blank lines aren't necessary before a close brace '}'.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the checks reported by checkpatch.pl
for Please don't use multiple blank lines.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

There are over-commenting in the wilc_wlan.c file and most of them
are not helpful to explain what the code does and generate 80 ending
line over warnings. So, all of comments are removed in this patch and the
comments will later be added if necessary with the preferred Linux style.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the error reported by checkpatch.pl
for that open brace { should be on the previous line.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the checks reported by checkpatch.pl
for prefer kmalloc(sizeof(*tx_data)...) over kmalloc(sizeof(struct tx_complete_data)...)
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes the checks reported by checkpatch.pl
for alignment should match open parenthesis
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch fixes build warning and error reported by kbuild test robot.
It is fixed by including netdevice.h.

>> drivers/staging/wilc1000/wilc_wlan_if.h:940:27: warning: 'struct net_device'
 declared inside parameter list
    int wilc_wlan_init(struct net_device *dev, wilc_wlan_inp_t *inp);
>> drivers/staging/wilc1000/wilc_wlan_if.h:940:27: warning: its scope is only
 this definition or declaration, which is probably not what you want
>> drivers/staging/wilc1000/wilc_wlan.c:1954:5: error: conflicting types for
 'wilc_wlan_init'
    int wilc_wlan_init(struct net_device *dev, wilc_wlan_inp_t *inp)

Fixes: 30135ce ("staging: wilc1000: wilc_wlan_init: add argument struct net_device")
Reported-by: Nkbuild test robot <fengguang.wu@intel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename pu8UdpBuffer variable of mac_xmit function to udp_buf
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename QueueCount variable of mac_xmit function to queue_count
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename s32Error variable of mac_ioctl function to ret
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename host_int_get_MacAddress function to hif_get_mac_address
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch rename Set_machw_change_vir_if function to set_machw_change_vir_if
to avoid CamelCase naming convention.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>

This patch removes goto from mac_open function. If address is invalid, goto
handles deinit process and return result.
So, just call deinit process and return the error value directly instead of
goto statement.
Signed-off-by: NLeo Kim <leo.kim@atmel.com>
Signed-off-by: NGlen Lee <glen.lee@atmel.com>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>