[PATCH RESEND v8 20/21] media: i2c: remove MAX96717 driver

Dumitru Ceclan via B4 Relay posted 21 patches 2 months ago
Only 20 patches received!
[PATCH RESEND v8 20/21] media: i2c: remove MAX96717 driver
Posted by Dumitru Ceclan via B4 Relay 2 months ago
From: Cosmin Tanislav <demonsingur@gmail.com>

The previous MAX96717 driver has been removed and its functionality has
been moved to a MAX96717 driver which makes use of the Maxim GMSL2/3
serializer framework.

Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com>
---
 MAINTAINERS                |  1 -
 drivers/media/i2c/Kconfig  | 16 ----------------
 drivers/media/i2c/Makefile |  1 -
 3 files changed, 18 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 6a6a04ce2787..ad7e28897062 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15235,7 +15235,6 @@ M:	Julien Massot <julien.massot@collabora.com>
 L:	linux-media@vger.kernel.org
 S:	Maintained
 F:	Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
-F:	drivers/media/i2c/max96717.c
 
 MAX9860 MONO AUDIO VOICE CODEC DRIVER
 M:	Peter Rosin <peda@axentia.se>
diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
index 37f86e6de969..4d3a861ee2f0 100644
--- a/drivers/media/i2c/Kconfig
+++ b/drivers/media/i2c/Kconfig
@@ -1701,22 +1701,6 @@ config VIDEO_MAX96714
 	  To compile this driver as a module, choose M here: the
 	  module will be called max96714.
 
-config VIDEO_MAX96717
-	tristate "Maxim MAX96717 GMSL2 Serializer support"
-	depends on I2C && VIDEO_DEV && COMMON_CLK
-	select I2C_MUX
-	select MEDIA_CONTROLLER
-	select GPIOLIB
-	select V4L2_CCI_I2C
-	select V4L2_FWNODE
-	select VIDEO_V4L2_SUBDEV_API
-	help
-	  Device driver for the Maxim MAX96717 GMSL2 Serializer.
-	  MAX96717 serializers convert video on a MIPI CSI-2
-	  input to a GMSL2 output.
-
-	  To compile this driver as a module, choose M here: the
-	  module will be called max96717.
 
 source "drivers/media/i2c/maxim-serdes/Kconfig"
 
diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile
index be3200b23819..96bc49339341 100644
--- a/drivers/media/i2c/Makefile
+++ b/drivers/media/i2c/Makefile
@@ -70,7 +70,6 @@ obj-$(CONFIG_VIDEO_M52790) += m52790.o
 obj-$(CONFIG_VIDEO_MAX9271_LIB) += max9271.o
 obj-$(CONFIG_VIDEO_MAX9286) += max9286.o
 obj-$(CONFIG_VIDEO_MAX96714) += max96714.o
-obj-$(CONFIG_VIDEO_MAX96717) += max96717.o
 obj-$(CONFIG_VIDEO_MAXIM_SERDES) += maxim-serdes/
 obj-$(CONFIG_VIDEO_ML86V7667) += ml86v7667.o
 obj-$(CONFIG_VIDEO_MSP3400) += msp3400.o

-- 
2.43.0
Re: [PATCH RESEND v8 20/21] media: i2c: remove MAX96717 driver
Posted by Sakari Ailus 2 weeks, 2 days ago
Hi Cosmin, Dumitru,

On Mon, Dec 08, 2025 at 04:13:12PM +0200, Dumitru Ceclan via B4 Relay wrote:
> From: Cosmin Tanislav <demonsingur@gmail.com>
> 
> The previous MAX96717 driver has been removed and its functionality has
> been moved to a MAX96717 driver which makes use of the Maxim GMSL2/3
> serializer framework.

Please use imperative form when describing what the patch does.

> 
> Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com>
> ---
>  MAINTAINERS                |  1 -
>  drivers/media/i2c/Kconfig  | 16 ----------------
>  drivers/media/i2c/Makefile |  1 -

This patch doesn't actually remove the driver, same for the next one.

It'd be good to have comments (acks hopefully!) from the authors (and
users) of the existing drivers on how does the new driver work for them.

There's also no need to resend patchsets to get more attention, please just
ping instead.

-- 
Kind regards,

Sakari Ailus