On Tue, Apr 24, 2018 at 06:34:54PM +0200, Johan Hovold wrote: > +static inline void *gnss_serial_get_drvdata(struct gnss_serial *gserial) > +{ > + return &gserial[1]; > +}
Oh that's the best hack I have seen in a long time. I need to remember that one for next time. Anyway, in reading this driver, I don't think the module reference counting is being handled here, so you might want to add that to the gnss core to keep things from going south if modules are unloaded. thanks, greg k-h