devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: display: mediatek: dpi: Add power domain for MT8195 DP_INTF
@ 2024-09-11  7:17 Rohit Agarwal
  2024-09-16 17:11 ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Rohit Agarwal @ 2024-09-11  7:17 UTC (permalink / raw)
  To: chunkuang.hu, krzk+dt, ck.hu
  Cc: linux-mediatek, devicetree, linux-kernel, linux-arm-kernel,
	Rohit Agarwal, kernel test robot

Add power domain binding for MT8195 DP_INTF that resolves the following
error and many more similar ones:

arch/arm64/boot/dts/mediatek/mt8195-cherry-dojo-r1.dtb: dp-intf@1c113000:
power-domains: False schema does not allow [[55, 18]]

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202409102119.AYvaTjUi-lkp@intel.com/
Signed-off-by: Rohit Agarwal <rohiagar@chromium.org>
---
 .../devicetree/bindings/display/mediatek/mediatek,dpi.yaml       | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
index 3a82aec9021c..07acc8a76bfc 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
@@ -89,6 +89,7 @@ allOf:
                 - mediatek,mt6795-dpi
                 - mediatek,mt8173-dpi
                 - mediatek,mt8186-dpi
+                - mediatek,mt8195-dp-intf
     then:
       properties:
         power-domains: false
-- 
2.46.0.662.g92d0881bb0-goog


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: display: mediatek: dpi: Add power domain for MT8195 DP_INTF
  2024-09-11  7:17 [PATCH] dt-bindings: display: mediatek: dpi: Add power domain for MT8195 DP_INTF Rohit Agarwal
@ 2024-09-16 17:11 ` Rob Herring
  2024-09-18 10:56   ` Rohit Agarwal
  0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2024-09-16 17:11 UTC (permalink / raw)
  To: Rohit Agarwal
  Cc: chunkuang.hu, krzk+dt, ck.hu, linux-mediatek, devicetree,
	linux-kernel, linux-arm-kernel, kernel test robot

On Wed, Sep 11, 2024 at 07:17:21AM +0000, Rohit Agarwal wrote:
> Add power domain binding for MT8195 DP_INTF that resolves the following
> error and many more similar ones:
> 
> arch/arm64/boot/dts/mediatek/mt8195-cherry-dojo-r1.dtb: dp-intf@1c113000:
> power-domains: False schema does not allow [[55, 18]]
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202409102119.AYvaTjUi-lkp@intel.com/

Fixes?

> Signed-off-by: Rohit Agarwal <rohiagar@chromium.org>
> ---
>  .../devicetree/bindings/display/mediatek/mediatek,dpi.yaml       | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> index 3a82aec9021c..07acc8a76bfc 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> @@ -89,6 +89,7 @@ allOf:
>                  - mediatek,mt6795-dpi
>                  - mediatek,mt8173-dpi
>                  - mediatek,mt8186-dpi
> +                - mediatek,mt8195-dp-intf

mt8183 is also missing. There's a patch[1] to fix it, but it's wrong 
given the recent changes. It would be best to fix both in one 
patch/series as 2 separate patches will have conflicts.

Rob


[1] https://lore.kernel.org/all/20240912144430.3161717-2-treapking@chromium.org/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] dt-bindings: display: mediatek: dpi: Add power domain for MT8195 DP_INTF
  2024-09-16 17:11 ` Rob Herring
@ 2024-09-18 10:56   ` Rohit Agarwal
  0 siblings, 0 replies; 3+ messages in thread
From: Rohit Agarwal @ 2024-09-18 10:56 UTC (permalink / raw)
  To: Rob Herring
  Cc: chunkuang.hu, krzk+dt, ck.hu, linux-mediatek, devicetree,
	linux-kernel, linux-arm-kernel, kernel test robot


On 16/09/24 10:41 PM, Rob Herring wrote:
> On Wed, Sep 11, 2024 at 07:17:21AM +0000, Rohit Agarwal wrote:
>> Add power domain binding for MT8195 DP_INTF that resolves the following
>> error and many more similar ones:
>>
>> arch/arm64/boot/dts/mediatek/mt8195-cherry-dojo-r1.dtb: dp-intf@1c113000:
>> power-domains: False schema does not allow [[55, 18]]
>>
>> Reported-by: kernel test robot <lkp@intel.com>
>> Closes: https://lore.kernel.org/oe-kbuild-all/202409102119.AYvaTjUi-lkp@intel.com/
> Fixes?
Will add in the next version.
>
>> Signed-off-by: Rohit Agarwal <rohiagar@chromium.org>
>> ---
>>   .../devicetree/bindings/display/mediatek/mediatek,dpi.yaml       | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
>> index 3a82aec9021c..07acc8a76bfc 100644
>> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
>> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
>> @@ -89,6 +89,7 @@ allOf:
>>                   - mediatek,mt6795-dpi
>>                   - mediatek,mt8173-dpi
>>                   - mediatek,mt8186-dpi
>> +                - mediatek,mt8195-dp-intf
> mt8183 is also missing. There's a patch[1] to fix it, but it's wrong
> given the recent changes. It would be best to fix both in one
> patch/series as 2 separate patches will have conflicts.
>
> Rob
>
>
> [1] https://lore.kernel.org/all/20240912144430.3161717-2-treapking@chromium.org/

Ok, Sure will have a new (separate) patch added here to include the 
compatible strings. Thanks for pointing out.

Thanks,
Rohit.



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-09-18 10:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-11  7:17 [PATCH] dt-bindings: display: mediatek: dpi: Add power domain for MT8195 DP_INTF Rohit Agarwal
2024-09-16 17:11 ` Rob Herring
2024-09-18 10:56   ` Rohit Agarwal

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