Re: [PATCH v2 02/20] dt-bindings: power: Add power-domain header for RV1126

On Thu, 4 Aug 2022 at 04:03, Rob Herring [off-list ref] wrote:

On Wed, Aug 03, 2022 at 03:54:56PM +0530, Jagan Teki wrote:

quoted

On Tue, 2 Aug 2022 at 16:08, Krzysztof Kozlowski
[off-list ref] wrote:

quoted

On 31/07/2022 19:47, Jagan Teki wrote:

quoted

Add power-domain header for RV1126 SoC from description in TRM.

Signed-off-by: Elaine Zhang <redacted>
Signed-off-by: Jagan Teki <jagan@edgeble.ai>
---
Changes for v2:
- add blank line

 include/dt-bindings/power/rv1126-power.h | 35 ++++++++++++++++++++++++
 1 file changed, 35 insertions(+)
 create mode 100644 include/dt-bindings/power/rv1126-power.h

diff --git a/include/dt-bindings/power/rv1126-power.h b/include/dt-bindings/power/rv1126-power.h

Vendor in filename is needed.

Yes this follows how similar rockchip vendor notation

$ ls include/dt-bindings/power/rk3* -l
 include/dt-bindings/power/rk3036-power.h
 include/dt-bindings/power/rk3066-power.h
 include/dt-bindings/power/rk3128-power.h
 include/dt-bindings/power/rk3188-power.h
 include/dt-bindings/power/rk3228-power.h
 include/dt-bindings/power/rk3288-power.h
 include/dt-bindings/power/rk3328-power.h
 include/dt-bindings/power/rk3366-power.h
 include/dt-bindings/power/rk3368-power.h
 include/dt-bindings/power/rk3399-power.h
 include/dt-bindings/power/rk3568-power.h

quoted

quoted

new file mode 100644
index 000000000000..38a68e000d38

--- /dev/null
+++ b/include/dt-bindings/power/rv1126-power.h

@@ -0,0 +1,35 @@
+/* SPDX-License-Identifier: GPL-2.0 */

Dual license, please, unless some copyrights are no allowing this.

Yes, as we discussed in v1 - I'm not liable to change so I keep this as it is.

I assume that would be Rockchip. Can someone from Rockchip please give
an okay here to dual license?

Kever or Heiko - any comments on this?

Thanks,
Jagan.