Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur
@ 2025-09-25  0:20 Sibi Sankar
  2025-09-25 17:22 ` Bjorn Andersson
  0 siblings, 1 reply; 4+ messages in thread
From: Sibi Sankar @ 2025-09-25  0:20 UTC (permalink / raw)
  To: robh, krzk+dt, conor+dt, jingyi.wang, andersson, mathieu.poirier
  Cc: linux-kernel, linux-arm-msm, devicetree

Document compatible for Qualcomm Glymur SoC SoCCP (SoC Control Processor)
PAS which is fully compatible with Kaanapali.

Signed-off-by: Sibi Sankar <sibi.sankar@oss.qualcomm.com>
---

Dependencies:
[1] Add initial remoteproc support for Kaanapali SoC
https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t

This patch depends on patch 4/5 of ^^ series

[2] Add support for remoteproc early attach
https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t

 .../bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml    | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
index 79f678f5f7d9..8089e0869ed2 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
@@ -17,8 +17,13 @@ description:
 
 properties:
   compatible:
-    enum:
-      - qcom,kaanapali-soccp-pas
+    oneOf:
+      - items:
+          - enum:
+              - qcom,glymur-soccp-pas
+          - const: qcom,kaanapali-soccp-pas
+      - enum:
+          - qcom,kaanapali-soccp-pas
 
   reg:
     maxItems: 1
-- 
2.34.1


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

* Re: [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur
  2025-09-25  0:20 [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur Sibi Sankar
@ 2025-09-25 17:22 ` Bjorn Andersson
  2025-10-09 10:20   ` Sibi Sankar
  2025-10-09 10:36   ` Krzysztof Kozlowski
  0 siblings, 2 replies; 4+ messages in thread
From: Bjorn Andersson @ 2025-09-25 17:22 UTC (permalink / raw)
  To: Sibi Sankar
  Cc: robh, krzk+dt, conor+dt, jingyi.wang, mathieu.poirier,
	linux-kernel, linux-arm-msm, devicetree

On Thu, Sep 25, 2025 at 05:50:34AM +0530, Sibi Sankar wrote:
> Document compatible for Qualcomm Glymur SoC SoCCP (SoC Control Processor)
> PAS which is fully compatible with Kaanapali.
> 
> Signed-off-by: Sibi Sankar <sibi.sankar@oss.qualcomm.com>
> ---
> 
> Dependencies:
> [1] Add initial remoteproc support for Kaanapali SoC
> https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t

Please don't send out patches that depend on other patches sent out the
same day. This just assumes that the maintainer will keep track of
dependencies and figure out when a future version of that dependency
meets the needs of this version.

Work with Jingyi to handle dependencies on your side.

> 
> This patch depends on patch 4/5 of ^^ series
> 
> [2] Add support for remoteproc early attach
> https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t

This is the same link as above and what you call "early attach" that
series calls "late attach".

Thanks,
Bjorn

>  .../bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml    | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
> index 79f678f5f7d9..8089e0869ed2 100644
> --- a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
> @@ -17,8 +17,13 @@ description:
>  
>  properties:
>    compatible:
> -    enum:
> -      - qcom,kaanapali-soccp-pas
> +    oneOf:
> +      - items:
> +          - enum:
> +              - qcom,glymur-soccp-pas
> +          - const: qcom,kaanapali-soccp-pas
> +      - enum:
> +          - qcom,kaanapali-soccp-pas
>  
>    reg:
>      maxItems: 1
> -- 
> 2.34.1
> 

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

