Hello Hans, > -----Original Message----- > From: Hans Verkuil <hverkuil@xxxxxxxxx> > Sent: Wednesday, September 27, 2023 10:24 PM > To: ishikawa yuji(石川 悠司 ○RDC□AITC○EA開) > <yuji2.ishikawa@xxxxxxxxxxxxx>; Laurent Pinchart > <laurent.pinchart@xxxxxxxxxxxxxxxx>; Mauro Carvalho Chehab > <mchehab@xxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Krzysztof > Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>; Conor Dooley > <conor+dt@xxxxxxxxxx>; iwamatsu nobuhiro(岩松 信洋 ○DITC□DIT○ > OST) <nobuhiro1.iwamatsu@xxxxxxxxxxxxx> > Cc: linux-media@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v8 0/5] Add Toshiba Visconti Video Input Interface driver > > Hi Yuji Ishikawa, > > On 26/09/2023 01:28, Yuji Ishikawa wrote: > > This series is the Video Input Interface driver for Toshiba's ARM SoC, > > Visconti. > > This provides DT binding documentation, device driver, documentation > > and MAINTAINER files. > > > > A visconti VIIF driver instance exposes > > 1 media control device file and 3 video device files for a VIIF > > hardware. > > Detailed HW/SW are described in documentation directory. > > The VIIF hardware has CSI2 receiver, > > image signal processor and DMAC inside. > > The subdevice for image signal processor provides vendor specific V4L2 > > controls. > > > > The device driver depends on two other drivers under development; > > clock framework driver and IOMMU driver. > > Corresponding features will be added later. > > Building this series results in a set of sparse warnings: I set up the latest sparse tool and reproduced the warnings. > drivers/media/platform/toshiba/visconti/viif_capture.c:235:29: warning: symbol > 'out_format_spec' was not declared. Should it be static? I'll set static qualifier. > drivers/media/platform/toshiba/visconti/viif_capture.c:517:13: warning: > context imbalance in 'viif_regbuf_access_start' - wrong count at exit > drivers/media/platform/toshiba/visconti/viif_capture.c:523:13: warning: > context imbalance in 'viif_regbuf_access_end' - unexpected unlock I'll expand these inline functions. The same sequences are already expanded in viif_controls.c > drivers/media/platform/toshiba/visconti/viif_csi2rx.c:458:15: warning: Using > plain integer as NULL pointer > drivers/media/platform/toshiba/visconti/viif_csi2rx.c:458:15: warning: Using > plain integer as NULL pointer I'll fix them. > And I also get a Documentation build warning: > > Documentation/output/videodev2.h.rst:6: WARNING: undefined label: > 'v4l2-ctrl-type-visconti-isp' It seems the label is generated with a script (kerneldoc->ReST). I also found that the other members of enum v4l2_ctrl_type have labels pointing at legal targets. I'll try further to get a legal label. > Also note that you need to rebase: the v4l2-controls.h header changed. It seems that media-staging tree was updated after I fixed my patchset. I should have re-check before submission. I'll rebase to the latest HEAD and format v9 patch. > Regards, > > Hans > Regards, Yuji