Linux IEEE 802.15.4 and 6LoWPAN development
 help / color / mirror / Atom feed
From: Varka Bhadram <varkab@cdac.in>
To: Stefan Schmidt <stefan@osg.samsung.com>,
	Varka Bhadram <varkabhadram@gmail.com>,
	linux-wpan@vger.kernel.org
Subject: Re: [PATCH bluetooth-next 2/3] cc2520: update initial transmit power value
Date: Thu, 28 May 2015 14:08:25 +0530	[thread overview]
Message-ID: <5566D401.6040004@cdac.in> (raw)
In-Reply-To: <5566CF25.4040206@osg.samsung.com>

Hello,

On 05/28/2015 01:47 PM, Stefan Schmidt wrote:
> Hello.
>
> On 28/05/15 09:27, Varka Bhadram wrote:
>> On 05/28/2015 12:20 PM, Varka Bhadram wrote:
>>
>>> CC2520 has the default 0dBm transmit power level  on reset.
>>> This patch update initial value of transmit power with 0dBm value.
>>
>> CC2520 has the 0dBm value on reset, but in the driver we are setting
>> tx power value to 5dBm.
>
> Why?
>
> In most cases it makes sense to use the same default values the device 
> uses as reset values. Why did you decide to use a different value here?
>
Correct. I need to change that.

Initially we dont have tx power setting from the user-space.
So i just hard coded that value (5dBm) at the time of
hardware initialization.

Now we can set the values from the user-space.


Thanks

-- 
Varka Bhadram


-------------------------------------------------------------------------------------------------------------------------------
[ C-DAC is on Social-Media too. Kindly follow us at:
Facebook: https://www.facebook.com/CDACINDIA & Twitter: @cdacindia ]

This e-mail is for the sole use of the intended recipient(s) and may
contain confidential and privileged information. If you are not the
intended recipient, please contact the sender by reply e-mail and destroy
all copies and the original message. Any unauthorized review, use,
disclosure, dissemination, forwarding, printing or copying of this email
is strictly prohibited and appropriate legal action will be taken.
-------------------------------------------------------------------------------------------------------------------------------


  reply	other threads:[~2015-05-28  8:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-28  6:50 [PATCH bluetooth-next 1/3] cc2520: add set transmit power setting support Varka Bhadram
2015-05-28  6:50 ` [PATCH bluetooth-next 2/3] cc2520: update initial transmit power value Varka Bhadram
2015-05-28  7:27   ` Varka Bhadram
2015-05-28  8:17     ` Stefan Schmidt
2015-05-28  8:38       ` Varka Bhadram [this message]
2015-05-28  6:51 ` [PATCH bluetooth-next 3/3] cc2520: update current channel Varka Bhadram

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=5566D401.6040004@cdac.in \
    --to=varkab@cdac.in \
    --cc=linux-wpan@vger.kernel.org \
    --cc=stefan@osg.samsung.com \
    --cc=varkabhadram@gmail.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