* [net-next:master 37/50] net/ipv6/sit.c:1243:34: sparse: incorrect type in assignment (different base types)
From: kbuild test robot @ 2012-11-15 5:42 UTC (permalink / raw)
To: Nicolas Dichtel; +Cc: netdev
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
head: 702ed3c1a9dfe4dfe112f13542d0c9d689f5008b
commit: f37234160233561f2a2e3332272ae5b3725b620b [37/50] sit: add support of link creation via rtnl
sparse warnings:
net/ipv6/sit.c:231:21: sparse: restricted __be16 degrades to integer
+ net/ipv6/sit.c:1243:34: sparse: incorrect type in assignment (different base types)
net/ipv6/sit.c:1243:34: expected restricted __be32 [usertype] saddr
net/ipv6/sit.c:1243:34: got unsigned int
net/ipv6/sit.c:1246:34: sparse: incorrect type in assignment (different base types)
net/ipv6/sit.c:1246:34: expected restricted __be32 [usertype] daddr
net/ipv6/sit.c:1246:34: got unsigned int
net/ipv6/sit.c:1261:32: sparse: incorrect type in assignment (different base types)
net/ipv6/sit.c:1261:32: expected restricted __be16 [usertype] i_flags
net/ipv6/sit.c:1261:32: got unsigned short
net/ipv6/sit.c:1340:52: sparse: incorrect type in argument 3 (different base types)
net/ipv6/sit.c:1340:52: expected unsigned short [unsigned] [usertype] value
net/ipv6/sit.c:1340:52: got restricted __be16 [usertype] i_flags
vim +1243 net/ipv6/sit.c
f3723416 Nicolas Dichtel 2012-11-14 1227 struct ip_tunnel_parm *parms)
f3723416 Nicolas Dichtel 2012-11-14 1228 {
f3723416 Nicolas Dichtel 2012-11-14 1229 memset(parms, 0, sizeof(*parms));
f3723416 Nicolas Dichtel 2012-11-14 1230
f3723416 Nicolas Dichtel 2012-11-14 1231 parms->iph.version = 4;
f3723416 Nicolas Dichtel 2012-11-14 1232 parms->iph.protocol = IPPROTO_IPV6;
f3723416 Nicolas Dichtel 2012-11-14 1233 parms->iph.ihl = 5;
f3723416 Nicolas Dichtel 2012-11-14 1234 parms->iph.ttl = 64;
f3723416 Nicolas Dichtel 2012-11-14 1235
f3723416 Nicolas Dichtel 2012-11-14 1236 if (!data)
f3723416 Nicolas Dichtel 2012-11-14 1237 return;
f3723416 Nicolas Dichtel 2012-11-14 1238
f3723416 Nicolas Dichtel 2012-11-14 1239 if (data[IFLA_IPTUN_LINK])
f3723416 Nicolas Dichtel 2012-11-14 1240 parms->link = nla_get_u32(data[IFLA_IPTUN_LINK]);
f3723416 Nicolas Dichtel 2012-11-14 1241
f3723416 Nicolas Dichtel 2012-11-14 1242 if (data[IFLA_IPTUN_LOCAL])
f3723416 Nicolas Dichtel 2012-11-14 @1243 parms->iph.saddr = nla_get_u32(data[IFLA_IPTUN_LOCAL]);
f3723416 Nicolas Dichtel 2012-11-14 1244
f3723416 Nicolas Dichtel 2012-11-14 1245 if (data[IFLA_IPTUN_REMOTE])
f3723416 Nicolas Dichtel 2012-11-14 1246 parms->iph.daddr = nla_get_u32(data[IFLA_IPTUN_REMOTE]);
f3723416 Nicolas Dichtel 2012-11-14 1247
f3723416 Nicolas Dichtel 2012-11-14 1248 if (data[IFLA_IPTUN_TTL]) {
f3723416 Nicolas Dichtel 2012-11-14 1249 parms->iph.ttl = nla_get_u8(data[IFLA_IPTUN_TTL]);
f3723416 Nicolas Dichtel 2012-11-14 1250 if (parms->iph.ttl)
f3723416 Nicolas Dichtel 2012-11-14 1251 parms->iph.frag_off = htons(IP_DF);
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu Intel Corporation
^ permalink raw reply
* [net-next:master 35/50] net/ipv6/sit.c:1232:52: sparse: incorrect type in argument 3 (different base types)
From: kbuild test robot @ 2012-11-15 5:31 UTC (permalink / raw)
To: Nicolas Dichtel; +Cc: netdev
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
head: 702ed3c1a9dfe4dfe112f13542d0c9d689f5008b
commit: a12c9a85813e927e1143989876d70697eb85aac9 [35/50] sit/rtnl: add missing parameters on dump
sparse warnings:
net/ipv6/sit.c:263:18: sparse: restricted __be16 degrades to integer
+ net/ipv6/sit.c:1232:52: sparse: incorrect type in argument 3 (different base types)
net/ipv6/sit.c:1232:52: expected unsigned short [unsigned] [usertype] value
net/ipv6/sit.c:1232:52: got restricted __be16 [usertype] i_flags
vim +1232 net/ipv6/sit.c
a12c9a85 Nicolas Dichtel 2012-11-14 1216 nla_total_size(2) +
ba3e3f50 Nicolas Dichtel 2012-11-09 1217 0;
ba3e3f50 Nicolas Dichtel 2012-11-09 1218 }
ba3e3f50 Nicolas Dichtel 2012-11-09 1219
ba3e3f50 Nicolas Dichtel 2012-11-09 1220 static int sit_fill_info(struct sk_buff *skb, const struct net_device *dev)
ba3e3f50 Nicolas Dichtel 2012-11-09 1221 {
ba3e3f50 Nicolas Dichtel 2012-11-09 1222 struct ip_tunnel *tunnel = netdev_priv(dev);
ba3e3f50 Nicolas Dichtel 2012-11-09 1223 struct ip_tunnel_parm *parm = &tunnel->parms;
ba3e3f50 Nicolas Dichtel 2012-11-09 1224
ba3e3f50 Nicolas Dichtel 2012-11-09 1225 if (nla_put_u32(skb, IFLA_IPTUN_LINK, parm->link) ||
ba3e3f50 Nicolas Dichtel 2012-11-09 1226 nla_put_be32(skb, IFLA_IPTUN_LOCAL, parm->iph.saddr) ||
ba3e3f50 Nicolas Dichtel 2012-11-09 1227 nla_put_be32(skb, IFLA_IPTUN_REMOTE, parm->iph.daddr) ||
ba3e3f50 Nicolas Dichtel 2012-11-09 1228 nla_put_u8(skb, IFLA_IPTUN_TTL, parm->iph.ttl) ||
a12c9a85 Nicolas Dichtel 2012-11-14 1229 nla_put_u8(skb, IFLA_IPTUN_TOS, parm->iph.tos) ||
a12c9a85 Nicolas Dichtel 2012-11-14 1230 nla_put_u8(skb, IFLA_IPTUN_PMTUDISC,
a12c9a85 Nicolas Dichtel 2012-11-14 1231 !!(parm->iph.frag_off & htons(IP_DF))) ||
a12c9a85 Nicolas Dichtel 2012-11-14 @1232 nla_put_u16(skb, IFLA_IPTUN_FLAGS, parm->i_flags))
ba3e3f50 Nicolas Dichtel 2012-11-09 1233 goto nla_put_failure;
ba3e3f50 Nicolas Dichtel 2012-11-09 1234 return 0;
ba3e3f50 Nicolas Dichtel 2012-11-09 1235
ba3e3f50 Nicolas Dichtel 2012-11-09 1236 nla_put_failure:
ba3e3f50 Nicolas Dichtel 2012-11-09 1237 return -EMSGSIZE;
ba3e3f50 Nicolas Dichtel 2012-11-09 1238 }
ba3e3f50 Nicolas Dichtel 2012-11-09 1239
ba3e3f50 Nicolas Dichtel 2012-11-09 1240 static struct rtnl_link_ops sit_link_ops __read_mostly = {
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu Intel Corporation
^ permalink raw reply
* [net-next:master 33/50] net/ipv4/ipip.c:867:34: sparse: incorrect type in assignment (different base types)
From: kbuild test robot @ 2012-11-15 5:16 UTC (permalink / raw)
To: Nicolas Dichtel; +Cc: netdev
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
head: 702ed3c1a9dfe4dfe112f13542d0c9d689f5008b
commit: be42da0e1012bf67d8f6899b7d9162e35527da4b [33/50] ipip: add support of link creation via rtnl
sparse warnings:
+ net/ipv4/ipip.c:867:34: sparse: incorrect type in assignment (different base types)
net/ipv4/ipip.c:867:34: expected restricted __be32 [usertype] saddr
net/ipv4/ipip.c:867:34: got unsigned int
net/ipv4/ipip.c:870:34: sparse: incorrect type in assignment (different base types)
net/ipv4/ipip.c:870:34: expected restricted __be32 [usertype] daddr
net/ipv4/ipip.c:870:34: got unsigned int
vim +867 net/ipv4/ipip.c
be42da0e Nicolas Dichtel 2012-11-14 851 static void ipip_netlink_parms(struct nlattr *data[],
be42da0e Nicolas Dichtel 2012-11-14 852 struct ip_tunnel_parm *parms)
be42da0e Nicolas Dichtel 2012-11-14 853 {
be42da0e Nicolas Dichtel 2012-11-14 854 memset(parms, 0, sizeof(*parms));
be42da0e Nicolas Dichtel 2012-11-14 855
be42da0e Nicolas Dichtel 2012-11-14 856 parms->iph.version = 4;
be42da0e Nicolas Dichtel 2012-11-14 857 parms->iph.protocol = IPPROTO_IPIP;
be42da0e Nicolas Dichtel 2012-11-14 858 parms->iph.ihl = 5;
be42da0e Nicolas Dichtel 2012-11-14 859
be42da0e Nicolas Dichtel 2012-11-14 860 if (!data)
be42da0e Nicolas Dichtel 2012-11-14 861 return;
be42da0e Nicolas Dichtel 2012-11-14 862
be42da0e Nicolas Dichtel 2012-11-14 863 if (data[IFLA_IPTUN_LINK])
be42da0e Nicolas Dichtel 2012-11-14 864 parms->link = nla_get_u32(data[IFLA_IPTUN_LINK]);
be42da0e Nicolas Dichtel 2012-11-14 865
be42da0e Nicolas Dichtel 2012-11-14 866 if (data[IFLA_IPTUN_LOCAL])
be42da0e Nicolas Dichtel 2012-11-14 @867 parms->iph.saddr = nla_get_u32(data[IFLA_IPTUN_LOCAL]);
be42da0e Nicolas Dichtel 2012-11-14 868
be42da0e Nicolas Dichtel 2012-11-14 869 if (data[IFLA_IPTUN_REMOTE])
be42da0e Nicolas Dichtel 2012-11-14 870 parms->iph.daddr = nla_get_u32(data[IFLA_IPTUN_REMOTE]);
be42da0e Nicolas Dichtel 2012-11-14 871
be42da0e Nicolas Dichtel 2012-11-14 872 if (data[IFLA_IPTUN_TTL]) {
be42da0e Nicolas Dichtel 2012-11-14 873 parms->iph.ttl = nla_get_u8(data[IFLA_IPTUN_TTL]);
be42da0e Nicolas Dichtel 2012-11-14 874 if (parms->iph.ttl)
be42da0e Nicolas Dichtel 2012-11-14 875 parms->iph.frag_off = htons(IP_DF);
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu Intel Corporation
^ permalink raw reply
* [net-next:master 30/50] net/ipv6/ip6_tunnel.c:1571:33: sparse: incorrect type in assignment (different base types)
From: kbuild test robot @ 2012-11-15 4:53 UTC (permalink / raw)
To: Nicolas Dichtel; +Cc: netdev
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
head: 702ed3c1a9dfe4dfe112f13542d0c9d689f5008b
commit: 0b112457229d8a17198a02f3cca32922d2e374f1 [30/50] ip6tnl: add support of link creation via rtnl
sparse warnings:
+ net/ipv6/ip6_tunnel.c:1571:33: sparse: incorrect type in assignment (different base types)
net/ipv6/ip6_tunnel.c:1571:33: expected restricted __be32 [usertype] flowinfo
net/ipv6/ip6_tunnel.c:1571:33: got unsigned int
vim +1571 net/ipv6/ip6_tunnel.c
0b112457 Nicolas Dichtel 2012-11-14 1555
0b112457 Nicolas Dichtel 2012-11-14 1556 if (data[IFLA_IPTUN_LOCAL])
0b112457 Nicolas Dichtel 2012-11-14 1557 nla_memcpy(&parms->laddr, data[IFLA_IPTUN_LOCAL],
0b112457 Nicolas Dichtel 2012-11-14 1558 sizeof(struct in6_addr));
0b112457 Nicolas Dichtel 2012-11-14 1559
0b112457 Nicolas Dichtel 2012-11-14 1560 if (data[IFLA_IPTUN_REMOTE])
0b112457 Nicolas Dichtel 2012-11-14 1561 nla_memcpy(&parms->raddr, data[IFLA_IPTUN_REMOTE],
0b112457 Nicolas Dichtel 2012-11-14 1562 sizeof(struct in6_addr));
0b112457 Nicolas Dichtel 2012-11-14 1563
0b112457 Nicolas Dichtel 2012-11-14 1564 if (data[IFLA_IPTUN_TTL])
0b112457 Nicolas Dichtel 2012-11-14 1565 parms->hop_limit = nla_get_u8(data[IFLA_IPTUN_TTL]);
0b112457 Nicolas Dichtel 2012-11-14 1566
0b112457 Nicolas Dichtel 2012-11-14 1567 if (data[IFLA_IPTUN_ENCAP_LIMIT])
0b112457 Nicolas Dichtel 2012-11-14 1568 parms->encap_limit = nla_get_u8(data[IFLA_IPTUN_ENCAP_LIMIT]);
0b112457 Nicolas Dichtel 2012-11-14 1569
0b112457 Nicolas Dichtel 2012-11-14 1570 if (data[IFLA_IPTUN_FLOWINFO])
0b112457 Nicolas Dichtel 2012-11-14 @1571 parms->flowinfo = nla_get_u32(data[IFLA_IPTUN_FLOWINFO]);
0b112457 Nicolas Dichtel 2012-11-14 1572
0b112457 Nicolas Dichtel 2012-11-14 1573 if (data[IFLA_IPTUN_FLAGS])
0b112457 Nicolas Dichtel 2012-11-14 1574 parms->flags = nla_get_u32(data[IFLA_IPTUN_FLAGS]);
0b112457 Nicolas Dichtel 2012-11-14 1575
0b112457 Nicolas Dichtel 2012-11-14 1576 if (data[IFLA_IPTUN_PROTO])
0b112457 Nicolas Dichtel 2012-11-14 1577 parms->proto = nla_get_u8(data[IFLA_IPTUN_PROTO]);
0b112457 Nicolas Dichtel 2012-11-14 1578 }
0b112457 Nicolas Dichtel 2012-11-14 1579
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu Intel Corporation
^ permalink raw reply
* Re: [PATCHES] Networking fixes for -stable.
From: Greg KH @ 2012-11-15 4:06 UTC (permalink / raw)
To: David Miller; +Cc: ben, peter.senna, stable, netdev
In-Reply-To: <20121114.223145.1718907430767854932.davem@davemloft.net>
On Wed, Nov 14, 2012 at 10:31:45PM -0500, David Miller wrote:
> From: Ben Hutchings <ben@decadent.org.uk>
> Date: Tue, 13 Nov 2012 19:51:26 +0000
>
> > On Mon, 2012-11-12 at 00:25 -0500, David Miller wrote:
> >> Please queue up the following networking bug fixes for
> >> 3.0.x, 3.2.x, 3.4.x, and 3.6.x, respectively.
> > [...]
> >> From 2204849a85383fbde75680aa199142abe504adbb Mon Sep 17 00:00:00 2001
> >> From: Peter Senna Tschudin <peter.senna@gmail.com>
> >> Date: Sun, 28 Oct 2012 06:12:01 +0000
> >> Subject: [PATCH 7/9] drivers/net/phy/mdio-bitbang.c: Call mdiobus_unregister
> >> before mdiobus_free
> >>
> >> [ Upstream commit aa731872f7d33dcb8b54dad0cfb82d4e4d195d7e ]
> >>
> >> Based on commit b27393aecf66199f5ddad37c302d3e0cfadbe6c0
> >>
> >> Calling mdiobus_free without calling mdiobus_unregister causes
> >> BUG_ON(). This patch fixes the issue.
> > [...]
> >
> > This introduces a regresssion, as mdiobus_unregister() is not safe to
> > call if the bus isn't registered. Registration is controlled by the
> > drivers that use mdio-bitbang, and they should take care of
> > unregistration too - and most of them do.
> >
> > This should be reverted in mainline and not applied to any stable
> > series.
>
> Ok, I'll revert.
>
> Greg, please toss it from your -stable queue as well.
Now tossed, thanks.
greg k-h
^ permalink raw reply
* Re: Latest 3.6.6 are not compiling due tg3 network driver, hwmon_device_unregister
From: Nithin Nayak Sujir @ 2012-11-15 3:43 UTC (permalink / raw)
To: David Rientjes
Cc: Paul Gortmaker, Denys Fedoryshchenko, Michael Chan, netdev,
linux-kernel
In-Reply-To: <alpine.DEB.2.00.1211141929510.14414@chino.kir.corp.google.com>
On 11/14/2012 07:30 PM, David Rientjes wrote:
> On Wed, 14 Nov 2012, Nithin Nayak Sujir wrote:
>
>> This was fixed by
>>
>> commit de0a41484c47d783dd4d442914815076aa2caac2
>> Author: Paul Gortmaker <paul.gortmaker@windriver.com>
>> Date: Mon Oct 1 11:43:49 2012 -0400
>>
>> tg3: unconditionally select HWMON support when tg3 is enabled.
>>
> Would you mind submitting this for stable by following the procedure
> described in Documentation/stable_kernel_rules.txt?
>
Will do. Thank you for bringing this to our attention.
Nithin.
^ permalink raw reply
* [PATCH] Revert "drivers/net/phy/mdio-bitbang.c: Call mdiobus_unregister before mdiobus_free"
From: David Miller @ 2012-11-15 3:34 UTC (permalink / raw)
To: netdev
This reverts commit aa731872f7d33dcb8b54dad0cfb82d4e4d195d7e.
As pointed out by Ben Hutchings, this change is not correct.
mdiobus_unregister() can't be called if the bus isn't registered yet,
however this change can result in situations which cause that to
happen.
Part of the confusion here revolves around the fact that the
callers of this module control registration/unregistration,
rather than the module itself.
Signed-off-by: David S. Miller <davem@davemloft.net>
---
drivers/net/phy/mdio-bitbang.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/phy/mdio-bitbang.c b/drivers/net/phy/mdio-bitbang.c
index 6428fcb..daec9b0 100644
--- a/drivers/net/phy/mdio-bitbang.c
+++ b/drivers/net/phy/mdio-bitbang.c
@@ -234,7 +234,6 @@ void free_mdio_bitbang(struct mii_bus *bus)
struct mdiobb_ctrl *ctrl = bus->priv;
module_put(ctrl->ops->owner);
- mdiobus_unregister(bus);
mdiobus_free(bus);
}
EXPORT_SYMBOL(free_mdio_bitbang);
--
1.7.11.7
^ permalink raw reply related
* Re: [PATCHES] Networking fixes for -stable.
From: David Miller @ 2012-11-15 3:31 UTC (permalink / raw)
To: ben; +Cc: gregkh, peter.senna, stable, netdev
In-Reply-To: <1352836286.16264.102.camel@deadeye.wl.decadent.org.uk>
From: Ben Hutchings <ben@decadent.org.uk>
Date: Tue, 13 Nov 2012 19:51:26 +0000
> On Mon, 2012-11-12 at 00:25 -0500, David Miller wrote:
>> Please queue up the following networking bug fixes for
>> 3.0.x, 3.2.x, 3.4.x, and 3.6.x, respectively.
> [...]
>> From 2204849a85383fbde75680aa199142abe504adbb Mon Sep 17 00:00:00 2001
>> From: Peter Senna Tschudin <peter.senna@gmail.com>
>> Date: Sun, 28 Oct 2012 06:12:01 +0000
>> Subject: [PATCH 7/9] drivers/net/phy/mdio-bitbang.c: Call mdiobus_unregister
>> before mdiobus_free
>>
>> [ Upstream commit aa731872f7d33dcb8b54dad0cfb82d4e4d195d7e ]
>>
>> Based on commit b27393aecf66199f5ddad37c302d3e0cfadbe6c0
>>
>> Calling mdiobus_free without calling mdiobus_unregister causes
>> BUG_ON(). This patch fixes the issue.
> [...]
>
> This introduces a regresssion, as mdiobus_unregister() is not safe to
> call if the bus isn't registered. Registration is controlled by the
> drivers that use mdio-bitbang, and they should take care of
> unregistration too - and most of them do.
>
> This should be reverted in mainline and not applied to any stable
> series.
Ok, I'll revert.
Greg, please toss it from your -stable queue as well.
Thanks!
^ permalink raw reply
* Re: Latest 3.6.6 are not compiling due tg3 network driver, hwmon_device_unregister
From: David Rientjes @ 2012-11-15 3:30 UTC (permalink / raw)
To: Nithin Nayak Sujir, Paul Gortmaker
Cc: Denys Fedoryshchenko, Michael Chan, netdev, linux-kernel
In-Reply-To: <50A4606D.4020402@broadcom.com>
On Wed, 14 Nov 2012, Nithin Nayak Sujir wrote:
> This was fixed by
>
> commit de0a41484c47d783dd4d442914815076aa2caac2
> Author: Paul Gortmaker <paul.gortmaker@windriver.com>
> Date: Mon Oct 1 11:43:49 2012 -0400
>
> tg3: unconditionally select HWMON support when tg3 is enabled.
>
Would you mind submitting this for stable by following the procedure
described in Documentation/stable_kernel_rules.txt?
^ permalink raw reply
* Re: Latest 3.6.6 are not compiling due tg3 network driver, hwmon_device_unregister
From: Nithin Nayak Sujir @ 2012-11-15 3:24 UTC (permalink / raw)
To: Denys Fedoryshchenko; +Cc: Michael Chan, netdev, linux-kernel
In-Reply-To: <8b566c4a9ee622737212dc17291ee238@visp.net.lb>
This was fixed by
commit de0a41484c47d783dd4d442914815076aa2caac2
Author: Paul Gortmaker <paul.gortmaker@windriver.com>
Date: Mon Oct 1 11:43:49 2012 -0400
tg3: unconditionally select HWMON support when tg3 is enabled.
Nithin.
On 11/14/2012 07:03 PM, Denys Fedoryshchenko wrote:
> Hi
>
> During compiling i am getting that:
>
> drivers/built-in.o: In function `tg3_close':
> tg3.c:(.text+0x12902e): undefined reference to `hwmon_device_unregister'
> drivers/built-in.o: In function `tg3_hwmon_open':
> tg3.c:(.text+0x12ae09): undefined reference to `hwmon_device_register'
>
> Kernel config are at http://nuclearcat.com/config-tg3.txt , but i noticed this error are
> appearing at x86, x64 kernels, probably also other 3.6.x series kernels.
> Disabling TG3 driver helps, but i believe it is not a solution.
>
> ---
> Denys Fedoryshchenko, Network Engineer, Virtual ISP S.A.L.
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply
* Re: [PATCH v2 0/3] macb add support for phy gpio interrupt
From: David Miller @ 2012-11-15 3:23 UTC (permalink / raw)
To: manabian; +Cc: nicolas.ferre, plagnioj, netdev
In-Reply-To: <1352678188-18647-1-git-send-email-manabian@gmail.com>
From: Joachim Eastwood <manabian@gmail.com>
Date: Mon, 12 Nov 2012 00:56:25 +0100
> Main feature of this patch set is phy gpio interrupt for macb.
...
> Patch series was tested on a custom board with DM9161AEP and AT91RM9200 EMAC.
Applied.
If there are problems found during testing we can fix or revert.
^ permalink raw reply
* Re: pull request: batman-adv 2012-11-14
From: David Miller @ 2012-11-15 3:11 UTC (permalink / raw)
To: ordex; +Cc: netdev, b.a.t.m.a.n
In-Reply-To: <1352924189-18843-1-git-send-email-ordex@autistici.org>
From: Antonio Quartulli <ordex@autistici.org>
Date: Wed, 14 Nov 2012 21:16:18 +0100
> here is again our new patchset intended for net-next/linux-3.8.
> Patch 1/11 has been modified to address the problems you pointed out last time;
> however hash_bytes has kept its inline tag and it has been moved to hash.h (in
> this way it becomes usable by the rest of the batman-adv code for new changes).
Pulled, thanks.
^ permalink raw reply
* Re: [PATCH V5 0/7] ARM: AM33XX: net: Add DT support to CPSW and MDIO driver
From: David Miller @ 2012-11-15 3:09 UTC (permalink / raw)
To: mugunthanvnm
Cc: netdev, devicetree-discuss, linux-arm-kernel, linux-omap,
b-cousson, paul
In-Reply-To: <1352920080-6179-1-git-send-email-mugunthanvnm@ti.com>
From: Mugunthan V N <mugunthanvnm@ti.com>
Date: Thu, 15 Nov 2012 00:37:53 +0530
> This patch-series adds support for,
>
> [1/7]: Typo mistake in CPSW driver while invoking runtime_pm api's
>
> [2/7]: Adds parent<->child relation between CPSW & MDIO module inside cpsw
> driver, as in case of AM33XX, the resources are shared and common
> register bit-field is provided to control module/clock enable/disable,
> makes it difficult to handle common resource.
>
> So the solution here is, to create parent<->child relation between them.
>
> [3/7]: cpsw: simplify the setup of the register pointers
>
> [4/7]: cpsw: Kernel warn fix during suspend
>
> [5/7]: Add hwmod entry for MDIO module, required for MDIO driver.
>
> [6/7]: Enable CPSW support to omap2plus_defconfig
>
> [7/7]: Add DT device nodes for both CPSW and MDIO modules in am33xx.dtsi,
> am335x-evm.dts and am335x-bone.dts file
>
> This patch series has been created on top of net-next/master and tested
> on BeagleBone platform for NFS boot and basic ping test cases.
All applied, thanks.
^ permalink raw reply
* Re: pull request: wireless-next 2012-11-14
From: David Miller @ 2012-11-15 3:07 UTC (permalink / raw)
To: linville; +Cc: linux-wireless, netdev, linux-kernel
In-Reply-To: <20121114190510.GA6128@tuxdriver.com>
From: "John W. Linville" <linville@tuxdriver.com>
Date: Wed, 14 Nov 2012 14:05:11 -0500
> This pull request is intended for the 3.8 stream...
Pulled, thanks John.
^ permalink raw reply
* Re: [PATCH net-next 4/4] tg3: Use tp->rxq_cnt when checking RSS tables.
From: David Miller @ 2012-11-15 3:05 UTC (permalink / raw)
To: mchan; +Cc: netdev, nsujir
In-Reply-To: <1352940269-23821-4-git-send-email-mchan@broadcom.com>
From: "Michael Chan" <mchan@broadcom.com>
Date: Wed, 14 Nov 2012 16:44:29 -0800
> irq_cnt is no longer reliable since rxq_cnt can be independently configured.
>
> Update version to 3.127.
>
> Signed-off-by: Michael Chan <mchan@broadcom.com>
Applied.
^ permalink raw reply
* Re: [PATCH net-next 3/4] tg3: Cleanup hardcoded ethtool test array indexes
From: David Miller @ 2012-11-15 3:05 UTC (permalink / raw)
To: mchan; +Cc: netdev, nsujir
In-Reply-To: <1352940269-23821-3-git-send-email-mchan@broadcom.com>
From: "Michael Chan" <mchan@broadcom.com>
Date: Wed, 14 Nov 2012 16:44:28 -0800
> From: Nithin Nayak Sujir <nsujir@broadcom.com>
>
> Signed-off-by: Nithin Nayak Sujir <nsujir@broadcom.com>
> Signed-off-by: Michael Chan <mchan@broadcom.com>
Applied.
^ permalink raw reply
* Re: [PATCH net-next 2/4] tg3: Prevent spurious tx timeout by setting carrier off before tx disable.
From: David Miller @ 2012-11-15 3:05 UTC (permalink / raw)
To: mchan; +Cc: netdev, nsujir
In-Reply-To: <1352940269-23821-2-git-send-email-mchan@broadcom.com>
From: "Michael Chan" <mchan@broadcom.com>
Date: Wed, 14 Nov 2012 16:44:27 -0800
> From: Nithin Nayak Sujir <nsujir@broadcom.com>
>
> The watchdog will not trigger when the carrier is off when reconfiguring
> the device. Because carrier state is now off during reset, we need to
> introduce a link_up flag to keep track of link state during PHY setup.
>
> Signed-off-by: Nithin Nayak Sujir <nsujir@broadcom.com>
> Signed-off-by: Michael Chan <mchan@broadcom.com>
Applied.
^ permalink raw reply
* Re: [PATCH net-next 1/4] tg3: Set 10_100_ONLY flag for additional 10/100 Mbps devices
From: David Miller @ 2012-11-15 3:05 UTC (permalink / raw)
To: mchan; +Cc: netdev, nsujir
In-Reply-To: <1352940269-23821-1-git-send-email-mchan@broadcom.com>
From: "Michael Chan" <mchan@broadcom.com>
Date: Wed, 14 Nov 2012 16:44:26 -0800
> From: Nithin Nayak Sujir <nsujir@broadcom.com>
>
> - Also refactor the conditional to use the existing tg3_pci_tbl array.
> - Set flags in the driver_data field of the pci_device_id structure to
> identify these devices.
> - Add PCI_DEVICE_SUB() to pci.h to declare PCI 4-part IDs to match these
> devices.
>
> Signed-off-by: Nithin Nayak Sujir <nsujir@broadcom.com>
> Signed-off-by: Michael Chan <mchan@broadcom.com>
Applied.
^ permalink raw reply
* Latest 3.6.6 are not compiling due tg3 network driver, hwmon_device_unregister
From: Denys Fedoryshchenko @ 2012-11-15 3:03 UTC (permalink / raw)
To: Matt Carlson, Michael Chan, netdev, linux-kernel
Hi
During compiling i am getting that:
drivers/built-in.o: In function `tg3_close':
tg3.c:(.text+0x12902e): undefined reference to
`hwmon_device_unregister'
drivers/built-in.o: In function `tg3_hwmon_open':
tg3.c:(.text+0x12ae09): undefined reference to `hwmon_device_register'
Kernel config are at http://nuclearcat.com/config-tg3.txt , but i
noticed this error are appearing at x86, x64 kernels, probably also
other 3.6.x series kernels.
Disabling TG3 driver helps, but i believe it is not a solution.
---
Denys Fedoryshchenko, Network Engineer, Virtual ISP S.A.L.
^ permalink raw reply
* Re: [PATCH net-next 0/10] Add support of tunnel management via rtnetlink
From: David Miller @ 2012-11-15 3:03 UTC (permalink / raw)
To: nicolas.dichtel; +Cc: netdev, eric.dumazet
In-Reply-To: <1352906047-11604-1-git-send-email-nicolas.dichtel@6wind.com>
From: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Date: Wed, 14 Nov 2012 16:13:57 +0100
> The goal of this serie is to add the support of ipip, sit and ip6tnl tunnels
> via rtnetlink.
>
> The patch against iproute2 will be sent once the patches are included and
> net-next merged. I can send it on demand.
Great work, all applied to net-next.
Thanks Nicolas!
^ permalink raw reply
* Re: [PATCH v7] Network driver for the Armada 370 and Armada XP ARM Marvell SoCs
From: David Miller @ 2012-11-15 2:59 UTC (permalink / raw)
To: thomas.petazzoni
Cc: romieu, kernel, netdev, linux-arm-kernel, jason, andrew,
gregory.clement, alior, dima
In-Reply-To: <1352905010-24172-1-git-send-email-thomas.petazzoni@free-electrons.com>
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Date: Wed, 14 Nov 2012 15:56:44 +0100
> The previous versions of this patch set have been sent on September
> 4th (v1), October 11th (v2), October 23rd (v3), October 26th (v4),
> November 12th (v5), November 13th (v6) and now comes the v7 of the
> driver. The number of comments over the last versions have been really
> small, and I would really appreciate if this driver could land into
> the 3.8 kernel release.
I can't apply this patch to net-next, because for one thing there
are missing dependencies. For example, the file:
arch/arm/boot/dts/armada-xp-openblocks-ax3-4.dts
doesn't exist there, therefore patch #6 won't apply.
^ permalink raw reply
* Re: [patch net] net: correct check in dev_addr_del()
From: David Miller @ 2012-11-15 2:52 UTC (permalink / raw)
To: jiri; +Cc: netdev, eric.dumazet, shemminger, john.r.fastabend
In-Reply-To: <1352897464-832-1-git-send-email-jiri@resnulli.us>
From: Jiri Pirko <jiri@resnulli.us>
Date: Wed, 14 Nov 2012 13:51:04 +0100
> Check (ha->addr == dev->dev_addr) is always true because dev_addr_init()
> sets this. Correct the check to behave properly on addr removal.
>
> Signed-off-by: Jiri Pirko <jiri@resnulli.us>
I'm pretty sure this is very intentional.
It's trying to prevent deletion of the implicit dev->dev_addr
entry. But it will allow decementing the reference count to
1, but no further.
I'm not applying this.
^ permalink raw reply
* Re: [PATCH] net/smsc911x: Fix ready check in cases where WORD_SWAP is needed
From: David Miller @ 2012-11-15 2:51 UTC (permalink / raw)
To: kamlakant.patel; +Cc: netdev, steve, linus.walleij, robert.marklund
In-Reply-To: <1352893298-20058-1-git-send-email-kamlakant.patel@broadcom.com>
From: "Kamlakant Patel" <kamlakant.patel@broadcom.com>
Date: Wed, 14 Nov 2012 17:11:38 +0530
> The chip ready check added by the commit 3ac3546e [Always wait for
> the chip to be ready] does not work when the register read/write
> is word swapped. This check has been added before the WORD_SWAP
> register is programmed, so we need to check for swapped register
> value as well.
>
> Bit 16 is marked as RESERVED in SMSC datasheet, Steve Glendinning
> <steve@shawell.net> checked with SMSC and wrote:
>
> The chip architects have concluded we should be reading PMT_CTRL
> until we see any of bits 0, 8, 16 or 24 set. Then we should read
> BYTE_TEST to check the byte order is correct (as we already do).
>
> The rationale behind this is that some of the chip variants have
> word order swapping features too, so the READY bit could actually
> be in any of the 4 possible locations. The architects have confirmed
> that if any of these 4 positions is set the chip is ready. The other
> 3 locations will either never be set or can only go high after READY
> does (so also indicate the device is ready).
>
> This change will check for the READY bit at the 16th position. We do
> not check the other two cases (bit 8 and 24) since the driver does not
> support byte-swapped register read/write.
>
> Signed-off-by: Kamlakant Patel <kamlakant.patel@broadcom.com>
Applied, thank you.
^ permalink raw reply
* Re: [PATCH] drivers/net: fix tasklet misuse issue
From: David Miller @ 2012-11-15 2:50 UTC (permalink / raw)
To: xtfeng; +Cc: dannyfeng, netdev, linux-kernel
In-Reply-To: <1352872056-5637-1-git-send-email-xtfeng@gmail.com>
From: Xiaotian Feng <xtfeng@gmail.com>
Date: Wed, 14 Nov 2012 13:47:36 +0800
> In commit 175c0dff, drivers uses tasklet_kill to avoid put disabled tasklet
> on the tasklet vec. But some of the drivers uses tasklet_init & tasklet_disable
> in the driver init code, then tasklet_enable when it is opened. This makes
> tasklet_enable on a killed tasklet and make ksoftirqd crazy then. Normally,
> drivers should use tasklet_init/tasklet_kill on device open/remove, and use
> tasklet_disable/tasklet_enable on device suspend/resume.
>
> Reported-by: Peter Wu <lekensteyn@gmail.com>
> Tested-by: Peter Wu <lekensteyn@gmail.com>
> Signed-off-by: Xiaotian Feng <dannyfeng@tencent.com>
Applied, thanks.
^ permalink raw reply
* Re: [PATCH net-next] vmxnet3: fix indentation
From: David Miller @ 2012-11-15 2:48 UTC (permalink / raw)
To: shemminger; +Cc: sbhatewara, netdev
In-Reply-To: <20121113155328.0914d6f1@nehalam.linuxnetplumber.net>
From: Stephen Hemminger <shemminger@vyatta.com>
Date: Tue, 13 Nov 2012 15:53:28 -0800
> Minor indentation out of alignment.
>
> Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Applied, thanks.
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox