mbox series

[v3,0/4] Add support for MAX96714F and MAX96717F GMSL2 ser/des

Message ID 20240111130349.2776699-1-julien.massot@collabora.com
Headers
Series Add support for MAX96714F and MAX96717F GMSL2 ser/des |

Message

Julien Massot Jan. 11, 2024, 1:03 p.m. UTC 
  Change since v2:
- Convert drivers to use CCI helpers
- Use generic node name
- Use 'powerdown' as gpio name instead of 'enable'
- Add pattern generator support for MAX96714

These patches add support for Maxim MAX96714F deserializer and
MAX96717F serializer.

MAX96714F has one GMSL2 input port and one CSI2 4 lanes output port,
MAX96717F has one CSI2 input port and one GMSL2 output port.

The drivers support the tunnel mode where all the
CSI2 traffic coming from an imager is replicated through the deserializer
output port.

Both MAX96714F and MAX96717F are limited to a 3Gbps forward link rate
leaving a maximum of 2.6Gbps for the video payload.

Julien Massot (4):
  dt-bindings: media: add Maxim MAX96717F GMSL2 Serializer
  dt-bindings: media: add Maxim MAX96714F GMSL2 Deserializer
  media: i2c: add MAX96717 driver
  media: i2c: add MAX96714 driver

 .../bindings/media/i2c/maxim,max96714f.yaml   |  171 +++
 .../bindings/media/i2c/maxim,max96717f.yaml   |  147 +++
 MAINTAINERS                                   |   14 +
 drivers/media/i2c/Kconfig                     |   26 +
 drivers/media/i2c/Makefile                    |    2 +
 drivers/media/i2c/max96714.c                  | 1077 +++++++++++++++++
 drivers/media/i2c/max96717.c                  |  956 +++++++++++++++
 7 files changed, 2393 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/maxim,max96714f.yaml
 create mode 100644 Documentation/devicetree/bindings/media/i2c/maxim,max96717f.yaml
 create mode 100644 drivers/media/i2c/max96714.c
 create mode 100644 drivers/media/i2c/max96717.c

Comments

Krzysztof Kozlowski Jan. 12, 2024, 8:10 a.m. UTC | #1 
On 11/01/2024 14:03, Julien Massot wrote:
> Change since v2:
> - Convert drivers to use CCI helpers
> - Use generic node name
> - Use 'powerdown' as gpio name instead of 'enable'
> - Add pattern generator support for MAX96714

It's incomplete changelog or I miss something. b4 diff shows me huge
amount of changes, including in examples, many properties, copyrights
and so on. It does not make it easier for review.

Best regards,
Krzysztof