[RESEND v5 0/6] Add resets for ADSP based audio clock controller driver

Srinivasa Rao Mandadapu posted 6 patches 2 years, 7 months ago
There is a newer version of this series
.../devicetree/bindings/clock/qcom,sc7280-lpasscc.yaml   |  7 +++++++
drivers/clk/qcom/lpassaudiocc-sc7280.c                   | 16 +++++++++++-----
drivers/clk/qcom/lpasscc-sc7280.c                        | 12 +++++++-----
drivers/clk/qcom/lpasscorecc-sc7280.c                    |  3 +++
include/dt-bindings/clock/qcom,lpassaudiocc-sc7280.h     |  2 ++
5 files changed, 30 insertions(+), 10 deletions(-)
[RESEND v5 0/6] Add resets for ADSP based audio clock controller driver
Posted by Srinivasa Rao Mandadapu 2 years, 7 months ago
Add resets and remove qdsp6ss clcok controller for audioreach based platforms. 

Changes since v4:
    -- Update Fixes tag in Merge lpasscc into lpass_aon patch.
    -- Revert removal of clk_regmap structure in Merge lpasscc into lpass_aon patch.

Changes since v3:
    -- Remove duplicate clock resets patch.
    -- Add binding headers for q6 clocks.
    -- Create new patch for merging lpasscc q6 clocks into lpass_aon.
    -- Create new patches for handling conflicts of ADSP and bypass solution.

Changes since v2:
    -- Revert removing qdsp6ss clock control.
    -- Add Conditional check for qdsp6ss clock registration.
Changes since v1:
    -- Update commit message.
    -- Remove qdsp6ss clock control.

Srinivasa Rao Mandadapu (6):
  dt-bindings: clock: qcom,sc7280-lpasscc: Add qcom,adsp-pil-mode
    property
  dt-bindings: clock: lpassaudiocc-sc7280: Add binding headers for
    lpasscc
  clk: qcom: lpasscc-sc7280: Skip qdsp6ss clock registration
  clk: qcom: lpasscorecc-sc7280: Skip lpasscorecc registration
  clk: qcom: lpassaudiocc-sc7280: Merge lpasscc into lpass_aon
  clk: qcom: lpassaudiocc-sc7280: Skip lpass_aon_cc_pll config

 .../devicetree/bindings/clock/qcom,sc7280-lpasscc.yaml   |  7 +++++++
 drivers/clk/qcom/lpassaudiocc-sc7280.c                   | 16 +++++++++++-----
 drivers/clk/qcom/lpasscc-sc7280.c                        | 12 +++++++-----
 drivers/clk/qcom/lpasscorecc-sc7280.c                    |  3 +++
 include/dt-bindings/clock/qcom,lpassaudiocc-sc7280.h     |  2 ++
 5 files changed, 30 insertions(+), 10 deletions(-)

-- 
2.7.4
Re: [RESEND v5 0/6] Add resets for ADSP based audio clock controller driver
Posted by Krzysztof Kozlowski 2 years, 7 months ago
On 21/01/2023 18:32, Srinivasa Rao Mandadapu wrote:
> Add resets and remove qdsp6ss clcok controller for audioreach based platforms. 
> 
> Changes since v4:
>     -- Update Fixes tag in Merge lpasscc into lpass_aon patch.
>     -- Revert removal of clk_regmap structure in Merge lpasscc into lpass_aon patch.
> 

Your v5 before resend have build issues. Please fix.

Best regards,
Krzysztof
Re: [RESEND v5 0/6] Add resets for ADSP based audio clock controller driver
Posted by Srinivasa Rao Mandadapu 2 years, 7 months ago
On 1/22/2023 7:21 PM, Krzysztof Kozlowski wrote:
Thanks for Your time Krzysztof!!!
> On 21/01/2023 18:32, Srinivasa Rao Mandadapu wrote:
>> Add resets and remove qdsp6ss clcok controller for audioreach based platforms.
>>
>> Changes since v4:
>>      -- Update Fixes tag in Merge lpasscc into lpass_aon patch.
>>      -- Revert removal of clk_regmap structure in Merge lpasscc into lpass_aon patch.
>>
> Your v5 before resend have build issues. Please fix.

In v5, one commit missed to amend local changes.Hence resent it.

on Resent patch series no build issues.

>
> Best regards,
> Krzysztof
>
Re: [RESEND v5 0/6] Add resets for ADSP based audio clock controller driver
Posted by Krzysztof Kozlowski 2 years, 7 months ago
On 23/01/2023 06:13, Srinivasa Rao Mandadapu wrote:
> 
> On 1/22/2023 7:21 PM, Krzysztof Kozlowski wrote:
> Thanks for Your time Krzysztof!!!
>> On 21/01/2023 18:32, Srinivasa Rao Mandadapu wrote:
>>> Add resets and remove qdsp6ss clcok controller for audioreach based platforms.
>>>
>>> Changes since v4:
>>>      -- Update Fixes tag in Merge lpasscc into lpass_aon patch.
>>>      -- Revert removal of clk_regmap structure in Merge lpasscc into lpass_aon patch.
>>>
>> Your v5 before resend have build issues. Please fix.
> 
> In v5, one commit missed to amend local changes.Hence resent it.
> 
> on Resent patch series no build issues.

Resend means you send the same patches. If you change patches, it's not
resend but new version.

Best regards,
Krzysztof
Re: [RESEND v5 0/6] Add resets for ADSP based audio clock controller driver
Posted by Stephen Boyd 2 years, 7 months ago
Quoting Krzysztof Kozlowski (2023-01-23 00:01:02)
> On 23/01/2023 06:13, Srinivasa Rao Mandadapu wrote:
> > 
> > On 1/22/2023 7:21 PM, Krzysztof Kozlowski wrote:
> > Thanks for Your time Krzysztof!!!
> >> On 21/01/2023 18:32, Srinivasa Rao Mandadapu wrote:
> >>> Add resets and remove qdsp6ss clcok controller for audioreach based platforms.
> >>>
> >>> Changes since v4:
> >>>      -- Update Fixes tag in Merge lpasscc into lpass_aon patch.
> >>>      -- Revert removal of clk_regmap structure in Merge lpasscc into lpass_aon patch.
> >>>
> >> Your v5 before resend have build issues. Please fix.
> > 
> > In v5, one commit missed to amend local changes.Hence resent it.
> > 
> > on Resent patch series no build issues.
> 
> Resend means you send the same patches. If you change patches, it's not
> resend but new version.
> 

I'm waiting for a new version now.