On 06/28/17 08:38, Arvind Yadav wrote: > of_device_ids are not supposed to change at runtime. All functions > working with of_device_ids provided by <linux/of.h> work with const > of_device_ids. So mark the non-const structs as const. > > File size before: > text data bss dec hex filename > 1531 592 0 2123 84b drivers/gpu/drm/tilcdc/tilcdc_panel.o > > File size after constify: > text data bss dec hex filename > 1915 176 0 2091 82b drivers/gpu/drm/tilcdc/tilcdc_panel.o > > Signed-off-by: Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> Thanks, I pick this up and push it in next merge window (for 4.14). However, both this and the bundled tfp410 driver is going away as soon as I have time to do it, but it is unlikely to happen before v4.14. Best regards, Jyri > --- > Changes in v1: > Subject was not correct. > > drivers/gpu/drm/tilcdc/tilcdc_panel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > index 28c3e2f..2abe7c4 100644 > --- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c > +++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > @@ -419,7 +419,7 @@ static int panel_remove(struct platform_device *pdev) > return 0; > } > > -static struct of_device_id panel_of_match[] = { > +static const struct of_device_id panel_of_match[] = { > { .compatible = "ti,tilcdc,panel", }, > { }, > }; > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel