David Lechner

OK, hopefully I got this working with all combinations now. Also rebased on current master, so other recent status light/bluetooth light changes should be included. Amending my previous summary of...

> Will test using build 3542 for this PR Yup, that should be the latest and greatest.

> I am understanding correctly that the following quirk should no longer be used anywhere? No, it is still needed. It is implemented here: ```c bool pbdrv_ioport_power_off(void) { init_ports(); //...

> I don't see the pin being set to input anywhere anymore? Ah, it doesn't need to be set to input, `pbdrv_ioport_enable_vcc(true);` has the same effect. > . I'm wondering...

> Seems like it should work the same as what I had done. I can test it this weekend. Done. Tested both movehub and cityhub. Confirmed problem exists before commit...

In the firmware, the `set_volume` method needs to be added to [pb_type_speaker.c](https://github.com/pybricks/pybricks-micropython/blob/master/pybricks/common/pb_type_speaker.c). Compared to the EV3 version, we will leave out the `which` parameter since the new hubs don't have...

It's really not that bad. But I have doubts to how well this method of volume control would work anyway since it essentially lowers the sound quality as it lowers...

From Homebrew you will need to install `dfu-util` or `libusb` to make flashing the firmware work. And `brew install --cask gcc-arm-bembedded` to install the cross-compiler. And I assume `poetry` is...

> 1. How is the Python API created for the `C` code? Is it auto-generated? It actually works much like regular Python. A type (class) is basically a lookup table...

The hubs without those don't have USB. If we remove the check, then it could be possible to install the wrong firmware on a device.