On Wed, 2018-10-17 at 11:49 +0300, Stanimir Varbanov wrote: > On 10/08/2018 04:32 PM, Vikash Garodia wrote: > > Add routine to reset the ARM9 and brings it out of reset. Also > > abstract the Venus CPU state handling with a new function. This > > is in preparation to add PIL functionality in venus driver. [] > > diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h [] > > @@ -129,6 +130,7 @@ struct venus_core { > > struct device *dev; > > struct device *dev_dec; > > struct device *dev_enc; > > + bool use_tz; > > could you make it unsigned? For more info please run checkpatch --strict. > > I know that we have structure members of type bool already - that should > be fixed with follow-up patches, I guess. That's probably not necessary. I personally have no issue with bool struct members that are only used on a transitory basis and not used by hardware or shared between multiple cpus with different hardware alignment requirements. Nothing in this struct is saved or shared. Perhaps the checkpatch message should be expanded to enumerate when bool use in a struct is acceptable.