lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f8f19cf1-6b83-4fee-92a0-b9934a0a6c64@gmail.com>
Date: Thu, 15 Feb 2024 20:53:12 +0100
From: Philipp Hortmann <philipp.g.hortmann@...il.com>
To: Aaron Parfitt <aaronparfitt123@...il.com>, gregkh@...uxfoundation.org
Cc: linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] Staging: rtl8192e: rtl819x_HTProc: Removed braces from
 single statement block in ht_on_assoc_rsp

Hi Aaron,

the changelog needs to be below the three minus "---".

Look for examples: https://lore.kernel.org/linux-staging/

Please omit the changelog line with my name... that is not relevant...

Is the length of the subject line within the expected range?

Thanks for your support.

Bye Philipp


On 15.02.24 20:40, Aaron Parfitt wrote:
> Remove braces from single statement blocks in the rtl8192e staging
> driver to clear checkpatch.pl warnings, improving code readability
> and adhering to coding style guidelines.
>
> Changelog since v1:
> - Updated the subject line to make it more unique and descriptive.
> - Refined the patch description for clarity and conciseness.
> - Incorporated feedback from Philipp to improve code style compliance.
>
> Signed-off-by: Aaron Parfitt <aaronparfitt123@...il.com>
> ---
>   drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/rtl8192e/rtl819x_HTProc.c b/drivers/staging/rtl8192e/rtl819x_HTProc.c
> index 6d0912f90198..49b882c363bf 100644
> --- a/drivers/staging/rtl8192e/rtl819x_HTProc.c
> +++ b/drivers/staging/rtl8192e/rtl819x_HTProc.c
> @@ -480,9 +480,9 @@ void ht_on_assoc_rsp(struct rtllib_device *ieee)
>   	}
>   
>   	ht_info->current_mpdu_density = pPeerHTCap->MPDUDensity;
> -	if (ht_info->iot_action & HT_IOT_ACT_TX_USE_AMSDU_8K) {
> +	if (ht_info->iot_action & HT_IOT_ACT_TX_USE_AMSDU_8K)
>   		ht_info->current_ampdu_enable = false;
> -	}
> +
>   	ht_info->cur_rx_reorder_enable = 1;
>   
>   	if (pPeerHTCap->MCS[0] == 0)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