Re: [PATCH 2/4] Staging: gdm724x: hci: Changed camel-case to snake-case.

From: Greg KH
Date: Fri Feb 23 2018 - 03:15:31 EST


On Thu, Feb 22, 2018 at 05:32:59PM -0800, Quytelda Kahja wrote:
> Changed a variable name from camel to snake case to fix a coding style
> issue.
>
> Signed-off-by: Quytelda Kahja <quytelda@xxxxxxxxxxx>
> ---
> drivers/staging/gdm724x/hci_packet.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/gdm724x/hci_packet.h b/drivers/staging/gdm724x/hci_packet.h
> index 22ce8b9477b6..f8837c7103ac 100644
> --- a/drivers/staging/gdm724x/hci_packet.h
> +++ b/drivers/staging/gdm724x/hci_packet.h
> @@ -50,7 +50,7 @@ struct tlv {
> struct sdu_header {
> __dev16 cmd_evt;
> __dev16 len;
> - __dev32 dftEpsId;
> + __dev32 dft_eps_ID;
> __dev32 bearer_ID;
> __dev32 nic_type;
> } __packed;

Why not just delete the whole structure if no one is using it?

thanks,

greg k-h