[riot-notifications] [RIOT-OS/RIOT] drivers/isl29020 : Expose Configurations to Kconfig (#13995)

Akshai M (Augustus Diode) notifications at github.com
Fri May 1 06:41:30 CEST 2020


### Contribution description

This PR exposes compile configurations in ISL29020 Sensor Device driver to Kconfig.

### Testing procedure
1. New documentation was built using doxygen 

The build worked fine.

2. New macro was introduced in tests/driver_tmp00x/main.c for testing.

```
#define STR(x)   #x
#define SHOW_DEFINE(x) printf("%s=%s\n", #x, STR(x))
```
Firmware was uploaded to FIT IoT Lab test bed.

#### Default State:

##### Firmware Output

main(): This is RIOT! (Version: 2020.07-devel-311-gec19c-Kconfig_isl29020_tests)
CONFIG_ISL29020_DEFAULT_ADDRESS=0x44
ISL29020 light sensor test application

Initializing ISL29020 sensor
[Failed]

#### Usage with CFLAGS 

/tests/driver_isl29020/Makefile

> CFLAGS += -DCONFIG_ISL29020_DEFAULT_ADDRESS=0x45

##### Firmware Output

main(): This is RIOT! (Version: 2020.07-devel-311-gec19c-Kconfig_isl29020_tests)
CONFIG_ISL29020_DEFAULT_ADDRESS=0x45
ISL29020 light sensor test application

Initializing ISL29020 sensor
[Failed]

#### Usage with Kconfig

/tests/driver_isl29020/

> make menuconfig

##### Firmware Output

main(): This is RIOT! (Version: 2020.07-devel-312-gc0d85b-Kconfig_isl29020_tests)
CONFIG_ISL29020_DEFAULT_ADDRESS=0x45
ISL29020 light sensor test application

Initializing ISL29020 sensor
[Failed]

Note : The sensor is not available for interfacing hence configurability of macros were only tested.

### Issues/PRs references

#12888
@leandrolanzieri  Please merge this only after  #13981

You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/13995

-- Commit Summary --

  * drivers/isl29020 : Add CONFIG_
  * drivers/isl29020 : Make ADDR configurable
  * drivers/isl29020 : Expose to Kconfig

-- File Changes --

    M drivers/Kconfig (5)
    M drivers/include/isl29020.h (12)
    A drivers/isl29020/Kconfig (25)
    M drivers/isl29020/include/isl29020_params.h (2)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/13995.patch
https://github.com/RIOT-OS/RIOT/pull/13995.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/13995
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200430/f5352b0b/attachment.htm>


More information about the notifications mailing list