* Update xsns_11_veml6070.ino
in Veml6070Detect check for the presence of both addresses in the bus to avoid misdetection of ATH20/21 (anyone with the device, please confirm this change still detect the device when present)
* Update xsns_16_tsl2561.ino
check the correct ID was returned to avoid misdetection of other sensors.
* Update xsns_57_tsl2591.ino - report channel values in JSON as in TSL2561
Add the raw infrared and broadband channels of the sensor to the JSON report like is done in the driver for TSL2561
* Update xsns_57_tsl2591.ino
Fix var name
* Update xsns_57_tsl2591.ino
Fix variable name
Command pointer table broken after refactoring 3a28938b91
The error with a comma before an entry instead of after, to match the other pointers, looks very obvious to me. Hence, I'm submitting this PR without having the hardware, and thus not able to do a runtime test.
I observed that the definition of command names became mangled with commit 2bc5f682b0 due to vertical bar not being placed where it should. With no bar after `D_CMND_CPU_FREQUENCY`, the command name became concatenated with the next `D_CMND_SETSENSOR` with the "funny" effect that command name `CpuFrequencySetSensor` got matched with `&CmndSetSensor`
The reason for it not being matched with `&CmndCpuFrequency` is that there is a bar too many before the `Info` command name, as previous command name lines all finish with `"|"`. Hence an extra command table entry was created, meaning that the mismatch "only" affected command names `Info`, `TouchCal`, `TouchThres`, `CpuFrequency` and `SetSensor`.
Issue is only for ESP32, as both mishaps happened in a block conditional on `#ifdef ESP32`.
Arguably, it could make sense to have all the lines with command names start with `"|"` as a consistent way of formatting the lines without the special case of ending `"|"` to be omitted for the very last command name line, but I wanted to only make minimal changes
