[GitHub] mlaz opened a new pull request #1378: [WIP - Do not merge] hw/drivers/led/lp5523: Fixing lp5523 bugs

[GitBox <gi...@apache.org>]

mlaz opened a new pull request #1378: [WIP - Do not merge] hw/drivers/led/lp5523: Fixing lp5523 bugs
URL: https://github.com/apache/mynewt-core/pull/1378
 
 
   So far I have found several issues with this driver:
   * Not being able to use the driver(not even the basic stuff like simple PWM) without having auto-increment enabled;  
   * Not passing selftest when D9 is enabled (also not being able to enable it);
   * Not passing verify_program after under any conditions;
   * lp5523 register set doesn't work on lp5523_shell; 
   
   Most of this is solved with:
   * Making lp5523_set_output_on() write only a single register, it will only set a bit at a time anyway;
   * Fixing lp5523_set_n_regs();
   * WIP;
   
   Some odd behaviours (which probably are undocumented but normal) I noticed so far:
   * Reading a single register from the program memory region(addr >= 0x50) always returns 0x00. This might be related to the fact that when I tried it auto-increment is enabled;
   * To be continued;

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services