Messages in this thread | | | Date | Fri, 29 Jul 2016 17:42:21 +0100 | From | Mark Brown <> | Subject | Re: [PATCH] regmap: cache: Fix num_reg_defaults computation from reg_defaults_raw |
| |
On Fri, Jul 29, 2016 at 04:46:43PM +0200, Maarten ter Huurne wrote:
> for (count = 0, i = 0; i < map->num_reg_defaults_raw; i++) > - if (!regmap_volatile(map, i * map->reg_stride)) > + if (regmap_readable(map, i * map->reg_stride) && > + !regmap_volatile(map, i * map->reg_stride))
This seems fine but please use a normal coding style, there's a *huge* amount of indentation on the second line for some reason. It should align with the ( in the if statement. [unhandled content-type:application/pgp-signature] | |