Asustor AS6302T And AS5304T fans and leds working under Proxmox

[Tki2000]

I have managed to get gpios for disks from my Asustor AS6302T and AS5304T and make them work under Proxmox.

I use Proxmox as I can assign a NAS VM instance the desired number of disks and manage them separately if I like. I can make backups in almost no time and get the rest of NASes working.
For backups, I only backup the main NAS VM without disks as they are assigned as a passthrough disk from Proxmox. The data on the disks is replicated from within VM to another location. No RAID nor fancy ZFS configurations. Just plain mirrors.

This is a "brief" tutorial to make the it87 chip work with proxmox so we can have fan control, and led signals with AS6302T and AS5304T.

In Proxmox, prepare the development environment:

apt install pve-headers build-essential git dkms
git clone https://github.com/mafredri/asustor-platform-driver
cd asustor-platform-driver

Now we have to edit file Makefile to reflect Proxmox headers location as they are not where expected:
Find:
KERNEL_MODULES := /lib/modules/$(TARGET)
Replace:
KERNEL_MODULES := /usr/lib/modules/$(TARGET)

I am writting the whole file asustor.c here as I do not have time to make a diff patch. If someone is willing to do, please do. Anyway, it is conflicting with the AS66 detection mechanism as the DMI info returned is the same as the AS53. I have just overridden the AS66 detection to the AS53 gpios. Tried to use another DMI info string as UUID but I could not get working and anyway I could not assure that it is not in other Asustor products. I have found it is common on another PCs and led me to believe it is a generic UUID anyway. So that leaves me without actions for making separate AS66 and AS53 detections.
I tried to separate the most code for the detected devices.
Here is the complete asustor.c code:

---

// SPDX-License-Identifier: GPL-2.0-or-later
/*
 * asustor.c - Platform driver for ASUSTOR NAS hardware
 *
 * Copyright (C) 2021 Mathias Fredriksson <[email protected]>
 */

#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt

#include <linux/dmi.h>
#include <linux/errno.h>
#include <linux/gpio/driver.h>
#include <linux/gpio/machine.h>
#include <linux/gpio_keys.h>
#include <linux/input.h>
#include <linux/kernel.h>
#include <linux/leds.h>
#include <linux/module.h>
#include <linux/platform_device.h>
#include <linux/version.h>

#define GPIO_IT87 "asustor_gpio_it87"
#define GPIO_ICH "gpio_ich"
#define GPIO_AS6100 "INT33FF:01"
#define GPIO_AS6300 "INT3452:00"
#define GPIO_AS5304 "INT3453:02"

#define DISK_ACT_LED(_name)                                                    \
	{                                                                      \
		.name            = _name ":green:disk",                        \
		.default_state   = LEDS_GPIO_DEFSTATE_ON,                      \
		.default_trigger = "disk-activity"                             \
	}
#define DISK_ERR_LED(_name)                                                    \
	{                                                                      \
		.name          = _name ":red:disk",                            \
		.default_state = LEDS_GPIO_DEFSTATE_OFF                        \
	}
#define NVME_ACT_LED(_name)                                                    \
	{                                                                      \
		.name          = _name ":green:disk",                          \
		.default_state = LEDS_GPIO_DEFSTATE_OFF                        \
	}
#define NVME_ERR_LED(_name)                                                    \
	{                                                                      \
		.name          = _name ":red:disk",                            \
		.default_state = LEDS_GPIO_DEFSTATE_OFF                        \
	}

// ASUSTOR Leds.
// If ledtrig-blkdev ever lands, use that instead of disk-activity:
// https://lore.kernel.org/linux-leds/[email protected]/
static struct gpio_led asustor_leds[] = {
	{ .name          = "power:front_panel",
	  .default_state = LEDS_GPIO_DEFSTATE_ON },                         // 0
	{ .name = "power:lcd", .default_state = LEDS_GPIO_DEFSTATE_ON },    // 1
	{ .name = "blue:power", .default_state = LEDS_GPIO_DEFSTATE_ON },   // 2
	{ .name = "red:power", .default_state = LEDS_GPIO_DEFSTATE_OFF },   // 3
	{ .name = "green:status", .default_state = LEDS_GPIO_DEFSTATE_ON }, // 4
	{
		.name            = "red:status",
		.default_state   = LEDS_GPIO_DEFSTATE_OFF,
		.panic_indicator = 1,
		.default_trigger = "panic",
	},                                                                // 5
	{ .name = "blue:usb", .default_state = LEDS_GPIO_DEFSTATE_OFF },  // 6
	{ .name = "green:usb", .default_state = LEDS_GPIO_DEFSTATE_OFF }, // 7
	{ .name = "blue:lan", .default_state = LEDS_GPIO_DEFSTATE_ON },   // 8
	DISK_ACT_LED("sata1"),                                            // 9
	DISK_ERR_LED("sata1"),                                            // 10
	DISK_ACT_LED("sata2"),                                            // 11
	DISK_ERR_LED("sata2"),                                            // 12
	DISK_ACT_LED("sata3"),                                            // 13
	DISK_ERR_LED("sata3"),                                            // 14
	DISK_ACT_LED("sata4"),                                            // 15
	DISK_ERR_LED("sata4"),                                            // 16
	DISK_ACT_LED("sata5"),                                            // 17
	DISK_ERR_LED("sata5"),                                            // 18
	DISK_ACT_LED("sata6"),                                            // 19
	DISK_ERR_LED("sata6"),                                            // 20
	NVME_ACT_LED("nvme1"),                                            // 21
	NVME_ERR_LED("nvme1"),                                            // 22
};

static const struct gpio_led_platform_data asustor_leds_pdata = {
	.leds     = asustor_leds,
	.num_leds = ARRAY_SIZE(asustor_leds),
};

// clang-format off
static struct gpiod_lookup_table asustor_fs6700_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
										// Red LEDs to the left and right of the
										// power button on the side of the unit
		GPIO_LOOKUP_IDX(GPIO_IT87, 52, NULL,  0, GPIO_ACTIVE_LOW),	// power:front_panel
		// 1
										// blue:power also controls the red LED
										// inside the power button on the side
		GPIO_LOOKUP_IDX(GPIO_IT87, 56, NULL,  2, GPIO_ACTIVE_LOW),	// blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,  8, NULL,  3, GPIO_ACTIVE_LOW),	// red:power
		GPIO_LOOKUP_IDX(GPIO_IT87, 31, NULL,  4, GPIO_ACTIVE_LOW),	// green:status
		GPIO_LOOKUP_IDX(GPIO_IT87, 49, NULL,  5, GPIO_ACTIVE_LOW),	// red:status
		// 6
		// 7
		GPIO_LOOKUP_IDX(GPIO_IT87, 55, NULL,  8, GPIO_ACTIVE_HIGH),	// blue:lan
		// 9
		// 10
		// 11
		// 12
		// 13
		// 14
		// 15
		// 16
		// 17
		// 18
		// 19
		// 20
		GPIO_LOOKUP_IDX(GPIO_IT87, 12, NULL, 21, GPIO_ACTIVE_LOW),	// nvme1:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 13, NULL, 22, GPIO_ACTIVE_LOW),	// nvme1:red:disk
		{}
	},
};

static struct gpiod_lookup_table asustor_6700_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87, 29, NULL,  0, GPIO_ACTIVE_HIGH),	// power:front_panel
		GPIO_LOOKUP_IDX(GPIO_IT87, 59, NULL,  1, GPIO_ACTIVE_HIGH),	// power:lcd
		GPIO_LOOKUP_IDX(GPIO_IT87, 56, NULL,  2, GPIO_ACTIVE_LOW),	// blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,  8, NULL,  3, GPIO_ACTIVE_LOW),	// red:power
		GPIO_LOOKUP_IDX(GPIO_IT87, 31, NULL,  4, GPIO_ACTIVE_LOW),	// green:status
		GPIO_LOOKUP_IDX(GPIO_IT87, 49, NULL,  5, GPIO_ACTIVE_LOW),	// red:status
		// 6
		GPIO_LOOKUP_IDX(GPIO_IT87, 21, NULL,  7, GPIO_ACTIVE_LOW),	// green:usb
		GPIO_LOOKUP_IDX(GPIO_IT87, 55, NULL,  8, GPIO_ACTIVE_HIGH),	// blue:lan
		GPIO_LOOKUP_IDX(GPIO_IT87, 12, NULL,  9, GPIO_ACTIVE_HIGH),	// sata1:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 13, NULL, 10, GPIO_ACTIVE_LOW),	// sata1:red:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 46, NULL, 11, GPIO_ACTIVE_HIGH),	// sata2:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 47, NULL, 12, GPIO_ACTIVE_LOW),	// sata2:red:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 51, NULL, 13, GPIO_ACTIVE_HIGH),	// sata3:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 52, NULL, 14, GPIO_ACTIVE_LOW),	// sata3:red:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 63, NULL, 15, GPIO_ACTIVE_HIGH),	// sata4:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 48, NULL, 16, GPIO_ACTIVE_LOW),	// sata4:red:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 61, NULL, 17, GPIO_ACTIVE_HIGH),	// sata5:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 62, NULL, 18, GPIO_ACTIVE_LOW),	// sata5:red:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 58, NULL, 19, GPIO_ACTIVE_HIGH),	// sata6:green:disk
		GPIO_LOOKUP_IDX(GPIO_IT87, 60, NULL, 20, GPIO_ACTIVE_LOW),	// sata6:red:disk
		{}
	},
};

static struct gpiod_lookup_table asustor_6100_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87,   29, NULL,  0, GPIO_ACTIVE_HIGH), // power:front_panel
		GPIO_LOOKUP_IDX(GPIO_IT87,   59, NULL,  1, GPIO_ACTIVE_HIGH), // power:lcd
		GPIO_LOOKUP_IDX(GPIO_IT87,   56, NULL,  2, GPIO_ACTIVE_LOW),  // blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,    8, NULL,  3, GPIO_ACTIVE_LOW),  // red:power
		GPIO_LOOKUP_IDX(GPIO_IT87,   31, NULL,  4, GPIO_ACTIVE_LOW),  // green:status
		GPIO_LOOKUP_IDX(GPIO_AS6100, 21, NULL,  5, GPIO_ACTIVE_HIGH), // red:status
		// 6
		GPIO_LOOKUP_IDX(GPIO_IT87,   21, NULL,  7, GPIO_ACTIVE_LOW),  // green:usb
		GPIO_LOOKUP_IDX(GPIO_IT87,   52, NULL,  8, GPIO_ACTIVE_HIGH), // blue:lan
		GPIO_LOOKUP_IDX(GPIO_AS6100, 24, NULL,  9, GPIO_ACTIVE_LOW),  // sata1:green:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 15, NULL, 10, GPIO_ACTIVE_HIGH), // sata1:red:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 22, NULL, 11, GPIO_ACTIVE_LOW),  // sata2:green:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 19, NULL, 12, GPIO_ACTIVE_HIGH), // sata2:red:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 25, NULL, 13, GPIO_ACTIVE_LOW),  // sata3:green:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 16, NULL, 14, GPIO_ACTIVE_HIGH), // sata3:red:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 18, NULL, 15, GPIO_ACTIVE_LOW),  // sata4:green:disk
		GPIO_LOOKUP_IDX(GPIO_AS6100, 17, NULL, 16, GPIO_ACTIVE_HIGH), // sata4:red:disk
		{}
	},
};

static struct gpiod_lookup_table asustor_6300_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87,   29, NULL,  0, GPIO_ACTIVE_HIGH), // power:front_panel
		GPIO_LOOKUP_IDX(GPIO_IT87,   59, NULL,  1, GPIO_ACTIVE_HIGH), // power:lcd
		GPIO_LOOKUP_IDX(GPIO_IT87,   56, NULL,  2, GPIO_ACTIVE_LOW),  // blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,    8, NULL,  3, GPIO_ACTIVE_LOW),  // red:power
		GPIO_LOOKUP_IDX(GPIO_IT87,   31, NULL,  4, GPIO_ACTIVE_LOW),  // green:status
		GPIO_LOOKUP_IDX(GPIO_AS6300, 20, NULL,  5, GPIO_ACTIVE_HIGH), // red:status
		// 6
		GPIO_LOOKUP_IDX(GPIO_IT87,   21, NULL,  7, GPIO_ACTIVE_LOW),  // green:usb
		GPIO_LOOKUP_IDX(GPIO_IT87,   55, NULL,  8, GPIO_ACTIVE_HIGH), // blue:lan
		GPIO_LOOKUP_IDX(GPIO_AS6300, 23, NULL,  9, GPIO_ACTIVE_HIGH), // sata1:green:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS6300, 24, NULL, 10, GPIO_ACTIVE_HIGH), // sata1:red:disk 	++
		GPIO_LOOKUP_IDX(GPIO_AS6300, 22, NULL, 11, GPIO_ACTIVE_HIGH), // sata2:green:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS6300, 21, NULL, 12, GPIO_ACTIVE_HIGH), // sata2:red:disk 	++
		{}
		// GPIO_LOOKUP_IDX(GPIO_IT87,   53, NULL, 15, GPIO_ACTIVE_HIGH), // beep:status <<-- It can beep too!!!
	},
};

static struct gpiod_lookup_table asustor_5304_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87,   29, NULL,  0, GPIO_ACTIVE_HIGH), // power:front_panel
		GPIO_LOOKUP_IDX(GPIO_IT87,   59, NULL,  1, GPIO_ACTIVE_HIGH), // power:lcd
		GPIO_LOOKUP_IDX(GPIO_IT87,   56, NULL,  2, GPIO_ACTIVE_LOW),  // blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,    8, NULL,  3, GPIO_ACTIVE_LOW),  // red:power
		GPIO_LOOKUP_IDX(GPIO_IT87,   31, NULL,  4, GPIO_ACTIVE_LOW),  // green:status
		GPIO_LOOKUP_IDX(GPIO_AS5304, 20, NULL,  5, GPIO_ACTIVE_HIGH), // red:status
		//GPIO_LOOKUP_IDX(GPIO_IT87,   21, NULL,  6, GPIO_ACTIVE_LOW),  // blue:usb
		GPIO_LOOKUP_IDX(GPIO_IT87,   21, NULL,  7, GPIO_ACTIVE_LOW),  // green:usb
		GPIO_LOOKUP_IDX(GPIO_IT87,   55, NULL,  8, GPIO_ACTIVE_HIGH), // blue:lan

		GPIO_LOOKUP_IDX(GPIO_AS5304, 1,  NULL,  9, GPIO_ACTIVE_HIGH), // sata1:green:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 9,  NULL, 10, GPIO_ACTIVE_LOW),  // sata1:red:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 2,  NULL, 11, GPIO_ACTIVE_HIGH), // sata2:green:disk 	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 10, NULL, 12, GPIO_ACTIVE_LOW),  // sata2:red:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 4,  NULL, 13, GPIO_ACTIVE_HIGH), // sata3:green:disk 	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 11, NULL, 14, GPIO_ACTIVE_LOW),  // sata3:red:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 6,  NULL, 15, GPIO_ACTIVE_HIGH), // sata4:green:disk	++
		GPIO_LOOKUP_IDX(GPIO_AS5304, 13, NULL, 16, GPIO_ACTIVE_LOW),  // sata4:red:disk 	++
		{}
	},
};

