* [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).