Re: [PATCH v2] staging: rtl8723bs: fix constant on left side of test checkpatch warnings
From: Greg KH
Date: Mon Mar 30 2026 - 12:16:30 EST
On Sat, Mar 28, 2026 at 11:17:16PM +0530, Prithvi wrote:
> Hello all,
>
> Thank you very much for taking time to review this patch.
>
> I understand, that this patch may be considered to be majorly mechanical
> changes, but still, the scope of the patch as well as the changes
> involved make testing important for this patch.
>
> I agree that, in case of unavailability of hardware for testing, I should
> have mentioned it and used the RFT tag for the patch since v1. I
> apologize for missing this detail & understand that compile-time testing
> can't be sufficient here with all the changes introduced by this patch
> and also the concern of the possibility of regressions getting introduced.
>
> Going forward, I will be meticulous about clearly disclosing the testing
> status of the patch and if I am not able to test a patch, I will be sure
> to add RFT tag since v1 of the patch itself.
>
> Lastly, I wanted to kindly ask if it will be alright to send a v3 patch
> with the RFT tag, incorporating the changes discussed in this thread?
Yes please.
But remember to do only one logical thing per patch, I see multiple
things happening in this one :(
thanks,
greg k-h