From: "Tobin C. Harding" <me@tobin.cc>
To: Shiva Kerdel <shiva@exdev.nl>
Cc: gregkh@linuxfoundation.org, wsa+renesas@sang-engineering.com,
p.hoefflin@posteo.de, karniksayli1995@gmail.com,
sabitha.george@gmail.com, kdunne@mail.ccsf.edu,
bhumirks@gmail.com, mattkilgore12@gmail.com,
georgiana.chelu93@gmail.com, goudapatilk@gmail.com,
mihaela.muraru21@gmail.com, devel@driverdev.osuosl.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t'
Date: Sun, 12 Mar 2017 14:00:12 +1100 [thread overview]
Message-ID: <20170312030012.GA1012@eros> (raw)
In-Reply-To: <20170312023001.6621-1-shiva@exdev.nl>
On Sun, Mar 12, 2017 at 03:29:59AM +0100, Shiva Kerdel wrote:
> Fix prefer kernel type 'u8' over 'uint8_t' checks.
>
> Signed-off-by: Shiva Kerdel <shiva@exdev.nl>
> ---
> drivers/staging/ks7010/ks_hostif.c | 4 +-
> drivers/staging/ks7010/ks_hostif.h | 114 +++++++++++++++++------------------
> drivers/staging/ks7010/ks_wlan_net.c | 2 +-
> 3 files changed, 60 insertions(+), 60 deletions(-)
>
> diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010/ks_hostif.c
> index 6522bc3d30d5..0d6bbe61131c 100644
> --- a/drivers/staging/ks7010/ks_hostif.c
> +++ b/drivers/staging/ks7010/ks_hostif.c
> @@ -2384,8 +2384,8 @@ void hostif_sme_set_pmksa(struct ks_wlan_private *priv)
> struct pmk_cache_t {
> uint16_t size;
> struct {
> - uint8_t bssid[ETH_ALEN];
> - uint8_t pmkid[IW_PMKID_LEN];
> + u8 bssid[ETH_ALEN];
> + u8 pmkid[IW_PMKID_LEN];
> } __packed list[PMK_LIST_MAX];
> } __packed pmkcache;
> struct pmk_t *pmk;
> diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/ks7010/ks_hostif.h
> index 35bfadf4665d..be8ded44c1ac 100644
> --- a/drivers/staging/ks7010/ks_hostif.h
> +++ b/drivers/staging/ks7010/ks_hostif.h
> @@ -72,7 +72,7 @@ struct hostif_data_request_t {
> #define TYPE_DATA 0x0000
> #define TYPE_AUTH 0x0001
> uint16_t reserved;
> - uint8_t data[0];
> + u8 data[0];
> } __packed;
>
> struct hostif_data_indication_t {
> @@ -83,14 +83,14 @@ struct hostif_data_indication_t {
> #define TYPE_GMK1 0x0002
> #define TYPE_GMK2 0x0003
> uint16_t reserved;
> - uint8_t data[0];
> + u8 data[0];
> } __packed;
>
> #define CHANNEL_LIST_MAX_SIZE 14
> struct channel_list_t {
> - uint8_t size;
> - uint8_t body[CHANNEL_LIST_MAX_SIZE];
> - uint8_t pad;
> + u8 size;
> + u8 body[CHANNEL_LIST_MAX_SIZE];
> + u8 pad;
> } __packed;
>
> /* MIB Attribute */
> @@ -154,7 +154,7 @@ struct hostif_mib_value_t {
> #define MIB_VALUE_TYPE_BOOL 2
> #define MIB_VALUE_TYPE_COUNT32 3
> #define MIB_VALUE_TYPE_OSTRING 4
> - uint8_t body[0];
> + u8 body[0];
> } __packed;
>
> struct hostif_mib_get_confirm_t {
> @@ -227,32 +227,32 @@ struct hostif_start_confirm_t {
>
> #define SSID_MAX_SIZE 32
> struct ssid_t {
> - uint8_t size;
> - uint8_t body[SSID_MAX_SIZE];
> - uint8_t ssid_pad;
> + u8 size;
> + u8 body[SSID_MAX_SIZE];
> + u8 ssid_pad;
> } __packed;
>
> #define RATE_SET_MAX_SIZE 16
> struct rate_set8_t {
> - uint8_t size;
> - uint8_t body[8];
> - uint8_t rate_pad;
> + u8 size;
> + u8 body[8];
> + u8 rate_pad;
> } __packed;
>
> struct FhParms_t {
> uint16_t dwellTime;
> - uint8_t hopSet;
> - uint8_t hopPattern;
> - uint8_t hopIndex;
> + u8 hopSet;
> + u8 hopPattern;
> + u8 hopIndex;
> } __packed;
>
> struct DsParms_t {
> - uint8_t channel;
> + u8 channel;
> } __packed;
>
> struct CfParms_t {
> - uint8_t count;
> - uint8_t period;
> + u8 count;
> + u8 period;
> uint16_t maxDuration;
> uint16_t durRemaining;
> } __packed;
> @@ -262,27 +262,27 @@ struct IbssParms_t {
> } __packed;
>
> struct rsn_t {
> - uint8_t size;
> + u8 size;
> #define RSN_BODY_SIZE 64
> - uint8_t body[RSN_BODY_SIZE];
> + u8 body[RSN_BODY_SIZE];
> } __packed;
>
> struct ErpParams_t {
> - uint8_t erp_info;
> + u8 erp_info;
> } __packed;
>
> struct rate_set16_t {
> - uint8_t size;
> - uint8_t body[16];
> - uint8_t rate_pad;
> + u8 size;
> + u8 body[16];
> + u8 rate_pad;
> } __packed;
>
> struct ap_info_t {
> - uint8_t bssid[6]; /* +00 */
> - uint8_t rssi; /* +06 */
> - uint8_t sq; /* +07 */
> - uint8_t noise; /* +08 */
> - uint8_t pad0; /* +09 */
> + u8 bssid[6]; /* +00 */
> + u8 rssi; /* +06 */
> + u8 sq; /* +07 */
> + u8 noise; /* +08 */
> + u8 pad0; /* +09 */
> uint16_t beacon_period; /* +10 */
> uint16_t capability; /* +12 */
> #define BSS_CAP_ESS BIT(0)
> @@ -295,21 +295,21 @@ struct ap_info_t {
> #define BSS_CAP_CHANNEL_AGILITY BIT(7)
> #define BSS_CAP_SHORT_SLOT_TIME BIT(10)
> #define BSS_CAP_DSSS_OFDM BIT(13)
> - uint8_t frame_type; /* +14 */
> - uint8_t ch_info; /* +15 */
> + u8 frame_type; /* +14 */
> + u8 ch_info; /* +15 */
> #define FRAME_TYPE_BEACON 0x80
> #define FRAME_TYPE_PROBE_RESP 0x50
> uint16_t body_size; /* +16 */
> - uint8_t body[1024]; /* +18 */
> + u8 body[1024]; /* +18 */
> /* +1032 */
> } __packed;
>
> struct link_ap_info_t {
> - uint8_t bssid[6]; /* +00 */
> - uint8_t rssi; /* +06 */
> - uint8_t sq; /* +07 */
> - uint8_t noise; /* +08 */
> - uint8_t pad0; /* +09 */
> + u8 bssid[6]; /* +00 */
> + u8 rssi; /* +06 */
> + u8 sq; /* +07 */
> + u8 noise; /* +08 */
> + u8 pad0; /* +09 */
> uint16_t beacon_period; /* +10 */
> uint16_t capability; /* +12 */
> struct rate_set8_t rate_set; /* +14 */
> @@ -318,16 +318,16 @@ struct link_ap_info_t {
> struct CfParms_t cf_parameter; /* +30 */
> struct IbssParms_t ibss_parameter; /* +36 */
> struct ErpParams_t erp_parameter; /* +38 */
> - uint8_t pad1; /* +39 */
> + u8 pad1; /* +39 */
> struct rate_set8_t ext_rate_set; /* +40 */
> - uint8_t DTIM_period; /* +50 */
> - uint8_t rsn_mode; /* +51 */
> + u8 DTIM_period; /* +50 */
> + u8 rsn_mode; /* +51 */
> #define RSN_MODE_NONE 0
> #define RSN_MODE_WPA 1
> #define RSN_MODE_WPA2 2
> struct {
> - uint8_t size; /* +52 */
> - uint8_t body[128]; /* +53 */
> + u8 size; /* +52 */
> + u8 body[128]; /* +53 */
> } __packed rsn;
> } __packed;
>
> @@ -400,7 +400,7 @@ struct hostif_infrastructure_set2_request_t {
> #define AUTH_TYPE_SHARED_KEY 1
> struct channel_list_t channel_list;
> uint16_t scan_type;
> - uint8_t bssid[ETH_ALEN];
> + u8 bssid[ETH_ALEN];
> } __packed;
>
> struct hostif_infrastructure_set_confirm_t {
> @@ -431,7 +431,7 @@ struct hostif_adhoc_set2_request_t {
> * bit10:ShortSlotTime bit13:DSSS-OFDM DSSS-OFDM not supported always 0 */
> uint16_t scan_type;
> struct channel_list_t channel_list;
> - uint8_t bssid[ETH_ALEN];
> + u8 bssid[ETH_ALEN];
> } __packed;
>
> struct hostif_adhoc_set_confirm_t {
> @@ -440,26 +440,26 @@ struct hostif_adhoc_set_confirm_t {
> } __packed;
>
> struct last_associate_t {
> - uint8_t type;
> - uint8_t status;
> + u8 type;
> + u8 status;
> } __packed;
>
> struct association_request_t {
> - uint8_t type;
> + u8 type;
> #define FRAME_TYPE_ASSOC_REQ 0x00
> #define FRAME_TYPE_REASSOC_REQ 0x20
> - uint8_t pad;
> + u8 pad;
> uint16_t capability;
> uint16_t listen_interval;
> - uint8_t ap_address[6];
> + u8 ap_address[6];
> uint16_t reqIEs_size;
> } __packed;
>
> struct association_response_t {
> - uint8_t type;
> + u8 type;
> #define FRAME_TYPE_ASSOC_RESP 0x10
> #define FRAME_TYPE_REASSOC_RESP 0x30
> - uint8_t pad;
> + u8 pad;
> uint16_t capability;
> uint16_t status;
> uint16_t association_id;
> @@ -476,10 +476,10 @@ struct hostif_associate_indication_t {
>
> struct hostif_bss_scan_request_t {
> struct hostif_hdr header;
> - uint8_t scan_type;
> + u8 scan_type;
> #define ACTIVE_SCAN 0
> #define PASSIVE_SCAN 1
> - uint8_t pad[3];
> + u8 pad[3];
> uint32_t ch_time_min;
> uint32_t ch_time_max;
> struct channel_list_t channel_list;
> @@ -502,10 +502,10 @@ struct hostif_phy_information_request_t {
>
> struct hostif_phy_information_confirm_t {
> struct hostif_hdr header;
> - uint8_t rssi;
> - uint8_t sq;
> - uint8_t noise;
> - uint8_t link_speed;
> + u8 rssi;
> + u8 sq;
> + u8 noise;
> + u8 link_speed;
> uint32_t tx_frame;
> uint32_t rx_frame;
> uint32_t tx_error;
> diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c
> index d06dfe0f6732..2b4d6c1cb037 100644
> --- a/drivers/staging/ks7010/ks_wlan_net.c
> +++ b/drivers/staging/ks7010/ks_wlan_net.c
> @@ -2767,7 +2767,7 @@ static int ks_wlan_set_wps_probe_req(struct net_device *dev,
> struct iw_request_info *info,
> struct iw_point *dwrq, char *extra)
> {
> - uint8_t *p = extra;
> + u8 *p = extra;
> unsigned char len;
> struct ks_wlan_private *priv =
> (struct ks_wlan_private *)netdev_priv(dev);
> --
> 2.12.0
>
Why did you only do these three files? uint8_t appears in other files
in ks7010/ also. Perhaps you could do all the files in one patch.
If you like you could do a patch set changing all the standard types
to kernel types. i.e one patch per type (uint8_t, uint16_t ...).
I have a patch set in flight that includes these type changes with
review asking me to do exactly this. However, if you would like to do
it I will remove the type changes from my patch set and wait until yours is applied before
re-submitting.
Please let me know if you intend to do this.
Oh, and if you really want cudos you could add a bit more detail to
your git log, for example
Checkpatch emits warning ...
Change standard type uint8_t to kernel type u8.
Well done and good luck.
thanks,
Tobin.
next prev parent reply other threads:[~2017-03-12 3:00 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-12 2:29 [PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t' Shiva Kerdel
2017-03-12 2:30 ` [PATCH 2/3] Staging: ks7010: ks_hostif.*: Use preferred 'u16' kernel type over 'uint16_t' Shiva Kerdel
2017-03-12 2:30 ` [PATCH 3/3] Staging: ks7010: ks_hostif.*: Use preferred 'u32' kernel type over 'uint32_t' Shiva Kerdel
2017-03-12 3:00 ` Tobin C. Harding [this message]
2017-03-14 15:12 ` [PATCH 1/3] Staging: ks7010: ks_*: Use preferred 'u8' kernel type over 'uint8_t' Shiva Kerdel
2017-03-14 20:33 ` Tobin C. Harding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170312030012.GA1012@eros \
--to=me@tobin.cc \
--cc=bhumirks@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=georgiana.chelu93@gmail.com \
--cc=goudapatilk@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=karniksayli1995@gmail.com \
--cc=kdunne@mail.ccsf.edu \
--cc=linux-kernel@vger.kernel.org \
--cc=mattkilgore12@gmail.com \
--cc=mihaela.muraru21@gmail.com \
--cc=p.hoefflin@posteo.de \
--cc=sabitha.george@gmail.com \
--cc=shiva@exdev.nl \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox