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
This commit is contained in:
Jens Arnold 2008-10-31 20:06:08 +00:00
parent cd9b1379ff
commit 3aa5832808
2 changed files with 7 additions and 2 deletions

View file

@ -147,7 +147,6 @@ static const struct plugin_api rockbox_api = {
gui_scrollbar_draw,
#endif
#ifdef HAVE_BACKLIGHT
backlight_on,
backlight_off,
backlight_set_timeout,
@ -159,7 +158,6 @@ static const struct plugin_api rockbox_api = {
backlight_set_timeout_plugged,
#endif
is_backlight_on,
#endif /* HAVE_BACKLIGHT */
splash,
splashf,

View file

@ -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);