Re: [PATCH 03/12] cx231xx: delete i2c_client per bus

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewed-by: Antti Palosaari <crope@xxxxxx>

Good catch, certainly pointless to add "dummy" I2C client per adapter. Just wastes memory.

Missing patch description.

regards
Antti

On 09/25/2014 08:07 AM, Matthias Schwarzott wrote:
Signed-off-by: Matthias Schwarzott <zzam@xxxxxxxxxx>
---
  drivers/media/usb/cx231xx/cx231xx-i2c.c | 7 -------
  drivers/media/usb/cx231xx/cx231xx.h     | 1 -
  2 files changed, 8 deletions(-)

diff --git a/drivers/media/usb/cx231xx/cx231xx-i2c.c b/drivers/media/usb/cx231xx/cx231xx-i2c.c
index 67a1391..a30d400 100644
--- a/drivers/media/usb/cx231xx/cx231xx-i2c.c
+++ b/drivers/media/usb/cx231xx/cx231xx-i2c.c
@@ -455,10 +455,6 @@ static struct i2c_adapter cx231xx_adap_template = {
  	.algo = &cx231xx_algo,
  };

-static struct i2c_client cx231xx_client_template = {
-	.name = "cx231xx internal",
-};
-
  /* ----------------------------------------------------------- */

  /*
@@ -514,7 +510,6 @@ int cx231xx_i2c_register(struct cx231xx_i2c *bus)
  	BUG_ON(!dev->cx231xx_send_usb_command);

  	bus->i2c_adap = cx231xx_adap_template;
-	bus->i2c_client = cx231xx_client_template;
  	bus->i2c_adap.dev.parent = &dev->udev->dev;

  	strlcpy(bus->i2c_adap.name, bus->dev->name, sizeof(bus->i2c_adap.name));
@@ -523,8 +518,6 @@ int cx231xx_i2c_register(struct cx231xx_i2c *bus)
  	i2c_set_adapdata(&bus->i2c_adap, &dev->v4l2_dev);
  	i2c_add_adapter(&bus->i2c_adap);

-	bus->i2c_client.adapter = &bus->i2c_adap;
-
  	if (0 == bus->i2c_rc) {
  		if (i2c_scan)
  			cx231xx_do_i2c_scan(dev, bus->nr);
diff --git a/drivers/media/usb/cx231xx/cx231xx.h b/drivers/media/usb/cx231xx/cx231xx.h
index 5efc93e..c92382f 100644
--- a/drivers/media/usb/cx231xx/cx231xx.h
+++ b/drivers/media/usb/cx231xx/cx231xx.h
@@ -472,7 +472,6 @@ struct cx231xx_i2c {

  	/* i2c i/o */
  	struct i2c_adapter i2c_adap;
-	struct i2c_client i2c_client;
  	u32 i2c_rc;

  	/* different settings for each bus */


--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux