Hi Yang Li, Please repost with a proper subject line prefix: "media: rzg2l-cru:". The 'media:' prefix suggests a media-wide patch, not a patch for a single driver. Ditto for your other patch for sun6i_isp.c. Why is this so hard to do? Regards, Hans On 29/11/2022 02:04, Yang Li wrote: > ./drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c:409:2-3: Unneeded semicolon > ./drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c:407:2-3: Unneeded semicolon > > Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=3273 > Reported-by: Abaci Robot <abaci@xxxxxxxxxxxxxxxxx> > Signed-off-by: Yang Li <yang.lee@xxxxxxxxxxxxxxxxx> > --- > drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c | 2 +- > drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c > index aa752b80574c..5b80feaad927 100644 > --- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c > +++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c > @@ -406,7 +406,7 @@ static void rzg2l_csi2_mipi_link_disable(struct rzg2l_csi2 *csi2) > if (!(rzg2l_csi2_read(csi2, CSI2nRTST) & CSI2nRTST_VSRSTS)) > break; > usleep_range(100, 200); > - }; > + } > > if (!timeout) > dev_err(csi2->dev, "Clearing CSI2nRTST.VSRSTS timed out\n"); > diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > index 9533e4069ecd..cec4d20963a8 100644 > --- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > +++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-video.c > @@ -404,7 +404,7 @@ void rzg2l_cru_stop_image_processing(struct rzg2l_cru_dev *cru) > break; > > usleep_range(10, 20); > - }; > + } > > /* Notify that AXI bus can not stop here */ > if (!retries)