mbox series

[0/5] dma/arm64: qcom: use one compatible also for 0x10000 offset

Message ID 20221015140447.55221-1-krzysztof.kozlowski@linaro.org
Headers show
Series dma/arm64: qcom: use one compatible also for 0x10000 offset | expand

Message

Krzysztof Kozlowski Oct. 15, 2022, 2:04 p.m. UTC 
Hi,

Dependencies
============
1. DT bindings and DMA driver change depends contextually on:
   https://lore.kernel.org/all/20220927014846.32892-1-mailingradian@gmail.com/

2. DTS patches are independent, although they will spark some dtbs_check
   warnings (due to change in bindings).

Best regards,
Krzysztof

Krzysztof Kozlowski (5):
  dt-bindings: dma: qcom: gpi: use sm6350 fallback
  dmaengine: qcom: gpi: document preferred SM6350 binding
  arm64: dts: qcom: sc7280: Add GPI DMA compatible fallback
  arm64: dts: qcom: sm8350: Add GPI DMA compatible fallback
  arm64: dts: qcom: sm8450: Add GPI DMA compatible fallback

 Documentation/devicetree/bindings/dma/qcom,gpi.yaml | 10 ++++++----
 arch/arm64/boot/dts/qcom/sc7280.dtsi                |  4 ++--
 arch/arm64/boot/dts/qcom/sm8350.dtsi                |  6 +++---
 arch/arm64/boot/dts/qcom/sm8450.dtsi                |  6 +++---
 drivers/dma/qcom/gpi.c                              |  7 ++++---
 5 files changed, 18 insertions(+), 15 deletions(-)

Comments

Krzysztof Kozlowski Oct. 17, 2022, 9:34 p.m. UTC | #1 
On 17/10/2022 17:10, Richard Acayan wrote:
>> Several devices like SM6350, SM8150 and SC7280 are actually compatible,
>> so use one compatible fallback for all of them.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>  Documentation/devicetree/bindings/dma/qcom,gpi.yaml | 10 ++++++----
>>  1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
>> index 750b40c32213..0c2894498845 100644
>> --- a/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
>> +++ b/Documentation/devicetree/bindings/dma/qcom,gpi.yaml
>> @@ -20,12 +20,14 @@ properties:
>>    compatible:
>>      oneOf:
>>        - enum:
>> -          - qcom,sc7280-gpi-dma
>>            - qcom,sdm845-gpi-dma
>>            - qcom,sm6350-gpi-dma
>> -          - qcom,sm8350-gpi-dma
>> -          - qcom,sm8450-gpi-dma
>> -
> 
> If you don't want this empty line here, you can still ask for it to be removed
> in the original patch. It doesn't look like it was applied yet.
> 

Sure.

Best regards,
Krzysztof