Re: [PATCH V14 8/9] arm64: dts: qcom: pm8008: Add base dts file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luca,


On 6/7/2022 6:46 PM, Luca Weiss wrote:
Hi Satya,

On Tue Jun 7, 2022 at 1:50 PM CEST, Satya Priya wrote:
Add base DTS file for pm8008.

Signed-off-by: Satya Priya <quic_c_skakit@xxxxxxxxxxx>
Reviewed-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>
---
Changes in V14:
  - None.

Changes in V13:
  - None.

Changes in V12:
  - None.

Changes in V11:
  - Remove intermediate regulators node and add the ldos under
    pm8008@8 node.
  - change the address cells as 2 for pm8008 parent mfd node.
  - add compatible to register the ldos.
  - add reg with i2c client offset and address.

  arc
  arch/arm64/boot/dts/qcom/pm8008.dtsi | 54 ++++++++++++++++++++++++++++++++++++
  1 file changed, 54 insertions(+)
  create mode 100644 arch/arm64/boot/dts/qcom/pm8008.dtsi

diff --git a/arch/arm64/boot/dts/qcom/pm8008.dtsi b/arch/arm64/boot/dts/qcom/pm8008.dtsi
new file mode 100644
index 0000000..6f37e4d
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/pm8008.dtsi
@@ -0,0 +1,54 @@
+// SPDX-License-Identifier: GPL-2.0-only
+// Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.
I noticed this file was licensed as BSD-3-Clause in V13 and
this change was not mentioned in the "Changes" part.
Any specific reason? Especially newer dts files should be BSD-licensed,
and having this GPL dtsi included in otherwise BSD dts files is not
good.


I changed it as per the suggestions of Qualcomm internal review team but i see that many of the drivers use BSD license, will change this back to BSD-3-Clause.


Regards
Luca



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux