Commit e4600d69 authored by huangdaode's avatar huangdaode Committed by David S. Miller

net: Fix Hisilicon Network Subsystem Support Compilation

This patch fixes the compilation error with arm allmodconfig, this error
generated due to unavailability of readq() on 32-bit platform which was
found during net-next daily compilation. In the same time, fix all the
hns drivers compilation warnings.
Signed-off-by: default avatarhuangdaode <huangdaode@hisilicon.com>
Signed-off-by: default avatarzhaungyuzeng <Yisen.zhuang@huawei.com>
Signed-off-by: default avatarkenneth Lee <liguozhu@hisilicon.com>
Signed-off-by: default avataryankejian <yankejian@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1273bc57
...@@ -86,7 +86,7 @@ int hns_mac_get_sfp_prsnt(struct hns_mac_cb *mac_cb, int *sfp_prsnt) ...@@ -86,7 +86,7 @@ int hns_mac_get_sfp_prsnt(struct hns_mac_cb *mac_cb, int *sfp_prsnt)
if (!mac_cb->cpld_vaddr) if (!mac_cb->cpld_vaddr)
return -ENODEV; return -ENODEV;
*sfp_prsnt = !dsaf_read_b((u64)mac_cb->cpld_vaddr *sfp_prsnt = !dsaf_read_b((u8 *)mac_cb->cpld_vaddr
+ MAC_SFP_PORT_OFFSET); + MAC_SFP_PORT_OFFSET);
return 0; return 0;
......
...@@ -43,7 +43,7 @@ struct hns_mac_cb; ...@@ -43,7 +43,7 @@ struct hns_mac_cb;
#define DSAF_DUMP_REGS_NUM 504 #define DSAF_DUMP_REGS_NUM 504
#define DSAF_STATIC_NUM 28 #define DSAF_STATIC_NUM 28
#define DSAF_STATS_READ(p, offset) (*((u64 *)((u64)(p) + (offset)))) #define DSAF_STATS_READ(p, offset) (*((u64 *)((u8 *)(p) + (offset))))
enum hal_dsaf_mode { enum hal_dsaf_mode {
HRD_DSAF_NO_DSAF_MODE = 0x0, HRD_DSAF_NO_DSAF_MODE = 0x0,
...@@ -302,7 +302,7 @@ struct dsaf_device { ...@@ -302,7 +302,7 @@ struct dsaf_device {
static inline void *hns_dsaf_dev_priv(const struct dsaf_device *dsaf_dev) static inline void *hns_dsaf_dev_priv(const struct dsaf_device *dsaf_dev)
{ {
return (void *)((u64)dsaf_dev + sizeof(*dsaf_dev)); return (void *)((u8 *)dsaf_dev + sizeof(*dsaf_dev));
} }
struct dsaf_drv_tbl_tcam_key { struct dsaf_drv_tbl_tcam_key {
......
...@@ -192,8 +192,7 @@ static void hns_rcb_ring_init(struct ring_pair_cb *ring_pair, int ring_type) ...@@ -192,8 +192,7 @@ static void hns_rcb_ring_init(struct ring_pair_cb *ring_pair, int ring_type)
dsaf_write_dev(q, RCB_RING_RX_RING_BASEADDR_L_REG, dsaf_write_dev(q, RCB_RING_RX_RING_BASEADDR_L_REG,
(u32)dma); (u32)dma);
dsaf_write_dev(q, RCB_RING_RX_RING_BASEADDR_H_REG, dsaf_write_dev(q, RCB_RING_RX_RING_BASEADDR_H_REG,
(u32)(dma >> 32)); (u32)((dma >> 31) >> 1));
dsaf_write_dev(q, RCB_RING_RX_RING_BD_LEN_REG, dsaf_write_dev(q, RCB_RING_RX_RING_BD_LEN_REG,
bd_size_type); bd_size_type);
dsaf_write_dev(q, RCB_RING_RX_RING_BD_NUM_REG, dsaf_write_dev(q, RCB_RING_RX_RING_BD_NUM_REG,
...@@ -204,8 +203,7 @@ static void hns_rcb_ring_init(struct ring_pair_cb *ring_pair, int ring_type) ...@@ -204,8 +203,7 @@ static void hns_rcb_ring_init(struct ring_pair_cb *ring_pair, int ring_type)
dsaf_write_dev(q, RCB_RING_TX_RING_BASEADDR_L_REG, dsaf_write_dev(q, RCB_RING_TX_RING_BASEADDR_L_REG,
(u32)dma); (u32)dma);
dsaf_write_dev(q, RCB_RING_TX_RING_BASEADDR_H_REG, dsaf_write_dev(q, RCB_RING_TX_RING_BASEADDR_H_REG,
(u32)(dma >> 32)); (u32)((dma >> 31) >> 1));
dsaf_write_dev(q, RCB_RING_TX_RING_BD_LEN_REG, dsaf_write_dev(q, RCB_RING_TX_RING_BD_LEN_REG,
bd_size_type); bd_size_type);
dsaf_write_dev(q, RCB_RING_TX_RING_BD_NUM_REG, dsaf_write_dev(q, RCB_RING_TX_RING_BD_NUM_REG,
......
...@@ -967,6 +967,6 @@ static inline u32 dsaf_get_reg_field(void *base, u32 reg, u32 mask, u32 shift) ...@@ -967,6 +967,6 @@ static inline u32 dsaf_get_reg_field(void *base, u32 reg, u32 mask, u32 shift)
readb((__iomem unsigned char *)(addr)) readb((__iomem unsigned char *)(addr))
#define hns_mac_reg_read64(drv, offset) \ #define hns_mac_reg_read64(drv, offset) \
readq((__iomem void *)(((u64)(drv)->io_base + 0xc00 + (offset)))) readq((__iomem void *)(((u8 *)(drv)->io_base + 0xc00 + (offset))))
#endif /* _DSAF_REG_H */ #endif /* _DSAF_REG_H */
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
* (at your option) any later version. * (at your option) any later version.
*/ */
#include <asm-generic/io-64-nonatomic-hi-lo.h>
#include <linux/of_mdio.h> #include <linux/of_mdio.h>
#include "hns_dsaf_main.h" #include "hns_dsaf_main.h"
#include "hns_dsaf_mac.h" #include "hns_dsaf_mac.h"
......
...@@ -31,8 +31,6 @@ ...@@ -31,8 +31,6 @@
#define NIC_TX_CLEAN_MAX_NUM 256 #define NIC_TX_CLEAN_MAX_NUM 256
#define NIC_RX_CLEAN_MAX_NUM 64 #define NIC_RX_CLEAN_MAX_NUM 64
#define RCB_ERR_PRINT_CYCLE 1000
#define RCB_IRQ_NOT_INITED 0 #define RCB_IRQ_NOT_INITED 0
#define RCB_IRQ_INITED 1 #define RCB_IRQ_INITED 1
...@@ -434,21 +432,12 @@ static int hns_nic_poll_rx_skb(struct hns_nic_ring_data *ring_data, ...@@ -434,21 +432,12 @@ static int hns_nic_poll_rx_skb(struct hns_nic_ring_data *ring_data,
if (unlikely((!desc->rx.pkt_len) || if (unlikely((!desc->rx.pkt_len) ||
hnae_get_bit(bnum_flag, HNS_RXD_DROP_B))) { hnae_get_bit(bnum_flag, HNS_RXD_DROP_B))) {
if (!(ring->stats.err_pkt_len % RCB_ERR_PRINT_CYCLE))
netdev_dbg(ndev,
"pkt_len(%u),drop(%u),%#llx,%#llx\n",
le16_to_cpu(desc->rx.pkt_len),
hnae_get_bit(bnum_flag, HNS_RXD_DROP_B),
((u64 *)desc)[0], ((u64 *)desc)[1]);
ring->stats.err_pkt_len++; ring->stats.err_pkt_len++;
dev_kfree_skb_any(skb); dev_kfree_skb_any(skb);
return -EFAULT; return -EFAULT;
} }
if (unlikely(hnae_get_bit(bnum_flag, HNS_RXD_L2E_B))) { if (unlikely(hnae_get_bit(bnum_flag, HNS_RXD_L2E_B))) {
if (!(ring->stats.l2_err % RCB_ERR_PRINT_CYCLE))
netdev_dbg(ndev, "L2 check err,%#llx,%#llx\n",
((u64 *)desc)[0], ((u64 *)desc)[1]);
ring->stats.l2_err++; ring->stats.l2_err++;
dev_kfree_skb_any(skb); dev_kfree_skb_any(skb);
return -EFAULT; return -EFAULT;
...@@ -459,12 +448,6 @@ static int hns_nic_poll_rx_skb(struct hns_nic_ring_data *ring_data, ...@@ -459,12 +448,6 @@ static int hns_nic_poll_rx_skb(struct hns_nic_ring_data *ring_data,
if (unlikely(hnae_get_bit(bnum_flag, HNS_RXD_L3E_B) || if (unlikely(hnae_get_bit(bnum_flag, HNS_RXD_L3E_B) ||
hnae_get_bit(bnum_flag, HNS_RXD_L4E_B))) { hnae_get_bit(bnum_flag, HNS_RXD_L4E_B))) {
if (!(ring->stats.l3l4_csum_err % RCB_ERR_PRINT_CYCLE))
netdev_dbg(ndev,
"check err(%#x),%#llx,%#llx\n",
hnae_get_bit(bnum_flag, HNS_RXD_L3E_B) |
hnae_get_bit(bnum_flag, HNS_RXD_L4E_B),
((u64 *)desc)[0], ((u64 *)desc)[1]);
ring->stats.l3l4_csum_err++; ring->stats.l3l4_csum_err++;
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment