mbox series

[0/9] arm64: dts: qcom: msm8916/39: Reserve firmware memory dynamically

Message ID 20230911-msm8916-rmem-v1-0-b7089ec3e3a1@gerhold.net (mailing list archive)
Headers show
Series arm64: dts: qcom: msm8916/39: Reserve firmware memory dynamically | expand

Message

Stephan Gerhold Sept. 11, 2023, 5:41 p.m. UTC
Refactor the MSM8916 and MSM8939 device trees to make use of dynamic 
memory reservations for the firmware regions, rather than hardcoding 
fixed addresses for each device. This allows to keep most of the 
definitions in the SoC.dtsi while defining the board-specific 
properties (such as firmware size) separately for each device.

The main motivation for this patch set is to simplify enabling the 
modem on the various MSM8916/39 devices. The modem firmware size 
differs on almost each device, which requires redefining *all* of
the firmware reservations with newly calculated addresses to make
room for the larger modem firmware. I've explained this in detail
in a previous RFC:
https://lore.kernel.org/linux-arm-msm/20230510-dt-resv-bottom-up-v1-4-3bf68873dbed@gerhold.net/

Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
---
Stephan Gerhold (9):
      arm64: dts: qcom: msm8916: Disable venus by default
      arm64: dts: qcom: msm8916/39: Disable GPU by default
      arm64: dts: qcom: msm8916-ufi: Drop gps_mem for now
      arm64: dts: qcom: msm8916: Reserve firmware memory dynamically
      arm64: dts: qcom: msm8916: Reserve MBA memory dynamically
      arm64: dts: qcom: msm8939: Reserve firmware memory dynamically
      arm64: dts: qcom: msm8916/39: Disable unneeded firmware reservations
      arm64: dts: qcom: msm8916/39: Move mpss_mem size to boards
      arm64: dts: qcom: msm8916/39: Fix venus memory size

 arch/arm64/boot/dts/qcom/apq8016-sbc.dts           | 25 ++++++++++++++
 arch/arm64/boot/dts/qcom/apq8039-t2.dts            |  8 +++++
 arch/arm64/boot/dts/qcom/msm8916-acer-a1-724.dts   | 12 +++++++
 .../boot/dts/qcom/msm8916-alcatel-idol347.dts      | 12 +++++++
 arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dts     | 12 +++++++
 arch/arm64/boot/dts/qcom/msm8916-gplus-fl8005a.dts | 12 +++++++
 arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dts     | 12 +++++++
 .../boot/dts/qcom/msm8916-longcheer-l8150.dts      | 21 ++++++++++--
 .../boot/dts/qcom/msm8916-longcheer-l8910.dts      | 12 +++++++
 .../dts/qcom/msm8916-samsung-a2015-common.dtsi     | 12 +++++++
 .../boot/dts/qcom/msm8916-samsung-a3u-eur.dts      |  4 +++
 .../boot/dts/qcom/msm8916-samsung-a5u-eur.dts      |  4 +++
 .../dts/qcom/msm8916-samsung-e2015-common.dtsi     |  4 +++
 .../boot/dts/qcom/msm8916-samsung-gt5-common.dtsi  | 12 +++++++
 .../boot/dts/qcom/msm8916-samsung-j5-common.dtsi   | 12 +++++++
 .../boot/dts/qcom/msm8916-samsung-serranove.dts    | 12 +++++++
 arch/arm64/boot/dts/qcom/msm8916-ufi.dtsi          | 33 +++++++++++-------
 .../boot/dts/qcom/msm8916-wingtech-wt88047.dts     | 12 +++++++
 arch/arm64/boot/dts/qcom/msm8916.dtsi              | 39 +++++++++++++++++-----
 arch/arm64/boot/dts/qcom/msm8939-samsung-a7.dts    |  4 +++
 .../dts/qcom/msm8939-sony-xperia-kanuti-tulip.dts  |  8 +++++
 arch/arm64/boot/dts/qcom/msm8939.dtsi              | 37 +++++++++++++++-----
 22 files changed, 288 insertions(+), 31 deletions(-)
---
base-commit: 0bb80ecc33a8fb5a682236443c1e740d5c917d1d
change-id: 20230720-msm8916-rmem-f88fff6bcded

Best regards,

Comments

Bjorn Andersson Sept. 20, 2023, 5:14 p.m. UTC | #1
On Mon, 11 Sep 2023 19:41:42 +0200, Stephan Gerhold wrote:
> Refactor the MSM8916 and MSM8939 device trees to make use of dynamic
> memory reservations for the firmware regions, rather than hardcoding
> fixed addresses for each device. This allows to keep most of the
> definitions in the SoC.dtsi while defining the board-specific
> properties (such as firmware size) separately for each device.
> 
> The main motivation for this patch set is to simplify enabling the
> modem on the various MSM8916/39 devices. The modem firmware size
> differs on almost each device, which requires redefining *all* of
> the firmware reservations with newly calculated addresses to make
> room for the larger modem firmware. I've explained this in detail
> in a previous RFC:
> https://lore.kernel.org/linux-arm-msm/20230510-dt-resv-bottom-up-v1-4-3bf68873dbed@gerhold.net/
> 
> [...]

Applied, thanks!

[1/9] arm64: dts: qcom: msm8916: Disable venus by default
      commit: 29589248420766cd492e6db0632d6f59ec216e92
[2/9] arm64: dts: qcom: msm8916/39: Disable GPU by default
      commit: 0ce5bb825d54c904b217cc7f1131e084e02ed037
[3/9] arm64: dts: qcom: msm8916-ufi: Drop gps_mem for now
      commit: 40eb256e5fd1fd49813a27a252b7f45ccca89fde
[4/9] arm64: dts: qcom: msm8916: Reserve firmware memory dynamically
      commit: 0ed3d82862e8451cc2b14ddb1b064e72ba3e5a60
[5/9] arm64: dts: qcom: msm8916: Reserve MBA memory dynamically
      commit: b4f3a410061bf5a8cc148c9435479da580abf85b
[6/9] arm64: dts: qcom: msm8939: Reserve firmware memory dynamically
      commit: b22bef3dbc3a67a796f82f49a6df9e413211cb40
[7/9] arm64: dts: qcom: msm8916/39: Disable unneeded firmware reservations
      commit: 0ece6438a8c0492a7df036d57ac299500a25cb70
[8/9] arm64: dts: qcom: msm8916/39: Move mpss_mem size to boards
      commit: 35efa1be51bd6db067d7380b34538b17b9f250a8
[9/9] arm64: dts: qcom: msm8916/39: Fix venus memory size
      commit: e3c6386c6a5d0187f103fc9bf39850ac15c01690

Best regards,