[PATCH 5/6] ath10k: add memory dump support for QCA6174/QCA9377

[Ben Greear]

On 08/15/2017 01:23 AM, Kalle Valo wrote:
> From: Alan Liu <alanliu at qca.qualcomm.com>
>
> Add memory dump to the firmware crash data file which is provided to user space
> via devcoredump interface. This makes it easier for firmware engineers to debug
> firmware crashes.
>
> Due to increased memory consumption the memory dump is disabled by default. To
> enable it make sure that bit 3 is set in coredump_mask module parameter:
>
> modprobe ath10k_core coredump_mask=0xffffffff
>
> When RAMDUMP is enabled a buffer for the dump is allocated with vmalloc during
> device probe. The actual memory layout is different in hardware versions and
> the layouts are defined in coredump.c. The memory is split to regions and, to
> get even finegrained control of what to copy, the region can split to smaller
> sections as not all registers are readable (which could cause the whole system
> to stall).
>
> Signed-off-by: Alan Liu <alanliu at qca.qualcomm.com>
> [kvalo at qca.qualcomm.com: refactoring and cleanup]
> Signed-off-by: Kalle Valo <kvalo at qca.qualcomm.com>

>  #include "debug.h"
> +#include "hw.h"
> +
> +static const struct ath10k_mem_section qca6174_hw21_register_sections[] = {
> +	{0x800, 0x810},
> +	{0x820, 0x82C},
> +	{0x830, 0x8F4},
> +	{0x90C, 0x91C},
> +	{0xA14, 0xA18},

This looks pretty fragile.  Any chance this type of info could be packaged
into the firmware image and then the driver just pulls it out of there
instead of having all these hard coded magic values in the driver?

Thanks,
Ben

-- 
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc  http://www.candelatech.com