* [PATCH v2] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry
@ 2024-04-12 14:19 Luca Weiss
2024-06-03 6:39 ` Luca Weiss
0 siblings, 1 reply; 3+ messages in thread
From: Luca Weiss @ 2024-04-12 14:19 UTC (permalink / raw
To: Bjorn Andersson, Konrad Dybcio, Stanimir Varbanov, Vikash Garodia,
Bryan O'Donoghue, Mauro Carvalho Chehab, Conor Dooley,
Rob Herring, Krzysztof Kozlowski
Cc: ~postmarketos/upstreaming, phone-devel, Stanimir Varbanov,
linux-arm-msm, linux-media, devicetree, linux-kernel,
Krzysztof Kozlowski, Luca Weiss
Some SC7280-based boards crash when providing the "secure_non_pixel"
context bank, so allow only one iommu in the bindings also.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
Reference:
https://lore.kernel.org/linux-arm-msm/20231201-sc7280-venus-pas-v3-2-bc132dc5fc30@fairphone.com/
---
Changes in v2:
- Pick up tags
- Otherwise just a resend, v1 was sent in January
- Link to v1: https://lore.kernel.org/r/20240129-sc7280-venus-bindings-v1-1-20a9ba194c60@fairphone.com
---
Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
index 8f9b6433aeb8..10c334e6b3dc 100644
--- a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
+++ b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
@@ -43,6 +43,7 @@ properties:
- const: vcodec_bus
iommus:
+ minItems: 1
maxItems: 2
interconnects:
---
base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
change-id: 20240129-sc7280-venus-bindings-6e62a99620de
Best regards,
--
Luca Weiss <luca.weiss@fairphone.com>
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry
2024-04-12 14:19 [PATCH v2] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry Luca Weiss
@ 2024-06-03 6:39 ` Luca Weiss
2024-06-17 7:28 ` Luca Weiss
0 siblings, 1 reply; 3+ messages in thread
From: Luca Weiss @ 2024-06-03 6:39 UTC (permalink / raw
To: Luca Weiss, Bjorn Andersson, Konrad Dybcio, Stanimir Varbanov,
Vikash Garodia, Bryan O'Donoghue, Mauro Carvalho Chehab,
Conor Dooley, Rob Herring, Krzysztof Kozlowski
Cc: ~postmarketos/upstreaming, phone-devel, Stanimir Varbanov,
linux-arm-msm, linux-media, devicetree, linux-kernel,
Krzysztof Kozlowski
On Fri Apr 12, 2024 at 4:19 PM CEST, Luca Weiss wrote:
> Some SC7280-based boards crash when providing the "secure_non_pixel"
> context bank, so allow only one iommu in the bindings also.
Hi all,
This patch is still pending and not having it causes dt validation
warnings for some qcom-sc7280 boards.
Regards
Luca
>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
> Reference:
> https://lore.kernel.org/linux-arm-msm/20231201-sc7280-venus-pas-v3-2-bc132dc5fc30@fairphone.com/
> ---
> Changes in v2:
> - Pick up tags
> - Otherwise just a resend, v1 was sent in January
> - Link to v1: https://lore.kernel.org/r/20240129-sc7280-venus-bindings-v1-1-20a9ba194c60@fairphone.com
> ---
> Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> index 8f9b6433aeb8..10c334e6b3dc 100644
> --- a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> +++ b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> @@ -43,6 +43,7 @@ properties:
> - const: vcodec_bus
>
> iommus:
> + minItems: 1
> maxItems: 2
>
> interconnects:
>
> ---
> base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> change-id: 20240129-sc7280-venus-bindings-6e62a99620de
>
> Best regards,
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry
2024-06-03 6:39 ` Luca Weiss
@ 2024-06-17 7:28 ` Luca Weiss
0 siblings, 0 replies; 3+ messages in thread
From: Luca Weiss @ 2024-06-17 7:28 UTC (permalink / raw
To: Luca Weiss, Bjorn Andersson, Konrad Dybcio, Stanimir Varbanov,
Vikash Garodia, Bryan O'Donoghue, Mauro Carvalho Chehab,
Conor Dooley, Rob Herring, Krzysztof Kozlowski
Cc: ~postmarketos/upstreaming, phone-devel, linux-arm-msm,
linux-media, devicetree, linux-kernel, Krzysztof Kozlowski
On Mon Jun 3, 2024 at 8:39 AM CEST, Luca Weiss wrote:
> On Fri Apr 12, 2024 at 4:19 PM CEST, Luca Weiss wrote:
> > Some SC7280-based boards crash when providing the "secure_non_pixel"
> > context bank, so allow only one iommu in the bindings also.
>
> Hi all,
>
> This patch is still pending and not having it causes dt validation
> warnings for some qcom-sc7280 boards.
Hi Rob,
Could you please pick up this patch? Mauro seems to ignore this patch
either on purpose or by accident and I'd like for this dtbs_check
failure to finally be fixed.
Regards
Luca
>
> Regards
> Luca
>
> >
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> > ---
> > Reference:
> > https://lore.kernel.org/linux-arm-msm/20231201-sc7280-venus-pas-v3-2-bc132dc5fc30@fairphone.com/
> > ---
> > Changes in v2:
> > - Pick up tags
> > - Otherwise just a resend, v1 was sent in January
> > - Link to v1: https://lore.kernel.org/r/20240129-sc7280-venus-bindings-v1-1-20a9ba194c60@fairphone.com
> > ---
> > Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> > index 8f9b6433aeb8..10c334e6b3dc 100644
> > --- a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> > +++ b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> > @@ -43,6 +43,7 @@ properties:
> > - const: vcodec_bus
> >
> > iommus:
> > + minItems: 1
> > maxItems: 2
> >
> > interconnects:
> >
> > ---
> > base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> > change-id: 20240129-sc7280-venus-bindings-6e62a99620de
> >
> > Best regards,
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-06-17 7:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-12 14:19 [PATCH v2] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry Luca Weiss
2024-06-03 6:39 ` Luca Weiss
2024-06-17 7:28 ` Luca Weiss
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.