Search Linux Wireless

Re: [PATCH V3] ath6kl: ath6kl: Add debugfs interface to dump diagnostic registers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 02, 2011 at 11:21:24AM +0300, Kalle Valo wrote:
> On 09/02/2011 11:01 AM, Vasanthakumar Thiagarajan wrote:
> > On Fri, Sep 02, 2011 at 10:55:03AM +0300, Kalle Valo wrote:
> >> On 09/01/2011 01:09 PM, Vasanthakumar Thiagarajan wrote:
> >>
> >> Thanks, applied. But I did minor changes:
> >>
> >> commit 91d57de5adfc40184ef7cb8974104459c5211add
> >> Author: Vasanthakumar Thiagarajan <vthiagar@xxxxxxxxxxxxxxxx>
> >> Date:   Fri Sep 2 10:40:06 2011 +0300
> >>
> >>     ath6kl: Add debugfs interface to dump diagnostic registers from firmware
> >>
> >>     To dump a particular register:
> >>
> >>         echo <reg_addr> > <debugfs_root>/ieee80211/phyX/ath6kl/reg_addr
> >>
> >>     To dump the entire register set:
> >>
> >>         echo 0 > <debugfs_root>/ieee80211/phyX/ath6kl/reg_addr
> >>
> >>     Register values will be available at:
> >>
> >>         cat <debugfs_root>/ieee80211/phyX/ath6kl/reg_dump
> >>
> >>     kvalo: commit log cleanup, renamed few functions, removed a warning
> >>     message
> > 
> > Thanks, what is the warning?
> 
> I just removed the ath6kl_warn() about invalid register address in
> ath6kl_dbg_is_diag_reg_valid(). Kernel should not send any debug logs in
> that case.

Ok, thanks.
> 
> > Did you apply my other patch which fixes endianess?.
> 
> No, not yet. I still have patches pending for review. Will finish them
> today.

No problem, i thought it was a sparse warning.

Vasanth
--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux