Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ESP32-S3 Add OneWire #1546

Open
28106830 opened this issue Oct 29, 2024 · 4 comments
Open

ESP32-S3 Add OneWire #1546

28106830 opened this issue Oct 29, 2024 · 4 comments

Comments

@28106830
Copy link

Library/API/IoT binding

nanoFramework.Device.OneWire1.4.22

Visual Studio version

vs2022(17.11.5)

.NET nanoFramework extension version

2022.3.0.101

Target name(s)

ESP32-S3,ESP32-S3_ALL,ESP32_S3_BLE

Firmware version

1.12.0.192,1.12.0.119

Device capabilities

These prompts when using DS18B20 examples
2>The connected target does not have support for the following assembly(ies):
2>
2> 'nanoFramework.Device. OneWire'
2>

Description

Hello, I am using
ESP32_S3_BLE-1.12.0.192
ESP32_S3-1.12.0.119
ESP32_S3_ALL-1.12.0.119
These prompts when using DS18B20 examples
2>The connected target does not have support for the following assembly(ies):
2>
2> 'nanoFramework.Device. OneWire'
2>
Did this feature not be enabled during compilation
I am using ESP32 S3 Dev Board A, which is functioning properly. I have tested IIC, SPI, and WIFI

How to reproduce

No response

Expected behaviour

I would like to add OneWire functionality to ESP32-S3

Screenshots

No response

Sample project or code

https://github.com/nanoframework/nanoFramework.IoT.Device/tree/develop/devices/Ds18b20

Aditional information

No response

@josesimoes
Copy link
Member

@AdrianSoundy I don't think this has been discussed before... was there a reason for disabling this on S3 builds? Memory issue, maybe?

@AdrianSoundy
Copy link
Member

@josesimoes I don't remember disabling it for any reason.
It was either never there before or some how got removed.

@josesimoes
Copy link
Member

josesimoes commented Oct 30, 2024

@AdrianSoundy got it.
So, nothing against enabling this for all ESP32 targets?

@AdrianSoundy
Copy link
Member

@AdrianSoundy got it. So, nothing against enabling this for all ESP32 targets?

No, go ahead and enable it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants