The API takes a signed int as a parameter as I usually just like using ‘int’ as 
opposed to ‘int8_t’ in functions. However, the lower-layer phy driver stores 
the power as a signed, 8-bit integer as that is all that is needed at the lower 
layer (just as you say Kevin).

Certainly we can change the API to int8_t if folks think that is proper…

Will

> On May 19, 2016, at 11:17 AM, Christopher Collins <[email protected]> wrote:
> 
> On Thu, May 19, 2016 at 11:12:39AM -0700, James Howarth wrote:
>> Hi Chris,
>> 
>> I think it needs to be a signed int right, as txpwer can be negative, does
>> that sound right?
> 
> Yes, good catch (thanks also, Kevin!).
> 
> In that case, you should declare the tx power variable as an int8_t.
> 
> Chris

Reply via email to