The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH 0/4] Dead structs in media/i2c
@ 2024-05-24 16:48 linux
  2024-05-24 16:48 ` [PATCH 1/4] media: i2c: dw9768: remove unused struct 'regval_list' linux
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: linux @ 2024-05-24 16:48 UTC (permalink / raw)
  To: mchehab, sakari.ailus, hverkuil-cisco
  Cc: linux-media, linux-kernel, Dr. David Alan Gilbert

From: "Dr. David Alan Gilbert" <linux@treblig.org>

Clean out a bunch of dead structs in media/i2c.
Build tested only.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>

Dr. David Alan Gilbert (4):
  media: i2c: dw9768: remove unused struct 'regval_list'
  media: i2c: ks0127: remove unused struct 'adjust'
  media: i2c: tw9910: remove unused strust 'regval_list'
  media: i2c: adv7511: remove unused struct 'i2c_reg_value'

 drivers/media/i2c/adv7511-v4l2.c | 5 -----
 drivers/media/i2c/dw9768.c       | 5 -----
 drivers/media/i2c/ks0127.c       | 8 --------
 drivers/media/i2c/tw9910.c       | 5 -----
 4 files changed, 23 deletions(-)

-- 
2.45.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 1/4] media: i2c: dw9768: remove unused struct 'regval_list'
  2024-05-24 16:48 [PATCH 0/4] Dead structs in media/i2c linux
@ 2024-05-24 16:48 ` linux
  2024-05-24 16:48 ` [PATCH 2/4] media: i2c: ks0127: remove unused struct 'adjust' linux
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: linux @ 2024-05-24 16:48 UTC (permalink / raw)
  To: mchehab, sakari.ailus, hverkuil-cisco
  Cc: linux-media, linux-kernel, Dr. David Alan Gilbert

From: "Dr. David Alan Gilbert" <linux@treblig.org>

'regval_list' is unused since the original
commit 859891228e56 ("media: i2c: dw9768: Add DW9768 VCM driver")

Remove it.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/media/i2c/dw9768.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/media/i2c/dw9768.c b/drivers/media/i2c/dw9768.c
index daabbece8c7e..18ef2b35c9aa 100644
--- a/drivers/media/i2c/dw9768.c
+++ b/drivers/media/i2c/dw9768.c
@@ -115,11 +115,6 @@ static inline struct dw9768 *sd_to_dw9768(struct v4l2_subdev *subdev)
 	return container_of(subdev, struct dw9768, sd);
 }
 
-struct regval_list {
-	u8 reg_num;
-	u8 value;
-};
-
 struct dw9768_aac_mode_ot_multi {
 	u32 aac_mode_enum;
 	u32 ot_multi_base100;
-- 
2.45.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/4] media: i2c: ks0127: remove unused struct 'adjust'
  2024-05-24 16:48 [PATCH 0/4] Dead structs in media/i2c linux
  2024-05-24 16:48 ` [PATCH 1/4] media: i2c: dw9768: remove unused struct 'regval_list' linux
