Таймаут подсветки слоёв сбрасывается с 0 до 4 минут после перезапуска

[suregoodru]

Описание

На клавиатуре M4CR0Pad v3 настройка Layer LED timeout визуально сохраняется после изменения с 4 минут на 0, но после перезапуска Entropy снова отображается значение 4 минуты.

Проблема воспроизводится повторно.

Шаги воспроизведения

1. Подключить M4CR0Pad v3.
2. Открыть в Entropy раздел Layer LEDs.
3. Изменить таймаут с 4 минут на 0.
4. Дождаться сообщения Entropy об успешном сохранении.
5. Закрыть и снова запустить Entropy либо переподключить устройство.
6. Снова открыть раздел Layer LEDs.

Ожидаемый результат

После перезапуска или переподключения таймаут остаётся равным 0.

Фактический результат

Entropy снова отображает таймаут 4 минуты.

Дополнительный контекст

• Прошивку клавиатуры менять нельзя; исследуется возможность корректного исправления на стороне Entropy.
• Обычные настройки Vial RGB управляются отдельным путём и, по имеющимся данным, не включают Layer LED timeout.
• Пока рассматриваются варианты с кодированием значения и выбором QSID из схемы, сохранением в устройстве, отсутствием readback, завершением асинхронной очереди и обработкой полученного нуля. Первопричина ещё не заявлена.
• Если проблема подтвердится как исправимая в Entropy, будет подготовлен узкий PR с regression test и результатами проверки на реальном M4CR0Pad v3.

[suregoodru]

Проверено на реальном M4CR0Pad v3 с firmware 4.0.5 по USB.

Первопричина состоит из двух частей:

• QSID 317, width 1 и кодирование значения 0 в Entropy корректны; 0 не заменяется значением по умолчанию.
• Клавиатура объявляет настройку и подтверждает/эхом возвращает успешный CMD_VIAL_QMK_SETTINGS_SET, но последующий GET возвращает прежнее значение 4. Это воспроизводится и для 0, и для обычного значения 5. Отдельной команды commit/save для QMK Settings в протоколе нет.
• Entropy использовал для Layer LED режим SetOnly, поэтому принимал подтверждение SET за сохранение и мог показывать ложный Saved без проверки фактического значения.

Открыт draft PR #147: #147

Исправление включает отложенный readback для Layer LED по USB, возврат UI к фактическому значению и честную ошибку при несовпадении. Bluetooth остаётся на SetOnly, чтобы не вернуть ранее устранённую гонку с асинхронной записью flash в RMK.

Ручная проверка после исправления:

• запрос 0 -> read back 4, expected 0, UI возвращается к 4m;
• запрос 5 -> read back 4, expected 5, UI возвращается к 4m;
• после полного перезапуска Entropy отображается фактическое значение 4m;
• фоновый MacropadDisplay был изолирован на время HID-проверки.

Ограничение: firmware 4.0.5 фактически не применяет объявленную настройку, поэтому Entropy-side изменение не может заставить 0 сохраняться без исправления прошивки. PR устраняет ложный статус успешного сохранения и делает проблему видимой пользователю.

[suregoodru]

Обновление после исследования исходников firmware и совместной аппаратной проверки.

Подтверждённая причина в M4CR0Pad v3: устройство использует RGB_MATRIX, а реализация QSID 317 с EEPROM находилась только в RGBLIGHT-модуле. Поэтому вызывались weak getter/setter: getter возвращал значение по умолчанию, setter подтверждал команду, но ничего не сохранял. Отдельно Vial schema M4CR0Pad v3 не описывала этот параметр, из-за чего Entropy 0.3.4 мог показывать legacy-значение 128m.

Подготовлены два связанных исправления:

• Entropy Корректно проверять и отображать таймаут Layer LED #147 — readback после USB-записи и корректное отображение select-вариантов: Корректно проверять и отображать таймаут Layer LED #147
• vial-qmk Sync duplicate layer LED color settings #39 — EEPROM-backed runtime timeout для RGB Matrix M4CR0Pad v3 и Vial schema: Исправить сохранение таймаута RGB на M4CR0Pad v3 vial-qmk#39

Допустимые значения после исправления firmware: Never, 1 min, 2 min, 5 min, 10 min, 15 min, 30 min, 60 min. Never соответствует значению 0. Настройка относится к RGB-подсветке клавиш; таймаут дисплея остаётся отдельным и этим изменением не затрагивается.

Проверено на реальном M4CR0Pad v3:

• SET 317=0 сохраняется после полного USB-переподключения;
• ненулевые 5 min и 10 min также сохраняются;
• исправленная Entropy после перезапуска показывает Never или 10 min, а не 128m;
• после тестов пользовательское значение возвращено в Never и подтверждено GET 317=0;
• MacropadDisplay изолировался на время финального теста и не использует QSID 317.