mbox series

[0/5] media: i2c: max9286: Add configuration properties

Message ID 20200316202757.529740-1-jacopo+renesas@jmondi.org
Headers show
Series media: i2c: max9286: Add configuration properties | expand

Message

Jacopo Mondi March 16, 2020, 8:27 p.m. UTC
Hello,
   this small series applies on top of Kieran's
[PATCH v7 0/2] MAX9286 GMSL support
and instruments the driver and expands its dt bindings to support
configuring a few deserializer parameters to enable support for different
remote serializers.

In particular for our use case RDACM20 and RDACM21 camera modules require
different configurations of the deserializer reverse channel amplitude and
disabling of the device overlap window.

Hyun has reported he needs to disable the deserializer overlap window to
have this driver work with max96705 serializer.

The series expands the device bindings to require two additional properties
to control overlap window and channel amplitude, and instruments the driver
to parse those properties and use them to configure it parameter.

What 'overlap window' controls is not totally clear at the moment and it's
poorly documented. In all our cases it could stay disabled, but as long as
its precise meaning it's not clarified, a required property ensures that
all current DTB are fully specified, so that in future we select a default
value, old users continue to work as intended/

The reverse channel amplitude describes the initially programmed signal
amplitude of the low bandwidth control channel. The amplitude is made
configurable to accommodate different serializers that might need an initial
different amplitude configuration to establish reliable communications.

On top of the series, a small fixup to be brought in next max9286 version.

Tested on R-Car R8A77970 Eagle with RDACM21.
I would have liked to test the same on Salvator-X with RDACM20. Kieran could
we sync and test it?

RDACM21 and RDACM20 drivers available at
git://jmondi.org/linux #gmsl/jmondi/platform/rdacm21
with integration in Eagle and Salvator-x from Kieran's platform branch, with
additional properties added by this series.


Jacopo Mondi (5):
  media: i2c: max9286: Put of node on error
  dt-bindings: media: max9286: Add overlap window
  media: i2c: max9286: Parse overlap window value
  dt-bindings: media: max9286: Add reverse channel amplitude
  media: i2c: max9286: Parse channel amplitude

 .../bindings/media/i2c/maxim,max9286.yaml     | 32 +++++++++
 MAINTAINERS                                   |  1 +
 drivers/media/i2c/max9286.c                   | 66 +++++++++++++++++--
 include/dt-bindings/media/maxim-gmsl.h        |  9 +++
 4 files changed, 103 insertions(+), 5 deletions(-)
 create mode 100644 include/dt-bindings/media/maxim-gmsl.h

--
2.25.1