hiby: backlight: use framebuffer blanking instead of PWM on hosted targets

This enables deeper sleep. On touchscreen devices (HiBy R1) it shuts down both screen and touch (touchscreen needs few seconds before it's actually disabled) Change-Id: I5e8a7dae840227ddf4433daa461cc7124db7676a
Fix red in 7e31f2a6d5
2026-05-12 11:43:16 -04:00 · 2026-01-19 14:23:07 -05:00 · 2026-01-19 13:49:51 -05:00 · 2026-01-19 12:47:57 -05:00 · 2026-01-19 16:58:02 +01:00 · 2026-01-19 09:24:41 -05:00
15 changed files with 48 additions and 14 deletions
--- a/firmware/export/config.h
+++ b/firmware/export/config.h
@ -386,17 +386,6 @@ Lyre prototype 1 */
 #define CPU 0
 #define COP 1

-/* imx233 specific: IMX233_PACKAGE */
-#define IMX233_BGA100   0
-#define IMX233_BGA169   1
-#define IMX233_TQFP100  2
-#define IMX233_TQFP128  3
-#define IMX233_LQFP100  4
-
-/* IMX233_PARTITIONS */
-#define IMX233_FREESCALE    (1 << 0) /* Freescale I.MX233 nonstandard two-level MBR */
-#define IMX233_CREATIVE     (1 << 1) /* Creative MBLK windowing */
-
 /* CONFIG_BUFLIB_BACKEND */
 #define BUFLIB_BACKEND_MEMPOOL      0 /* Default memory pool backed buflib */
 #define BUFLIB_BACKEND_MALLOC       1 /* malloc() buflib (for debugging) */
--- a/firmware/export/config/creativezen.h
+++ b/firmware/export/config/creativezen.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative Zen
 */
--- a/firmware/export/config/creativezenmozaic.h
+++ b/firmware/export/config/creativezenmozaic.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative ZEN Mozaic
 */
--- a/firmware/export/config/creativezenv.h
+++ b/firmware/export/config/creativezenv.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative ZEN V
 */
--- a/firmware/export/config/creativezenxfi.h
+++ b/firmware/export/config/creativezenxfi.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative Zen X-Fi
 */
--- a/firmware/export/config/creativezenxfi2.h
+++ b/firmware/export/config/creativezenxfi2.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative Zen X-Fi2
 */
--- a/firmware/export/config/creativezenxfi3.h
+++ b/firmware/export/config/creativezenxfi3.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative Zen X-Fi3
 */
--- a/firmware/export/config/creativezenxfistyle.h
+++ b/firmware/export/config/creativezenxfistyle.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Creative Zen X-Fi Style
 */
--- a/firmware/export/config/hibyr3proii.h
+++ b/firmware/export/config/hibyr3proii.h
@ -51,7 +51,6 @@

 /* Battery doesn't update its charging status, but the charger does. */
 #define POWER_DEV_NAME "mp2731-charger"
-#define BATTERY_DEV_NAME "mp2731-charger"

 /* Battery */
 #define BATTERY_TYPES_COUNT  1
--- a/firmware/export/config/imx233-config.h
+++ b/firmware/export/config/imx233-config.h
@ -0,0 +1,14 @@
+/*
+ * To be included by config/TARGET.h
+ */
+
+/* IMX233_PACKAGE */
+#define IMX233_BGA100   0
+#define IMX233_BGA169   1
+#define IMX233_TQFP100  2
+#define IMX233_TQFP128  3
+#define IMX233_LQFP100  4
+
+/* IMX233_PARTITIONS */
+#define IMX233_FREESCALE    (1 << 0) /* Freescale I.MX233 nonstandard two-level MBR */
+#define IMX233_CREATIVE     (1 << 1) /* Creative MBLK windowing */
--- a/firmware/export/config/sansafuzeplus.h
+++ b/firmware/export/config/sansafuzeplus.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Sandisk Sansa Fuze+
 */
--- a/firmware/export/config/sonynwze360.h
+++ b/firmware/export/config/sonynwze360.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Sony NWZ-E360 series
 */
--- a/firmware/export/config/sonynwze370.h
+++ b/firmware/export/config/sonynwze370.h
@ -1,3 +1,6 @@
+/* Pull in SoC-specific defines */
+#include "imx233-config.h"
+
 /*
 * This config file is for the Sony NWZ-E370/E380 series
 */
--- a/firmware/target/hosted/backlight-unix.c
+++ b/firmware/target/hosted/backlight-unix.c
@ -43,7 +43,9 @@ static const char * const sysfs_bl_brightness =
    "/sys/class/backlight/backlight_pwm0/brightness";

 static const char * const sysfs_bl_power =
-    "/sys/class/backlight/backlight_pwm0/bl_power";
+    /* Framebuffer powers off both touch (if available) and screen */
+    "/sys/class/graphics/fb0/blank";
+
 #else
 static const char * const sysfs_bl_brightness =
    "/sys/class/backlight/pwm-backlight.0/brightness";
--- a/firmware/usbstack/usb_audio.c
+++ b/firmware/usbstack/usb_audio.c
@ -938,7 +938,7 @@ static int db_to_usb_audio_volume(int db, int numdecimals)
    return tmp;
 }

-#if defined(LOGF_ENABLE) && defined(ROCKBOX_HAS_LOGF)
+#if defined(DEBUG) || (defined(LOGF_ENABLE) && defined(ROCKBOX_HAS_LOGF))
 static const char *usb_audio_ac_ctl_req_str(uint8_t cmd)
 {
    switch(cmd)
Author	SHA1	Message	Date
Roman Artiukhin	7bc856046b	hiby: backlight: use framebuffer blanking instead of PWM on hosted targets This enables deeper sleep. On touchscreen devices (HiBy R1) it shuts down both screen and touch (touchscreen needs few seconds before it's actually disabled) Change-Id: I5e8a7dae840227ddf4433daa461cc7124db7676a	2026-01-19 14:23:07 -05:00
Solomon Peachy	476d23ac3a	Fix red in `7e31f2a6d5` While the binary firmware build succeeded, the 'make zip' process failed because the script that put the default wps image together runs the device config header through CPP while only looking into the configuration header directory. So move the imx233-config.h header out of the target/arm/imx233 and into export/config where everything is copacetic. Change-Id: I9914558a892f8ff7ad839818f0a5ef687cc7b997	2026-01-19 13:49:51 -05:00
Aidan MacDonald	7e31f2a6d5	imx233: move SoC-specific config symbols to separate header Change-Id: Id044c59045712f1b96b93b5a93b07ddaa46bab7c	2026-01-19 12:47:57 -05:00
Marc Aarts	ef31aae6f4	Remove wrong define for HiBy R3 Pro II BATTERY_DEV_NAME resulting in the device shutting down almost instantly. Battery device name is 'battery' which is defaulted in hibylinux.h. Should fix FS#13760. Change-Id: I0aa3dc2c025fad94d70b37603ea42f93b442866e	2026-01-19 16:58:02 +01:00
Solomon Peachy	e13619b888	usb_audio: Fix compile failure with a (debug && !logf) build Change-Id: Ie571020b0187c791d02b00c82af2d0dbc2237227	2026-01-19 09:24:41 -05:00