Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-16 Thread Lee Jones
Are you planning on re-submitting this patch-set anytime soon? I thought Tony said it was important? -- 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

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-11 Thread Uwe Kleine-König
On Mon, Dec 02, 2013 at 09:42:54PM -0600, Felipe Balbi wrote: we could build that as a driver. What is that. How can it not be a driver? Do you mean modular? In that case there is no problem, the only thing that doesn't work is unloading the module. Best regards Uwe Signed-off-by: Felipe

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-10 Thread Lee Jones
The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. example ? All of your commit messages. that macro just helps removing some extra ^-

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-10 Thread Felipe Balbi
Hi, On Tue, Dec 10, 2013 at 08:50:07AM +, Lee Jones wrote: The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. example ? All of your commit

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-09 Thread Lee Jones
The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. example ? All of your commit messages. that macro just helps removing some extra ^- Sentences start with an

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-09 Thread Felipe Balbi
Hi, On Mon, Dec 09, 2013 at 09:37:48AM +, Lee Jones wrote: The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. example ? All of your commit messages.

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-08 Thread Felipe Balbi
On Tue, Dec 03, 2013 at 09:51:36AM +, Lee Jones wrote: The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. example ? -- balbi signature.asc Description: Digital

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-06 Thread Aaro Koskinen
Hi, For the whole series: Tested-by: Aaro Koskinen aaro.koski...@iki.fi A. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-03 Thread Lee Jones
The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro:

Re: [PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-03 Thread Lee Jones
The code looks mostly fine, but the implementation of the commit logs seems lazy. Please submit a v3 using coherent sentences with full explanations and correct punctuation. Also use the full length of the provided buffer, which I believe is 72 chars, but happy to be correct on that one. It's

[PATCH v2 01/15] mfd: menelaus: Drop __exit section annotation

2013-12-02 Thread Felipe Balbi
we could build that as a driver. Signed-off-by: Felipe Balbi ba...@ti.com --- drivers/mfd/menelaus.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mfd/menelaus.c b/drivers/mfd/menelaus.c index ad25bfa..975ff9e 100644 --- a/drivers/mfd/menelaus.c +++