diff mbox series

[net-next] sungem: Prepare cleanup of powerpc's asm/prom.h

Message ID fa778bf9c0a23df8a9e6fe2e2b20d936bd0a89af.1648833433.git.christophe.leroy@csgroup.eu (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series [net-next] sungem: Prepare cleanup of powerpc's asm/prom.h | expand

Checks

Context Check Description
netdev/tree_selection success Clearly marked for net-next
netdev/fixes_present success Fixes tag not required for -next series
netdev/subject_prefix success Link
netdev/cover_letter success Single patches do not need cover letters
netdev/patch_count success Link
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 0 this patch: 0
netdev/cc_maintainers success CCed 4 of 4 maintainers
netdev/build_clang success Errors and warnings before: 0 this patch: 0
netdev/module_param success Was 0 now: 0
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 0 this patch: 0
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 12 lines checked
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Christophe Leroy April 2, 2022, 10:17 a.m. UTC
powerpc's asm/prom.h brings some headers that it doesn't
need itself.

In order to clean it up, first add missing headers in
users of asm/prom.h

Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
---
 drivers/net/sungem_phy.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Comments

Jakub Kicinski April 5, 2022, 8:22 p.m. UTC | #1
On Sat,  2 Apr 2022 12:17:13 +0200 Christophe Leroy wrote:
> powerpc's asm/prom.h brings some headers that it doesn't
> need itself.
> 
> In order to clean it up, first add missing headers in
> users of asm/prom.h

Could you resend the net-next patches you had?

They got dropped from patchwork due to net-next being closed during 
the merge window.
Christophe Leroy April 6, 2022, 5:53 a.m. UTC | #2
Le 05/04/2022 à 22:22, Jakub Kicinski a écrit :
> On Sat,  2 Apr 2022 12:17:13 +0200 Christophe Leroy wrote:
>> powerpc's asm/prom.h brings some headers that it doesn't
>> need itself.
>>
>> In order to clean it up, first add missing headers in
>> users of asm/prom.h
> 
> Could you resend the net-next patches you had?

Sure I can but,

> 
> They got dropped from patchwork due to net-next being closed during
> the merge window.

https://patchwork.kernel.org/project/netdevbpf/list/?series=&submitter=192363&state=*&q=&archive=&delegate=

As far as I can see they are in patchwork and two of them have been 
accepted, and this one is tagged as 'deferred', so do I have to resend it ?

Thanks
Christophe
Jakub Kicinski April 6, 2022, 4:39 p.m. UTC | #3
On Wed, 6 Apr 2022 05:53:46 +0000 Christophe Leroy wrote:
> Le 05/04/2022 à 22:22, Jakub Kicinski a écrit :
> > On Sat,  2 Apr 2022 12:17:13 +0200 Christophe Leroy wrote:  
> >> powerpc's asm/prom.h brings some headers that it doesn't
> >> need itself.
> >>
> >> In order to clean it up, first add missing headers in
> >> users of asm/prom.h  
> > 
> > Could you resend the net-next patches you had?  
> 
> Sure I can but,
> 
> > 
> > They got dropped from patchwork due to net-next being closed during
> > the merge window.  
> 
> https://patchwork.kernel.org/project/netdevbpf/list/?series=&submitter=192363&state=*&q=&archive=&delegate=
> 
> As far as I can see they are in patchwork and two of them have been 
> accepted, and this one is tagged as 'deferred', so do I have to resend it ?

Erm, perhaps a clerical error? I don't see them in the tree.
I'd resend both.
Christophe Leroy April 15, 2022, 8:45 a.m. UTC | #4
Le 05/04/2022 à 22:22, Jakub Kicinski a écrit :
> On Sat,  2 Apr 2022 12:17:13 +0200 Christophe Leroy wrote:
>> powerpc's asm/prom.h brings some headers that it doesn't
>> need itself.
>>
>> In order to clean it up, first add missing headers in
>> users of asm/prom.h
> 
> Could you resend the net-next patches you had?
> 
> They got dropped from patchwork due to net-next being closed during
> the merge window.

Ok, I sent v2, with a bit more detailed description.

Thanks
Christophe
diff mbox series

Patch

diff --git a/drivers/net/sungem_phy.c b/drivers/net/sungem_phy.c
index 4daac5fda073..ff22b6b1c686 100644
--- a/drivers/net/sungem_phy.c
+++ b/drivers/net/sungem_phy.c
@@ -29,11 +29,7 @@ 
 #include <linux/mii.h>
 #include <linux/ethtool.h>
 #include <linux/delay.h>
-
-#ifdef CONFIG_PPC_PMAC
-#include <asm/prom.h>
-#endif
-
+#include <linux/of.h>
 #include <linux/sungem_phy.h>
 
 /* Link modes of the BCM5400 PHY */