Hi all,
Just something I noticed here that I'll pass on to the group.
Mynewt has special directory structure to avoid name conflicts with header
files. For example,
Hal/hal_adc.h
And
Mcu/hal_adc.h
won't conflict because the user has to include the directory name in the path.
#include
The culprit is the security code that was recently added to the host.
This new code caused ~10kB of mbedtls code to get pulled in to the
image.
Since security support is incomplete, I have disabled the security
manager by default. The latest code in the develop branch should fit on
the nRF51
On 3/28/16 9:44 AM, p...@wrada.com wrote:
> Thanks for the reviews.
>
> 1) regarding names. I can shorten. I really like having the units in the
> names, but can shorted to
>
> hal_adc_refmv();
> hal_adc_bits();
OK.
I'm fine if we chose milivolts as our primary unit. But if we believe
MV
Hello:
This is just an FYI: the nrf51 bletiny build is over the code size limit, so if
you pull develop and build this project it will not link. This will get fixed
today (hopefully).
Thanks for the reviews.
1) regarding names. I can shorten. I really like having the units in the
names, but can shorted to
hal_adc_refmv();
hal_adc_bits();
As far as convert. To convert from adc output to molts requires the
resolution and the reference voltage. I agree though that
This
I will second the motion to abbreviate things more :-) While I do like the
simplicity of the mbed HAL I do realize that it does not support everything we
want the HAL to do. And unless I am mistaken, it shouldnt be hard to map
between the two. So I guess this means a +1 from me.
Will
> On Mar