static struct gpiod_lookup_table asustor_600_gpio_leds_lookup = {
	.dev_id = "leds-gpio",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87, 29, NULL, 0, GPIO_ACTIVE_HIGH), // power:front_panel
		GPIO_LOOKUP_IDX(GPIO_IT87, 59, NULL, 1, GPIO_ACTIVE_HIGH), // power:lcd
		GPIO_LOOKUP_IDX(GPIO_IT87, 56, NULL, 2, GPIO_ACTIVE_LOW),  // blue:power
		GPIO_LOOKUP_IDX(GPIO_IT87,  8, NULL, 3, GPIO_ACTIVE_LOW),  // red:power
		GPIO_LOOKUP_IDX(GPIO_IT87, 31, NULL, 4, GPIO_ACTIVE_LOW),  // green:status
		GPIO_LOOKUP_IDX(GPIO_ICH,  27, NULL, 5, GPIO_ACTIVE_HIGH), // red:status
		GPIO_LOOKUP_IDX(GPIO_IT87, 21, NULL, 6, GPIO_ACTIVE_LOW),  // blue:usb
		// 7
		GPIO_LOOKUP_IDX(GPIO_IT87, 52, NULL, 8, GPIO_ACTIVE_HIGH), // blue:lan
		{}
	},
};
// clang-format on

// ASUSTOR Buttons.
// Unfortunately, gpio-keys-polled does not use gpio lookup tables.
static struct gpio_keys_button asustor_gpio_keys_table[] = {
	{
		.desc       = "USB Copy Button",
		.code       = KEY_COPY,
		.type       = EV_KEY,
		.active_low = 1,
		.gpio       = -1, // Invalid, set in init.
	},
	{
		.desc       = "Power Button",
		.code       = KEY_POWER,
		.type       = EV_KEY,
		.active_low = 1,
		.gpio       = -1, // Invalid, set in init.
	},
};

static struct gpio_keys_platform_data asustor_keys_pdata = {
	.buttons       = asustor_gpio_keys_table,
	.nbuttons      = ARRAY_SIZE(asustor_gpio_keys_table),
	.poll_interval = 50,
	.name          = "asustor-keys",
};

// clang-format off
static struct gpiod_lookup_table asustor_fs6700_gpio_keys_lookup = {
	.dev_id = "gpio-keys-polled",
	.table = {
		// 0 (There is no USB Copy Button).
		// 1 (Power Button is already handled properly via ACPI).
		{}
	},
};

static struct gpiod_lookup_table asustor_6100_gpio_keys_lookup = { // same for 6700
	.dev_id = "gpio-keys-polled",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87, 20, NULL, 0, GPIO_ACTIVE_LOW),
		// 1 (Power Button is already handled properly via ACPI).
		{}
	},
};

static struct gpiod_lookup_table asustor_600_gpio_keys_lookup = {
	.dev_id = "gpio-keys-polled",
	.table = {
		GPIO_LOOKUP_IDX(GPIO_IT87, 20, NULL, 0, GPIO_ACTIVE_LOW),
		GPIO_LOOKUP_IDX(GPIO_IT87, 27, NULL, 1, GPIO_ACTIVE_LOW),
		{}
	},
};
// clang-format on

// ASUSTOR Platform.
struct asustor_driver_data {
	struct gpiod_lookup_table *leds;
	struct gpiod_lookup_table *keys;
};

static struct asustor_driver_data asustor_fs6700_driver_data = {
	.leds = &asustor_fs6700_gpio_leds_lookup,
	.keys = &asustor_fs6700_gpio_keys_lookup,
};

static struct asustor_driver_data asustor_6700_driver_data = {
	.leds = &asustor_6700_gpio_leds_lookup,
	.keys = &asustor_6100_gpio_keys_lookup,
};

static struct asustor_driver_data asustor_6100_driver_data = {
	.leds = &asustor_6100_gpio_leds_lookup,
	.keys = &asustor_6100_gpio_keys_lookup,
};

