Re: [PATCH] staging: rtl8723bs: xmit: fix grammar in comment

[weibu@redadmin.org]

Thanks, Greg.

Sorry for the off-list reply; resending to the lists.
You're right about the extra space; I dropped it. To avoid the "number 
of"
confusion I rephrased the sentence to "Total page count is not 
available, so
update current FIFO status." Sent as v2.

Best,
Akiyoshi

2025-09-17 14:52 に Greg KH さんは書きました:
> On Wed, Sep 17, 2025 at 02:39:44PM +0900, Akiyoshi Kurita wrote:
>> Fix 'number of page' -> 'number of pages' in a comment.
>> 
>> Signed-off-by: Akiyoshi Kurita <weibu@redadmin.org>
>> ---
>>  drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c 
>> b/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
>> index 842e19b53421..7ba8981f4652 100644
>> --- a/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
>> +++ b/drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
>> @@ -76,7 +76,7 @@ static s32 rtl8723_dequeue_writeport(struct adapter 
>> *padapter)
>>  	/*  check if hardware tx fifo page is enough */
>>  	if (!rtw_hal_sdio_query_tx_freepage(pri_padapter, PageIdx, 
>> pxmitbuf->pg_num)) {
>>  		if (!bUpdatePageNum) {
>> -			/*  Total number of page is NOT available, so update current FIFO 
>> status */
>> +			/*  Total number of pages is NOT available, so update current FIFO 
>> status */
> 
> As "pages" is now plural, shouldn't "is" become "are"?
> 
> Also, you can drop the extra leading space in the comment.
> 
> thanks,
> 
> greg "english grammer is rough" k-h