Hi,
people found out that for platform_driver, we don't need to set the
.owner field because this is done by the platform driver core. So far,
so good. However, now I got patches removing the .owner field for this
single i2c driver or for that one. To prevent getting thousands of
patches fixing si
Replace "Assignment of bool to 0/1" by "Assignment of 0/1 to bool"
Suggested-by: Dan Mick
Signed-off-by: Fabian Frederick
---
scripts/coccinelle/misc/boolinit.cocci | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/scripts/coccinelle/misc/boolinit.cocci
b/scripts/cocc
> @match1@
> declarer name module_platform_driver;
> declarer name module_platform_driver_probe;
> identifier __driver;
> @@
> (
> module_platform_driver(__driver);
> |
> module_platform_driver_probe(__driver, ...);
> )
>
> @fix1 depends on match1@
> identifier match1.__driver;
> @@
>
Acked-by: Julia Lawall
Thanks!
On Fri, 10 Oct 2014, Fabian Frederick wrote:
> Replace "Assignment of bool to 0/1" by "Assignment of 0/1 to bool"
>
> Suggested-by: Dan Mick
> Signed-off-by: Fabian Frederick
> ---
> scripts/coccinelle/misc/boolinit.cocci | 8
> 1 file changed, 4 inser
On Friday 10 October 2014 09:24:39 Wolfram Sang wrote:
> people found out that for platform_driver, we don't need to set the
> .owner field because this is done by the platform driver core. So far,
> so good. However, now I got patches removing the .owner field for this
> single i2c driver or for t
On Fri, Oct 10, 2014 at 09:24:39AM +0200, Wolfram Sang wrote:
> people found out that for platform_driver, we don't need to set the
> .owner field because this is done by the platform driver core. So far,
> so good. However, now I got patches removing the .owner field for this
> single i2c driver o
> The semantic patch looks fine.
Wow, nothing to improve on the semantic patch? Now I am proud :) Thanks
Julia for your support, as always!
> If you think that it would be useful to have this in the Linux kernel, so
> people don't add the owner initializer back in the future, you can try
>
> co
Hi Arnd,
thanks for taking a look!
On Fri, Oct 10, 2014 at 10:30:08AM +0200, Arnd Bergmann wrote:
> On Friday 10 October 2014 09:24:39 Wolfram Sang wrote:
> > people found out that for platform_driver, we don't need to set the
> > .owner field because this is done by the platform driver core. So
On Fri, Oct 10, 2014 at 09:36:27AM +0100, Russell King - ARM Linux wrote:
> On Fri, Oct 10, 2014 at 09:24:39AM +0200, Wolfram Sang wrote:
> > people found out that for platform_driver, we don't need to set the
> > .owner field because this is done by the platform driver core. So far,
> > so good. H
On Friday 10 October 2014 20:12:21 Wolfram Sang wrote:
> Hi Arnd,
>
> thanks for taking a look!
>
> On Fri, Oct 10, 2014 at 10:30:08AM +0200, Arnd Bergmann wrote:
> > On Friday 10 October 2014 09:24:39 Wolfram Sang wrote:
> > > people found out that for platform_driver, we don't need to set the
>
On Friday 10 October 2014 20:26:05 Wolfram Sang wrote:
>
> You got me wondering, though, that it could not be correct to call
> platform_driver_register() from the platform core instead of module
> init. I will check tomorrow. Still, this would be a bug independent of
> my series. Although I'd nee
On Fri, Oct 10, 2014 at 08:26:05PM +0200, Wolfram Sang wrote:
> platform_create_bundle() calls platform_driver_probe().
> platform_driver_probe() calls platform_driver_register().
> platform_driver_register() modifies driver.owner.
>
> So, it is correct from the point of view that it doesn't make
12 matches
Mail list logo