Hi Phiipp,
On 26 July 2018 at 07:59, Philipp Tomsich philipp.tomsich@theobroma-systems.com wrote:
With the recent changes of the underlying types for the ram size, we need to cast the constant used in clamping to u64 instead of size_t.
Signed-off-by: Philipp Tomsich philipp.tomsich@theobroma-systems.com
drivers/ram/rockchip/dmc-rk3368.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
diff --git a/drivers/ram/rockchip/dmc-rk3368.c b/drivers/ram/rockchip/dmc-rk3368.c index 8d1b9fa..35d7605 100644 --- a/drivers/ram/rockchip/dmc-rk3368.c +++ b/drivers/ram/rockchip/dmc-rk3368.c @@ -960,7 +960,7 @@ static int rk3368_dmc_probe(struct udevice *dev) * is SoC register space (i.e. reserved), and 0xfe000000~0xfeffffff is * inaccessible for some IP controller. */
priv->info.size = min(priv->info.size, (size_t)0xfe000000);
priv->info.size = min(priv->info.size, (u64)0xfe000000);
Why case at all? Is it because min() is a macro?
return 0;}
2.1.4
Regards, Simon