@ 2024-05-24 16:48 ` linux
  2024-05-24 16:48 ` [PATCH 3/4] media: i2c: tw9910: remove unused strust 'regval_list' linux
  2024-05-24 16:48 ` [PATCH 4/4] media: i2c: adv7511: remove unused struct 'i2c_reg_value' linux
  3 siblings, 0 replies; 5+ messages in thread
From: linux @ 2024-05-24 16:48 UTC (permalink / raw)
  To: mchehab, sakari.ailus, hverkuil-cisco
  Cc: linux-media, linux-kernel, Dr. David Alan Gilbert

From: "Dr. David Alan Gilbert" <linux@treblig.org>

'adjust' has been unused since original
commit fbe60daac4c3 ("V4L/DVB (3916): AverMedia 6 Eyes AVS6EYES
support").

Remove it.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/media/i2c/ks0127.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/drivers/media/i2c/ks0127.c b/drivers/media/i2c/ks0127.c
index 5c583f57e3f3..9d0a763cd503 100644
--- a/drivers/media/i2c/ks0127.c
+++ b/drivers/media/i2c/ks0127.c
@@ -175,14 +175,6 @@ MODULE_LICENSE("GPL");
 * mga_dev : represents one ks0127 chip.
 ****************************************************************************/
 
-struct adjust {
-	int	contrast;
-	int	bright;
-	int	hue;
-	int	ugain;
-	int	vgain;
-};
-
 struct ks0127 {
 	struct v4l2_subdev sd;
 	v4l2_std_id	norm;
-- 
2.45.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 3/4] media: i2c: tw9910: remove unused strust 'regval_list'
  2024-05-24 16:48 [PATCH 0/4] Dead structs in media/i2c linux
  2024-05-24 16:48 ` [PATCH 1/4] media: i2c: dw9768: remove unused struct 'regval_list' linux
  2024-05-24 16:48 ` [PATCH 2/4] media: i2c: ks0127: remove unused struct 'adjust' linux
@ 2024-05-24 16:48 ` linux
  2024-05-24 16:48 ` [PATCH 4/4] media: i2c: adv7511: remove unused struct 'i2c_reg_value' linux
  3 siblings, 0 replies; 5+ messages in thread
From: linux @ 2024-05-24 16:48 UTC (permalink / raw)
  To: mchehab, sakari.ailus, hverkuil-cisco
  Cc: linux-media, linux-kernel, Dr. David Alan Gilbert

From: "Dr. David Alan Gilbert" <linux@treblig.org>

'regval_list' has been unused since
commit 398994c1e104 ("V4L/DVB (13666): tw9910: modify V/H outpit pin
setting to use VALID").

Remove it.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/media/i2c/tw9910.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/media/i2c/tw9910.c b/drivers/media/i2c/tw9910.c
index 905af98c7d53..6dffaaa9ed56 100644
--- a/drivers/media/i2c/tw9910.c
+++ b/drivers/media/i2c/tw9910.c
@@ -212,11 +212,6 @@
  * structure
  */
 
-struct regval_list {
-	unsigned char reg_num;
-	unsigned char value;
-};
-
 struct tw9910_scale_ctrl {
 	char           *name;
 	unsigned short  width;
-- 
2.45.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 4/4] media: i2c: adv7511: remove unused struct 'i2c_reg_value'
  2024-05-24 16:48 [PATCH 0/4] Dead structs in media/i2c linux
                   ` (2 preceding siblings ...)
  2024-05-24 16:48 ` [PATCH 3/4] media: i2c: tw9910: remove unused strust 'regval_list' linux
@ 2024-05-24 16:48 ` linux
  3 siblings, 0 replies; 5+ messages in thread
From: linux @ 2024-05-24 16:48 UTC (permalink / raw)
  To: mchehab, sakari.ailus, hverkuil-cisco
  Cc: linux-media, linux-kernel, Dr. David Alan Gilbert

From: "Dr. David Alan Gilbert" <linux@treblig.org>

'i2c_reg_value' is unused since the original
commit 5a544cce2177 ("[media] adv7511: add new video encoder").

Remove it.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/media/i2c/adv7511-v4l2.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/media/i2c/adv7511-v4l2.c b/drivers/media/i2c/adv7511-v4l2.c
index 79946e9c7401..261871be833f 100644
--- a/drivers/media/i2c/adv7511-v4l2.c
+++ b/drivers/media/i2c/adv7511-v4l2.c
@@ -62,11 +62,6 @@ MODULE_LICENSE("GPL v2");
 **********************************************************************
 */
 
-struct i2c_reg_value {
-	unsigned char reg;
-	unsigned char value;
-};
-
 struct adv7511_state_edid {
 	/* total number of blocks */
 	u32 blocks;
-- 
2.45.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-05-24 16:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-24 16:48 [PATCH 0/4] Dead structs in media/i2c linux
2024-05-24 16:48 ` [PATCH 1/4] media: i2c: dw9768: remove unused struct 'regval_list' linux
2024-05-24 16:48 ` [PATCH 2/4] media: i2c: ks0127: remove unused struct 'adjust' linux
2024-05-24 16:48 ` [PATCH 3/4] media: i2c: tw9910: remove unused strust 'regval_list' linux
2024-05-24 16:48 ` [PATCH 4/4] media: i2c: adv7511: remove unused struct 'i2c_reg_value' linux

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox