On 3/16/2020 5:01 PM, Andrey Smirnov wrote: > @@ -564,6 +579,26 @@ static void caam_remove_debugfs(void *root) > } > #endif > > +#ifdef CONFIG_FSL_MC_BUS > +static bool check_version(struct fsl_mc_version *mc_version, u32 major, > + u32 minor, u32 revision) > +{ > + if (mc_version->major > major) > + return true; > + > + if (mc_version->major == major) { > + if (mc_version->minor > minor) > + return true; > + > + if (mc_version->minor == minor && mc_version->revision > 0) > + return true; > + } > + > + return false; > +} > +#endif > + > + Nitpick - checkpatch complains here: CHECK: Please don't use multiple blank lines Horia