On Fri, Feb 16, 2018 at 01:40:08PM -0800, Quytelda Kahja wrote:
> Fix a coding style problem causing warnings from checkpatch.pl.
>
> Signed-off-by: Quytelda Kahja <quytelda@xxxxxxxxxxx>
> ---
> drivers/staging/gdm724x/gdm_lte.c | 50 ++++++++++++++++++++++-----------------
> 1 file changed, 28 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/gdm724x/gdm_lte.c
> index 0527b0d1c1d0..ff3fed9c4a81 100644
> --- a/drivers/staging/gdm724x/gdm_lte.c
> +++ b/drivers/staging/gdm724x/gdm_lte.c
> @@ -204,9 +204,12 @@ static __sum16 icmp6_checksum(struct ipv6hdr *ipv6, u16 *ptr, int len)
> pseudo_header.ph.ph_nxt = ipv6->nexthdr;
>
> w = (u16 *)&pseudo_header;
> - for (i = 0; i < ARRAY_SIZE(pseudo_header.pa); i++)
> - sum = csum_add(sum, csum_unfold(
> - (__force __sum16)pseudo_header.pa[i]));
> + for (i = 0; i < ARRAY_SIZE(pseudo_header.pa); i++) {
> + __wsum tmp_sum;
> +
> + tmp_sum = csum_unfold((__force __sum16)pseudo_header.pa[i]);
> + sum = csum_add(sum, tmp_sum);
> + }
>
> w = ptr;
> while (len > 1) {
> @@ -510,18 +513,18 @@ static int gdm_lte_event_send(struct net_device *dev, char *buf, int len)
> {
> struct nic *nic = netdev_priv(dev);
> struct hci_packet *hci = (struct hci_packet *)buf;
> + struct gdm_endian *phy_endian;
> int idx;
> + u16 cpu;
> int ret;
>
> ret = sscanf(dev->name, "lte%d", &idx);
> if (ret != 1)
> return -EINVAL;
>
> - return netlink_send(lte_event.sock, idx, 0, buf,
> - gdm_dev16_to_cpu(
> - nic->phy_dev->get_endian(
> - nic->phy_dev->priv_dev), hci->len)
> - + HCI_HEADER_SIZE);
> + phy_endian = nic->phy_dev->get_endian(nic->phy_dev->priv_dev);
> + cpu = gdm_dev16_to_cpu(phy_endian, hci->len);
> + return netlink_send(lte_event.sock, idx, 0, buf, cpu + HCI_HEADER_SIZE);
> }
>
> static void gdm_lte_event_rcv(struct net_device *dev, u16 type,
> @@ -728,17 +731,21 @@ static void gdm_lte_pdn_table(struct net_device *dev, char *buf, int len)
> {
> struct nic *nic = netdev_priv(dev);
> struct hci_pdn_table_ind *pdn_table = (struct hci_pdn_table_ind *)buf;
> + struct gdm_endian *dft_endian;
> + struct gdm_endian *nic_endian;
>
> if (pdn_table->activate) {
> nic->pdn_table.activate = pdn_table->activate;
> - 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);
> +
> + dft_endian = nic->phy_dev->get_endian(nic->phy_dev->priv_dev);
> + nic_endian = nic->phy_dev->get_endian(nic->phy_dev->priv_dev);
These are the same values :(
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
