absalom-muc / MHI-AC-Ctrl

Reads and writes data (e.g. power, mode, fan status etc.) from/to a Mitsubishi Heavy Industries (MHI) air conditioner (AC) via SPI controlled by MQTT
MIT License
279 stars 62 forks source link

KiCAD: J1 Connector pinout is wrong #167

Closed AlexCPU closed 10 months ago

AlexCPU commented 1 year ago

The KiCAD design was "fixed" in #155 to model a female connector correctly on the reverse of the board, which involved flipping the connector. However the schematic wasn't updated, so the PCB layout has the pins defined incorrectly: image You can see the unconnected nets, and the fact that different named nets are connected to the pins (easiest to see on the GND and +12V pins). I'll see if I can raise a PR for this.

Unrelated question: I'm making a new version of the board with an ESP32-C3 integrated (i.e. no extra D1 Mini) with mostly SMD parts, any preference in the repo where to put it?

glsf91 commented 1 year ago

Regarding the the esp32: Did you already tried to run MHI-AC-Ctrl with this ESP ? Because I got some troubles getting it running (without errors) on a ESP32.

AlexCPU commented 1 year ago

I haven't tried. And in all honesty, I'm going to be using the ESPHome fork rather than this native codebase, so I expect to just change the board type and rebuild in ESPHome.

glsf91 commented 1 year ago

That epshome fork is using a part of the mhi-ac-ctrl code. I have also tried with that esphome fork without success. So I suggest you first trying to get this up and running without errors before making a new version of the board :-)

My goal was to use a ESP32 because making it possible to use a temperature sensor with bluetooth. A lot of troubles getting this up and running.

oliv3r commented 9 months ago

Sorry for not noticing this !! Thanks @AlexCPU for the fix :)

Did you actually order some PCB's? :)