Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-25 Thread Michel Dänzer
On 2021-10-25 14:28, Javier Martinez Canillas wrote: > Hello Michel, > > On 10/25/21 12:45, Michel Dänzer wrote: >> On 2021-10-24 22:32, Javier Martinez Canillas wrote: >>> Hello Ville, >>> >>> On 10/22/21 21:12, Ville Syrjälä wrote: On Fri, Oct 22, 2021 at 04:40:40PM +0200, Javier Martinez C

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-25 Thread Neal Gompa
On Mon, Oct 25, 2021 at 8:28 AM Javier Martinez Canillas wrote: > > Hello Michel, > > On 10/25/21 12:45, Michel Dänzer wrote: > > On 2021-10-24 22:32, Javier Martinez Canillas wrote: > >> Hello Ville, > >> > >> On 10/22/21 21:12, Ville Syrjälä wrote: > >>> On Fri, Oct 22, 2021 at 04:40:40PM +0200,

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-25 Thread Javier Martinez Canillas
Hello Michel, On 10/25/21 12:45, Michel Dänzer wrote: > On 2021-10-24 22:32, Javier Martinez Canillas wrote: >> Hello Ville, >> >> On 10/22/21 21:12, Ville Syrjälä wrote: >>> On Fri, Oct 22, 2021 at 04:40:40PM +0200, Javier Martinez Canillas wrote: The simpledrm driver allows to use the frame

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-25 Thread Michel Dänzer
On 2021-10-24 22:32, Javier Martinez Canillas wrote: > Hello Ville, > > On 10/22/21 21:12, Ville Syrjälä wrote: >> On Fri, Oct 22, 2021 at 04:40:40PM +0200, Javier Martinez Canillas wrote: >>> The simpledrm driver allows to use the frame buffer that was set-up by the >>> firmware. This gives early

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-24 Thread Neal Gompa
On Sun, Oct 24, 2021 at 4:40 PM Javier Martinez Canillas wrote: > > Hello Thomas, > > Thanks a lot for your feedback. > > On 10/22/21 21:05, Thomas Zimmermann wrote: > > > There's still the question of the semantics of this parameter. It's a > > bit fuzzy. > > > > If you use 'disable_handover' (as

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-24 Thread Javier Martinez Canillas
Hello Thomas, Thanks a lot for your feedback. On 10/22/21 21:05, Thomas Zimmermann wrote: [snip] >> >> +static bool drm_aperture_remove_fb = true; > > Global variables should default to zero if somehow possible. This way, > they can all be stored in the BSS segment and backed by a single s

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-24 Thread Javier Martinez Canillas
Hello Ville, On 10/22/21 21:12, Ville Syrjälä wrote: > On Fri, Oct 22, 2021 at 04:40:40PM +0200, Javier Martinez Canillas wrote: >> The simpledrm driver allows to use the frame buffer that was set-up by the >> firmware. This gives early video output before the platform DRM driver is >> probed and

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Ville Syrjälä
On Fri, Oct 22, 2021 at 04:40:40PM +0200, Javier Martinez Canillas wrote: > The simpledrm driver allows to use the frame buffer that was set-up by the > firmware. This gives early video output before the platform DRM driver is > probed and takes over. > > But it would be useful to have a way to di

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Thomas Zimmermann
Hi, thanks for sending the patch out quickly. Am 22.10.21 um 16:40 schrieb Javier Martinez Canillas: The simpledrm driver allows to use the frame buffer that was set-up by the firmware. This gives early video output before the platform DRM driver is probed and takes over. But it would be usefu

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Neal Gompa
On Fri, Oct 22, 2021 at 11:16 AM Javier Martinez Canillas wrote: > > Hello Neal, > > Thanks for your feedback. > > On 10/22/21 16:56, Neal Gompa wrote: > > On Fri, Oct 22, 2021 at 10:40 AM Javier Martinez Canillas > > wrote: > >> > >> The simpledrm driver allows to use the frame buffer that was s

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Neal Gompa
On Fri, Oct 22, 2021 at 10:40 AM Javier Martinez Canillas wrote: > > The simpledrm driver allows to use the frame buffer that was set-up by the > firmware. This gives early video output before the platform DRM driver is > probed and takes over. > > But it would be useful to have a way to disable t

Re: [RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Javier Martinez Canillas
Hello Neal, Thanks for your feedback. On 10/22/21 16:56, Neal Gompa wrote: > On Fri, Oct 22, 2021 at 10:40 AM Javier Martinez Canillas > wrote: >> >> The simpledrm driver allows to use the frame buffer that was set-up by the >> firmware. This gives early video output before the platform DRM driv

[RFC PATCH] drm/aperture: Add param to disable conflicting framebuffers removal

2021-10-22 Thread Javier Martinez Canillas
The simpledrm driver allows to use the frame buffer that was set-up by the firmware. This gives early video output before the platform DRM driver is probed and takes over. But it would be useful to have a way to disable this take over by the real DRM drivers. For example, there may be bugs in the