Hi, 09/07/2019 06:08, Qiming Yang: > Fixed error return value check and wrong error message. > > Fixes: d76116a4678f ("net/ice: add generic flow API") > > Signed-off-by: Qiming Yang <qiming.y...@intel.com>
Comments about the formatting of the title: The verb must be in infinitive form: "fix". The title must say which behaviour is fixed, not the code detail (we don't care in the title if it because of a return value or variable assignment). Here it is about fixing flow rule validation, right? One more thing, the word "wrong" can always be avoided after "fix", because we know you are not fixing something which worked perfectly :) Do not hesitate to share these tips around you. Thank you