>> + if (encoder->encoder_type == DRM_MODE_ENCODER_TVDAC)>> + val = 1;>> + else>> + val = 0;Isn't this better written as val = (encoder->encoder_type == DRM_MODE_ENCODER_TVDAC);-- Stefan