I tested the patch on NXP iMX8MP platform and no issues found. Test-by: Guoniu.zhou <guoniu.zhou@xxxxxxx> > -----Original Message----- > From: G.N. Zhou > Sent: 2023年5月5日 16:03 > To: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > Cc: slongerbeam@xxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; > mchehab@xxxxxxxxxx > Subject: RE: [EXT] [PATCH 0/2] media: ov5640: drive-by frame_interval cleanups > > Sure, will update later. > > > -----Original Message----- > > From: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > > Sent: 2023年5月5日 15:16 > > To: G.N. Zhou <guoniu.zhou@xxxxxxx> > > Cc: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx>; > > slongerbeam@xxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; > mchehab@xxxxxxxxxx > > Subject: [EXT] [PATCH 0/2] media: ov5640: drive-by frame_interval > > cleanups > > > > Caution: This is an external email. Please take care when clicking > > links or opening attachments. When in doubt, report the message using the > 'Report this email' > > button > > > > > > While looking at Guoniu Zhou patches I noticed that there were a few > > cleanups related to the usage of frame_interval fileds for MIPI CSI-2 > > framerate calculations. > > > > No functional changes intended, just cleanups. > > > > Guoniu: could you please test these on your setup as well ? A > > tested-by tag would be useful! > > > > Thanks > > j > > > > Jacopo Mondi (2): > > media: ov5640: Remove unused 'framerate' parameter > > media: ov5640: Drop dead code using frame_interval > > > > drivers/media/i2c/ov5640.c | 17 +---------------- > > 1 file changed, 1 insertion(+), 16 deletions(-) > > > > -- > > 2.40.1