static struct asustor_driver_data asustor_6300_driver_data = {
	.leds = &asustor_6300_gpio_leds_lookup,
	.keys = &asustor_6100_gpio_keys_lookup,
};

static struct asustor_driver_data asustor_5304_driver_data = {
	.leds = &asustor_5304_gpio_leds_lookup,
	.keys = &asustor_6100_gpio_keys_lookup,
};

static struct asustor_driver_data asustor_600_driver_data = {
	.leds = &asustor_600_gpio_leds_lookup,
	.keys = &asustor_600_gpio_keys_lookup,
};

static const struct dmi_system_id asustor_systems[] = {
	{
		// Note: This uses the BIOS release date to help match the FS67xx,
		//       because otherwise it matches the AS670xT, AS540xT and others
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Jasper Lake Client Platform"),
			DMI_EXACT_MATCH(DMI_BIOS_DATE, "09/15/2023"),
		},
		.driver_data = &asustor_fs6700_driver_data,
	},
	{
		// Note: This not only matches (and works with) AS670xT (Lockerstore Gen2),
		//       but also AS540xT (Nimbustor Gen2)
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Jasper Lake Client Platform"),
		},
		.driver_data = &asustor_6700_driver_data,
	},
	{
		// AS6
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "ASUSTOR Inc."),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "AS-6xxT"),
		},
		.driver_data = &asustor_600_driver_data,
	},
	{
		// AS61
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Insyde"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "AS61xx"),
		},
		.driver_data = &asustor_6100_driver_data,
	},
	{
		// AS63
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Apollolake I Platform"),
		},
		.driver_data = &asustor_6300_driver_data,
	},
	{
		// AS66
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Insyde"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "GeminiLake"),
		},
		.driver_data = &asustor_5304_driver_data,
		//.driver_data = &asustor_6700_driver_data,
	},
	{
		// AS5304 -- Not working properly
		.matches = {
			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Insyde"),
			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "GeminiLake"),
			//DMI_EXACT_MATCH(DMI_PRODUCT_UUID, "12345678-1234-5678-90ab-cddeefaabbcc"),
		},
		.driver_data = &asustor_5304_driver_data,
	},
	{}
};
MODULE_DEVICE_TABLE(dmi, asustor_systems);

static struct asustor_driver_data *driver_data;
static struct platform_device *asustor_leds_pdev;
static struct platform_device *asustor_keys_pdev;

static struct platform_device *__init asustor_create_pdev(const char *name,
                                                          const void *pdata,
                                                          size_t sz)
{
	struct platform_device *pdev;

	pdev = platform_device_register_data(NULL, name, PLATFORM_DEVID_NONE,
	                                     pdata, sz);
	if (IS_ERR(pdev))
		pr_err("failed registering %s: %ld\n", name, PTR_ERR(pdev));

	return pdev;
}

#if LINUX_VERSION_CODE < KERNEL_VERSION(6, 7, 0)
static int gpiochip_match_name(struct gpio_chip *chip, void *data)
{
	const char *name = data;

	return !strcmp(chip->label, name);
}

static struct gpio_chip *find_chip_by_name(const char *name)
{
	return gpiochip_find((void *)name, gpiochip_match_name);
}
#else
// DG: kernel 6.7 removed gpiochip_find() and introduced gpio_device_find()
//     and friends instead
static struct gpio_chip *find_chip_by_name(const char *name)
{
	struct gpio_device *dev = gpio_device_find_by_label(name);
	return (dev != NULL) ? gpio_device_get_chip(dev) : NULL;
}
#endif

// TODO(mafredri): Allow force model for testing.
static int __init asustor_init(void)
{
	const struct dmi_system_id *system;
	const struct gpiod_lookup *keys_table;
	int ret, i;

	system = dmi_first_match(asustor_systems);
	if (!system) {
		pr_info("No supported ASUSTOR mainboard found");
		return -ENODEV;
	}

	if (strlen(system->matches[2].substr))
		pr_info("Found %s/%s/%s\n", system->matches[0].substr,
	        	system->matches[1].substr, system->matches[2].substr);
	else
		pr_info("Found %s/%s\n", system->matches[0].substr,
	        	system->matches[1].substr);

	driver_data = system->driver_data;
	gpiod_add_lookup_table(driver_data->leds);
	gpiod_add_lookup_table(driver_data->keys);

	for (i = 0; i < ARRAY_SIZE(asustor_gpio_keys_table); i++) {
		// This is here simply because gpio-keys-polled does
		// not support gpio lookups.
		keys_table = driver_data->keys->table;
		for (; keys_table->key != NULL; keys_table++) {
			if (i == keys_table->idx) {
				// add the GPIO chip's base, so we get the absolute (global) gpio number
				struct gpio_chip *chip =
					find_chip_by_name(keys_table->key);
				if (chip == NULL)
					continue;
				asustor_gpio_keys_table[i].gpio =
					chip->base + keys_table->chip_hwnum;
			}
		}
	}

	// TODO(mafredri): Handle number of disk slots -> enabled LEDs.
	asustor_leds_pdev = asustor_create_pdev(
		"leds-gpio", &asustor_leds_pdata, sizeof(asustor_leds_pdata));
	if (IS_ERR(asustor_leds_pdev)) {
		ret = PTR_ERR(asustor_leds_pdev);
		goto err;
	}

	asustor_keys_pdev = asustor_create_pdev(
		"gpio-keys-polled", &asustor_keys_pdata, sizeof(asustor_keys_pdata));
	if (IS_ERR(asustor_keys_pdev)) {
		ret = PTR_ERR(asustor_keys_pdev);
		platform_device_unregister(asustor_leds_pdev);
		goto err;
	}

	return 0;

err:
	gpiod_remove_lookup_table(driver_data->leds);
	gpiod_remove_lookup_table(driver_data->keys);
	return ret;
}

