On Tue, Sep 6, 2011 at 12:12 PM, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote: > On 09/05/2011 12:42 PM, Vivek Natarajan wrote: >> Process the regulatory code from eeprom and pass the >> country information to cfg80211. ... >> + else if (!(((u16) reg_code & WORLD_SKU_MASK) == WORLD_SKU_PREFIX)) { >> + >> + regpair = ath6kl_get_regpair((u16) reg_code); >> + country = ath6kl_regd_find_country_by_rd((u16) reg_code); >> + printk(KERN_DEBUG "ath6kl: Regpair used: 0x%0x\n", >> + regpair->regDmnEnum); > > ath6kl_dbg() is more approriate here. > >> + regulatory_hint(wmi->parent_dev->wdev->wiphy, alpha2); >> + >> + printk(KERN_DEBUG "ath6kl: Country alpha2 being used: %c%c\n", >> + alpha2[0], alpha2[1]); > > Please use ath6kl_dbg() or, if we really need to log this, you can use > ath6kl_info(). But is there a reason to log this everytime? I had kept this print similar to ath9k. Anyhow, cfg80211 also prints the country code from the driver. So, I will change this to ath6kl_dbg. Thanks Vivek. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html