Messages in this thread | | | Date | Wed, 11 Dec 2013 09:17:39 +0000 | From | Lee Jones <> | Subject | Re: [PATCH v3 07/15] mfd: menelaus: Limit the usage of the_menelaus |
| |
On Tue, 10 Dec 2013, Felipe Balbi wrote:
> On Tue, Dec 10, 2013 at 08:57:01AM +0000, Lee Jones wrote: > > > Pass a menelaus_chip pointer as argument to most functions so we can > > > minimize the usage of the global the_menelaus pointer. > > > > > > Tested-by: Aaro Koskinen <aaro.koskinen@iki.fi> > > > Signed-off-by: Felipe Balbi <balbi@ti.com> > > > --- > > > drivers/mfd/menelaus.c | 265 ++++++++++++++++++++++++++----------------------- > > > 1 file changed, 142 insertions(+), 123 deletions(-) > > > > How is this different from patch 6? > > have you read the patch ? patch 6 converts *only* enable/disable irq > functions.
Well it actually converts menelaus_ack_irq() too, but that's not the point I was making.
I can see that the 'code' is different, but the functionality of the patch is the same i.e. converting functions to pass around the menelaus_chip pointer for eradication of the_menelaus global pointer.
I'm not concerned about the 'lines changed' count. The change your making is trivial, thus there is no requirement to split unnecessarily over 5 patches. Please squash them into a single functionality patch.
-- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |