| 6ffd02b8 | 24-Nov-2025 |
Nicolas Frattaroli <nicolas.frattaroli@collabora.com> |
interconnect: mediatek: Aggregate bandwidth with saturating add
By using a regular non-overflow-checking add, the MediaTek icc-emi driver will happy wrap at U32_MAX + 1 to 0. As it's common for the
interconnect: mediatek: Aggregate bandwidth with saturating add
By using a regular non-overflow-checking add, the MediaTek icc-emi driver will happy wrap at U32_MAX + 1 to 0. As it's common for the interconnect core to fill in INT_MAX values, this is not a hypothetical situation, but something that actually happens in regular use. This would be pretty disasterous if anything used this driver.
Replace the addition with an overflow-checked addition from overflow.h, and saturate to U32_MAX if an overflow is detected.
Fixes: b45293799f75 ("interconnect: mediatek: Add MediaTek MT8183/8195 EMI Interconnect driver") Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com> Link: https://lore.kernel.org/r/20251124-mt8196-dvfsrc-v2-13-d9c1334db9f3@collabora.com Signed-off-by: Georgi Djakov <djakov@kernel.org>
show more ...
|
| 510f8214 | 24-Nov-2025 |
Nicolas Frattaroli <nicolas.frattaroli@collabora.com> |
interconnect: mediatek: Don't hijack parent device
If the intention is that users of the interconnect declare their relationship to the child icc_emi node of the dvfsrc controller, then this code ne
interconnect: mediatek: Don't hijack parent device
If the intention is that users of the interconnect declare their relationship to the child icc_emi node of the dvfsrc controller, then this code never worked. That's because it uses the parent dvfsrc device as the device it passes to the interconnect core framework, which means all the OF parsing is broken.
Use the actual device instead, and pass the dvfsrc parent into the dvfsrc calls.
Fixes: b45293799f75 ("interconnect: mediatek: Add MediaTek MT8183/8195 EMI Interconnect driver") Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Signed-off-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com> Link: https://lore.kernel.org/r/20251124-mt8196-dvfsrc-v2-12-d9c1334db9f3@collabora.com Signed-off-by: Georgi Djakov <djakov@kernel.org>
show more ...
|
| 9265ea04 | 24-Jun-2024 |
Jiapeng Chong <jiapeng.chong@linux.alibaba.com> |
interconnect: mediatek: remove unneeded semicolon
No functional modification involved.
./drivers/interconnect/mediatek/icc-emi.c:71:2-3: Unneeded semicolon.
Reported-by: Abaci Robot <abaci@linux.a
interconnect: mediatek: remove unneeded semicolon
No functional modification involved.
./drivers/interconnect/mediatek/icc-emi.c:71:2-3: Unneeded semicolon.
Reported-by: Abaci Robot <abaci@linux.alibaba.com> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9401 Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com> Link: https://lore.kernel.org/r/20240624025326.69551-1-jiapeng.chong@linux.alibaba.com Signed-off-by: Georgi Djakov <djakov@kernel.org>
show more ...
|