Staging: gdm724x: Remove confusing macro gdm_dev_endian in gdm_lte.c
authorRashika Kheria <rashika.kheria@gmail.com>
Thu, 7 Nov 2013 14:58:40 +0000 (20:28 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 10 Nov 2013 20:22:34 +0000 (12:22 -0800)
This patch removes confusing macro gdm_dev_endian as stated in TODO list
in file gdm_lte.c. It then fixes the place where the macro is used.

Signed-off-by: Rashika Kheria <rashika.kheria@gmail.com>
Reviewed-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/gdm724x/gdm_lte.c

index d061cd553053b99aee3942e28f3ca7a2e6f33dc1..5cf0f6f7f8d3c4b8fc3729aa77b428dc024c7390 100644 (file)
@@ -44,9 +44,6 @@
  */
 #define DEFAULT_MTU_SIZE 1500
 
-#define gdm_dev_endian(n) (\
-       n->phy_dev->get_endian(n->phy_dev->priv_dev))
-
 #define gdm_lte_hci_send(n, d, l) (\
        n->phy_dev->send_hci_func(n->phy_dev->priv_dev, d, l, NULL, NULL))
 
@@ -500,7 +497,10 @@ static int gdm_lte_event_send(struct net_device *dev, char *buf, int len)
        sscanf(dev->name, "lte%d", &idx);
 
        return netlink_send(lte_event.sock, idx, 0, buf,
-                           gdm_dev16_to_cpu(gdm_dev_endian(nic), hci->len) + HCI_HEADER_SIZE);
+                           gdm_dev16_to_cpu(
+                                   nic->phy_dev->get_endian(
+                                           nic->phy_dev->priv_dev), hci->len)
+                           + HCI_HEADER_SIZE);
 }
 
 static void gdm_lte_event_rcv(struct net_device *dev, u16 type, void *msg, int len)
@@ -685,8 +685,14 @@ static void gdm_lte_pdn_table(struct net_device *dev, char *buf, int len)
 
        if (pdn_table->activate) {
                nic->pdn_table.activate = pdn_table->activate;
-               nic->pdn_table.dft_eps_id = gdm_dev32_to_cpu(gdm_dev_endian(nic), pdn_table->dft_eps_id);
-               nic->pdn_table.nic_type = gdm_dev32_to_cpu(gdm_dev_endian(nic), pdn_table->nic_type);
+               nic->pdn_table.dft_eps_id = gdm_dev32_to_cpu(
+                                               nic->phy_dev->get_endian(
+                                                       nic->phy_dev->priv_dev),
+                                               pdn_table->dft_eps_id);
+               nic->pdn_table.nic_type = gdm_dev32_to_cpu(
+                                               nic->phy_dev->get_endian(
+                                                       nic->phy_dev->priv_dev),
+                                               pdn_table->nic_type);
 
                netdev_info(dev, "pdn activated, nic_type=0x%x\n",
                            nic->pdn_table.nic_type);