Messages in this thread | | | Date | Tue, 16 Mar 2021 00:15:16 +0200 | From | Laurent Pinchart <> | Subject | Re: [PATCH v2 1/4] dt-bindings: media: max9286: Describe gpio-hog |
| |
Hi Jacopo,
Thank you for the patch.
On Mon, Mar 15, 2021 at 05:30:25PM +0100, Jacopo Mondi wrote: > The MAX9286 GMSL deserializer features gpio controller capabilities, > as it provides 2 GPIO lines. > > As establishing a regulator that uses one of the GPIO lines and > enabling/disabling it at run-time in the max9286 won't work due to > a circular dependency on the gpio-controller/regulator creation, allow > the usage of a gpio-hog for that purpose. > > The usage of the gpio-hog is required in designs where the MAX9286 > GPIO lines control the remote cameras power. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org>
That's really a workaround until we can find a good solution, do we have to officially support it in the DT bindings ?
> --- > .../bindings/media/i2c/maxim,max9286.yaml | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > index ee16102fdfe7..9038300e373c 100644 > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml > @@ -177,6 +177,22 @@ properties: > > additionalProperties: false > > +patternProperties: > + "^(hog-[0-9]+|.+-hog(-[0-9]+)?)$": > + type: object > + properties: > + gpio-hog: true > + gpios: true > + output-low: true > + line-name: true > + > + required: > + - gpio-hog > + - gpios > + - output-low > + > + additionalProperties: false > + > required: > - compatible > - reg
-- Regards,
Laurent Pinchart
| |