Messages in this thread | | | Subject | Re: [PATCH v2 07/14] regulator: s2mps11: Copy supported regulators from initconst | From | Krzysztof Kozlowski <> | Date | Thu, 13 Feb 2014 13:35:23 +0100 |
| |
On Thu, 2014-02-13 at 17:51 +0530, Yadwinder Singh Brar wrote: > Hi, > > On Thu, Feb 13, 2014 at 2:44 PM, Krzysztof Kozlowski > <k.kozlowski@samsung.com> wrote: > > Add __initconst to 'regulator_desc' array with supported regulators. > > During probe choose how many and which regulators will be supported > > according to device ID. Then copy the 'regulator_desc' array to > > allocated memory so the regulator core can use it. > > > > Additionally allocate array of of_regulator_match() dynamically (based > > on number of regulators) instead of allocation on the stack. > > > > This is needed for supporting different devices in s2mps11 > > driver and actually prepares the regulator driver for supporting the > > S2MPS14 device. > > > > Code for supporting the S2MPS14 device will add its own array of > > 'regulator_desc' (also marked as __initconst). This way memory footprint > > of the driver will be reduced (approximately 'regulators_desc' array for > > S2MPS11 occupies 5 kB on 32-bit ARM, for S2MPS14 will occupy 3 kB). > > > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > > Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com> > > Cc: Mark Brown <broonie@kernel.org> > > Cc: Liam Girdwood <lgirdwood@gmail.com> > > Cc: Yadwinder Singh Brar <yadi.brar01@gmail.com> > > --- > > Just observed one trivial thing that in this patch, spacing is not > provided before and after multiplication binary operator ( _ * _ ), > which is recommended by linux spacing convention. > > Other than that it looks fine to me, pls feel free to add > > Reviewed-by: Yadwinder Singh Brar <yadi.brar@samsung.com> >
Thanks for review. I'll send a patch with proper spacing around '*'.
Best regards, Krzysztof
| |