Fix t-beam-s3-core display not working

[lewisxhe]

See #2519

https://meshtastic.discourse.group/t/new-lilygo-t-beam-supreme-product/7474
The conditional macro cannot recognize the enumeration variable, change it to a conditional statement to determine the hardware manufacturer model.

When the OLED power is not turned on, scanning the I2C port will cause the application to be stuck, because the I2C bus is not normal. It cannot scan to the device correctly.

[github-actions]

🤖 Pull request artifacts

file	commit
pr2520-firmware-2.1.13.78af6e2.zip	78af6e2

[geeksville]

This pull request has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/new-lilygo-t-beam-supreme-product/7474/28

[Tom9876]

Thank you very much for the quick fix. Display now working as expected.

`INFO | ??:??:?? 2 Scanning for i2c devices...

DEBUG | ??:??:?? 2 Scanning for i2c devices on port 2
DEBUG | ??:??:?? 3 I2C device found at address 0x34
INFO | ??:??:?? 3 axp192/axp2101 PMU found
DEBUG | ??:??:?? 3 I2C device found at address 0x51
INFO | ??:??:?? 3 PCF8563 RTC found
DEBUG | ??:??:?? 4 Scanning for i2c devices on port 1
DEBUG | ??:??:?? 4 I2C device found at address 0x1c
INFO | ??:??:?? 4 QMC6310 Highrate 3-Axis magnetic sensor found
DEBUG | ??:??:?? 5 I2C device found at address 0x3c
INFO | ??:??:?? 5 sh1106 display found
DEBUG | ??:??:?? 5 0x0 subtype probed in 1 tries
DEBUG | ??:??:?? 5 I2C device found at address 0x77
DEBUG | ??:??:?? 5 Wire.available() = 0
INFO | ??:??:?? 5 BMP-280 sensor found at address 0x77
INFO | ??:??:?? 6 5 I2C devices found`

Also the BMP-280 was detected. But - after activating telemetry in the settings - still no environmental data available. Any specific settings required?

[caveman99]

working for me too - i'm gonna fix the linter errors and accept the PR.

[geeksville]

This pull request has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/new-lilygo-t-beam-supreme-product/7474/33