Hello On Thu, May 11, 2023 at 09:14:11AM +0000, G.N. Zhou wrote: > I tested the patch on NXP iMX8MP platform and no issues found. > Thanks for testing > Test-by: Guoniu.zhou <guoniu.zhou@xxxxxxx> FYI the tag is meant to be Tested-by: Thanks for testing > > > -----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 >
