Hi, On 3/9/23 07:10, Jiapeng Chong wrote: > ./drivers/platform/x86/acerhdf.c:343:2-3: Unneeded semicolon. > > Reported-by: Abaci Robot <abaci@xxxxxxxxxxxxxxxxx> > Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=2271 > Signed-off-by: Jiapeng Chong <jiapeng.chong@xxxxxxxxxxxxxxxxx> Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/x86/acerhdf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c > index a48638ad2a8a..ec8cc780b822 100644 > --- a/drivers/platform/x86/acerhdf.c > +++ b/drivers/platform/x86/acerhdf.c > @@ -341,7 +341,7 @@ static void acerhdf_check_param(struct thermal_zone_device *thermal) > pr_err("fanoff temperature (%d) is above fanon temperature (%d), clamping to %d\n", > fanoff, fanon, fanon); > fanoff = fanon; > - }; > + } > > trips[0].temperature = fanon; > trips[0].hysteresis = fanon - fanoff;