On Tue, 2016-01-12 at 09:32 +0900, Chaehyun Lim wrote:
> This patch removes typedef from struct wilc_cfg_byte_t and renames it to
> wilc_cfg_byte.
Is this really a good name?
> diff --git a/drivers/staging/wilc1000/wilc_wlan_cfg.h
> b/drivers/staging/wilc1000/wilc_wlan_cfg.h
[]
> @@ -10,10 +10,10 @@
> #ifndef WILC_WLAN_CFG_H
> #define WILC_WLAN_CFG_H
>
> -typedef struct {
> +struct wilc_cfg_byte {
> u16 id;
> u16 val;
> -} wilc_cfg_byte_t;
> +};
val is a u16, calling it cfg_byte seems misleading.
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel