[bug report] ath10k: add memory dump support for QCA6174/QCA9377

[Kalle Valo]

Dan Carpenter <dan.carpenter at oracle.com> writes:

> Hello Alan Liu,
>
> The patch 703f261dd77f: "ath10k: add memory dump support for
> QCA6174/QCA9377" from Dec 22, 2017, leads to the following static
> checker warning:
>
> 	drivers/net/wireless/ath/ath10k/pci.c:1481 ath10k_pci_dump_memory_section()
> 	warn: unsigned 'mem_region->section_table.size' is never less than zero.
>
> drivers/net/wireless/ath/ath10k/pci.c
>   1470  static int ath10k_pci_dump_memory_section(struct ath10k *ar,
>   1471                                            const struct ath10k_mem_region *mem_region,
>   1472                                            u8 *buf, size_t buf_len)
>   1473  {
>   1474          const struct ath10k_mem_section *cur_section, *next_section;
>   1475          unsigned int count, section_size, skip_size;
>   1476          int ret, i, j;
>   1477  
>   1478          if (!mem_region || !buf)
>   1479                  return 0;
>   1480  
>   1481          if (mem_region->section_table.size < 0)
>                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> Impossible condition

Should be fixed by the commit below which I applied 20 minutes ago :)

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=ath-next&id=95f27a51ee02fdd9e7afc06f090c7171673a79ac

-- 
Kalle Valo