Received-SPF: pass (google.com: domain of linux-kernel+bounces-117721-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Mon, 25 Mar 2024 19:35:46 +0100
From: Greg KH <gregkh@linuxfoundation.org>
To: Ayush Tiwari <ayushtiw0110@gmail.com>
Cc: Larry.Finger@lwfinger.net, florian.c.schilhabel@googlemail.com,
	linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev,
	outreachy@lists.linux.dev
Subject: Re: [PATCH] staging: rtl8712: Fix instances of lines ending with '('
Message-ID: <2024032546-earthworm-ardently-ed12@gregkh>
References: <ZgG6UC1Fiac7pDfa@ayush-HP-Pavilion-Gaming-Laptop-15-ec0xxx>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ZgG6UC1Fiac7pDfa@ayush-HP-Pavilion-Gaming-Laptop-15-ec0xxx>

On Mon, Mar 25, 2024 at 11:24:24PM +0530, Ayush Tiwari wrote:
> Fix instances of lines ending with '(' to address checkpatch warning
> "lines should not add with a '(' " and conform to common kernel coding
> style.
> 
> Signed-off-by: Ayush Tiwari <ayushtiw0110@gmail.com>
> ---
>  drivers/staging/rtl8712/rtl871x_ioctl_rtl.h | 123 +++++++-------------
>  1 file changed, 41 insertions(+), 82 deletions(-)
> 
> diff --git a/drivers/staging/rtl8712/rtl871x_ioctl_rtl.h b/drivers/staging/rtl8712/rtl871x_ioctl_rtl.h
> index 7c0b880ac686..70d48e6b4112 100644
> --- a/drivers/staging/rtl8712/rtl871x_ioctl_rtl.h
> +++ b/drivers/staging/rtl8712/rtl871x_ioctl_rtl.h
> @@ -18,92 +18,51 @@
>  #include "drv_types.h"
>  
>  /*************** oid_rtl_seg_01_01 **************/
> -uint oid_rt_get_signal_quality_hdl(
> -			struct oid_par_priv *poid_par_priv);/*84*/
> -uint oid_rt_get_small_packet_crc_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_middle_packet_crc_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_large_packet_crc_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_tx_retry_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_rx_retry_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_rx_total_packet_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_tx_beacon_ok_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_tx_beacon_err_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_rx_icv_err_hdl(
> -			struct oid_par_priv *poid_par_priv);/*93*/
> -uint oid_rt_set_encryption_algorithm_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_preamble_mode_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_ap_ip_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_channelplan_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_set_channelplan_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_set_preamble_mode_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_set_bcn_intvl_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_dedicate_probe_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_total_tx_bytes_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_total_rx_bytes_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_current_tx_power_level_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_enc_key_mismatch_count_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_enc_key_match_count_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_channel_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_hardware_radio_off_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_key_mismatch_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_supported_wireless_mode_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_channel_list_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_scan_in_progress_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_forced_data_rate_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_wireless_mode_for_scan_list_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_get_bss_wireless_mode_hdl(
> -			struct oid_par_priv *poid_par_priv);
> -uint oid_rt_scan_with_magic_packet_hdl(
> -			struct oid_par_priv *poid_par_priv);
> +uint oid_rt_get_signal_quality_hdl(struct oid_par_priv *poid_par_priv); /*84*/
> +uint oid_rt_get_small_packet_crc_hdl(struct oid_par_priv *poid_par_priv);

<snip>

Why are you keeping these functions that no one calls around at all?

Why not remove the ones that are not needed, and then, after cleaning
that all up, then worry about the few that are left to get into proper
shape here.

I would bet most of these aren't needed at all, right?

thanks,

greg k-h