| Message ID | 20230327163249.1081824-1-quic_mohs@quicinc.com |
|---|---|
| Headers | show |
| Series | Remove the qdsp6ss register from lpasscc | expand |
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/
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/
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.
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(-)