mbox series

[v3,0/2] SM6115 interconnect

Message ID 20231125-topic-6115icc-v3-0-bd8907b8cfd7@linaro.org
Headers show
Series SM6115 interconnect | expand

Message

Konrad Dybcio Nov. 29, 2023, 2:41 p.m. UTC 
As it says on the can.

Georgi, Bjorn, can we please set up an immutable branch with the bindings?

Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
Changes in v3:
- Actually define clock-names before using it in conditional blocks (oops)
- Link to v2: https://lore.kernel.org/r/20231125-topic-6115icc-v2-0-69d05d90871b@linaro.org

Changes in v2:
bindings:
- Remove unnecessary '|' after description:
- sort property definitions in a manner that resembles their proper ordering
- define clock{s/-names} before using them in conditional blocks
- drop unnecessary allOf:
- move unevaluatedProperties after the conditional blocks
- disallow clock{s,-names} when unnecessary
- fix up the example
C:
- Apply my own fixup (ebi channel configuration)
- Link to v1: https://lore.kernel.org/r/20231125-topic-6115icc-v1-0-fa51c0b556c9@linaro.org

---
Konrad Dybcio (2):
      dt-bindings: interconnect: Add Qualcomm SM6115 NoC
      interconnect: qcom: Add SM6115 interconnect provider driver

 .../bindings/interconnect/qcom,sm6115.yaml         |  152 +++
 drivers/interconnect/qcom/Kconfig                  |    9 +
 drivers/interconnect/qcom/Makefile                 |    2 +
 drivers/interconnect/qcom/sm6115.c                 | 1427 ++++++++++++++++++++
 include/dt-bindings/interconnect/qcom,sm6115.h     |  111 ++
 5 files changed, 1701 insertions(+)
---
base-commit: 48bbaf8b793e0770798519f8ee1ea2908ff0943a
change-id: 20231125-topic-6115icc-a187f5989af7

Best regards,

Comments

Krzysztof Kozlowski Nov. 30, 2023, 8:50 a.m. UTC | #1 
On 29/11/2023 15:41, Konrad Dybcio wrote:
> Add bindings for Qualcomm SM6115 Network-On-Chip interconnect.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
>  .../bindings/interconnect/qcom,sm6115.yaml         | 152 +++++++++++++++++++++
>  include/dt-bindings/interconnect/qcom,sm6115.h     | 111 +++++++++++++++
>  2 files changed, 263 insertions(+)


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Georgi Djakov Dec. 7, 2023, 1:23 p.m. UTC | #2 
On 29.11.23 16:41, Konrad Dybcio wrote:
> As it says on the can.
> 
> Georgi, Bjorn, can we please set up an immutable branch with the bindings?

Thanks for the patches, Konrad. Here is the branch:
https://git.kernel.org/pub/scm/linux/kernel/git/djakov/icc.git/log/?h=icc-sm6115

BR,
Georgi

> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
> Changes in v3:
> - Actually define clock-names before using it in conditional blocks (oops)
> - Link to v2: https://lore.kernel.org/r/20231125-topic-6115icc-v2-0-69d05d90871b@linaro.org
> 
> Changes in v2:
> bindings:
> - Remove unnecessary '|' after description:
> - sort property definitions in a manner that resembles their proper ordering
> - define clock{s/-names} before using them in conditional blocks
> - drop unnecessary allOf:
> - move unevaluatedProperties after the conditional blocks
> - disallow clock{s,-names} when unnecessary
> - fix up the example
> C:
> - Apply my own fixup (ebi channel configuration)
> - Link to v1: https://lore.kernel.org/r/20231125-topic-6115icc-v1-0-fa51c0b556c9@linaro.org
> 
> ---
> Konrad Dybcio (2):
>        dt-bindings: interconnect: Add Qualcomm SM6115 NoC
>        interconnect: qcom: Add SM6115 interconnect provider driver
> 
>   .../bindings/interconnect/qcom,sm6115.yaml         |  152 +++
>   drivers/interconnect/qcom/Kconfig                  |    9 +
>   drivers/interconnect/qcom/Makefile                 |    2 +
>   drivers/interconnect/qcom/sm6115.c                 | 1427 ++++++++++++++++++++
>   include/dt-bindings/interconnect/qcom,sm6115.h     |  111 ++
>   5 files changed, 1701 insertions(+)
> ---
> base-commit: 48bbaf8b793e0770798519f8ee1ea2908ff0943a
> change-id: 20231125-topic-6115icc-a187f5989af7
> 
> Best regards,