On Fri, 12 Sep 2014, Chen, Alvin wrote: > > On Tue, 9 Sep 2014, Weike Chen wrote: > > > > > > > > struct dwapb_gpio; > > > +struct dwapb_context; > > > > > > struct dwapb_gpio_port { > > > struct bgpio_chip bgc; > > > bool is_registered; > > > struct dwapb_gpio *gpio; > > > + struct dwapb_context *ctx; > > > > Alvin, > > > > Will this build if CONFIG_PM_SLEEP is not defined? > Actually, PM_SLEEP is always set as 'y' in 'kerne/power/Kconfig'. But I manually change it to 'n', this module can be compiled correctly. > You may be concern with 'ctx', and you can see 'ctx' accessing is always in CONFIG_PM_SLEEP. Yes and in the case of 'struct dwapb_context *ctx;' it is ok for struct dwapb_context to be an incomplete type since that's just a pointer. Alan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html