Displaying 5 results from an estimated 5 matches for "drm_get_edid_acpi".
2020 Jul 27
6
[PATCH 0/4] drm: add support for retrieving EDID via ACPI _DDC
Some notebook systems provide the EDID for the internal panel via the
_DDC method in ACPI, instead of or in addition to providing the EDID via
DDC on LVDS/eDP. Add a DRM helper to search for an ACP _DDC method under
the ACPI namespace for each VGA/3D controller, and return the first EDID
successfully retrieved via _DDC. Update the i915, nouveau, and radeon
DRM-KMS drivers to fall back to
2020 Jul 27
0
[PATCH 1/4] drm: retrieve EDID via ACPI _DDC method
...w the mux is switched.
The _DDC method can be implemented for each individual display
output, so there could be an arbitrary number of outputs exposing
their EDIDs via _DDC; however, in practice, this has only been
implemented so far on systems with a single panel, so the current
implementation of drm_get_edid_acpi() walks the outputs listed by
each GPU's ACPI _DOD method and returns the first EDID successfully
retrieved by any attached _DDC method. It may be necessary in the
future to allow for the retrieval of distinct EDIDs for different
output devices, but in order to do so, it will first be necessary...
2020 Jul 27
1
[PATCH 4/4] radeon: fall back to ACPI EDID retrieval
...ios_hardcoded_edid) {
+ struct edid *edid;
edid = kmalloc(rdev->mode_info.bios_hardcoded_edid_size, GFP_KERNEL);
if (edid) {
memcpy((unsigned char *)edid,
@@ -412,7 +411,8 @@ radeon_bios_get_hardcoded_edid(struct radeon_device *rdev)
return edid;
}
}
- return NULL;
+
+ return drm_get_edid_acpi();
}
static struct radeon_i2c_bus_rec combios_setup_i2c_bus(struct radeon_device *rdev,
--
2.18.4
2020 Jul 28
0
[PATCH 4/4] radeon: fall back to ACPI EDID retrieval
...d = kmalloc(rdev->mode_info.bios_hardcoded_edid_size, GFP_KERNEL);
> if (edid) {
> memcpy((unsigned char *)edid,
> @@ -412,7 +411,8 @@ radeon_bios_get_hardcoded_edid(struct radeon_device *rdev)
> return edid;
> }
> }
> - return NULL;
> +
> + return drm_get_edid_acpi();
In general a good idea, but I'm wondering if we should really do this so
unconditionally here.
Regards,
Christian.
> }
>
> static struct radeon_i2c_bus_rec combios_setup_i2c_bus(struct radeon_device *rdev,
2020 Aug 08
2
[PATCH 1/4] drm: retrieve EDID via ACPI _DDC method
...want to correlate the display objects with
drm_connectors or at least constrain the search to Display Type
"Internal/Integrated Digital Flat Panel" instead of picking the
first EDID found. Otherwise we might erroneously use the DDC
for an externally attached display.
> +struct edid *drm_get_edid_acpi(void)
> +{
> +#if defined(CONFIG_ACPI) && defined(CONFIG_PCI)
No, put an empty inline stub in the header file instead of using #ifdef, see:
https://www.kernel.org/doc/html/latest/process/coding-style.html#conditional-compilation
Patches 2, 3 and 4 need a "drm/" prefix in...