Received-SPF: pass (google.com: domain of linux-kernel+bounces-67615-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <f8f19cf1-6b83-4fee-92a0-b9934a0a6c64@gmail.com>
Date: Thu, 15 Feb 2024 20:53:12 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2] Staging: rtl8192e: rtl819x_HTProc: Removed braces from
 single statement block in ht_on_assoc_rsp
To: Aaron Parfitt <aaronparfitt123@gmail.com>, gregkh@linuxfoundation.org
Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
References: <20240215194045.37205-3-aaronparfitt123@gmail.com>
Content-Language: en-US
From: Philipp Hortmann <philipp.g.hortmann@gmail.com>
In-Reply-To: <20240215194045.37205-3-aaronparfitt123@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

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@gmail.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)