static void __exit asustor_cleanup(void)
{
	platform_device_unregister(asustor_leds_pdev);
	platform_device_unregister(asustor_keys_pdev);

	gpiod_remove_lookup_table(driver_data->leds);
	gpiod_remove_lookup_table(driver_data->keys);
}

module_init(asustor_init);
module_exit(asustor_cleanup);

MODULE_AUTHOR("Mathias Fredriksson <[email protected]>");
MODULE_DESCRIPTION("Platform driver for ASUSTOR NAS hardware");
MODULE_LICENSE("GPL");
MODULE_ALIAS("platform:asustor");
MODULE_SOFTDEP("pre: asustor-it87 asustor-gpio-it87 gpio-ich"
               " platform:leds-gpio"
               " platform:gpio-keys-polled");

---

Now we can compile it:

make clean
make
make install
modprobe asustor_it87
modprobe asustor

If everything went ok, we can load the module at start up in /etc/modules with asustor_it87

Now install the fan control application:

apt-get install lm-sensors
apt-get install fancontrol
pwmconfig

Let the applicaction config the fan parameters
Here is my config file /etc/fancontrol for AS5304 just for reference:

# Configuration file generated by pwmconfig, changes will be lost
INTERVAL=10
DEVPATH=hwmon1=devices/platform/coretemp.0 hwmon2=devices/platform/asustor_it87.656
DEVNAME=hwmon1=coretemp hwmon2=it8625
FCTEMPS=hwmon2/pwm1=hwmon1/temp1_input
FCFANS= hwmon2/pwm1=hwmon2/fan1_input
MINTEMP=hwmon2/pwm1=40
MAXTEMP=hwmon2/pwm1=70
MINSTART=hwmon2/pwm1=50
MINSTOP=hwmon2/pwm1=50
MINPWM=hwmon2/pwm1=45
MAXPWM=hwmon2/pwm1=150

Now we can define custom scripts at Proxmox startup to configure disk LEDs and do some work.
I have found that I can assign the usb-host trigger to work on AS6302, but not on AS5304. Maybe something is wrong or they do not behave the same. Well not completely true as I can assign that trigger to the disk leds but not to the usb led... mmmm
By the way, Proxmox is booting from a M.2 disk attached to a USB-M.2 SATA enclosure, so the disk bays are completely free to be passed to VMs as a real NAS.

set_asustor_leds.bat

echo r8169-0-200:00:link | tee /sys/devices/platform/leds-gpio/leds/blue:lan/trigger
echo 1  | tee /sys/devices/platform/leds-gpio/leds/green:usb/brightness
echo usb-host | tee /sys/devices/platform/leds-gpio/leds/green:usb/trigger
echo 47 | tee /sys/devices/platform/asustor_it87.*/hwmon/hwmon*/gpled2_blink
echo 10 | tee /sys/devices/platform/asustor_it87.*/hwmon/hwmon*/gpled2_blink_freq
echo none | tee /sys/devices/platform/leds-gpio/leds/red:power/trigger

echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata4:green:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata3:green:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata2:green:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata1:green:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata4:green:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata3:green:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata2:green:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata1:green:disk/brightness
sleep 0.5

echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata4:red:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata3:red:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata2:red:disk/brightness
sleep 0.5
echo 1 | tee /sys/devices/platform/leds-gpio/leds/sata1:red:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata4:red:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata3:red:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata2:red:disk/brightness
sleep 0.5
echo 0 | tee /sys/devices/platform/leds-gpio/leds/sata1:red:disk/brightness

set_asustor_disks_leds.bat

lsblk -o NAME,SERIAL,WWN,HCTL | grep 0:0 | awk '{ print $1 " " $4 }' > /var/log/disks_present.log
sleep 1
if grep -q "0:0:0:0" /var/log/disks_present.log; then
   echo 1 > /sys/devices/platform/leds-gpio/leds/sata1:green:disk/brightness
   echo disk-activity > /sys/devices/platform/leds-gpio/leds/sata1:green:disk/trigger
