OP-TEE / optee_os

Trusted side of the TEE
Other
1.59k stars 1.07k forks source link

drivers: scmi-msg: correct voltage domain protocol version & clock rates enumeration #6951

Closed etienne-lms closed 2 months ago

etienne-lms commented 4 months ago

Fix the version ID of the implemented SCMI voltage domain protocol that is v2.0 (ID 0x20000), not v3.0 (ID 0x30000). (Fixes: 006d89b8f49f ("drivers: scmi-msg: add SCMI Voltage Domain protocol"))

Fix value of remaining clocks to describe on SCMI clock protocol message CLOCK_DESCRIBE_RATES that does not take into account the number of returned clock in the response. (Fixes: 90252e2a52c7 ("drivers: scmi-msg: clock adapts to output buffer size"))

jforissier commented 2 months ago

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

etienne-lms commented 2 months ago

Fixup commit squashed and review tag applied. @GseoC, do you want to provide a review tag?

pkcs11_1007 test sporadic failure faced during Ci test for this P-R but it's unrelated to this change (I still don't understand exactly what happens there, still tracking through https://github.com/OP-TEE/optee_os/issues/6952).

GseoC commented 2 months ago

Sure, Acked-by: Gatien Chevallier <gatien.chevallier@foss.st.com>

etienne-lms commented 2 months ago

Review tags applied. Thanks.