Message ID | 1232517048.9701.21.camel@brick |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Tue, 2009-01-20 at 21:50 -0800, Harvey Harrison wrote: > Remove a completely unused structure definition as well. > > Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> [...] > > diff --git a/drivers/net/bnx2x_main.c b/drivers/net/bnx2x_main.c > index 2f8d64c..021c31c 100644 > --- a/drivers/net/bnx2x_main.c > +++ b/drivers/net/bnx2x_main.c > @@ -987,8 +987,8 @@ static inline void bnx2x_free_rx_sge(struct bnx2x *bp, > __free_pages(page, PAGES_PER_SGE_SHIFT); > > sw_buf->page = NULL; > - sge->addr_hi = 0; > - sge->addr_lo = 0; > + sge->addr_hi = cpu_to_le32(0); > + sge->addr_lo = cpu_to_le32(0); > } I prefer to keep zeros as zeros. I find it less readable with the cpu_to_le macro. I also noticed that it does not add a sparse warning, so on the combined image I will keep the zeros as zeros. Other than that, the patch looks fine -- 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
On Thu, 2009-01-22 at 19:49 +0200, Eilon Greenstein wrote: > On Tue, 2009-01-20 at 21:50 -0800, Harvey Harrison wrote: > > Remove a completely unused structure definition as well. > > > > Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> > > [...] > > > > diff --git a/drivers/net/bnx2x_main.c b/drivers/net/bnx2x_main.c > > index 2f8d64c..021c31c 100644 > > --- a/drivers/net/bnx2x_main.c > > +++ b/drivers/net/bnx2x_main.c > > @@ -987,8 +987,8 @@ static inline void bnx2x_free_rx_sge(struct bnx2x *bp, > > __free_pages(page, PAGES_PER_SGE_SHIFT); > > > > sw_buf->page = NULL; > > - sge->addr_hi = 0; > > - sge->addr_lo = 0; > > + sge->addr_hi = cpu_to_le32(0); > > + sge->addr_lo = cpu_to_le32(0); > > } > > I prefer to keep zeros as zeros. I find it less readable with the > cpu_to_le macro. I also noticed that it does not add a sparse warning, > so on the combined image I will keep the zeros as zeros. > > Other than that, the patch looks fine Sparse will only warn if CHECK_ENDIAN is set as addr_hi/lo was just marked as a le32, how do you think I noticed this line needed changing ;-) Harvey -- 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
diff --git a/drivers/net/bnx2x_hsi.h b/drivers/net/bnx2x_hsi.h index a3c1068..b62d45e 100644 --- a/drivers/net/bnx2x_hsi.h +++ b/drivers/net/bnx2x_hsi.h @@ -1496,16 +1496,16 @@ struct ustorm_eth_st_context_config { * The eth Rx Buffer Descriptor */ struct eth_rx_bd { - u32 addr_lo; - u32 addr_hi; + __le32 addr_lo; + __le32 addr_hi; }; /* * The eth Rx SGE Descriptor */ struct eth_rx_sge { - u32 addr_lo; - u32 addr_hi; + __le32 addr_lo; + __le32 addr_hi; }; /* @@ -1908,8 +1908,8 @@ struct eth_tx_bd_flags { * The eth Tx Buffer Descriptor */ struct eth_tx_bd { - u32 addr_lo; - u32 addr_hi; + __le32 addr_lo; + __le32 addr_hi; u16 nbd; u16 nbytes; u16 vlan; @@ -2262,17 +2262,6 @@ union eth_ramrod_data { struct ramrod_data general; }; - -/* - * Rx Last BD in page (in ETH) - */ -struct eth_rx_bd_next_page { - u32 addr_lo; - u32 addr_hi; - u8 reserved[8]; -}; - - /* * Eth Rx Cqe structure- general structure for ramrods */ @@ -2297,8 +2286,8 @@ struct common_ramrod_eth_rx_cqe { * Rx Last CQE in page (in ETH) */ struct eth_rx_cqe_next_page { - u32 addr_lo; - u32 addr_hi; + __le32 addr_lo; + __le32 addr_hi; u32 reserved[6]; }; diff --git a/drivers/net/bnx2x_main.c b/drivers/net/bnx2x_main.c index 2f8d64c..021c31c 100644 --- a/drivers/net/bnx2x_main.c +++ b/drivers/net/bnx2x_main.c @@ -987,8 +987,8 @@ static inline void bnx2x_free_rx_sge(struct bnx2x *bp, __free_pages(page, PAGES_PER_SGE_SHIFT); sw_buf->page = NULL; - sge->addr_hi = 0; - sge->addr_lo = 0; + sge->addr_hi = cpu_to_le32(0); + sge->addr_lo = cpu_to_le32(0); } static inline void bnx2x_free_rx_sge_range(struct bnx2x *bp, @@ -9419,7 +9419,7 @@ static inline u32 bnx2x_xmit_type(struct bnx2x *bp, struct sk_buff *skb) rc = XMIT_PLAIN; else { - if (skb->protocol == ntohs(ETH_P_IPV6)) { + if (skb->protocol == htons(ETH_P_IPV6)) { rc = XMIT_CSUM_V6; if (ipv6_hdr(skb)->nexthdr == IPPROTO_TCP) rc |= XMIT_CSUM_TCP; @@ -9627,7 +9627,7 @@ static int bnx2x_start_xmit(struct sk_buff *skb, struct net_device *dev) /* for now NS flag is not used in Linux */ pbd->global_data = (hlen | - ((skb->protocol == ntohs(ETH_P_8021Q)) << + ((skb->protocol == cpu_to_be16(ETH_P_8021Q)) << ETH_TX_PARSE_BD_LLC_SNAP_EN_SHIFT)); pbd->ip_hlen = (skb_transport_header(skb) -
Remove a completely unused structure definition as well. Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> --- drivers/net/bnx2x_hsi.h | 27 ++++++++------------------- drivers/net/bnx2x_main.c | 8 ++++---- 2 files changed, 12 insertions(+), 23 deletions(-)