Thread (21 messages) 21 messages, 6 authors, 2024-04-04

Re: [PATCH v2 2/3] dt-bindings: power: Add mediatek larb definition

From: Yu-chang Lee (李禹璋) <hidden>
Date: 2024-03-27 10:39:53
Also in: linux-devicetree, linux-mediatek, linux-pm, lkml

On Wed, 2024-03-27 at 10:59 +0100, Krzysztof Kozlowski wrote:
 	 
External email : Please do not click links or open attachments until
you have verified the sender or the content.
 On 27/03/2024 10:34, Yu-chang Lee (李禹璋) wrote:
quoted
On Wed, 2024-03-27 at 10:23 +0100, Krzysztof Kozlowski wrote:
quoted
  
External email : Please do not click links or open attachments
until
quoted
quoted
you have verified the sender or the content.
 On 27/03/2024 09:39, Krzysztof Kozlowski wrote:
quoted
On 27/03/2024 06:57, yu-chang.lee wrote:
quoted
Add Smart Multimedia Interface Local Arbiter to mediatek
power domain.

Signed-off-by: yu-chang.lee <redacted>
---
 .../devicetree/bindings/power/mediatek,power-controller.yaml  | 
4
quoted
quoted
++++
quoted
quoted
 1 file changed, 4 insertions(+)

diff --git
a/Documentation/devicetree/bindings/power/mediatek,power-
controller.yaml
b/Documentation/devicetree/bindings/power/mediatek,power-
controller.yaml
quoted
quoted
index 8985e2df8a56..228c0dec5253 100644
--- a/Documentation/devicetree/bindings/power/mediatek,power-
controller.yaml
quoted
quoted
+++ b/Documentation/devicetree/bindings/power/mediatek,power-
controller.yaml
quoted
quoted
@@ -125,6 +125,10 @@ $defs:
         $ref: /schemas/types.yaml#/definitions/phandle
         description: phandle to the device containing the SMI
register range.
quoted
quoted
 
+     mediatek,larb:
+        $ref: /schemas/types.yaml#/definitions/phandle
+        description: phandle to the device containing the LARB
register range.
quoted
Why do you need it?

Plus I also see mediatek,larbs and mediatek,larb-id... so now we
have
quoted
third one similar.
... and not even tested!

Best regards,
Krzysztof
Hi,

I will double check the format of yaml for the next version, sorry
for
quoted
inconvenience. But I did test it on mt8188 chromebook, the reason
why

How do you test a binding on chromebook?
quoted
power domain need larb node is that when mtcmos power on, signal
glitch
quoted
may produce. Power domain driver must reset larb when this happen
to 
quoted
prevent dummy transaction on bus. That why I need larb node in dts.
No one talks here about larb node...
Sorry, May you elaborate on what information I need to provide to you
or it is just a syntax problem I need to fix?

Thanks
Best Regards,

Yu-chang
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help