* Re: [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur
  2025-09-25 17:22 ` Bjorn Andersson
@ 2025-10-09 10:20   ` Sibi Sankar
  2025-10-09 10:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 4+ messages in thread
From: Sibi Sankar @ 2025-10-09 10:20 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: robh, krzk+dt, conor+dt, jingyi.wang, mathieu.poirier,
	linux-kernel, linux-arm-msm, devicetree


On 9/25/2025 10:52 PM, Bjorn Andersson wrote:
> On Thu, Sep 25, 2025 at 05:50:34AM +0530, Sibi Sankar wrote:
>> Document compatible for Qualcomm Glymur SoC SoCCP (SoC Control Processor)
>> PAS which is fully compatible with Kaanapali.
>>
>> Signed-off-by: Sibi Sankar <sibi.sankar@oss.qualcomm.com>
>> ---
>>
>> Dependencies:
>> [1] Add initial remoteproc support for Kaanapali SoC
>> https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t
> Please don't send out patches that depend on other patches sent out the
> same day. This just assumes that the maintainer will keep track of
> dependencies and figure out when a future version of that dependency
> meets the needs of this version.
>
> Work with Jingyi to handle dependencies on your side.

Hey Bjorn,

Thanks for taking time to review the series :)

Ack, will make sure it's included in the other series.

>
>> This patch depends on patch 4/5 of ^^ series
>>
>> [2] Add support for remoteproc early attach
>> https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t
> This is the same link as above and what you call "early attach" that
> series calls "late attach".

Ack, my bad will stick with late attach.


-Sibi Sankar

>
> Thanks,
> Bjorn
>
>>   .../bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml    | 9 +++++++--
>>   1 file changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
>> index 79f678f5f7d9..8089e0869ed2 100644
>> --- a/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
>> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,kaanapali-soccp-pas.yaml
>> @@ -17,8 +17,13 @@ description:
>>   
>>   properties:
>>     compatible:
>> -    enum:
>> -      - qcom,kaanapali-soccp-pas
>> +    oneOf:
>> +      - items:
>> +          - enum:
>> +              - qcom,glymur-soccp-pas
>> +          - const: qcom,kaanapali-soccp-pas
>> +      - enum:
>> +          - qcom,kaanapali-soccp-pas
>>   
>>     reg:
>>       maxItems: 1
>> -- 
>> 2.34.1
>>

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

* Re: [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur
  2025-09-25 17:22 ` Bjorn Andersson
  2025-10-09 10:20   ` Sibi Sankar
@ 2025-10-09 10:36   ` Krzysztof Kozlowski
  1 sibling, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-10-09 10:36 UTC (permalink / raw)
  To: Bjorn Andersson, Sibi Sankar
  Cc: robh, krzk+dt, conor+dt, jingyi.wang, mathieu.poirier,
	linux-kernel, linux-arm-msm, devicetree

On 26/09/2025 02:22, Bjorn Andersson wrote:
> On Thu, Sep 25, 2025 at 05:50:34AM +0530, Sibi Sankar wrote:
>> Document compatible for Qualcomm Glymur SoC SoCCP (SoC Control Processor)
>> PAS which is fully compatible with Kaanapali.
>>
>> Signed-off-by: Sibi Sankar <sibi.sankar@oss.qualcomm.com>
>> ---
>>
>> Dependencies:
>> [1] Add initial remoteproc support for Kaanapali SoC
>> https://lore.kernel.org/linux-arm-msm/20250924-knp-remoteproc-v1-0-611bf7be8329@oss.qualcomm.com/T/#t
> 
> Please don't send out patches that depend on other patches sent out the
> same day. This just assumes that the maintainer will keep track of
> dependencies and figure out when a future version of that dependency
> meets the needs of this version.
> 
> Work with Jingyi to handle dependencies on your side.
@Bjorn, it is getting better.

This part of Glymur depends on Kaanapali patches. But there are
Kaanapali patches depending on Glymur!

There is absolutely zero coordination between these two and total mess
of multiplying patches instead of sending ONE thing. Submitting Glymur
and Kaanapali was really screwed.

Best regards,
Krzysztof

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

end of thread, other threads:[~2025-10-09 10:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-25  0:20 [PATCH] dt-bindings: remoteproc: qcom,pas: Document SoCCP PAS for Glymur Sibi Sankar
2025-09-25 17:22 ` Bjorn Andersson
2025-10-09 10:20   ` Sibi Sankar
2025-10-09 10:36   ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox