On Mon, Apr 3, 2017 at 4:51 AM, Sathyanarayanan Kuppuswamy Natarajan <sathyaosid@xxxxxxxxx> wrote: >>> +static inline int is_gcr_valid(u32 offset) >> >> Pointer to ipcdev should be a parameter to this function. > > But ipcdev is a static variable, visible across this file. So there is > no point in passing it as parameter. That's one of refactoring needed for this library. You need to pass a pointer to all internal functions and where it's possible to avoid reference to a global variable. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
