Michal Kazior <[email protected]> writes: > HTT version is already printed so print WMI > version as well for consistency. > > Signed-off-by: Michal Kazior <[email protected]> > --- > drivers/net/wireless/ath/ath10k/debug.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath10k/debug.c > b/drivers/net/wireless/ath/ath10k/debug.c > index 3756feb..f1e5916 100644 > --- a/drivers/net/wireless/ath/ath10k/debug.c > +++ b/drivers/net/wireless/ath/ath10k/debug.c > @@ -126,14 +126,18 @@ EXPORT_SYMBOL(ath10k_info); > > void ath10k_print_driver_info(struct ath10k *ar) > { > - ath10k_info(ar, "%s (0x%08x, 0x%08x) fw %s api %d htt %d.%d\n", > + ath10k_info(ar, "%s (0x%08x, 0x%08x) fw %s api %d htt %d.%d wmi > %d.%d.%d.%d\n",
Heh, I have been thinking about to add something similar. But if in the future we add the u32 FW IE for WMI version I would like to print that as well. I guess we could do that with "%d/%d.%d.%d.%d", the %d being the u32. The idea here being that both version numbers are good to know. But of course for now the above is good, just thinking aloud here. -- Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
