History log of /rk3399_ARM-atf/plat/qti/common/src/spmi_arb.c (Results 1 – 6 of 6)
Revision Date Author Comments
# 8b3e1b79 19-Feb-2021 Madhukar Pappireddy <madhukar.pappireddy@arm.com>

Merge "qti: spmi_arb: Fix NUM_APID and REG_APID_MAP() argument" into integration


# de67080f 28-Jan-2021 Julius Werner <jwerner@chromium.org>

qti: spmi_arb: Fix NUM_APID and REG_APID_MAP() argument

The NUM_APID value was derived from kernel device tree sources, but I
made a conversion mistake: the amount of bytes in the APID map is the
to

qti: spmi_arb: Fix NUM_APID and REG_APID_MAP() argument

The NUM_APID value was derived from kernel device tree sources, but I
made a conversion mistake: the amount of bytes in the APID map is the
total size of the "core" register range (0x1100) minus the offset of the
APID map in that range (0x900). This is of course 0x1100 - 0x900 = 0x800
and not 0x200, so the amount of 4-byte integers it can fit is not 0x80
but 0x200. Fix this and make the math more explicit so it can be more
easily factored out and adjusted if that becomes necessary for a future
SoC.

Also fix a dangerous typo in REG_APID_MAP() where the macro would
reference a random variable `i` rather than its argument (`apid`), and
we just got lucky that the only caller in the current code happened to
pass in a variable called `i` as that argument.

Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: I049dd044fa5aeb65be0e7b12150afd6eb4bac0fa

show more ...


# ddf28700 31-Aug-2020 Madhukar Pappireddy <madhukar.pappireddy@arm.com>

Merge "qti: spmi_arb: Fix coverity integer conversion warnings" into integration


# 9694c210 27-Aug-2020 Julius Werner <jwerner@chromium.org>

qti: spmi_arb: Fix coverity integer conversion warnings

Coverity warns about the risk of unintended sign-exension in some of the
calculations in spmi_arb.c. While the actual numbers used are small
e

qti: spmi_arb: Fix coverity integer conversion warnings

Coverity warns about the risk of unintended sign-exension in some of the
calculations in spmi_arb.c. While the actual numbers used are small
enough that this cannot happen in practice, it's still a good idea to
clean them up by explicitly making the constants used unsigned.

Signed-off-by: Julius Werner <jwerner@chromium.org>
Change-Id: Ia169e0f7c6b01b8041e8029e8c8d30ee596ba30d

show more ...


# a4c979ad 26-Aug-2020 Madhukar Pappireddy <madhukar.pappireddy@arm.com>

Merge changes I6bf1db15,I8631c34a,Id76ada14 into integration

* changes:
qti/sc7180: Do shutdown handling outside qtiseclib
qti: Add SPMI PMIC arbitrator driver
qti/sc7180: Fix GIC-600 support

Merge changes I6bf1db15,I8631c34a,Id76ada14 into integration

* changes:
qti/sc7180: Do shutdown handling outside qtiseclib
qti: Add SPMI PMIC arbitrator driver
qti/sc7180: Fix GIC-600 support setting

show more ...


# f40008a4 05-Jun-2019 Julius Werner <jwerner@chromium.org>

qti: Add SPMI PMIC arbitrator driver

This patch adds a very rudimentary driver for the SPMI arbitrator used
to access the PMIC. It doesn't support all the controller's actual
arbitration features, s

qti: Add SPMI PMIC arbitrator driver

This patch adds a very rudimentary driver for the SPMI arbitrator used
to access the PMIC. It doesn't support all the controller's actual
arbitration features, so it should probably not be used concurrently
with a running kernel (and it's also not optimized for performance). But
it can be used to set a few registers during boot or on shutdown to
control reset handling, which is all we need it for.

Change-Id: I8631c34a2a89ac71aa1ec9b8266e818c922fe34a
Signed-off-by: Julius Werner <jwerner@chromium.org>

show more ...