amba_id are not supposed to change at runtime. All functions working with const amba_id. So mark the non-const structs as const.
Signed-off-by: Arvind Yadav arvind.yadav.cs@gmail.com --- drivers/gpu/drm/pl111/pl111_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/pl111/pl111_drv.c b/drivers/gpu/drm/pl111/pl111_drv.c index ac8771b..522875c 100644 --- a/drivers/gpu/drm/pl111/pl111_drv.c +++ b/drivers/gpu/drm/pl111/pl111_drv.c @@ -243,7 +243,7 @@ static int pl111_amba_remove(struct amba_device *amba_dev) return 0; }
-static struct amba_id pl111_id_table[] = { +static const struct amba_id pl111_id_table[] = { { .id = 0x00041111, .mask = 0x000fffff,
Arvind Yadav arvind.yadav.cs@gmail.com writes:
amba_id are not supposed to change at runtime. All functions working with const amba_id. So mark the non-const structs as const.
Signed-off-by: Arvind Yadav arvind.yadav.cs@gmail.com
drivers/gpu/drm/pl111/pl111_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/pl111/pl111_drv.c b/drivers/gpu/drm/pl111/pl111_drv.c index ac8771b..522875c 100644 --- a/drivers/gpu/drm/pl111/pl111_drv.c +++ b/drivers/gpu/drm/pl111/pl111_drv.c @@ -243,7 +243,7 @@ static int pl111_amba_remove(struct amba_device *amba_dev) return 0; }
-static struct amba_id pl111_id_table[] = { +static const struct amba_id pl111_id_table[] = { { .id = 0x00041111, .mask = 0x000fffff, -- 2.7.4
Sorry about dropping your patch here -- Linus was working on a bigger series for pl110 support that also happened to fix this.
dri-devel@lists.freedesktop.org