summaryrefslogtreecommitdiff
path: root/kms++/src
diff options
context:
space:
mode:
authorTomi Valkeinen <tomi.valkeinen@ti.com>2019-01-17 17:24:25 +0200
committerTomi Valkeinen <tomi.valkeinen@ti.com>2019-01-17 17:43:48 +0200
commitdb0da1f33c7d688e84e9cd7ff0551842bb68ca6f (patch)
tree58abaa24d29b2c592b9de03e1d4c1aaa23e22806 /kms++/src
parentd49777b00e844ac77e0f3b0cfb9b9e07257b4c26 (diff)
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>
Diffstat (limited to 'kms++/src')
-rw-r--r--kms++/src/card.cpp37
-rw-r--r--kms++/src/omap/omapcard.cpp5
2 files changed, 23 insertions, 19 deletions
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)