mbox series

[0/4] i2c: tegra: Add support for the VI I2C on Tegra210

Message ID 20200506191511.2791107-1-thierry.reding@gmail.com
Headers show
Series i2c: tegra: Add support for the VI I2C on Tegra210 | expand

Message

Thierry Reding May 6, 2020, 7:15 p.m. UTC
From: Thierry Reding <treding@nvidia.com>

Hi Wolfram,

This small set of patches cleans up a few things in the I2C driver and
then adds support for the VI instance of the I2C controller which is a
regular I2C controller wrapped by some multimedia-specific logic. As a
result, the regular I2C controller register interface is offset by a
couple of bytes.

I can pick up patch 4/4 into the Tegra tree. These patches also don't
have any conflicts with Dmitry's patches that we've been discussing.

That said, I could also put this set of patches (except the DTS change)
into the same branch I already have for Dmitry's patches (plus the fixes
that we had discussed) and take it into linux-next and let it brew for a
couple of days to get more testing and then send everything to you with
a PR, say, sometime next week? How does that sound?

Thierry

Thierry Reding (4):
  dt-bindings: i2c: tegra: Document Tegra210 VI I2C
  i2c: tegra: Use FIELD_PREP/FIELD_GET macros
  i2c: tegra: Add support for the VI I2C on Tegra210
  arm64: tegra: Enable VI I2C on Jetson Nano

 .../bindings/i2c/nvidia,tegra20-i2c.txt       |   6 +
 .../boot/dts/nvidia/tegra210-p3450-0000.dts   |   4 +
 drivers/i2c/busses/i2c-tegra.c                | 178 +++++++++++++-----
 3 files changed, 145 insertions(+), 43 deletions(-)

Comments

Wolfram Sang May 12, 2020, 1:14 p.m. UTC | #1
Hi Thierry,

> That said, I could also put this set of patches (except the DTS change)
> into the same branch I already have for Dmitry's patches (plus the fixes
> that we had discussed) and take it into linux-next and let it brew for a
> couple of days to get more testing and then send everything to you with
> a PR, say, sometime next week? How does that sound?

Well, it is already next week when you get my answer... Yes, sure this
is fine with me. Makes dependency handling and testing easier.

Kind regards,

   Wolfram