On Mon, Nov 11, 2019 at 12:14:24PM -0500, Marcelo Diop-Gonzalez wrote: > @@ -109,7 +108,8 @@ int vchiq_platform_init(struct platform_device *pdev, struct vchiq_state *state) > if (err < 0) > return err; > > - g_cache_line_size = drvdata->cache_line_size; > + if (of_device_is_compatible(dev->of_node, "brcm,bcm2836-vchiq")) > + g_cache_line_size = 64; Could you remove the initialization and do it like this: if (of_device_is_compatible(dev->of_node, "brcm,bcm2836-vchiq")) g_cache_line_size = 64; else g_cache_line_size = 32; Maybe we could even just remove the global? regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel