Messages in this thread | | | Subject | Re: [PATCH] regulator: 88pm8607: Fix off-by-one value range checking in the case of no id is matched | From | Liam Girdwood <> | Date | Mon, 08 Aug 2011 17:39:53 +0100 |
| |
On Thu, 2011-08-04 at 16:18 +0800, Axel Lin wrote: > In the case of no id is matched, the variable i is equal to > ARRAY_SIZE(pm8607_regulator_info). > > Signed-off-by: Axel Lin <axel.lin@gmail.com> > --- > I think compare variable i with ARRAY_SIZE(pm8607_regulator_info) here is slightly > better in readability than compare with PM8607_ID_RG_MAX. > Axel >
Applied.
Thanks
Liam
| |