On Fri, 15 Jul 2016, Laxman Dewangan wrote: > > On Wednesday 13 July 2016 04:56 PM, Lee Jones wrote: > > Cc: Laxman Dewangan <[email protected]> > > Signed-off-by: Lee Jones <[email protected]> > > --- > > include/linux/mfd/core.h | 29 +++++++++++++++++++++++++++++ > > 1 file changed, 29 insertions(+) > > > > diff --git a/include/linux/mfd/core.h b/include/linux/mfd/core.h > > index 99c0395..c6018f4 100644 > > --- a/include/linux/mfd/core.h > > +++ b/include/linux/mfd/core.h > > @@ -16,6 +16,35 @@ > > #include <linux/platform_device.h> > > > +#define MFD_CELL_BASIC(_name, _res, _pdata, _pdsize, _id) \ > > + MFD_CELL_ALL(_name, _res, _pdata, _pdsize, _id, NULL, NULL) \ > > + > > +#define MFD_CELL_RES(_name, _res) \ > > + MFD_CELL_ALL(_name, _res, _pdata, 0, 0, NULL, NULL) \ > > + > Here _pdata should not be used. Instead it should be NULL.
Yes, it should. Thanks for the review. > I corrected it locally and then compiled and teste. Wonderful, thanks. > Once above fixed, you can take my ack and tested by > > Acked-by: Laxman Dewangan <[email protected]> > Tested-by: Laxman Dewangan <[email protected]> > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
