Commit 36e238cd authored by Vlad Zahorodnii's avatar Vlad Zahorodnii
Browse files

backends/drm: Remove EdidOverwrite config option

This is a too niche feature. It also doesn't have to be implemented in
the compositor. The kernel provides a way to overwrite the edid blob,
which is not specific to the running compositor.
parent 96f3a5ee
Pipeline #208432 passed with stage
in 12 minutes and 37 seconds
......@@ -15,10 +15,6 @@
#include "drm_pipeline.h"
#include "drm_pointer.h"
#include <main.h>
// frameworks
#include <KConfigGroup>
#include <cerrno>
#include <cstring>
#include <libxcvt/libxcvt.h>
......@@ -323,16 +319,6 @@ bool DrmConnector::updateProperties()
m_physicalSize = m_edid.physicalSize();
}
// the size might be completely borked. E.g. Samsung SyncMaster 2494HS reports 160x90 while in truth it's 520x292
// as this information is used to calculate DPI info, it's going to result in everything being huge
const QByteArray unknown = QByteArrayLiteral("unknown");
KConfigGroup group = kwinApp()->config()->group("EdidOverwrite").group(m_edid.eisaId().isEmpty() ? unknown : m_edid.eisaId()).group(m_edid.monitorName().isEmpty() ? unknown : m_edid.monitorName()).group(m_edid.serialNumber().isEmpty() ? unknown : m_edid.serialNumber());
if (group.hasKey("PhysicalSize")) {
const QSize overwriteSize = group.readEntry("PhysicalSize", m_physicalSize);
qCWarning(KWIN_DRM) << "Overwriting monitor physical size for" << m_edid.eisaId() << "/" << m_edid.monitorName() << "/" << m_edid.serialNumber() << " from " << m_physicalSize << "to " << overwriteSize;
m_physicalSize = overwriteSize;
}
// update modes
bool equal = m_conn->count_modes == m_driverModes.count();
for (int i = 0; equal && i < m_conn->count_modes; i++) {
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment