Message ID | 20231012071329.2542003-6-yuji2.ishikawa@toshiba.co.jp (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Hans Verkuil |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1qqq0R-00DJI8-5F; Thu, 12 Oct 2023 07:21:16 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347098AbjJLHVM (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Thu, 12 Oct 2023 03:21:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49518 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343609AbjJLHVF (ORCPT <rfc822;linux-media@vger.kernel.org>); Thu, 12 Oct 2023 03:21:05 -0400 Received: from mo-csw.securemx.jp (mo-csw1801.securemx.jp [210.130.202.135]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 054A2BA; Thu, 12 Oct 2023 00:21:02 -0700 (PDT) Received: by mo-csw.securemx.jp (mx-mo-csw1801) id 39C7KNoR494726; Thu, 12 Oct 2023 16:20:24 +0900 X-Iguazu-Qid: 2yAbIh4tauXUF3Sx6o X-Iguazu-QSIG: v=2; s=0; t=1697095223; q=2yAbIh4tauXUF3Sx6o; m=67wGgEapaJ/fwJyDfXsshyrFu7oVJ3EkD1jwBCxdIq8= Received: from imx12-a.toshiba.co.jp ([38.106.60.135]) by relay.securemx.jp (mx-mr1801) id 39C7KKJL1872873 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT); Thu, 12 Oct 2023 16:20:21 +0900 X-SA-MID: 8623576 From: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp> To: Hans Verkuil <hverkuil@xs4all.nl>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>, Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp> Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: [PATCH v9 5/5] MAINTAINERS: Add entries for Toshiba Visconti Video Input Interface Date: Thu, 12 Oct 2023 16:13:29 +0900 X-TSB-HOP2: ON Message-Id: <20231012071329.2542003-6-yuji2.ishikawa@toshiba.co.jp> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20231012071329.2542003-1-yuji2.ishikawa@toshiba.co.jp> References: <20231012071329.2542003-1-yuji2.ishikawa@toshiba.co.jp> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_LOW, SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -4.7 (----) X-LSpam-Report: No, score=-4.7 required=5.0 tests=BAYES_00=-1.9,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,RCVD_IN_DNSWL_MED=-2.3 autolearn=ham autolearn_force=no |
Series |
Add Toshiba Visconti Video Input Interface driver
|
|
Commit Message
Yuji Ishikawa
Oct. 12, 2023, 7:13 a.m. UTC
Added entries for visconti Video Input Interface driver, including;
* device tree bindings
* source files
* documentation files
Signed-off-by: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp>
---
Changelog v2:
- no change
Changelog v3:
- added entry for driver API documentation
Changelog v4:
- added entry for header file
Changelog v5:
- no change
Changelog v6:
- update path to VIIF driver source files
Changelog v7:
- no change
Changelog v8:
- rename bindings description file
Changelog v9:
- no change
MAINTAINERS | 4 ++++
1 file changed, 4 insertions(+)
Comments
On 12/10/2023 09:13, Yuji Ishikawa wrote: > Added entries for visconti Video Input Interface driver, including; > * device tree bindings > * source files > * documentation files > > Signed-off-by: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp> > --- > Changelog v2: > - no change > > Changelog v3: > - added entry for driver API documentation > > Changelog v4: > - added entry for header file > > Changelog v5: > - no change > > Changelog v6: > - update path to VIIF driver source files > > Changelog v7: > - no change > > Changelog v8: > - rename bindings description file > > Changelog v9: > - no change > > MAINTAINERS | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index eaa1a28a9d54..6035f711b3fb 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -2902,17 +2902,21 @@ F: Documentation/devicetree/bindings/arm/toshiba.yaml > F: Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pipllct.yaml > F: Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pismu.yaml > F: Documentation/devicetree/bindings/gpio/toshiba,gpio-visconti.yaml > +F: Documentation/devicetree/bindings/media/toshiba,visconti5-viif.yaml > F: Documentation/devicetree/bindings/net/toshiba,visconti-dwmac.yaml > F: Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml > F: Documentation/devicetree/bindings/pinctrl/toshiba,visconti-pinctrl.yaml > F: Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml > +F: Documentation/driver-api/media/drivers/visconti-viif.rst > F: arch/arm64/boot/dts/toshiba/ > F: drivers/clk/visconti/ > F: drivers/gpio/gpio-visconti.c > +F: drivers/media/platform/toshiba/visconti/ > F: drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c > F: drivers/pci/controller/dwc/pcie-visconti.c > F: drivers/pinctrl/visconti/ > F: drivers/watchdog/visconti_wdt.c > +F: include/uapi/linux/visconti_viif.h > N: visconti This is a bit odd: it adds the driver to an existing entry, but you do not add yourself as maintainer. Typically a large driver like this one will have its own entry. At minimum I need an Acked-by from Nobuhiro Iwamatsu for this change. But I think it might be better to make a separate entry for this driver. Up to you, though. Regards, Hans > > ARM/UNIPHIER ARCHITECTURE
Hello Hans, Thank you for the review. > -----Original Message----- > From: Hans Verkuil <hverkuil@xs4all.nl> > Sent: Tuesday, November 14, 2023 5:41 PM > To: ishikawa yuji(石川 悠司 ○RDC□AITC○EA開) > <yuji2.ishikawa@toshiba.co.jp>; Laurent Pinchart > <laurent.pinchart@ideasonboard.com>; Mauro Carvalho Chehab > <mchehab@kernel.org>; Rob Herring <robh+dt@kernel.org>; Krzysztof > Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley > <conor+dt@kernel.org>; iwamatsu nobuhiro(岩松 信洋 ○DITC□DIT○ > OST) <nobuhiro1.iwamatsu@toshiba.co.jp> > Cc: linux-media@vger.kernel.org; devicetree@vger.kernel.org; > linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH v9 5/5] MAINTAINERS: Add entries for Toshiba Visconti > Video Input Interface > > On 12/10/2023 09:13, Yuji Ishikawa wrote: > > Added entries for visconti Video Input Interface driver, including; > > * device tree bindings > > * source files > > * documentation files > > > > Signed-off-by: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp> > > --- > > Changelog v2: > > - no change > > > > Changelog v3: > > - added entry for driver API documentation > > > > Changelog v4: > > - added entry for header file > > > > Changelog v5: > > - no change > > > > Changelog v6: > > - update path to VIIF driver source files > > > > Changelog v7: > > - no change > > > > Changelog v8: > > - rename bindings description file > > > > Changelog v9: > > - no change > > > > MAINTAINERS | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS index > > eaa1a28a9d54..6035f711b3fb 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -2902,17 +2902,21 @@ F: > Documentation/devicetree/bindings/arm/toshiba.yaml > > F: > Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pipllct.y > aml > > F: > Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pismu.y > aml > > F: Documentation/devicetree/bindings/gpio/toshiba,gpio-visconti.yaml > > +F: > Documentation/devicetree/bindings/media/toshiba,visconti5-viif.yam > l > > F: > Documentation/devicetree/bindings/net/toshiba,visconti-dwmac.ya > ml > > F: Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml > > F: > Documentation/devicetree/bindings/pinctrl/toshiba,visconti-pinctrl.y > aml > > F: > Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt. > yaml > > +F: Documentation/driver-api/media/drivers/visconti-viif.rst > > F: arch/arm64/boot/dts/toshiba/ > > F: drivers/clk/visconti/ > > F: drivers/gpio/gpio-visconti.c > > +F: drivers/media/platform/toshiba/visconti/ > > F: drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c > > F: drivers/pci/controller/dwc/pcie-visconti.c > > F: drivers/pinctrl/visconti/ > > F: drivers/watchdog/visconti_wdt.c > > +F: include/uapi/linux/visconti_viif.h > > N: visconti > > This is a bit odd: it adds the driver to an existing entry, but you do not add > yourself as maintainer. Typically a large driver like this one will have its own > entry. > > At minimum I need an Acked-by from Nobuhiro Iwamatsu for this change. But I > think it might be better to make a separate entry for this driver. Up to you, > though. I'll make a separate entry for Visconti VIIF driver. I talked it over with Iwamatsu-san and he agreed. I need Acked-by tag from him at v10 patch submission; is it a correct understanding? > > Regards, > > Hans > > > > > ARM/UNIPHIER ARCHITECTURE Regards, Yuji
On 27/11/2023 01:38, yuji2.ishikawa@toshiba.co.jp wrote: > Hello Hans, > Thank you for the review. > >> -----Original Message----- >> From: Hans Verkuil <hverkuil@xs4all.nl> >> Sent: Tuesday, November 14, 2023 5:41 PM >> To: ishikawa yuji(石川 悠司 ○RDC□AITC○EA開) >> <yuji2.ishikawa@toshiba.co.jp>; Laurent Pinchart >> <laurent.pinchart@ideasonboard.com>; Mauro Carvalho Chehab >> <mchehab@kernel.org>; Rob Herring <robh+dt@kernel.org>; Krzysztof >> Kozlowski <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley >> <conor+dt@kernel.org>; iwamatsu nobuhiro(岩松 信洋 ○DITC□DIT○ >> OST) <nobuhiro1.iwamatsu@toshiba.co.jp> >> Cc: linux-media@vger.kernel.org; devicetree@vger.kernel.org; >> linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org >> Subject: Re: [PATCH v9 5/5] MAINTAINERS: Add entries for Toshiba Visconti >> Video Input Interface >> >> On 12/10/2023 09:13, Yuji Ishikawa wrote: >>> Added entries for visconti Video Input Interface driver, including; >>> * device tree bindings >>> * source files >>> * documentation files >>> >>> Signed-off-by: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp> >>> --- >>> Changelog v2: >>> - no change >>> >>> Changelog v3: >>> - added entry for driver API documentation >>> >>> Changelog v4: >>> - added entry for header file >>> >>> Changelog v5: >>> - no change >>> >>> Changelog v6: >>> - update path to VIIF driver source files >>> >>> Changelog v7: >>> - no change >>> >>> Changelog v8: >>> - rename bindings description file >>> >>> Changelog v9: >>> - no change >>> >>> MAINTAINERS | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/MAINTAINERS b/MAINTAINERS index >>> eaa1a28a9d54..6035f711b3fb 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -2902,17 +2902,21 @@ F: >> Documentation/devicetree/bindings/arm/toshiba.yaml >>> F: >> Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pipllct.y >> aml >>> F: >> Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pismu.y >> aml >>> F: Documentation/devicetree/bindings/gpio/toshiba,gpio-visconti.yaml >>> +F: >> Documentation/devicetree/bindings/media/toshiba,visconti5-viif.yam >> l >>> F: >> Documentation/devicetree/bindings/net/toshiba,visconti-dwmac.ya >> ml >>> F: Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml >>> F: >> Documentation/devicetree/bindings/pinctrl/toshiba,visconti-pinctrl.y >> aml >>> F: >> Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt. >> yaml >>> +F: Documentation/driver-api/media/drivers/visconti-viif.rst >>> F: arch/arm64/boot/dts/toshiba/ >>> F: drivers/clk/visconti/ >>> F: drivers/gpio/gpio-visconti.c >>> +F: drivers/media/platform/toshiba/visconti/ >>> F: drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c >>> F: drivers/pci/controller/dwc/pcie-visconti.c >>> F: drivers/pinctrl/visconti/ >>> F: drivers/watchdog/visconti_wdt.c >>> +F: include/uapi/linux/visconti_viif.h >>> N: visconti >> >> This is a bit odd: it adds the driver to an existing entry, but you do not add >> yourself as maintainer. Typically a large driver like this one will have its own >> entry. >> >> At minimum I need an Acked-by from Nobuhiro Iwamatsu for this change. But I >> think it might be better to make a separate entry for this driver. Up to you, >> though. > > I'll make a separate entry for Visconti VIIF driver. > I talked it over with Iwamatsu-san and he agreed. > I need Acked-by tag from him at v10 patch submission; is it a correct understanding? Yes, that's correct. Regards, Hans > >> >> Regards, >> >> Hans >> >>> >>> ARM/UNIPHIER ARCHITECTURE > > Regards, > Yuji
diff --git a/MAINTAINERS b/MAINTAINERS index eaa1a28a9d54..6035f711b3fb 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2902,17 +2902,21 @@ F: Documentation/devicetree/bindings/arm/toshiba.yaml F: Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pipllct.yaml F: Documentation/devicetree/bindings/clock/toshiba,tmpv770x-pismu.yaml F: Documentation/devicetree/bindings/gpio/toshiba,gpio-visconti.yaml +F: Documentation/devicetree/bindings/media/toshiba,visconti5-viif.yaml F: Documentation/devicetree/bindings/net/toshiba,visconti-dwmac.yaml F: Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml F: Documentation/devicetree/bindings/pinctrl/toshiba,visconti-pinctrl.yaml F: Documentation/devicetree/bindings/watchdog/toshiba,visconti-wdt.yaml +F: Documentation/driver-api/media/drivers/visconti-viif.rst F: arch/arm64/boot/dts/toshiba/ F: drivers/clk/visconti/ F: drivers/gpio/gpio-visconti.c +F: drivers/media/platform/toshiba/visconti/ F: drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c F: drivers/pci/controller/dwc/pcie-visconti.c F: drivers/pinctrl/visconti/ F: drivers/watchdog/visconti_wdt.c +F: include/uapi/linux/visconti_viif.h N: visconti ARM/UNIPHIER ARCHITECTURE