On Mon, Feb 06, 2017 at 12:20:07PM +0300, Dan Carpenter wrote: > On Fri, Feb 03, 2017 at 08:11:20PM +0200, Jarkko Sakkinen wrote: > > On Fri, Feb 03, 2017 at 01:30:40PM +0300, Dan Carpenter wrote: > > > We should check that we're within bounds first before checking that > > > "chip->active_banks[i] != TPM2_ALG_ERROR" so I've re-ordered the two > > > checks. > > > > > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > > > Thank you, good catch! Do you mind if I squash this. > > > > Not at all. > > regards, > dan carpenter Already went as a separate commit in a pull request to James :-) /Jarkko -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html