[PATCH v1 0/4] Remove the qdsp6ss register from lpasscc

Mohammad Rafi Shaik posted 4 patches 2 years, 10 months ago
.../bindings/clock/qcom,sc7280-lpasscc.yaml   |  8 +--
arch/arm64/boot/dts/qcom/sc7280.dtsi          |  7 +--
drivers/clk/qcom/lpasscc-sc7280.c             | 63 +------------------
3 files changed, 7 insertions(+), 71 deletions(-)
[PATCH v1 0/4] Remove the qdsp6ss register from lpasscc
Posted by Mohammad Rafi Shaik 2 years, 10 months ago
This patch set is to remove the qdsp6ss register from lpasscc to
resolve memory conflict's between lpascc and ADSP remoteproc driver.

Mohammad Rafi Shaik (4):
  arm64: dts: qcom: sc7280: Modify lpasscc node name
  dt-bindings: clock: qcom,sc7280-lpasscc: Remove qdsp6ss register
    region
  arm64: dts: qcom: sc7280: Remove qdsp6ss regmap region
  clk: qcom: lpasscc-sc7280: Remove qdsp6ss clock registration

 .../bindings/clock/qcom,sc7280-lpasscc.yaml   |  8 +--
 arch/arm64/boot/dts/qcom/sc7280.dtsi          |  7 +--
 drivers/clk/qcom/lpasscc-sc7280.c             | 63 +------------------
 3 files changed, 7 insertions(+), 71 deletions(-)

-- 
2.25.1
Re: [PATCH v1 0/4] Remove the qdsp6ss register from lpasscc
Posted by Stephen Boyd 2 years, 10 months ago
Quoting Mohammad Rafi Shaik (2023-03-27 09:32:45)
> This patch set is to remove the qdsp6ss register from lpasscc to
> resolve memory conflict's between lpascc and ADSP remoteproc driver.

Is this related to the other patch series[1] ("[PATCH v9 0/4] Add resets
for ADSP based audio clock controller driver")? Does it supersede those?

> 
> Mohammad Rafi Shaik (4):
>   arm64: dts: qcom: sc7280: Modify lpasscc node name
>   dt-bindings: clock: qcom,sc7280-lpasscc: Remove qdsp6ss register
>     region
>   arm64: dts: qcom: sc7280: Remove qdsp6ss regmap region
>   clk: qcom: lpasscc-sc7280: Remove qdsp6ss clock registration

[1] https://lore.kernel.org/all/20230317141622.1926573-1-quic_mohs@quicinc.com/
Re: [PATCH v1 0/4] Remove the qdsp6ss register from lpasscc
Posted by Mohammad Rafi Shaik 2 years, 10 months ago
On 3/27/2023 11:11 PM, Stephen Boyd wrote:
> Quoting Mohammad Rafi Shaik (2023-03-27 09:32:45)
>> This patch set is to remove the qdsp6ss register from lpasscc to
>> resolve memory conflict's between lpascc and ADSP remoteproc driver.
> Is this related to the other patch series[1] ("[PATCH v9 0/4] Add resets
> for ADSP based audio clock controller driver")? Does it supersede those?
Thanks for comment,

yes, its superseded form patch series[1] ("[PATCH v9 0/4] Add resets
for ADSP based audio clock controller driver") which is required many
changes.

As the qdsp6ss clocks are being enabled in remoteproc driver,
the qdsp6ss not required in lpasscc node.

For audioreach solution required to create the remoteproc_adsp
device tree node with base address 0x3000000 for remoteproc driver,
as already this address being used in lpasscc node it's causing memory
conflict.
>> Mohammad Rafi Shaik (4):
>>    arm64: dts: qcom: sc7280: Modify lpasscc node name
>>    dt-bindings: clock: qcom,sc7280-lpasscc: Remove qdsp6ss register
>>      region
>>    arm64: dts: qcom: sc7280: Remove qdsp6ss regmap region
>>    clk: qcom: lpasscc-sc7280: Remove qdsp6ss clock registration
> [1] https://lore.kernel.org/all/20230317141622.1926573-1-quic_mohs@quicinc.com/
Re: [PATCH v1 0/4] Remove the qdsp6ss register from lpasscc
Posted by Stephen Boyd 2 years, 10 months ago
Quoting Mohammad Rafi Shaik (2023-03-27 23:02:38)
> 
> On 3/27/2023 11:11 PM, Stephen Boyd wrote:
> > Quoting Mohammad Rafi Shaik (2023-03-27 09:32:45)
> >> This patch set is to remove the qdsp6ss register from lpasscc to
> >> resolve memory conflict's between lpascc and ADSP remoteproc driver.
> > Is this related to the other patch series[1] ("[PATCH v9 0/4] Add resets
> > for ADSP based audio clock controller driver")? Does it supersede those?
> Thanks for comment,
> 
> yes, its superseded form patch series[1] ("[PATCH v9 0/4] Add resets
> for ADSP based audio clock controller driver") which is required many
> changes.
> 
> As the qdsp6ss clocks are being enabled in remoteproc driver,
> the qdsp6ss not required in lpasscc node.
> 
> For audioreach solution required to create the remoteproc_adsp
> device tree node with base address 0x3000000 for remoteproc driver,
> as already this address being used in lpasscc node it's causing memory
> conflict.

Ok. Please add the details of superseded patch series to the cover
letter. It helps us understand what to do with the other patches on the
list.