Re: [RFC 00/12] TC358743 async subdev and dt support

2015-05-04 Thread Mats Randgaard (matrandg)

Hi Philipp,
Thank you for reviewing, testing and committing new patches to the 
driver for TC358743!


Fixed in latest version of the driver:

  * [media] tc358743: split set_csi into set_csi and start_csi
  * [media] tc358743: also set TCLK_TRAILCNT and TCLK_POSTCNT
  * [media] tc358743: don't return E2BIG from G_EDID

The code seems fine to me, but I am not able to test it:

  * [media] tc358743: add direct interrupt handling
  * [media] tc358743: allow event subscription

The rest of the patches are commented inline.

Regards,
Mats Randgaard



On 03/30/2015 01:10 PM, Philipp Zabel wrote:

Hi Mats,

>
> did you have time to work on the TC358743 driver some more in the
> meanwhile? These are the changes I have made locally to v1 to get it
> to work on i.MX6.
>
> regards Philipp
>
> Mats Randgaard (1): [media] Driver for Toshiba TC358743 CSI-2 to
> HDMI bridge
>
> Philipp Zabel (11): [media] tc358743: register v4l2 asynchronous
> subdevice [media] tc358743: support probe from device tree [media]
> tc358743: fix set_pll to enable PLL with default frequency [media]
> tc358743: fix lane number calculation to include blanking [media]
> tc358743: split set_csi into set_csi and start_csi [media] tc358743:
> also set TCLK_TRAILCNT and TCLK_POSTCNT [media] tc358743: parse MIPI
> CSI-2 endpoint, support noncontinuous clock [media] tc358743: add
> direct interrupt handling [media] tc358743: detect chip by ChipID
> instead of IntMask [media] tc358743: don't return E2BIG from G_EDID
> [media] tc358743: allow event subscription
>
> MAINTAINERS|6 +
> drivers/media/i2c/Kconfig |   12 + drivers/media/i2c/Makefile
> |1 + drivers/media/i2c/tc358743.c   | 1979
> 
> drivers/media/i2c/tc358743_regs.h  |  670 
> include/media/tc358743.h   |   89 ++
> include/uapi/linux/v4l2-controls.h |4 + 7 files changed, 2761
> insertions(+) create mode 100644 drivers/media/i2c/tc358743.c create
> mode 100644 drivers/media/i2c/tc358743_regs.h create mode 100644
> include/media/tc358743.h
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC 00/12] TC358743 async subdev and dt support

2015-04-08 Thread Mats Randgaard (matrandg)

Hi Philipp,
Hans forwarded this patch series to me now, since I haven't received if 
for some reason. I will prioritize upstreaming of the driver the next 
couple of days and create a new RFC. I will also go through your patches 
and give you feedback!


Regards,
Mats


On 03/30/2015 01:10 PM, Philipp Zabel wrote:

Hi Mats,

did you have time to work on the TC358743 driver some more in the meanwhile?
These are the changes I have made locally to v1 to get it to work on i.MX6.

regards
Philipp

Mats Randgaard (1):
   [media] Driver for Toshiba TC358743 CSI-2 to HDMI bridge

Philipp Zabel (11):
   [media] tc358743: register v4l2 asynchronous subdevice
   [media] tc358743: support probe from device tree
   [media] tc358743: fix set_pll to enable PLL with default frequency
   [media] tc358743: fix lane number calculation to include blanking
   [media] tc358743: split set_csi into set_csi and start_csi
   [media] tc358743: also set TCLK_TRAILCNT and TCLK_POSTCNT
   [media] tc358743: parse MIPI CSI-2 endpoint, support noncontinuous
 clock
   [media] tc358743: add direct interrupt handling
   [media] tc358743: detect chip by ChipID instead of IntMask
   [media] tc358743: don't return E2BIG from G_EDID
   [media] tc358743: allow event subscription

  MAINTAINERS|6 +
  drivers/media/i2c/Kconfig  |   12 +
  drivers/media/i2c/Makefile |1 +
  drivers/media/i2c/tc358743.c   | 1979 
  drivers/media/i2c/tc358743_regs.h  |  670 
  include/media/tc358743.h   |   89 ++
  include/uapi/linux/v4l2-controls.h |4 +
  7 files changed, 2761 insertions(+)
  create mode 100644 drivers/media/i2c/tc358743.c
  create mode 100644 drivers/media/i2c/tc358743_regs.h
  create mode 100644 include/media/tc358743.h



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


[RFC 00/12] TC358743 async subdev and dt support

2015-03-30 Thread Philipp Zabel
Hi Mats,

did you have time to work on the TC358743 driver some more in the meanwhile?
These are the changes I have made locally to v1 to get it to work on i.MX6.

regards
Philipp

Mats Randgaard (1):
  [media] Driver for Toshiba TC358743 CSI-2 to HDMI bridge

Philipp Zabel (11):
  [media] tc358743: register v4l2 asynchronous subdevice
  [media] tc358743: support probe from device tree
  [media] tc358743: fix set_pll to enable PLL with default frequency
  [media] tc358743: fix lane number calculation to include blanking
  [media] tc358743: split set_csi into set_csi and start_csi
  [media] tc358743: also set TCLK_TRAILCNT and TCLK_POSTCNT
  [media] tc358743: parse MIPI CSI-2 endpoint, support noncontinuous
clock
  [media] tc358743: add direct interrupt handling
  [media] tc358743: detect chip by ChipID instead of IntMask
  [media] tc358743: don't return E2BIG from G_EDID
  [media] tc358743: allow event subscription

 MAINTAINERS|6 +
 drivers/media/i2c/Kconfig  |   12 +
 drivers/media/i2c/Makefile |1 +
 drivers/media/i2c/tc358743.c   | 1979 
 drivers/media/i2c/tc358743_regs.h  |  670 
 include/media/tc358743.h   |   89 ++
 include/uapi/linux/v4l2-controls.h |4 +
 7 files changed, 2761 insertions(+)
 create mode 100644 drivers/media/i2c/tc358743.c
 create mode 100644 drivers/media/i2c/tc358743_regs.h
 create mode 100644 include/media/tc358743.h

-- 
2.1.4

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