Revert r18947, and add a comment why the backlight functions are wanted on all targets.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@18949 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 7 insertions, 0 deletions

diff --git a/apps/plugin.h b/apps/plugin.h
index 634bbed0d8..4f747ea2b1 100644
--- a/apps/plugin.h
+++ b/apps/plugin.h
@@ -246,6 +246,13 @@ struct plugin_api {
 #endif  /* HAVE_LCD_BITMAP */
 
     /* backlight */
+    /* The backlight_* functions must be present in the API regardless whether
+     * HAVE_BACKLIGHT is defined or not. The reason is that the stock Ondio has
+     * no backlight but can be modded to have backlight (it's prepared on the
+     * PCB). This makes backlight an all-target feature API wise, and keeps API
+     * compatible between stock and modded Ondio.
+     * For OLED targets like the Sansa Clip, the backlight_* functions control
+     * the display enable, which has essentially the same effect. */
     void (*backlight_on)(void);
     void (*backlight_off)(void);
     void (*backlight_set_timeout)(int index);