Re: [U-Boot] [PATCH v3 10/10] dm: i2c: tegra: Convert to driver model

Hi Masahiro,

On 3 December 2014 at 06:30, Masahiro Yamada yamada.m@jp.panasonic.com wrote:

Hi Simon,

Another comment for this patch.

On Mon, 24 Nov 2014 11:57:24 -0700 Simon Glass sjg@chromium.org wrote:

...

-/* Probe to see if a chip is present. */ -static int tegra_i2c_probe(struct i2c_adapter *adap, uchar chip) +static int tegra_i2c_set_offset_len(struct udevice *dev, const uint olen) {

• struct i2c_bus *bus;
  

• int rc;
  

• uchar reg;
  

• debug("i2c_probe: addr=0x%x\n", chip);
  

• bus = tegra_i2c_get_bus(adap);
  

• if (!bus)
  

•         return 1;
  

• reg = 0;
  

• rc = i2c_write_data(bus, chip, &reg, 1, false);
  

• if (rc) {
  

•         debug("Error probing 0x%x.\n", chip);
  

•         return 1;
  

• }
  

• return 0;
  

-}

• if (olen < 4)
  

•         return 0;
  

The maximum length 4 is not tegra-specific. I can't find good reason to have .set_offset_len handler.

It's not needed here as 4 is the maximum supported by the uclass. Some drivers may support less though, so perhaps we should keep the method?

Regards, Simon