else
   echo 0 > /sys/devices/platform/leds-gpio/leds/sata1:green:disk/brightness
   echo none > /sys/devices/platform/leds-gpio/leds/sata1:green:disk/trigger
fi
if grep -q "1:0:0:0" /var/log/disks_present.log; then
   echo 1 > /sys/devices/platform/leds-gpio/leds/sata2:green:disk/brightness
   echo disk-activity > /sys/devices/platform/leds-gpio/leds/sata2:green:disk/trigger
else
   echo 0 > /sys/devices/platform/leds-gpio/leds/sata2:green:disk/brightness
   echo none > /sys/devices/platform/leds-gpio/leds/sata2:green:disk/trigger
fi
if grep -q "2:0:0:0" /var/log/disks_present.log; then
   echo 1 > /sys/devices/platform/leds-gpio/leds/sata3:green:disk/brightness
   echo disk-activity > /sys/devices/platform/leds-gpio/leds/sata3:green:disk/trigger
else
   echo 0 > /sys/devices/platform/leds-gpio/leds/sata3:green:disk/brightness
   echo none > /sys/devices/platform/leds-gpio/leds/sata3:green:disk/trigger
fi
if grep -q "3:0:0:0" /var/log/disks_present.log; then
   echo 1 > /sys/devices/platform/leds-gpio/leds/sata4:green:disk/brightness
   echo disk-activity > /sys/devices/platform/leds-gpio/leds/sata4:green:disk/trigger
else
   echo 0 > /sys/devices/platform/leds-gpio/leds/sata4:green:disk/brightness
   echo none > /sys/devices/platform/leds-gpio/leds/sata4:green:disk/trigger
fi
if grep -q "4:0:0:0" /var/log/disks_present.log; then
   echo 1 > /sys/devices/platform/leds-gpio/leds/green:usb/brightness
   echo usb-host > /sys/devices/platform/leds-gpio/leds/green:usb/trigger
else
   echo 0 > /sys/devices/platform/leds-gpio/leds/green:usb/brightness
   echo none > /sys/devices/platform/leds-gpio/leds/green:usb/trigger
fi

set_asustor_disks_leds_error.bat

if grep -q "0:0:0:0" /var/log/disks_present.log; then
   disk1=`cat /var/log/disks_present.log | grep "0:0:0:0" | awk '{ print $1 }'`
else
   disk1=""
fi

if grep -q "1:0:0:0" /var/log/disks_present.log; then
   disk2=`cat /var/log/disks_present.log | grep "1:0:0:0" | awk '{ print $1 }'`
else
   disk2=""
fi

if grep -q "2:0:0:0" /var/log/disks_present.log; then
   disk3=`cat /var/log/disks_present.log | grep "2:0:0:0" | awk '{ print $1 }'`
else
   disk3=""
fi

if grep -q "3:0:0:0" /var/log/disks_present.log; then
   disk4=`cat /var/log/disks_present.log | grep "3:0:0:0" | awk '{ print $1 }'`
else
   disk4=""
fi

if grep -q "4:0:0:0" /var/log/disks_present.log; then
   disk5=`cat /var/log/disks_present.log | grep "3:0:0:0" | awk '{ print $1 }'`
else
   disk5=""
fi

if [ ! -z $disk1 ]; then
   error1=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Sector_Ct' | awk '{ print $10 }' | tr '\n' ' '`
   error12=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Event_Count' | awk '{ print $10 }' | tr '\n' ' '`
   error13=`smartctl -a /dev/$disk1 | grep -E 'Uncorrectable_Error_Cnt' | awk '{ print $10 }' | tr '\n' ' '`
   error14=`smartctl -a /dev/$disk1 | grep -E 'Current_Pending_Sector' | awk '{ print $10 }' | tr '\n' ' '`
   if [ ! -z $error12 ]; then
      error1=$((error1 + error12)) 
   fi
   if [ ! -z $error13 ]; then
      error1=$((error1 + error13)) 
   fi
   if [ ! -z $error14 ]; then
      error1=$((error1 + error14)) 
   fi
   if [ $error1 -gt 0 ]; then
      #echo "Error"
      echo 1 > /sys/devices/platform/leds-gpio/leds/sata1:red:disk/brightness
   else
      echo 0 > /sys/devices/platform/leds-gpio/leds/sata1:red:disk/brightness
   fi
fi

