card: another try with card constructors

Let's try again with the card constructors.

Card::open_modesetting_card() is removed.

The main constructor is Card(const std::string& dev_path = "").

If dev_path is set, the device node with that path is used. If dev_path
is not set, the behavior is similar as previously, except a modeset
capable card is used at the third step:

- If KMSXX_DEVICE env variable is set, the card device with that path is
  opened.
- If KMSXX_DRIVER env variable is set, the card with the given driver
  name and index is opened. The format is either "drvname" or
  "drvname:idx".
- If neither env variable is given, the first modeset capable card is
  opened.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

4 files changed, 25 insertions, 25 deletions

diff --git a/kms++/inc/kms++/card.h b/kms++/inc/kms++/card.h
index a2c27f8..099d5b5 100644
--- a/kms++/inc/kms++/card.h
+++ b/kms++/inc/kms++/card.h
@@ -14,10 +14,7 @@ class Card
 {
 	friend class Framebuffer;
 public:
-	static std::unique_ptr<Card> open_modesetting_card();
-
-	Card();
-	Card(const std::string& dev_path);
+	Card(const std::string& dev_path = "");
 	Card(const std::string& driver, uint32_t idx);
 	virtual ~Card();
 
diff --git a/kms++/inc/kms++/omap/omapcard.h b/kms++/inc/kms++/omap/omapcard.h
index 5c2f3a5..2f1f528 100644
--- a/kms++/inc/kms++/omap/omapcard.h
+++ b/kms++/inc/kms++/omap/omapcard.h
@@ -9,8 +9,7 @@ namespace kms
 class OmapCard : public Card
 {
 public:
-	OmapCard();
-	OmapCard(const std::string& device);
+	OmapCard(const std::string& device = "");
 	virtual ~OmapCard();
 
 	struct omap_device* dev() const { return m_omap_dev; }
diff --git a/kms++/src/card.cpp b/kms++/src/card.cpp
index 85c693b..8de8b82 100644
--- a/kms++/src/card.cpp
+++ b/kms++/src/card.cpp
@@ -39,15 +39,29 @@ static vector<string> glob(const string& pattern)
 	return filenames;
 }
 
-unique_ptr<Card> Card::open_modesetting_card()
+static int open_first_kms_device()
 {
 	vector<string> paths = glob("/dev/dri/card*");
 
 	for (const string& path : paths) {
-		unique_ptr<Card> card = unique_ptr<Card>(new Card(path));
+		int fd = open(path.c_str(), O_RDWR | O_CLOEXEC);
+		if (fd < 0)
+			throw invalid_argument(string(strerror(errno)) + " opening device " + path);
 
-		if (card->has_kms())
-			return card;
+		auto res = drmModeGetResources(fd);
+		if (!res) {
+			close(fd);
+			continue;
+		}
+
+		bool has_kms = res->count_crtcs > 0 && res->count_connectors > 0 && res->count_encoders > 0;
+
+		drmModeFreeResources(res);
+
+		if (has_kms)
+			return fd;
+
+		close(fd);
 	}
 
 	throw runtime_error("No modesetting DRM card found");
@@ -90,12 +104,14 @@ static int open_device_by_driver(string name, uint32_t idx)
 	throw invalid_argument("Failed to find a DRM device " + name + ":" + to_string(idx));
 }
 
-Card::Card()
+Card::Card(const std::string& dev_path)
 {
 	const char* drv_p = getenv("KMSXX_DRIVER");
 	const char* dev_p = getenv("KMSXX_DEVICE");
 
-	if (dev_p) {
+	if (!dev_path.empty()) {
+		m_fd = open_device_by_path(dev_path);
+	} else if (dev_p) {
 		string dev(dev_p);
 		m_fd = open_device_by_path(dev);
 	} else if (drv_p) {
@@ -119,7 +135,7 @@ Card::Card()
 
 		m_fd = open_device_by_driver(name, num);
 	} else {
-		m_fd = open_device_by_path("/dev/dri/card0");
+		m_fd = open_first_kms_device();
 	}
 
 	setup();
@@ -132,13 +148,6 @@ Card::Card(const std::string& driver, uint32_t idx)
 	setup();
 }
 
-Card::Card(const std::string& dev_path)
-{
-	m_fd = open_device_by_path(dev_path);
-
-	setup();
-}
-
 void Card::setup()
 {
 	drmVersionPtr ver = drmGetVersion(m_fd);
diff --git a/kms++/src/omap/omapcard.cpp b/kms++/src/omap/omapcard.cpp
index e811b6d..5e21c75 100644
--- a/kms++/src/omap/omapcard.cpp
+++ b/kms++/src/omap/omapcard.cpp
@@ -9,11 +9,6 @@ using namespace std;
 
 namespace kms
 {
-OmapCard::OmapCard()
-	: OmapCard("/dev/dri/card0")
-{
-
-}
 
 OmapCard::OmapCard(const string& device)
 	: Card(device)