From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Jacopo Mondi <jacopo+renesas@jmondi.org>
Cc: kieran.bingham+renesas@ideasonboard.com,
laurent.pinchart+renesas@ideasonboard.com,
niklas.soderlund+renesas@ragnatech.se, geert@linux-m68k.org,
robh@kernel.org, devicetree@vger.kernel.org,
linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
linux-kernel@vger.kernel.org, Hyun Kwon <hyunk@xilinx.com>,
Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
sergei.shtylyov@gmail.com
Subject: Re: [PATCH v6 2/5] dt-bindings: media: max9286: Document 'maxim,reverse-channel-microvolt'
Date: Wed, 16 Dec 2020 19:17:05 +0200 [thread overview]
Message-ID: <X9pBEe+da/8Y34Qv@pendragon.ideasonboard.com> (raw)
In-Reply-To: <X9o+XT3z1sVlh73x@pendragon.ideasonboard.com>
On Wed, Dec 16, 2020 at 07:05:34PM +0200, Laurent Pinchart wrote:
> Hi Jacopo,
>
> Thank you for the patch.
>
> On Tue, Dec 15, 2020 at 06:09:54PM +0100, Jacopo Mondi wrote:
> > Document the 'reverse-channel-microvolt' vendor property in the
> > bindings document of the max9286 driver.
> >
> > The newly introduced property allows to specifying the initial
> > configuration of the GMSL reverse control channel to accommodate
> > remote serializers pre-programmed with the high threshold power
> > supply noise immunity enabled.
> >
> > Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
> > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> > ---
> > v5->v6:
> > - Use standard unit suffix 'microvolt' for the custom property
> > - Drop '$ref' as according to 'example-schema.yaml':
> > "Vendor specific properties having a standard unit suffix don't need a type."
> > ---
> > .../bindings/media/i2c/maxim,max9286.yaml | 23 +++++++++++++++++++
> > 1 file changed, 23 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> > index 9ea827092fdd..b22ba3e0db4a 100644
> > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> > @@ -51,6 +51,26 @@ properties:
> > '#gpio-cells':
> > const: 2
> >
> > + maxim,reverse-channel-microvolt:
> > + minimum: 30000
> > + maximum: 200000
> > + default: 170000
> > + description: |
> > + Initial amplitude of the reverse control channel, in micro volts.
> > +
> > + The initial amplitude shall be adjusted to a value compatible with the
> > + configuration of the connected remote serializer.
> > +
> > + Some camera modules (for example RDACM20) include an on-board MCU that
> > + pre-programs the embedded serializer with power supply noise immunity
> > + (high-threshold) enabled. A typical value of the deserializer's reverse
> > + channel amplitude to communicate with pre-programmed serializers is
> > + 170000 micro volts.
> > +
> > + A typical value for the reverse channel amplitude to communicate with
> > + a remote serializer whose high-threshold noise immunity is not enabled
> > + is 100000 micro volts
> > +
> > ports:
> > type: object
> > description: |
> > @@ -221,6 +241,7 @@ required:
> > - ports
> > - i2c-mux
> > - gpio-controller
> > + - maxim,reverse-channel-microvolt
One comment though: You specify a default value above, which isn't very
useful when the property is required. Should we either drop the default
value, or make the property optional ?
> >
> > additionalProperties: false
> >
> > @@ -243,6 +264,8 @@ examples:
> > gpio-controller;
> > #gpio-cells = <2>;
> >
> > + maxim,reverse-channel-microvolt = <170000>;
> > +
> > ports {
> > #address-cells = <1>;
> > #size-cells = <0>;
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2020-12-16 17:17 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-15 17:09 [PATCH v6 0/5] media: i2c: Add RDACM21 camera module Jacopo Mondi
2020-12-15 17:09 ` [PATCH v6 1/5] media: i2c: Add driver for " Jacopo Mondi
2020-12-16 17:00 ` Laurent Pinchart
2020-12-15 17:09 ` [PATCH v6 2/5] dt-bindings: media: max9286: Document 'maxim,reverse-channel-microvolt' Jacopo Mondi
2020-12-16 17:05 ` Laurent Pinchart
2020-12-16 17:17 ` Laurent Pinchart [this message]
2020-12-21 18:58 ` Rob Herring
2020-12-22 8:53 ` Jacopo Mondi
2020-12-15 17:09 ` [PATCH v6 3/5] media: i2c: max9286: Break-out reverse channel setup Jacopo Mondi
2020-12-16 17:06 ` Laurent Pinchart
2020-12-15 17:09 ` [PATCH v6 4/5] media: i2c: max9286: Make channel amplitude programmable Jacopo Mondi
2020-12-16 17:14 ` Laurent Pinchart
2020-12-16 17:14 ` Laurent Pinchart
2020-12-15 17:09 ` [PATCH v6 5/5] media: i2c: max9286: Configure reverse channel amplitude Jacopo Mondi
2020-12-16 17:22 ` Laurent Pinchart
2021-01-11 10:43 ` Jacopo Mondi
2021-01-11 10:58 ` Laurent Pinchart
2021-01-11 11:20 ` Jacopo Mondi
2021-01-12 5:03 ` Laurent Pinchart
2021-01-12 9:08 ` Jacopo Mondi
2021-01-12 9:10 ` Geert Uytterhoeven
2021-01-12 10:00 ` Jacopo Mondi
2021-01-14 5:53 ` Laurent Pinchart
2021-01-14 8:09 ` Jacopo Mondi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=X9pBEe+da/8Y34Qv@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=devicetree@vger.kernel.org \
--cc=geert@linux-m68k.org \
--cc=hyunk@xilinx.com \
--cc=jacopo+renesas@jmondi.org \
--cc=kieran.bingham+renesas@ideasonboard.com \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=niklas.soderlund+renesas@ragnatech.se \
--cc=robh@kernel.org \
--cc=sergei.shtylyov@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).