if [ ! -z $disk2 ]; then
   error2=`smartctl -a /dev/$disk2 | grep -E 'Reallocated_Sector_Ct' | awk '{ print $10 }' | tr '\n' ' '`
   error22=`smartctl -a /dev/$disk2 | grep -E 'Reallocated_Event_Count' | awk '{ print $10 }' | tr '\n' ' '`
   error23=`smartctl -a /dev/$disk2 | grep -E 'Uncorrectable_Error_Cnt' | awk '{ print $10 }' | tr '\n' ' '`
   error24=`smartctl -a /dev/$disk2 | grep -E 'Current_Pending_Sector' | awk '{ print $10 }' | tr '\n' ' '`
   if [ ! -z $error22 ]; then
      error2=$((error2 + error22)) 
   fi
   if [ ! -z $error23 ]; then
      error2=$((error2 + error23)) 
   fi
   if [ ! -z $error24 ]; then
      error2=$((error2 + error24)) 
   fi
   if [ $error2 -gt 0 ]; then
      #echo "Error"
      echo 1 > /sys/devices/platform/leds-gpio/leds/sata2:red:disk/brightness
   else
      echo 0 > /sys/devices/platform/leds-gpio/leds/sata2:red:disk/brightness
   fi
fi

if [ ! -z $disk3 ]; then
   error3=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Sector_Ct' | awk '{ print $10 }' | tr '\n' ' '`
   error32=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Event_Count' | awk '{ print $10 }' | tr '\n' ' '`
   error33=`smartctl -a /dev/$disk1 | grep -E 'Uncorrectable_Error_Cnt' | awk '{ print $10 }' | tr '\n' ' '`
   error34=`smartctl -a /dev/$disk1 | grep -E 'Current_Pending_Sector' | awk '{ print $10 }' | tr '\n' ' '`
   if [ ! -z $error32 ]; then
      error3=$((error3 + error32)) 
   fi
   if [ ! -z $error33 ]; then
      error3=$((error3 + error33)) 
   fi
   if [ ! -z $error34 ]; then
      error3=$((error3 + error34)) 
   fi
   if [ $error3 -gt 0 ]; then
      #echo "Error"
      echo 1 > /sys/devices/platform/leds-gpio/leds/sata3:red:disk/brightness
   else
      echo 0 > /sys/devices/platform/leds-gpio/leds/sata3:red:disk/brightness
   fi
fi

if [ ! -z $disk4 ]; then
   error4=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Sector_Ct' | awk '{ print $10 }' | tr '\n' ' '`
   error42=`smartctl -a /dev/$disk1 | grep -E 'Reallocated_Event_Count' | awk '{ print $10 }' | tr '\n' ' '`
   error43=`smartctl -a /dev/$disk1 | grep -E 'Uncorrectable_Error_Cnt' | awk '{ print $10 }' | tr '\n' ' '`
   error44=`smartctl -a /dev/$disk1 | grep -E 'Current_Pending_Sector' | awk '{ print $10 }' | tr '\n' ' '`
   if [ ! -z $error42 ]; then
      error4=$((error4 + error42)) 
   fi
   if [ ! -z $error43 ]; then
      error4=$((error4 + error43)) 
   fi
   if [ ! -z $error44 ]; then
      error4=$((error4 + error44)) 
   fi
   if [ $error4 -gt 0 ]; then
      #echo "Error"
      echo 1 > /sys/devices/platform/leds-gpio/leds/sata4:red:disk/brightness
   else
      echo 0 > /sys/devices/platform/leds-gpio/leds/sata4:red:disk/brightness
   fi
fi

if [ ! -z $disk5 ]; then
   error5=`smartctl -a /dev/$disk3 | grep -E 'Reallocated_Sector_Ct' | awk '{ print $10 }' | tr '\n' ' '`
   error52=`smartctl -a /dev/$disk3 | grep -E 'Reallocated_Event_Count' | awk '{ print $10 }' | tr '\n' ' '`
   error53=`smartctl -a /dev/$disk3 | grep -E 'Uncorrectable_Error_Cnt' | awk '{ print $10 }' | tr '\n' ' '`
   error54=`smartctl -a /dev/$disk3 | grep -E 'Current_Pending_Sector' | awk '{ print $10 }' | tr '\n' ' '`
   if [ ! -z $error52 ]; then
      error5=$((error5 + error52)) 
   fi
   if [ ! -z $error33 ]; then
      error5=$((error5 + error53)) 
   fi
   if [ ! -z $error34 ]; then
      error5=$((error5 + error54)) 
   fi
   if [ $error5 -gt 0 ]; then
      #echo "Error"
      echo 1 > /sys/devices/platform/leds-gpio/leds/red:power/brightness
   else
      echo 0 > /sys/devices/platform/leds-gpio/leds/red:power/brightness
   fi
fi

---

Make the bat files executable with chmod +x *.bat
Yes, I use .bat extension to differentiate the custom made script files on a Linux system. With that extension I can do a whole search for custom made scripts on a system in a blink, and backup them all.
Now we can manage the disks leds by executing the script files periodically or at bootup time.
For example:
We can call a script from crontab at boot time for configuring the leds.
We can call a script every 24 hours to call the error leds check.

So you can get the idea for making the leds come to life.

I hope that it could be of use to anyone.