Re: [PATCH v2 1/2] drm: Implement DRM aperture helpers under video/

2022-06-21 Thread Javier Martinez Canillas
Hello Thomas, On 6/21/22 13:29, Thomas Zimmermann wrote: [...] >>> + >>> +static bool overlap(resource_size_t base1, resource_size_t end1, >>> + resource_size_t base2, resource_size_t end2) >>> +{ >>> + return (base1 < end2) && (end1 > base2); >>> +} >> >> There's a resource_over

Re: [PATCH v2 1/2] drm: Implement DRM aperture helpers under video/

2022-06-21 Thread Thomas Zimmermann
Hi Am 21.06.22 um 02:14 schrieb Javier Martinez Canillas: [...] Since we are talking about remove_conflicting_devices() here, a better code example could be for a platform device instead of a PCI device, like this: * static const struct platform_driver example_driver = { *

Re: [PATCH v2 1/2] drm: Implement DRM aperture helpers under video/

2022-06-20 Thread Javier Martinez Canillas
On 6/16/22 22:38, Alex Williamson wrote: > From: Thomas Zimmermann > > Implement DRM's aperture helpers under video/ for sharing with other > sub-systems. Remove DRM-isms from the interface. The helpers track > the ownership of framebuffer apertures and provide hand-over from > firmware, such as