(libretro) Go through new logger interface
authortwinaphex <libretro@gmail.com>
Tue, 17 Dec 2013 11:20:16 +0000 (12:20 +0100)
committertwinaphex <libretro@gmail.com>
Tue, 17 Dec 2013 11:20:16 +0000 (12:20 +0100)
platform/libretro.c
platform/libretro.h [changed mode: 0644->0755]

index 23f0694..67b4369 100644 (file)
 #include "common/version.h"
 #include "libretro.h"
 
+static retro_log_printf_t log_cb;
 static retro_video_refresh_t video_cb;
 static retro_input_poll_t input_poll_cb;
 static retro_input_state_t input_state_cb;
 static retro_environment_t environ_cb;
 static retro_audio_sample_batch_t audio_batch_cb;
 
-static FILE *emu_log;
-
 #define VOUT_MAX_WIDTH 320
 #define VOUT_MAX_HEIGHT 240
 static void *vout_buf;
@@ -176,13 +175,15 @@ void *plat_mmap(unsigned long addr, size_t size, int need_exec, int is_fixed)
    req = (void *)addr;
    ret = mmap(req, size, PROT_READ | PROT_WRITE, flags, -1, 0);
    if (ret == MAP_FAILED) {
-      lprintf("mmap(%08lx, %zd) failed: %d\n", addr, size, errno);
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "mmap(%08lx, %zd) failed: %d\n", addr, size, errno);
       return NULL;
    }
 
    if (addr != 0 && ret != (void *)addr) {
-      lprintf("warning: wanted to map @%08lx, got %p\n",
-            addr, ret);
+      if (log_cb)
+         log_cb(RETRO_LOG_WARN, "warning: wanted to map @%08lx, got %p\n",
+               addr, ret);
 
       if (is_fixed) {
          munmap(ret, size);
@@ -236,12 +237,12 @@ int plat_mem_set_exec(void *ptr, size_t size)
 {
 #ifdef _WIN32
    int ret = VirtualProtect(ptr,size,PAGE_EXECUTE_READWRITE,0);
-   if (ret == 0)
-      lprintf("mprotect(%p, %zd) failed: %d\n", ptr, size, 0);
+   if (ret == 0 && log_cb)
+      log_cb(RETRO_LOG_ERROR, "mprotect(%p, %zd) failed: %d\n", ptr, size, 0);
 #else
    int ret = mprotect(ptr, size, PROT_READ | PROT_WRITE | PROT_EXEC);
-   if (ret != 0)
-      lprintf("mprotect(%p, %zd) failed: %d\n", ptr, size, errno);
+   if (ret != 0 && log_cb)
+      log_cb(RETRO_LOG_ERROR, "mprotect(%p, %zd) failed: %d\n", ptr, size, errno);
 #endif
        return ret;
 }
@@ -260,34 +261,13 @@ void emu_32x_startup(void)
 {
 }
 
-#ifndef ANDROID
-
-void lprintf(const char *fmt, ...)
-{
-       va_list list;
-
-       va_start(list, fmt);
-       fprintf(emu_log, "PicoDrive: ");
-       vfprintf(emu_log, fmt, list);
-       va_end(list);
-       fflush(emu_log);
-}
-
-#else
-
-#include <android/log.h>
-
 void lprintf(const char *fmt, ...)
 {
-       va_list list;
-
-       va_start(list, fmt);
-       __android_log_vprint(ANDROID_LOG_INFO, "PicoDrive", fmt, list);
-       va_end(list);
+   /* TODO - add 'level' param for warning/error messages? */
+   if (log_cb)
+      log_cb(RETRO_LOG_INFO, fmt);
 }
 
-#endif
-
 /* libretro */
 void retro_set_environment(retro_environment_t cb)
 {
@@ -358,8 +338,9 @@ size_t state_read(void *p, size_t size, size_t nmemb, void *file)
        size_t bsize = size * nmemb;
 
        if (state->pos + bsize > state->size) {
-               lprintf("savestate error: %u/%u\n",
-                       state->pos + bsize, state->size);
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "savestate error: %u/%u\n",
+               state->pos + bsize, state->size);
                bsize = state->size - state->pos;
                if ((int)bsize <= 0)
                        return 0;
@@ -376,8 +357,9 @@ size_t state_write(void *p, size_t size, size_t nmemb, void *file)
        size_t bsize = size * nmemb;
 
        if (state->pos + bsize > state->size) {
-               lprintf("savestate error: %u/%u\n",
-                       state->pos + bsize, state->size);
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "savestate error: %u/%u\n",
+               state->pos + bsize, state->size);
                bsize = state->size - state->pos;
                if ((int)bsize <= 0)
                        return 0;
@@ -507,7 +489,8 @@ static bool disk_set_image_index(unsigned int index)
                return false;
 
        if (disks[index].fname == NULL) {
-               lprintf("missing disk #%u\n", index);
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "missing disk #%u\n", index);
 
                // RetroArch specifies "no disk" with index == count,
                // so don't fail here..
@@ -515,15 +498,17 @@ static bool disk_set_image_index(unsigned int index)
                return true;
        }
 
-       lprintf("switching to disk %u: \"%s\"\n", index,
-               disks[index].fname);
+   if (log_cb)
+      log_cb(RETRO_LOG_INFO, "switching to disk %u: \"%s\"\n", index,
+            disks[index].fname);
 
        ret = -1;
        cd_type = PicoCdCheck(disks[index].fname, NULL);
        if (cd_type != CIT_NOT_CD)
                ret = cdd_load(disks[index].fname, cd_type);
        if (ret != 0) {
-               lprintf("Load failed, invalid CD image?\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "Load failed, invalid CD image?\n");
                return 0;
        }
 
@@ -578,13 +563,15 @@ static struct retro_disk_control_callback disk_control = {
 
 static void disk_tray_open(void)
 {
-       lprintf("cd tray open\n");
+   if (log_cb)
+      log_cb(RETRO_LOG_INFO, "cd tray open\n");
        disk_ejected = 1;
 }
 
 static void disk_tray_close(void)
 {
-       lprintf("cd tray close\n");
+   if (log_cb)
+      log_cb(RETRO_LOG_INFO, "cd tray close\n");
        disk_ejected = 0;
 }
 
@@ -646,7 +633,8 @@ static const char *find_bios(int *region, const char *cd_fname)
        }
 
        if (f != NULL) {
-               lprintf("using bios: %s\n", path);
+      if (log_cb)
+         log_cb(RETRO_LOG_INFO, "using bios: %s\n", path);
                fclose(f);
                return path;
        }
@@ -662,12 +650,14 @@ bool retro_load_game(const struct retro_game_info *info)
 
        enum retro_pixel_format fmt = RETRO_PIXEL_FORMAT_RGB565;
        if (!environ_cb(RETRO_ENVIRONMENT_SET_PIXEL_FORMAT, &fmt)) {
-               lprintf("RGB565 support required, sorry\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "RGB565 support required, sorry\n");
                return false;
        }
 
        if (info == NULL || info->path == NULL) {
-               lprintf("info->path required\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "info->path required\n");
                return false;
        }
 
@@ -689,16 +679,20 @@ bool retro_load_game(const struct retro_game_info *info)
 
        switch (media_type) {
        case PM_BAD_DETECT:
-               lprintf("Failed to detect ROM/CD image type.\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "Failed to detect ROM/CD image type.\n");
                return false;
        case PM_BAD_CD:
-               lprintf("Invalid CD image\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "Invalid CD image\n");
                return false;
        case PM_BAD_CD_NO_BIOS:
-               lprintf("Missing BIOS\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "Missing BIOS\n");
                return false;
        case PM_ERROR:
-               lprintf("Load error\n");
+      if (log_cb)
+         log_cb(RETRO_LOG_ERROR, "Load error\n");
                return false;
        default:
                break;
@@ -797,7 +791,8 @@ static enum input_device input_name_to_val(const char *name)
        if (strcmp(name, "None") == 0)
                return PICO_INPUT_NOTHING;
 
-       lprintf("invalid picodrive_input: '%s'\n", name);
+   if (log_cb)
+      log_cb(RETRO_LOG_WARN, "invalid picodrive_input: '%s'\n", name);
        return PICO_INPUT_PAD_3BTN;
 }
 
@@ -869,19 +864,16 @@ void retro_run(void)
 
 void retro_init(void)
 {
+   struct retro_log_callback log;
        int level;
 
-#ifdef IOS
-       emu_log = fopen("/User/Documents/PicoDrive.log", "w");
-       if (emu_log == NULL)
-               emu_log = fopen("PicoDrive.log", "w");
-       if (emu_log == NULL)
-#endif
-       emu_log = stdout;
-
        level = 0;
        environ_cb(RETRO_ENVIRONMENT_SET_PERFORMANCE_LEVEL, &level);
 
+   environ_cb(RETRO_ENVIRONMENT_GET_LOG_INTERFACE, &log);
+   if (log.log)
+      log_cb = log.log;
+
        environ_cb(RETRO_ENVIRONMENT_SET_DISK_CONTROL_INTERFACE, &disk_control);
 
        PicoOpt = POPT_EN_STEREO|POPT_EN_FM|POPT_EN_PSG|POPT_EN_Z80
old mode 100644 (file)
new mode 100755 (executable)
index ff4f4fd..aaff348
@@ -94,6 +94,9 @@ extern "C" {
 // Eventually _PRESSED will return false for an index. No further presses are registered at this point.
 #define RETRO_DEVICE_POINTER      6
 
+// FIXME: Document this.
+#define RETRO_DEVICE_SENSOR_ACCELEROMETER 7
+
 // These device types are specializations of the base types above.
 // They should only be used in retro_set_controller_type() to inform libretro implementations
 // about use of a very specific device type.
@@ -150,6 +153,11 @@ extern "C" {
 #define RETRO_DEVICE_ID_POINTER_Y         1
 #define RETRO_DEVICE_ID_POINTER_PRESSED   2
 
+// Id values for SENSOR types.
+#define RETRO_DEVICE_ID_SENSOR_ACCELEROMETER_X      0
+#define RETRO_DEVICE_ID_SENSOR_ACCELEROMETER_Y      1
+#define RETRO_DEVICE_ID_SENSOR_ACCELEROMETER_Z      2
+
 // Returned from retro_get_region().
 #define RETRO_REGION_NTSC  0
 #define RETRO_REGION_PAL   1
@@ -357,6 +365,8 @@ enum retro_mod
 
 // If set, this call is not part of the public libretro API yet. It can change or be removed at any time.
 #define RETRO_ENVIRONMENT_EXPERIMENTAL 0x10000
+// Environment callback to be used internally in frontend.
+#define RETRO_ENVIRONMENT_PRIVATE 0x20000
 
 // Environment commands.
 #define RETRO_ENVIRONMENT_SET_ROTATION  1  // const unsigned * --
@@ -434,10 +444,8 @@ enum retro_mod
                                            // Sets an interface which frontend can use to eject and insert disk images.
                                            // This is used for games which consist of multiple images and must be manually
                                            // swapped out by the user (e.g. PSX).
-#define RETRO_ENVIRONMENT_SET_HW_RENDER    (14 | RETRO_ENVIRONMENT_EXPERIMENTAL)
+#define RETRO_ENVIRONMENT_SET_HW_RENDER 14
                                            // struct retro_hw_render_callback * --
-                                           // NOTE: This call is currently very experimental, and should not be considered part of the public API.
-                                           // The interface could be changed or removed at any time.
                                            // Sets an interface to let a libretro core render with hardware acceleration.
                                            // Should be called in retro_load_game().
                                            // If successful, libretro cores will be able to render to a frontend-provided framebuffer.
@@ -445,7 +453,7 @@ enum retro_mod
                                            // If HW rendering is used, pass only RETRO_HW_FRAME_BUFFER_VALID or NULL to retro_video_refresh_t.
 #define RETRO_ENVIRONMENT_GET_VARIABLE 15
                                            // struct retro_variable * --
-                                           // Interface to aquire user-defined information from environment
+                                           // Interface to acquire user-defined information from environment
                                            // that cannot feasibly be supported in a multi-system way.
                                            // 'key' should be set to a key which has already been set by SET_VARIABLES.
                                            // 'data' will be set to a value or NULL.
@@ -480,13 +488,222 @@ enum retro_mod
                                            // If true, the libretro implementation supports calls to retro_load_game() with NULL as argument.
                                            // Used by cores which can run without particular game data.
                                            // This should be called within retro_set_environment() only.
-                                          
+                                           //
+#define RETRO_ENVIRONMENT_GET_LIBRETRO_PATH 19
+                                           // const char ** --
+                                           // Retrieves the absolute path from where this libretro implementation was loaded.
+                                           // NULL is returned if the libretro was loaded statically (i.e. linked statically to frontend), or if the path cannot be determined.
+                                           // Mostly useful in cooperation with SET_SUPPORT_NO_GAME as assets can be loaded without ugly hacks.
+                                           //
+                                           //
+// Environment 20 was an obsolete version of SET_AUDIO_CALLBACK. It was not used by any known core at the time,
+// and was removed from the API.
+#define RETRO_ENVIRONMENT_SET_AUDIO_CALLBACK 22
+                                           // const struct retro_audio_callback * --
+                                           // Sets an interface which is used to notify a libretro core about audio being available for writing.
+                                           // The callback can be called from any thread, so a core using this must have a thread safe audio implementation.
+                                           // It is intended for games where audio and video are completely asynchronous and audio can be generated on the fly.
+                                           // This interface is not recommended for use with emulators which have highly synchronous audio.
+                                           //
+                                           // The callback only notifies about writability; the libretro core still has to call the normal audio callbacks
+                                           // to write audio. The audio callbacks must be called from within the notification callback.
+                                           // The amount of audio data to write is up to the implementation.
+                                           // Generally, the audio callback will be called continously in a loop.
+                                           //
+                                           // Due to thread safety guarantees and lack of sync between audio and video, a frontend
+                                           // can selectively disallow this interface based on internal configuration. A core using
+                                           // this interface must also implement the "normal" audio interface.
+                                           //
+                                           // A libretro core using SET_AUDIO_CALLBACK should also make use of SET_FRAME_TIME_CALLBACK.
+#define RETRO_ENVIRONMENT_SET_FRAME_TIME_CALLBACK 21
+                                           // const struct retro_frame_time_callback * --
+                                           // Lets the core know how much time has passed since last invocation of retro_run().
+                                           // The frontend can tamper with the timing to fake fast-forward, slow-motion, frame stepping, etc.
+                                           // In this case the delta time will use the reference value in frame_time_callback..
+                                           //
+#define RETRO_ENVIRONMENT_GET_RUMBLE_INTERFACE 23
+                                           // struct retro_rumble_interface * --
+                                           // Gets an interface which is used by a libretro core to set state of rumble motors in controllers.
+                                           // A strong and weak motor is supported, and they can be controlled indepedently.
+                                           //
+#define RETRO_ENVIRONMENT_GET_INPUT_DEVICE_CAPABILITIES 24
+                                           // uint64_t * --
+                                           // Gets a bitmask telling which device type are expected to be handled properly in a call to retro_input_state_t.
+                                           // Devices which are not handled or recognized always return 0 in retro_input_state_t.
+                                           // Example bitmask: caps = (1 << RETRO_DEVICE_JOYPAD) | (1 << RETRO_DEVICE_ANALOG).
+                                           // Should only be called in retro_run().
+                                           //
+#define RETRO_ENVIRONMENT_GET_SENSOR_INTERFACE (25 | RETRO_ENVIRONMENT_EXPERIMENTAL)
+                                           // struct retro_sensor_interface * --
+                                           // Gets access to the sensor interface.
+                                           // The purpose of this interface is to allow
+                                           // setting state related to sensors such as polling rate, enabling/disable it entirely, etc.
+                                           // Reading sensor state is done via the normal input_state_callback API.
+                                           //
+#define RETRO_ENVIRONMENT_GET_CAMERA_INTERFACE (26 | RETRO_ENVIRONMENT_EXPERIMENTAL)
+                                           // struct retro_camera_callback * --
+                                           // Gets an interface to a video camera driver.
+                                           // A libretro core can use this interface to get access to a video camera.
+                                           // New video frames are delivered in a callback in same thread as retro_run().
+                                           //
+                                           // GET_CAMERA_INTERFACE should be called in retro_load_game().
+                                           //
+                                           // Depending on the camera implementation used, camera frames will be delivered as a raw framebuffer,
+                                           // or as an OpenGL texture directly.
+                                           //
+                                           // The core has to tell the frontend here which types of buffers can be handled properly.
+                                           // An OpenGL texture can only be handled when using a libretro GL core (SET_HW_RENDER).
+                                           // It is recommended to use a libretro GL core when using camera interface.
+                                           //
+                                           // The camera is not started automatically. The retrieved start/stop functions must be used to explicitly
+                                           // start and stop the camera driver.
+                                           //
+#define RETRO_ENVIRONMENT_GET_LOG_INTERFACE 27
+                                           // struct retro_log_callback * --
+                                           // Gets an interface for logging. This is useful for logging in a cross-platform way
+                                           // as certain platforms cannot use use stderr for logging. It also allows the frontend to
+                                           // show logging information in a more suitable way.
+                                           // If this interface is not used, libretro cores should log to stderr as desired.
+
+enum retro_log_level
+{
+   RETRO_LOG_DEBUG = 0,
+   RETRO_LOG_INFO,
+   RETRO_LOG_WARN,
+   RETRO_LOG_ERROR,
+
+   RETRO_LOG_DUMMY = INT_MAX
+};
+
+// Logging function. Takes log level argument as well.
+typedef void (*retro_log_printf_t)(enum retro_log_level level, const char *fmt, ...);
+
+struct retro_log_callback
+{
+   retro_log_printf_t log;
+};
+
+// FIXME: Document the sensor API and work out behavior.
+// It will be marked as experimental until then.
+enum retro_sensor_action
+{
+   RETRO_SENSOR_ACCELEROMETER_ENABLE = 0,
+   RETRO_SENSOR_ACCELEROMETER_DISABLE,
+
+   RETRO_SENSOR_DUMMY = INT_MAX
+};
+
+typedef bool (*retro_set_sensor_state_t)(unsigned port, enum retro_sensor_action action, unsigned rate);
+struct retro_sensor_interface
+{
+   retro_set_sensor_state_t set_sensor_state;
+};
+////
+
+enum retro_camera_buffer
+{
+   RETRO_CAMERA_BUFFER_OPENGL_TEXTURE = 0,
+   RETRO_CAMERA_BUFFER_RAW_FRAMEBUFFER,
+
+   RETRO_CAMERA_BUFFER_DUMMY = INT_MAX
+};
+
+// Starts the camera driver. Can only be called in retro_run().
+typedef bool (*retro_camera_start_t)(void);
+// Stops the camera driver. Can only be called in retro_run().
+typedef void (*retro_camera_stop_t)(void);
+// Callback which signals when the camera driver is initialized and/or deinitialized.
+// retro_camera_start_t can be called in initialized callback.
+typedef void (*retro_camera_lifetime_status_t)(void);
+// A callback for raw framebuffer data. buffer points to an XRGB8888 buffer.
+// Width, height and pitch are similar to retro_video_refresh_t.
+// First pixel is top-left origin.
+typedef void (*retro_camera_frame_raw_framebuffer_t)(const uint32_t *buffer, unsigned width, unsigned height, size_t pitch);
+// A callback for when OpenGL textures are used.
+//
+// texture_id is a texture owned by camera driver.
+// Its state or content should be considered immutable, except for things like texture filtering and clamping.
+//
+// texture_target is the texture target for the GL texture.
+// These can include e.g. GL_TEXTURE_2D, GL_TEXTURE_RECTANGLE, and possibly more depending on extensions.
+// 
+// affine points to a packed 3x3 column-major matrix used to apply an affine transform to texture coordinates. (affine_matrix * vec3(coord_x, coord_y, 1.0))
+// After transform, normalized texture coord (0, 0) should be bottom-left and (1, 1) should be top-right (or (width, height) for RECTANGLE).
+//
+// GL-specific typedefs are avoided here to avoid relying on gl.h in the API definition.
+typedef void (*retro_camera_frame_opengl_texture_t)(unsigned texture_id, unsigned texture_target, const float *affine);
+struct retro_camera_callback
+{
+   uint64_t caps; // Set by libretro core. Example bitmask: caps = (1 << RETRO_CAMERA_BUFFER_OPENGL_TEXTURE) | (1 << RETRO_CAMERA_BUFFER_RAW_FRAMEBUFFER).
+
+   unsigned width; // Desired resolution for camera. Is only used as a hint.
+   unsigned height;
+   retro_camera_start_t start; // Set by frontend.
+   retro_camera_stop_t stop; // Set by frontend.
+
+   retro_camera_frame_raw_framebuffer_t frame_raw_framebuffer; // Set by libretro core if raw framebuffer callbacks will be used.
+   retro_camera_frame_opengl_texture_t frame_opengl_texture; // Set by libretro core if OpenGL texture callbacks will be used.
+
+   // Set by libretro core. Called after camera driver is initialized and ready to be started.
+   // Can be NULL, in which this callback is not called.
+   retro_camera_lifetime_status_t initialized;
+
+   // Set by libretro core. Called right before camera driver is deinitialized.
+   // Can be NULL, in which this callback is not called.
+   retro_camera_lifetime_status_t deinitialized;
+};
+
+enum retro_rumble_effect
+{
+   RETRO_RUMBLE_STRONG = 0,
+   RETRO_RUMBLE_WEAK = 1,
+
+   RETRO_RUMBLE_DUMMY = INT_MAX
+};
+
+// Sets rumble state for joypad plugged in port 'port'. Rumble effects are controlled independently,
+// and setting e.g. strong rumble does not override weak rumble.
+// Strength has a range of [0, 0xffff].
+//
+// Returns true if rumble state request was honored. Calling this before first retro_run() is likely to return false.
+typedef bool (*retro_set_rumble_state_t)(unsigned port, enum retro_rumble_effect effect, uint16_t strength);
+struct retro_rumble_interface
+{
+   retro_set_rumble_state_t set_rumble_state;
+};
+
+// Notifies libretro that audio data should be written.
+typedef void (*retro_audio_callback_t)(void);
+
+// True: Audio driver in frontend is active, and callback is expected to be called regularily.
+// False: Audio driver in frontend is paused or inactive. Audio callback will not be called until set_state has been called with true.
+// Initial state is false (inactive).
+typedef void (*retro_audio_set_state_callback_t)(bool enabled);
+struct retro_audio_callback
+{
+   retro_audio_callback_t callback;
+   retro_audio_set_state_callback_t set_state;
+};
+
+// Notifies a libretro core of time spent since last invocation of retro_run() in microseconds.
+// It will be called right before retro_run() every frame.
+// The frontend can tamper with timing to support cases like fast-forward, slow-motion and framestepping.
+// In those scenarios the reference frame time value will be used.
+typedef int64_t retro_usec_t;
+typedef void (*retro_frame_time_callback_t)(retro_usec_t usec);
+struct retro_frame_time_callback
+{
+   retro_frame_time_callback_t callback;
+   retro_usec_t reference; // Represents the time of one frame. It is computed as 1000000 / fps, but the implementation will resolve the rounding to ensure that framestepping, etc is exact.
+};
 
 // Pass this to retro_video_refresh_t if rendering to hardware.
 // Passing NULL to retro_video_refresh_t is still a frame dupe as normal.
 #define RETRO_HW_FRAME_BUFFER_VALID ((void*)-1)
 
 // Invalidates the current HW context.
+// Any GL state is lost, and must not be deinitialized explicitly. If explicit deinitialization is desired by the libretro core,
+// it should implement context_destroy callback.
 // If called, all GPU resources must be reinitialized.
 // Usually called when frontend reinits video driver.
 // Also called first time video driver is initialized, allowing libretro core to init resources.
@@ -501,8 +718,10 @@ typedef retro_proc_address_t (*retro_hw_get_proc_address_t)(const char *sym);
 enum retro_hw_context_type
 {
    RETRO_HW_CONTEXT_NONE = 0,
-   RETRO_HW_CONTEXT_OPENGL, // OpenGL 2.x. Latest version available before 3.x+.
+   RETRO_HW_CONTEXT_OPENGL, // OpenGL 2.x. Latest version available before 3.x+. Driver can choose to use latest compatibility context.
    RETRO_HW_CONTEXT_OPENGLES2, // GLES 2.0
+   RETRO_HW_CONTEXT_OPENGL_CORE, // Modern desktop core GL context. Use major/minor fields to set GL version.
+   RETRO_HW_CONTEXT_OPENGLES3, // GLES 3.0
 
    RETRO_HW_CONTEXT_DUMMY = INT_MAX
 };
@@ -510,10 +729,21 @@ enum retro_hw_context_type
 struct retro_hw_render_callback
 {
    enum retro_hw_context_type context_type; // Which API to use. Set by libretro core.
-   retro_hw_context_reset_t context_reset; // Set by libretro core.
+   retro_hw_context_reset_t context_reset; // Called when a context has been created or when it has been reset.
    retro_hw_get_current_framebuffer_t get_current_framebuffer; // Set by frontend.
    retro_hw_get_proc_address_t get_proc_address; // Set by frontend.
    bool depth; // Set if render buffers should have depth component attached.
+   bool stencil; // Set if stencil buffers should be attached.
+   // If depth and stencil are true, a packed 24/8 buffer will be added. Only attaching stencil is invalid and will be ignored.
+   bool bottom_left_origin; // Use conventional bottom-left origin convention. Is false, standard libretro top-left origin semantics are used.
+   unsigned version_major; // Major version number for core GL context.
+   unsigned version_minor; // Minor version number for core GL context.
+
+   bool cache_context; // If this is true, the frontend will go very far to avoid resetting context in scenarios like toggling fullscreen, etc.
+   // The reset callback might still be called in extreme situations such as if the context is lost beyond recovery. 
+   // For optimal stability, set this to false, and allow context to be reset at any time.
+   retro_hw_context_reset_t context_destroy; // A callback to be called before the context is destroyed. Resources can be deinitialized at this step. This can be set to NULL, in which resources will just be destroyed without any notification.
+   bool debug_context; // Creates a debug context.
 };
 
 // Callback type passed in RETRO_ENVIRONMENT_SET_KEYBOARD_CALLBACK. Called by the frontend in response to keyboard events.
@@ -521,6 +751,13 @@ struct retro_hw_render_callback
 // keycode is the RETROK value of the char.
 // character is the text character of the pressed key. (UTF-32).
 // key_modifiers is a set of RETROKMOD values or'ed together.
+//
+// The pressed/keycode state can be indepedent of the character.
+// It is also possible that multiple characters are generated from a single keypress.
+// Keycode events should be treated separately from character events.
+// However, when possible, the frontend should try to synchronize these.
+// If only a character is posted, keycode should be RETROK_UNKNOWN.
+// Similarily if only a keycode event is generated with no corresponding character, character should be 0.
 typedef void (*retro_keyboard_event_t)(bool down, unsigned keycode, uint32_t character, uint16_t key_modifiers);
 
 struct retro_keyboard_callback