Date: Wed, 3 Feb 2016 13:00:02 -0800 From: Adrian Chadd <adrian.chadd@gmail.com> To: Tobias Kortkamp <t@tobik.me> Cc: "freebsd-wireless@freebsd.org" <freebsd-wireless@freebsd.org> Subject: Re: [patch] Change ath(4) to report correct chip name for Qualcomm Atheros cards Message-ID: <CAJ-Vmom-8zKuorN2ubuXMUBqSe2oNnaaBTUubOLRyGkDYwFt_g@mail.gmail.com> In-Reply-To: <1454498827.3964610.510611394.3ED30F5C@webmail.messagingengine.com> References: <1454498827.3964610.510611394.3ED30F5C@webmail.messagingengine.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi! Thanks! I've been meaning to do this for a while but have been .. slack-ish. Can you attach it as a patch? I'll commit it tonight. -a On 3 February 2016 at 03:27, Tobias Kortkamp <t@tobik.me> wrote: > Hi, > > when using ath(4) the driver reports the wrong name for Qualcomm Atheros > chips on attach e.g. a QCA9565 card reports as being an AR9565. > ath_hal(4) however mentions QCA9565 and not AR9565. This is confusing > if you want to find out if your card is supposed to be supported or not. > I also saw some comments in sys/dev/ath/ath_hal/ah.c that is not > supposed to be that way. Fixing this seems simple enough (see inlined > patch). > > Thanks, > Tobias > > diff --git a/sys/dev/ath/ath_hal/ah.c b/sys/dev/ath/ath_hal/ah.c > index a0baa69..8333f10 100644 > --- a/sys/dev/ath/ath_hal/ah.c > +++ b/sys/dev/ath/ath_hal/ah.c > @@ -91,63 +91,60 @@ ath_hal_mac_name(struct ath_hal *ah) > switch (ah->ah_macVersion) { > case AR_SREV_VERSION_CRETE: > case AR_SREV_VERSION_MAUI_1: > - return "5210"; > + return "AR5210"; > case AR_SREV_VERSION_MAUI_2: > case AR_SREV_VERSION_OAHU: > - return "5211"; > + return "AR5211"; > case AR_SREV_VERSION_VENICE: > - return "5212"; > + return "AR5212"; > case AR_SREV_VERSION_GRIFFIN: > - return "2413"; > + return "AR2413"; > case AR_SREV_VERSION_CONDOR: > - return "5424"; > + return "AR5424"; > case AR_SREV_VERSION_EAGLE: > - return "5413"; > + return "AR5413"; > case AR_SREV_VERSION_COBRA: > - return "2415"; > + return "AR2415"; > case AR_SREV_2425: /* Swan */ > - return "2425"; > + return "AR2425"; > case AR_SREV_2417: /* Nala */ > - return "2417"; > + return "AR2417"; > case AR_XSREV_VERSION_OWL_PCI: > - return "5416"; > + return "AR5416"; > case AR_XSREV_VERSION_OWL_PCIE: > - return "5418"; > + return "AR5418"; > case AR_XSREV_VERSION_HOWL: > - return "9130"; > + return "AR9130"; > case AR_XSREV_VERSION_SOWL: > - return "9160"; > + return "AR9160"; > case AR_XSREV_VERSION_MERLIN: > if (AH_PRIVATE(ah)->ah_ispcie) > - return "9280"; > - return "9220"; > + return "AR9280"; > + return "AR9220"; > case AR_XSREV_VERSION_KITE: > - return "9285"; > + return "AR9285"; > case AR_XSREV_VERSION_KIWI: > if (AH_PRIVATE(ah)->ah_ispcie) > - return "9287"; > - return "9227"; > + return "AR9287"; > + return "AR9227"; > case AR_SREV_VERSION_AR9380: > if (ah->ah_macRev >= AR_SREV_REVISION_AR9580_10) > - return "9580"; > - return "9380"; > + return "AR9580"; > + return "AR9380"; > case AR_SREV_VERSION_AR9460: > - return "9460"; > + return "AR9460"; > case AR_SREV_VERSION_AR9330: > - return "9330"; > + return "AR9330"; > case AR_SREV_VERSION_AR9340: > - return "9340"; > + return "AR9340"; > case AR_SREV_VERSION_QCA9550: > - /* XXX should say QCA, not AR */ > - return "9550"; > + return "QCA9550"; > case AR_SREV_VERSION_AR9485: > - return "9485"; > + return "AR9485"; > case AR_SREV_VERSION_QCA9565: > - /* XXX should say QCA, not AR */ > - return "9565"; > + return "QCA9565"; > case AR_SREV_VERSION_QCA9530: > - /* XXX should say QCA, not AR */ > - return "9530"; > + return "QCA9530"; > } > return "????"; > } > diff --git a/sys/dev/ath/if_ath.c b/sys/dev/ath/if_ath.c > index 0dc7d3f..8b1580a 100644 > --- a/sys/dev/ath/if_ath.c > +++ b/sys/dev/ath/if_ath.c > @@ -6160,7 +6160,7 @@ ath_announce(struct ath_softc *sc) > { > struct ath_hal *ah = sc->sc_ah; > > - device_printf(sc->sc_dev, "AR%s mac %d.%d RF%s phy %d.%d\n", > + device_printf(sc->sc_dev, "%s mac %d.%d RF%s phy %d.%d\n", > ath_hal_mac_name(ah), ah->ah_macVersion, ah->ah_macRev, > ath_hal_rf_name(ah), ah->ah_phyRev >> 4, ah->ah_phyRev & > 0xf); > device_printf(sc->sc_dev, "2GHz radio: 0x%.4x; 5GHz radio: > 0x%.4x\n", > _______________________________________________ > freebsd-wireless@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/freebsd-wireless > To unsubscribe, send any mail to "freebsd-wireless-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmom-8zKuorN2ubuXMUBqSe2oNnaaBTUubOLRyGkDYwFt_g>