From 5147ed6d4b8b3e26f40590b200e5466a263a57c5 Mon Sep 17 00:00:00 2001 From: Mattias Andrée Date: Sun, 1 Jun 2014 07:36:57 +0200 Subject: m + doc + fix indentation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Mattias Andrée --- src/lib/gamma-linux-drm.c | 109 +++++++++++++++++++++++----------------------- 1 file changed, 55 insertions(+), 54 deletions(-) (limited to 'src/lib/gamma-linux-drm.c') diff --git a/src/lib/gamma-linux-drm.c b/src/lib/gamma-linux-drm.c index b97f94d..dba98b1 100644 --- a/src/lib/gamma-linux-drm.c +++ b/src/lib/gamma-linux-drm.c @@ -315,17 +315,21 @@ int libgamma_linux_drm_partition_initialise(libgamma_partition_state_t* restrict static void release_connectors_and_encoders(libgamma_drm_card_data_t* restrict this) { size_t i, n; + /* Release individual encoders. */ if (this->encoders != NULL) for (i = 0, n = (size_t)(this->res->count_connectors); i < n; i++) if (this->encoders[i] != NULL) drmModeFreeEncoder(this->encoders[i]); + /* Release encoder array. */ free(this->encoders); this->encoders = NULL; + /* Release individual connectors. */ if (this->connectors != NULL) for (i = 0, n = (size_t)(this->res->count_connectors); i < n; i++) if (this->connectors[i] != NULL) drmModeFreeConnector(this->connectors[i]); + /* Release connector array. */ free(this->connectors); this->connectors = NULL; } @@ -340,10 +344,8 @@ void libgamma_linux_drm_partition_destroy(libgamma_partition_state_t* restrict t { libgamma_drm_card_data_t* restrict data = this->data; release_connectors_and_encoders(data); - if (data->res != NULL) - drmModeFreeResources(data->res); - if (data->fd >= 0) - close(data->fd); + if (data->res != NULL) drmModeFreeResources(data->res); + if (data->fd >= 0) close(data->fd); free(data); } @@ -375,15 +377,12 @@ int libgamma_linux_drm_partition_restore(libgamma_partition_state_t* restrict th int libgamma_linux_drm_crtc_initialise(libgamma_crtc_state_t* restrict this, libgamma_partition_state_t* restrict partition, size_t crtc) { - libgamma_drm_card_data_t* restrict card; + libgamma_drm_card_data_t* restrict card = partition->data; uint32_t crtc_id; if (crtc >= partition->crtcs_available) return LIBGAMMA_NO_SUCH_CRTC; - - card = partition->data; - crtc_id = card->res->crtcs[crtc]; - this->data = (void*)(size_t)crtc_id; + this->data = (void*)(size_t)(card->res->crtcs[crtc]); return 0; } @@ -429,25 +428,29 @@ static drmModeConnector* find_connector(libgamma_crtc_state_t* restrict this, in /* Open connectors and encoders if not already opened. */ if (card->connectors == NULL) { - /* We use calloc so all non-loaded elements are `NULL` after an error. */ - if ((card->connectors = calloc(n, sizeof(drmModeConnector*))) == NULL) - goto fail; - if ((card->encoders = calloc(n, sizeof(drmModeEncoder*))) == NULL) - goto fail; + /* Allocate connector and encoder arrays. + We use `calloc` so all non-loaded elements are `NULL` after an error. */ + if ((card->connectors = calloc(n, sizeof(drmModeConnector*))) == NULL) goto fail; + if ((card->encoders = calloc(n, sizeof(drmModeEncoder*))) == NULL) goto fail; + /* Fill connector and encoder arrays. */ for (i = 0; i < n; i++) if (((card->connectors[i] = drmModeGetConnector(card->fd, card->res->connectors[i])) == NULL) || ((card->encoders[i] = drmModeGetEncoder(card->fd, card->connectors[i]->encoder_id)) == NULL)) goto fail; } - /* Find connector. */ + /* No error has occurred yet. */ *error = 0; + /* Find connector. */ for (i = 0; i < n; i++) if (card->encoders[i]->crtc_id == crtc_id) return card->connectors[i]; + /* We did not find the connector. */ *error = LIBGAMMA_CONNECTOR_UNKNOWN; return NULL; fail: + /* Report the error that got us here, release + resouces and exit with `NULL` for failure. */ *error = errno; release_connectors_and_encoders(card); return NULL; @@ -471,9 +474,7 @@ static int get_gamma_ramp_size(libgamma_crtc_information_t* restrict out, const out->gamma_size_error = crtc_info == NULL ? errno : 0; if (out->gamma_size_error == 0) { - out->red_gamma_size = (size_t)(crtc_info->gamma_size); - out->green_gamma_size = (size_t)(crtc_info->gamma_size); - out->blue_gamma_size = (size_t)(crtc_info->gamma_size); + out->red_gamma_size = out->green_gamma_size = out->blue_gamma_size = (size_t)(crtc_info->gamma_size); out->gamma_size_error = crtc_info->gamma_size < 2 ? LIBGAMMA_SINGLETON_GAMMA_RAMP : 0; drmModeFreeCrtc(crtc_info); } @@ -495,18 +496,18 @@ static void get_subpixel_order(libgamma_crtc_information_t* restrict out, out->subpixel_order = LIBGAMMA_SUBPIXEL_ORDER_##value; \ break - switch (connector->subpixel) - { - __select (UNKNOWN); - __select (HORIZONTAL_RGB); - __select (HORIZONTAL_BGR); - __select (VERTICAL_RGB); - __select (VERTICAL_BGR); - __select (NONE); - default: - out->subpixel_order_error = LIBGAMMA_SUBPIXEL_ORDER_NOT_RECOGNISED; - break; - } + switch (connector->subpixel) + { + __select (UNKNOWN); + __select (HORIZONTAL_RGB); + __select (HORIZONTAL_BGR); + __select (VERTICAL_RGB); + __select (VERTICAL_BGR); + __select (NONE); + default: + out->subpixel_order_error = LIBGAMMA_SUBPIXEL_ORDER_NOT_RECOGNISED; + break; + } #undef __select } @@ -529,36 +530,36 @@ static void get_connector_type(libgamma_crtc_information_t* restrict out, *connector_name_base = name; \ break - switch (connector->connector_type) - { + switch (connector->connector_type) + { #ifndef DRM_MODE_CONNECTOR_VIRTUAL # define DRM_MODE_CONNECTOR_VIRTUAL 15 #endif #ifndef DRM_MODE_CONNECTOR_DSI # define DRM_MODE_CONNECTOR_DSI 16 #endif - __select (Unknown, "Unknown" ); - __select (VGA, "VGA" ); - __select (DVII, "DVI-I" ); - __select (DVID, "DVI-D" ); - __select (DVIA, "DVI-A" ); - __select (Composite, "Composite"); - __select (SVIDEO, "SVIDEO" ); - __select (LVDS, "LVDS" ); - __select (Component, "Component"); - __select (9PinDIN, "DIN" ); - __select (DisplayPort, "DP" ); - __select (HDMIA, "HDMI-A" ); - __select (HDMIB, "HDMI-B" ); - __select (TV, "TV" ); - __select (eDP, "eDP" ); - __select (VIRTUAL, "VIRTUAL" ); - __select (DSI, "DSI" ); - default: - out->connector_type_error = LIBGAMMA_CONNECTOR_TYPE_NOT_RECOGNISED; - out->connector_name_error = LIBGAMMA_CONNECTOR_TYPE_NOT_RECOGNISED; - break; - } + __select (Unknown, "Unknown" ); + __select (VGA, "VGA" ); + __select (DVII, "DVI-I" ); + __select (DVID, "DVI-D" ); + __select (DVIA, "DVI-A" ); + __select (Composite, "Composite"); + __select (SVIDEO, "SVIDEO" ); + __select (LVDS, "LVDS" ); + __select (Component, "Component"); + __select (9PinDIN, "DIN" ); + __select (DisplayPort, "DP" ); + __select (HDMIA, "HDMI-A" ); + __select (HDMIB, "HDMI-B" ); + __select (TV, "TV" ); + __select (eDP, "eDP" ); + __select (VIRTUAL, "VIRTUAL" ); + __select (DSI, "DSI" ); + default: + out->connector_type_error = LIBGAMMA_CONNECTOR_TYPE_NOT_RECOGNISED; + out->connector_name_error = LIBGAMMA_CONNECTOR_TYPE_NOT_RECOGNISED; + break; + } #undef __select } -- cgit v1.2.3-70-g09d2