In commit 6e30a66433af ("driver core: class: remove struct module owner
out of struct class"), the module owner pointer was removed from struct
class, but this was missed for the mips vpe-cmp code due to lack of
build testing (and it being burried under a very unused config
settings.) Fix this up by removing the module pointer to resolve the
build error.
Note, there are other problems with the driver model usage in this file
(static struct device usage, empty device release function, etc.), so it
probably could use some good cleaning up, but odds are this driver
really isn't used so hopefully it will just be removed entirely someday
soon as part of the general "remove unused arches" cleanup that is
slowly happening.
Cc: Dengcheng Zhu <[email protected]>
Cc: John Crispin <[email protected]>
Cc: Qais Yousef <[email protected]>
Cc: Steven J. Hill <[email protected]>
Cc: Thomas Bogendoerfer <[email protected]>
Cc: Yang Yingliang <[email protected]>
Cc: [email protected]
Reported-by: kernel test robot <[email protected]>
Link: https://lore.kernel.org/oe-kbuild-all/[email protected]/
Fixes: 6e30a66433af ("driver core: class: remove struct module owner out of struct class")
Signed-off-by: Greg Kroah-Hartman <[email protected]>
---
arch/mips/kernel/vpe-cmp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/mips/kernel/vpe-cmp.c b/arch/mips/kernel/vpe-cmp.c
index 92140edb3ce3..4ef7f49a4c6d 100644
--- a/arch/mips/kernel/vpe-cmp.c
+++ b/arch/mips/kernel/vpe-cmp.c
@@ -79,7 +79,6 @@ static void vpe_device_release(struct device *cd)
static struct class vpe_class = {
.name = "vpe",
- .owner = THIS_MODULE,
.dev_release = vpe_device_release,
.dev_groups = vpe_groups,
};
--
2.40.0
On Sun, Apr 02, 2023 at 11:05:45AM +0200, Greg Kroah-Hartman wrote:
> In commit 6e30a66433af ("driver core: class: remove struct module owner
> out of struct class"), the module owner pointer was removed from struct
> class, but this was missed for the mips vpe-cmp code due to lack of
> build testing (and it being burried under a very unused config
> settings.) Fix this up by removing the module pointer to resolve the
> build error.
>
> Note, there are other problems with the driver model usage in this file
> (static struct device usage, empty device release function, etc.), so it
> probably could use some good cleaning up, but odds are this driver
> really isn't used so hopefully it will just be removed entirely someday
> soon as part of the general "remove unused arches" cleanup that is
> slowly happening.
>
> Cc: Dengcheng Zhu <[email protected]>
> Cc: John Crispin <[email protected]>
> Cc: Qais Yousef <[email protected]>
> Cc: Steven J. Hill <[email protected]>
> Cc: Thomas Bogendoerfer <[email protected]>
> Cc: Yang Yingliang <[email protected]>
> Cc: [email protected]
> Reported-by: kernel test robot <[email protected]>
> Link: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> Fixes: 6e30a66433af ("driver core: class: remove struct module owner out of struct class")
> Signed-off-by: Greg Kroah-Hartman <[email protected]>
> ---
> arch/mips/kernel/vpe-cmp.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/mips/kernel/vpe-cmp.c b/arch/mips/kernel/vpe-cmp.c
> index 92140edb3ce3..4ef7f49a4c6d 100644
> --- a/arch/mips/kernel/vpe-cmp.c
> +++ b/arch/mips/kernel/vpe-cmp.c
> @@ -79,7 +79,6 @@ static void vpe_device_release(struct device *cd)
>
> static struct class vpe_class = {
> .name = "vpe",
> - .owner = THIS_MODULE,
> .dev_release = vpe_device_release,
> .dev_groups = vpe_groups,
> };
> --
> 2.40.0
I guess this should go the same way as commit 6e30a66433af, therefore
Acked-by: Thomas Bogendoerfer <[email protected]>
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
On Mon, Apr 03, 2023 at 05:58:04PM +0200, Thomas Bogendoerfer wrote:
> On Sun, Apr 02, 2023 at 11:05:45AM +0200, Greg Kroah-Hartman wrote:
> > In commit 6e30a66433af ("driver core: class: remove struct module owner
> > out of struct class"), the module owner pointer was removed from struct
> > class, but this was missed for the mips vpe-cmp code due to lack of
> > build testing (and it being burried under a very unused config
> > settings.) Fix this up by removing the module pointer to resolve the
> > build error.
> >
> > Note, there are other problems with the driver model usage in this file
> > (static struct device usage, empty device release function, etc.), so it
> > probably could use some good cleaning up, but odds are this driver
> > really isn't used so hopefully it will just be removed entirely someday
> > soon as part of the general "remove unused arches" cleanup that is
> > slowly happening.
> >
> > Cc: Dengcheng Zhu <[email protected]>
> > Cc: John Crispin <[email protected]>
> > Cc: Qais Yousef <[email protected]>
> > Cc: Steven J. Hill <[email protected]>
> > Cc: Thomas Bogendoerfer <[email protected]>
> > Cc: Yang Yingliang <[email protected]>
> > Cc: [email protected]
> > Reported-by: kernel test robot <[email protected]>
> > Link: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> > Fixes: 6e30a66433af ("driver core: class: remove struct module owner out of struct class")
> > Signed-off-by: Greg Kroah-Hartman <[email protected]>
> > ---
> > arch/mips/kernel/vpe-cmp.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/arch/mips/kernel/vpe-cmp.c b/arch/mips/kernel/vpe-cmp.c
> > index 92140edb3ce3..4ef7f49a4c6d 100644
> > --- a/arch/mips/kernel/vpe-cmp.c
> > +++ b/arch/mips/kernel/vpe-cmp.c
> > @@ -79,7 +79,6 @@ static void vpe_device_release(struct device *cd)
> >
> > static struct class vpe_class = {
> > .name = "vpe",
> > - .owner = THIS_MODULE,
> > .dev_release = vpe_device_release,
> > .dev_groups = vpe_groups,
> > };
> > --
> > 2.40.0
>
> I guess this should go the same way as commit 6e30a66433af, therefore
>
> Acked-by: Thomas Bogendoerfer <[email protected]>
Thanks, I'll take it through my tree.
greg k-h