From 36f42004d8f084775efd92a559996ecaeefb0cda Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Fri, 15 May 2020 11:08:03 +0300 Subject: kmstest: allow enabling output without anything connected Allow kmstest to enable an output without anything connected, if the user gives a videomode. DRM framework allows this, and is needed for testing. Signed-off-by: Tomi Valkeinen --- kms++/inc/kms++/videomode.h | 2 ++ kms++/src/connector.cpp | 3 ++- kms++/src/videomode.cpp | 5 +++++ utils/kmstest.cpp | 8 ++++---- 4 files changed, 13 insertions(+), 5 deletions(-) diff --git a/kms++/inc/kms++/videomode.h b/kms++/inc/kms++/videomode.h index 74aa8f1..3aacd57 100644 --- a/kms++/inc/kms++/videomode.h +++ b/kms++/inc/kms++/videomode.h @@ -52,6 +52,8 @@ struct Videomode std::string to_string_short() const; std::string to_string_long() const; std::string to_string_long_padded() const; + + bool valid() const; }; struct Videomode videomode_from_timings(uint32_t clock_khz, diff --git a/kms++/src/connector.cpp b/kms++/src/connector.cpp index 72f8777..a408619 100644 --- a/kms++/src/connector.cpp +++ b/kms++/src/connector.cpp @@ -120,7 +120,8 @@ void Connector::restore_mode() Videomode Connector::get_default_mode() const { if (m_priv->drm_connector->count_modes == 0) - throw invalid_argument("no modes available\n"); + return Videomode(); + drmModeModeInfo drmmode = m_priv->drm_connector->modes[0]; return drm_mode_to_video_mode(drmmode); diff --git a/kms++/src/videomode.cpp b/kms++/src/videomode.cpp index b53f0c0..066896f 100644 --- a/kms++/src/videomode.cpp +++ b/kms++/src/videomode.cpp @@ -12,6 +12,11 @@ using namespace std; namespace kms { +bool Videomode::valid() const +{ + return !!clock; +} + unique_ptr Videomode::to_blob(Card& card) const { drmModeModeInfo drm_mode = video_mode_to_drm_mode(*this); diff --git a/utils/kmstest.cpp b/utils/kmstest.cpp index b6092cb..b4bf24b 100644 --- a/utils/kmstest.cpp +++ b/utils/kmstest.cpp @@ -88,9 +88,6 @@ static void get_connector(ResourceManager& resman, OutputInfo& output, const str if (!conn) EXIT("No connector '%s'", str.c_str()); - if (!conn->connected()) - EXIT("Connector '%s' not connected", conn->fullname().c_str()); - output.connector = conn; output.mode = output.connector->get_default_mode(); } @@ -646,7 +643,7 @@ static vector setups_to_outputs(Card& card, ResourceManager& resman, if (outputs.empty()) { // no outputs defined, show a pattern on all connected screens for (Connector* conn : card.get_connectors()) { - if (conn->connector_status() != ConnectorStatus::Connected) + if (!conn->connected()) continue; OutputInfo output = { }; @@ -668,6 +665,9 @@ static vector setups_to_outputs(Card& card, ResourceManager& resman, get_props(card, o.crtc_props, o.crtc); + if (!o.mode.valid()) + EXIT("Mode not valid for %s", o.connector->fullname().c_str()); + if (card.has_atomic()) { if (o.planes.empty()) add_default_planeinfo(&o); -- cgit v1.2.3