* [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction @ 2023-12-13 7:17 Julien Massot 2023-12-13 16:30 ` Conor Dooley 0 siblings, 1 reply; 5+ messages in thread From: Julien Massot @ 2023-12-13 7:17 UTC (permalink / raw) To: benjamin.mugnier, sylvain.petinot, mchehab, robh+dt, krzysztof.kozlowski+dt, conor+dt Cc: kernel, devicetree, linux-media, Julien Massot The ST VGXY61 sensors support multiple lane number, as well as lane mapping. Signed-off-by: Julien Massot <julien.massot@collabora.com> --- .../devicetree/bindings/media/i2c/st,st-vgxy61.yaml | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml index 8c28848b226a..733fac85a20f 100644 --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml +++ b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml @@ -67,11 +67,8 @@ properties: data-lanes: description: CSI lanes to use - items: - - const: 1 - - const: 2 - - const: 3 - - const: 4 + minItems: 1 + maxItems: 4 remote-endpoint: true -- 2.43.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction 2023-12-13 7:17 [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction Julien Massot @ 2023-12-13 16:30 ` Conor Dooley 2024-01-05 9:36 ` Julien Massot 0 siblings, 1 reply; 5+ messages in thread From: Conor Dooley @ 2023-12-13 16:30 UTC (permalink / raw) To: Julien Massot Cc: benjamin.mugnier, sylvain.petinot, mchehab, robh+dt, krzysztof.kozlowski+dt, conor+dt, kernel, devicetree, linux-media [-- Attachment #1: Type: text/plain, Size: 1057 bytes --] On Wed, Dec 13, 2023 at 08:17:37AM +0100, Julien Massot wrote: > The ST VGXY61 sensors support multiple lane number, as > well as lane mapping. > > Signed-off-by: Julien Massot <julien.massot@collabora.com> > --- > .../devicetree/bindings/media/i2c/st,st-vgxy61.yaml | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml > index 8c28848b226a..733fac85a20f 100644 > --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml > @@ -67,11 +67,8 @@ properties: > data-lanes: > description: > CSI lanes to use > - items: > - - const: 1 > - - const: 2 > - - const: 3 > - - const: 4 > + minItems: 1 > + maxItems: 4 So, it is now valid to have "data-lanes = <6 7 8>;" now? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction 2023-12-13 16:30 ` Conor Dooley @ 2024-01-05 9:36 ` Julien Massot 2024-02-09 9:24 ` Benjamin Mugnier 0 siblings, 1 reply; 5+ messages in thread From: Julien Massot @ 2024-01-05 9:36 UTC (permalink / raw) To: Conor Dooley Cc: benjamin.mugnier, sylvain.petinot, mchehab, robh+dt, krzysztof.kozlowski+dt, conor+dt, kernel, devicetree, linux-media Hi, On 12/13/23 17:30, Conor Dooley wrote: > On Wed, Dec 13, 2023 at 08:17:37AM +0100, Julien Massot wrote: >> The ST VGXY61 sensors support multiple lane number, as >> well as lane mapping. >> >> Signed-off-by: Julien Massot <julien.massot@collabora.com> >> --- >> .../devicetree/bindings/media/i2c/st,st-vgxy61.yaml | 7 ++----- >> 1 file changed, 2 insertions(+), 5 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >> index 8c28848b226a..733fac85a20f 100644 >> --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >> +++ b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >> @@ -67,11 +67,8 @@ properties: >> data-lanes: >> description: >> CSI lanes to use >> - items: >> - - const: 1 >> - - const: 2 >> - - const: 3 >> - - const: 4 >> + minItems: 1 >> + maxItems: 4 > > So, it is now valid to have "data-lanes = <6 7 8>;" now? Indeed it's not valid, I will send a v2 with a 'minimum: 1', 'maximum: 4' items properties. Thanks, -- Julien ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction 2024-01-05 9:36 ` Julien Massot @ 2024-02-09 9:24 ` Benjamin Mugnier 2024-03-15 9:06 ` Julien Massot 0 siblings, 1 reply; 5+ messages in thread From: Benjamin Mugnier @ 2024-02-09 9:24 UTC (permalink / raw) To: Julien Massot, Conor Dooley Cc: sylvain.petinot, mchehab, robh+dt, krzysztof.kozlowski+dt, conor+dt, kernel, devicetree, linux-media Hi Julien, On 1/5/24 10:36, Julien Massot wrote: > Hi, > > On 12/13/23 17:30, Conor Dooley wrote: >> On Wed, Dec 13, 2023 at 08:17:37AM +0100, Julien Massot wrote: >>> The ST VGXY61 sensors support multiple lane number, as >>> well as lane mapping. >>> >>> Signed-off-by: Julien Massot <julien.massot@collabora.com> >>> --- >>> .../devicetree/bindings/media/i2c/st,st-vgxy61.yaml | 7 ++----- >>> 1 file changed, 2 insertions(+), 5 deletions(-) >>> >>> diff --git >>> a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>> b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>> index 8c28848b226a..733fac85a20f 100644 >>> --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>> +++ b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>> @@ -67,11 +67,8 @@ properties: >>> data-lanes: >>> description: >>> CSI lanes to use >>> - items: >>> - - const: 1 >>> - - const: 2 >>> - - const: 3 >>> - - const: 4 >>> + minItems: 1 >>> + maxItems: 4 >> >> So, it is now valid to have "data-lanes = <6 7 8>;" now? > > Indeed it's not valid, I will send a v2 with a 'minimum: 1', 'maximum: > 4' items properties. I'm interested in this patch. Did you send a v2 yet that I might have missed ? Thank you. > > Thanks, -- Regards, Benjamin ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction 2024-02-09 9:24 ` Benjamin Mugnier @ 2024-03-15 9:06 ` Julien Massot 0 siblings, 0 replies; 5+ messages in thread From: Julien Massot @ 2024-03-15 9:06 UTC (permalink / raw) To: Benjamin Mugnier, Conor Dooley Cc: sylvain.petinot, mchehab, robh+dt, krzysztof.kozlowski+dt, conor+dt, kernel, devicetree, linux-media Hi Benjamin, On 2/9/24 10:24, Benjamin Mugnier wrote: > Hi Julien, > > On 1/5/24 10:36, Julien Massot wrote: >> Hi, >> >> On 12/13/23 17:30, Conor Dooley wrote: >>> On Wed, Dec 13, 2023 at 08:17:37AM +0100, Julien Massot wrote: >>>> The ST VGXY61 sensors support multiple lane number, as >>>> well as lane mapping. >>>> >>>> Signed-off-by: Julien Massot <julien.massot@collabora.com> >>>> --- >>>> .../devicetree/bindings/media/i2c/st,st-vgxy61.yaml | 7 ++----- >>>> 1 file changed, 2 insertions(+), 5 deletions(-) >>>> >>>> diff --git >>>> a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>>> b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>>> index 8c28848b226a..733fac85a20f 100644 >>>> --- a/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>>> +++ b/Documentation/devicetree/bindings/media/i2c/st,st-vgxy61.yaml >>>> @@ -67,11 +67,8 @@ properties: >>>> data-lanes: >>>> description: >>>> CSI lanes to use >>>> - items: >>>> - - const: 1 >>>> - - const: 2 >>>> - - const: 3 >>>> - - const: 4 >>>> + minItems: 1 >>>> + maxItems: 4 >>> >>> So, it is now valid to have "data-lanes = <6 7 8>;" now? >> >> Indeed it's not valid, I will send a v2 with a 'minimum: 1', 'maximum: >> 4' items properties. > > I'm interested in this patch. Did you send a v2 yet that I might have > missed ? > Thank you. I just sent a v2, which add minimum/maximum so that we can only use index 1..4. Regards, Julien ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-03-15 9:07 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-12-13 7:17 [PATCH] media: dt-bindings: st-vgxy61: relax data-lanes restriction Julien Massot 2023-12-13 16:30 ` Conor Dooley 2024-01-05 9:36 ` Julien Massot 2024-02-09 9:24 ` Benjamin Mugnier 2024-03-15 9:06 ` Julien Massot
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).