Hi Praveen, Marc, On Wednesday 12 of September 2012 13:08:26 Marc Kleine-Budde wrote: > > +static const struct of_device_id samsung_usbphy_dt_match[]; > > + > > +static inline int samsung_usbphy_get_driver_data(struct > > platform_device *pdev) +{ > > +#ifdef CONFIG_OF > > if (IS_ENABLED(CONFIG_OF) && pdev->dev.of_node) Is this check needed at all? In case of CONFIG_OF disabled, of_node should always be NULL and there is a dummy macro provided for of_match_node, so it should compile and work just fine. -- Best regards, Tomasz Figa -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html