| Message ID | 20240610150815.228790-2-benjamin.mugnier@foss.st.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | media: vgxy61: Remove vendor prefix from driver name | expand |
On 10/06/2024 17:08, Benjamin Mugnier wrote: > From 'st-vgxy61' to 'vgxy61'. > Align with other drivers to not use the vendor prefix. > Also the vendor prefix is already mentioned in the device tree > compatible string, being 'st,vgxy61', and does not need to be expressed twice. What bindings have anything to do with driver name? I think I made it clear last time. > > Signed-off-by: Benjamin Mugnier <benjamin.mugnier@foss.st.com> > --- > .../media/i2c/{st,st-vgxy61.yaml => st,vgxy61.yaml} | 6 +++--- > Documentation/userspace-api/media/drivers/index.rst | 2 +- > .../media/drivers/{st-vgxy61.rst => vgxy61.rst} | 0 > MAINTAINERS | 8 ++++---- > drivers/media/i2c/Kconfig | 2 +- > drivers/media/i2c/Makefile | 2 +- > drivers/media/i2c/{st-vgxy61.c => vgxy61.c} | 2 +- > 7 files changed, 11 insertions(+), 11 deletions(-) > rename Documentation/devicetree/bindings/media/i2c/{st,st-vgxy61.yaml => st,vgxy61.yaml} (95%) > rename Documentation/userspace-api/media/drivers/{st-vgxy61.rst => vgxy61.rst} (100%) > rename drivers/media/i2c/{st-vgxy61.c => vgxy61.c} (99%) > > diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml b/Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml > similarity index 95% > rename from Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml > rename to Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml > index 8c28848b226a..4e4c2c7ad168 100644 > --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml > @@ -2,7 +2,7 @@ > # Copyright (c) 2022 STMicroelectronics SA. > %YAML 1.2 > --- > -$id: http://devicetree.org/schemas/media/i2c/st,st-vgxy61.yaml# > +$id: http://devicetree.org/schemas/media/i2c/st,vgxy61.yaml# > $schema: http://devicetree.org/meta-schemas/core.yaml# > > title: STMicroelectronics VGxy61 HDR Global Shutter Sensor Family > @@ -23,7 +23,7 @@ description: |- > > properties: > compatible: > - const: st,st-vgxy61 > + const: st,vgxy61 Why? No. NAK. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml b/Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml similarity index 95% rename from Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml rename to Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml index 8c28848b226a..4e4c2c7ad168 100644 --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml +++ b/Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml @@ -2,7 +2,7 @@ # Copyright (c) 2022 STMicroelectronics SA. %YAML 1.2 --- -$id: http://devicetree.org/schemas/media/i2c/st,st-vgxy61.yaml# +$id: http://devicetree.org/schemas/media/i2c/st,vgxy61.yaml# $schema: http://devicetree.org/meta-schemas/core.yaml# title: STMicroelectronics VGxy61 HDR Global Shutter Sensor Family @@ -23,7 +23,7 @@ description: |- properties: compatible: - const: st,st-vgxy61 + const: st,vgxy61 reg: maxItems: 1 @@ -95,7 +95,7 @@ examples: #address-cells = <1>; #size-cells = <0>; vgxy61: csi2tx@10 { - compatible = "st,st-vgxy61"; + compatible = "st,vgxy61"; reg = <0x10>; clocks = <&clk_ext_camera>; VCORE-supply = <&v1v2>; diff --git a/Documentation/userspace-api/media/drivers/index.rst b/Documentation/userspace-api/media/drivers/index.rst index 2252063593bf..d706cb47b112 100644 --- a/Documentation/userspace-api/media/drivers/index.rst +++ b/Documentation/userspace-api/media/drivers/index.rst @@ -35,6 +35,6 @@ For more details see the file COPYING in the source distribution of Linux. max2175 npcm-video omap3isp-uapi - st-vgxy61 thp7312 uvcvideo + vgxy61 diff --git a/Documentation/userspace-api/media/drivers/st-vgxy61.rst b/Documentation/userspace-api/media/drivers/vgxy61.rst similarity index 100% rename from Documentation/userspace-api/media/drivers/st-vgxy61.rst rename to Documentation/userspace-api/media/drivers/vgxy61.rst diff --git a/MAINTAINERS b/MAINTAINERS index ef6be9d95143..3838f2ad77ca 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -20891,9 +20891,9 @@ M: Sylvain Petinot <sylvain.petinot@foss.st.com> L: linux-media@vger.kernel.org S: Maintained T: git git://linuxtv.org/media_tree.git -F: Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml -F: Documentation/userspace-api/media/drivers/st-vgxy61.rst -F: drivers/media/i2c/st-vgxy61.c +F: Documentation/devicetree/bindings/media/i2c/st,vgxy61.yaml +F: Documentation/userspace-api/media/drivers/vgxy61.rst +F: drivers/media/i2c/vgxy61.c ST VL53L0X ToF RANGER(I2C) IIO DRIVER M: Song Qiang <songqiang1304521@gmail.com> @@ -23210,7 +23210,7 @@ F: drivers/media/i2c/mt* F: drivers/media/i2c/og* F: drivers/media/i2c/ov* F: drivers/media/i2c/s5* -F: drivers/media/i2c/st-vgxy61.c +F: drivers/media/i2c/vgxy61.c VF610 NAND DRIVER M: Stefan Agner <stefan@agner.ch> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig index c6d3ee472d81..25619d5e29c4 100644 --- a/drivers/media/i2c/Kconfig +++ b/drivers/media/i2c/Kconfig @@ -659,7 +659,7 @@ config VIDEO_S5K6A3 This is a V4L2 sensor driver for Samsung S5K6A3 raw camera sensor. -config VIDEO_ST_VGXY61 +config VIDEO_VGXY61 tristate "ST VGXY61 sensor support" select V4L2_CCI_I2C depends on OF && GPIOLIB diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile index dfbe6448b549..d322ba161da5 100644 --- a/drivers/media/i2c/Makefile +++ b/drivers/media/i2c/Makefile @@ -124,7 +124,6 @@ obj-$(CONFIG_VIDEO_SAA717X) += saa717x.o obj-$(CONFIG_VIDEO_SAA7185) += saa7185.o obj-$(CONFIG_VIDEO_SONY_BTF_MPX) += sony-btf-mpx.o obj-$(CONFIG_VIDEO_ST_MIPID02) += st-mipid02.o -obj-$(CONFIG_VIDEO_ST_VGXY61) += st-vgxy61.o obj-$(CONFIG_VIDEO_TC358743) += tc358743.o obj-$(CONFIG_VIDEO_TC358746) += tc358746.o obj-$(CONFIG_VIDEO_TDA1997X) += tda1997x.o @@ -148,6 +147,7 @@ obj-$(CONFIG_VIDEO_TW9910) += tw9910.o obj-$(CONFIG_VIDEO_UDA1342) += uda1342.o obj-$(CONFIG_VIDEO_UPD64031A) += upd64031a.o obj-$(CONFIG_VIDEO_UPD64083) += upd64083.o +obj-$(CONFIG_VIDEO_VGXY61) += vgxy61.o obj-$(CONFIG_VIDEO_VP27SMPX) += vp27smpx.o obj-$(CONFIG_VIDEO_VPX3220) += vpx3220.o obj-$(CONFIG_VIDEO_WM8739) += wm8739.o diff --git a/drivers/media/i2c/st-vgxy61.c b/drivers/media/i2c/vgxy61.c similarity index 99% rename from drivers/media/i2c/st-vgxy61.c rename to drivers/media/i2c/vgxy61.c index b9e7c57027b1..30378e962016 100644 --- a/drivers/media/i2c/st-vgxy61.c +++ b/drivers/media/i2c/vgxy61.c @@ -1878,7 +1878,7 @@ static const struct dev_pm_ops vgxy61_pm_ops = { static struct i2c_driver vgxy61_i2c_driver = { .driver = { - .name = "st-vgxy61", + .name = "vgxy61", .of_match_table = vgxy61_dt_ids, .pm = &vgxy61_pm_ops, },
From 'st-vgxy61' to 'vgxy61'. Align with other drivers to not use the vendor prefix. Also the vendor prefix is already mentioned in the device tree compatible string, being 'st,vgxy61', and does not need to be expressed twice. Signed-off-by: Benjamin Mugnier <benjamin.mugnier@foss.st.com> --- .../media/i2c/{st,st-vgxy61.yaml => st,vgxy61.yaml} | 6 +++--- Documentation/userspace-api/media/drivers/index.rst | 2 +- .../media/drivers/{st-vgxy61.rst => vgxy61.rst} | 0 MAINTAINERS | 8 ++++---- drivers/media/i2c/Kconfig | 2 +- drivers/media/i2c/Makefile | 2 +- drivers/media/i2c/{st-vgxy61.c => vgxy61.c} | 2 +- 7 files changed, 11 insertions(+), 11 deletions(-) rename Documentation/devicetree/bindings/media/i2c/{st,st-vgxy61.yaml => st,vgxy61.yaml} (95%) rename Documentation/userspace-api/media/drivers/{st-vgxy61.rst => vgxy61.rst} (100%) rename drivers/media/i2c/{st-vgxy61.c => vgxy61.c} (99%)