gpio/uart pin use conflict on cozylife smartswitch (ln882h / ln-02)

[notzed]

Describe the bug

The ln822h port initialises uart0 and uart1 at some point and seems to use them for logging and/or stdio.

uart0 tx/rx uses GPIO A2/A3 (pin 2/3)
uart1 tx/rx uses GPIO B9/B8 (pin 25/24)

While it doesn't seem to affect basic operation, pin 24 is used as a GPIO input for the one button on this device. It also prevents the 3 other pins being used for gpio in a hacked device.

Firmware:

• Version 1.17.798
• Device (CozyLife) Mini Smart Switch (LN882H / LN-02)
• Chip/model: LN882H
• Device config {
  "vendor": "CozyLife",
  "bDetailed": "0",
  "name": "Mini Smart Switch",
  "model": "LN882H",
  "chip": "LN882H",
  "board": "LN-02",
  "flags": "1024",
  "keywords": [
  "switch module",
  "single relay",
  "single module"
  ],
  "pins": {
  "0": "TglChanOnTgl;0",
  "1": "Rel;0",
  "19": "LED_n;0",
  "24": "Btn;0"
  },
  "command": "PowerSave 1",
  "image": "https://obrazki.elektroda.pl/8132417200_1705759867.jpg",
  "wiki": "https://www.elektroda.pl/rtvforum/viewtopic.php?p=20919279#20919279"
  }

Additional context

I couldn't work out how to disable any output to these ports. But one fix is to ensure the alternative function is disabled in the pin setup functions so the pin is in the expected state since hal_gpio_init() doesn't do it. This is already done for pwm setup.

src/hal/ln882h/hal_pins_ln882h.c:My_LN882_Basic_GPIO_Setup ->

static void My_LN882_Basic_GPIO_Setup(lnPinMapping_t *pin, int direction) {
	gpio_init_t_def gpio_init = {
		.dir = direction,
		.pin = pin->pin,
		.speed = GPIO_HIGH_SPEED
	};

	hal_gpio_pin_afio_en(pin->base, pin->pin, HAL_DISABLE);
	hal_gpio_init(pin->base, &gpio_init);
}