linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: Andrey Borzenkov <arvidjaar@mail.ru>, linux-wireless@vger.kernel.org
Subject: Re: [PATCH] wireless: sysfs was displaying different values for level and noise than procfs
Date: Wed, 19 Nov 2008 16:51:58 -0500	[thread overview]
Message-ID: <1227131518.3571.16.camel@dv> (raw)
In-Reply-To: <20081119200856.GH3485@tuxdriver.com>

On Wed, 2008-11-19 at 15:08 -0500, John W. Linville wrote:

> > But positive dBm is not even possible to represnt right now. It is
> > u8 and is displayed as ((s32)level - 256); so it is *always*
> > negative.

I would force dBm to be between -192 and 63.  That would cover
everything even remotely possible.

> And what is your perspective regarding potential 'userland ABI'
> issues of either this patch or of Pavel's suggestion?

In my opinion, discrepancy between procfs and sysfs is permissible.  If
the sysfs interface has been wrong, we should be able to change the
format.  As for the procfs interface, it could be deprecated.

-- 
Regards,
Pavel Roskin

  reply	other threads:[~2008-11-19 21:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-15 15:31 [PATCH] wireless: sysfs was displaying different values for level and noise than procfs Andrey Borzenkov
2008-11-19 19:13 ` Pavel Roskin
2008-11-19 19:45   ` Andrey Borzenkov
2008-11-19 20:08     ` John W. Linville
2008-11-19 21:51       ` Pavel Roskin [this message]
2008-11-20  6:57         ` Johannes Berg
2008-11-20 16:15           ` Pavel Roskin
2008-11-20 12:08         ` Holger Schurig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1227131518.3571.16.camel@dv \
    --to=proski@gnu.org \
    --cc=arvidjaar@mail.ru \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).