From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 762DEC04AAF for ; Tue, 21 May 2019 12:43:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 36660217D7 for ; Tue, 21 May 2019 12:43:07 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ffwll.ch header.i=@ffwll.ch header.b="ISAP2GTd" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728135AbfEUMnG (ORCPT ); Tue, 21 May 2019 08:43:06 -0400 Received: from mail-ed1-f65.google.com ([209.85.208.65]:43134 "EHLO mail-ed1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726344AbfEUMnF (ORCPT ); Tue, 21 May 2019 08:43:05 -0400 Received: by mail-ed1-f65.google.com with SMTP id w33so29155056edb.10 for ; Tue, 21 May 2019 05:43:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=sender:date:from:to:cc:subject:message-id:mail-followup-to :references:mime-version:content-disposition :content-transfer-encoding:in-reply-to:user-agent; bh=8iobs6bKR0V0jviIyIn94LH1ldLUcwxTRpiup8U9bVE=; b=ISAP2GTdjSpoZIgSAiFoIruqS78MNxfKT0SAQ+XwvcB/3CSnjvjywTQCtIiLGS6BQ8 7BQdUCIuOKKJDinCsXKoDCLLcincKnHs/wTkWygThSCAroUwG51MNxnNVFmyc7LBUKVT BsaCOWgE08JXWZ2W2NmgPDlsqm1AlBeXGIoMM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :mail-followup-to:references:mime-version:content-disposition :content-transfer-encoding:in-reply-to:user-agent; bh=8iobs6bKR0V0jviIyIn94LH1ldLUcwxTRpiup8U9bVE=; b=EULI8wBtZhSBn+CrboO0ZI7k7E5F5/bfMYSgirN32++lCHlZ2lCdCFwRaLQ58Q2zN0 N6RTcmMuM7AGZs73IxCepM+wJSUch6XFaYXc2OrtZUMH2Cuvksq9Wdi6aTvhDXJiK6W+ GKGkRBypvJZBSFLSkKQKj8WHOTJ8slpNry7cwz9PxIJ8cnrimtCS4Pyf39oqiUE6rLHi 2A5BAeafP87M2yx6Tk1OKmpI03LgeshAw06FzMLRYv1KtWVEyoFLQS8Er1R9q465nHuk uxYWp225DA6fT8AZ6UMN06Tnz79pm3Wmh6w1ixG5CB9HcZi4AIw1kbvrHBVxYfynEY6X 74/Q== X-Gm-Message-State: APjAAAUG94tB3SoJgrP7MMt3yFl0pHrIF9fOLRWIyA+O0ARE+fTjGET3 KTVZl3AAxUfb1u99oRUO2IPSHg== X-Google-Smtp-Source: APXvYqzMrsDdLDH5tyYyz2KdETmLHcsyneRHpQkU+94+T7ZpPF9uux7Se6P4Ws/5azkq5DgtdGvodw== X-Received: by 2002:a17:906:b2d3:: with SMTP id cf19mr22060657ejb.10.1558442582598; Tue, 21 May 2019 05:43:02 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:569e:0:3106:d637:d723:e855]) by smtp.gmail.com with ESMTPSA id a61sm6295741edf.8.2019.05.21.05.43.01 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 21 May 2019 05:43:01 -0700 (PDT) Date: Tue, 21 May 2019 14:42:59 +0200 From: Daniel Vetter To: Maarten Lankhorst Cc: Daniel Vetter , DRI Development , Daniel Thompson , Bartlomiej Zolnierkiewicz , Intel Graphics Development , LKML , =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Yisheng Xie , Hans de Goede , Mikulas Patocka , linux-fbdev@vger.kernel.org, Jingoo Han , Daniel Vetter , Lee Jones , Peter Rosin Subject: Re: [PATCH 29/33] fbcon: replace FB_EVENT_MODE_CHANGE/_ALL with direct calls Message-ID: <20190521124259.GN21222@phenom.ffwll.local> Mail-Followup-To: Maarten Lankhorst , DRI Development , Daniel Thompson , Bartlomiej Zolnierkiewicz , Intel Graphics Development , LKML , =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Yisheng Xie , Hans de Goede , Mikulas Patocka , linux-fbdev@vger.kernel.org, Jingoo Han , Daniel Vetter , Lee Jones , Peter Rosin References: <20190520082216.26273-1-daniel.vetter@ffwll.ch> <20190520082216.26273-30-daniel.vetter@ffwll.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Operating-System: Linux phenom 4.14.0-3-amd64 User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 21, 2019 at 12:56:30PM +0200, Maarten Lankhorst wrote: > Op 20-05-2019 om 10:22 schreef Daniel Vetter: > > Create a new wrapper function for this, feels like there's some > > refactoring room here between the two modes. > > > > Signed-off-by: Daniel Vetter > > Cc: Lee Jones > > Cc: Daniel Thompson > > Cc: Jingoo Han > > Cc: Bartlomiej Zolnierkiewicz > > Cc: Daniel Vetter > > Cc: Hans de Goede > > Cc: Yisheng Xie > > Cc: "Michał Mirosław" > > Cc: Peter Rosin > > Cc: Mikulas Patocka > > Cc: linux-fbdev@vger.kernel.org > > --- > > drivers/video/backlight/lcd.c | 2 -- > > drivers/video/fbdev/core/fbcon.c | 15 +++++++++------ > > drivers/video/fbdev/core/fbmem.c | 13 ++----------- > > drivers/video/fbdev/sh_mobile_lcdcfb.c | 11 +---------- > > include/linux/fb.h | 4 ---- > > include/linux/fbcon.h | 2 ++ > > 6 files changed, 14 insertions(+), 33 deletions(-) > > > > diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c > > index 4b40c6a4d441..16298041b141 100644 > > --- a/drivers/video/backlight/lcd.c > > +++ b/drivers/video/backlight/lcd.c > > @@ -32,8 +32,6 @@ static int fb_notifier_callback(struct notifier_block *self, > > /* If we aren't interested in this event, skip it immediately ... */ > > switch (event) { > > case FB_EVENT_BLANK: > > - case FB_EVENT_MODE_CHANGE: > > - case FB_EVENT_MODE_CHANGE_ALL: > > case FB_EARLY_EVENT_BLANK: > > case FB_R_EARLY_EVENT_BLANK: > > break; > > Below it performs a call to set_mode() if it's none of the blanking events; it can be removed. :) Oops ... I think I'll insert a patch here to create a new MODESET event for backlights. We still need this one I think (maybe not for kms, but for old fbdev drivers). And maybe a wrapper for fb_backlight_notify on top ... Thanks for spotting this. -Daniel > > > diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c > > index c1a7476e980f..8cc62d340387 100644 > > --- a/drivers/video/fbdev/core/fbcon.c > > +++ b/drivers/video/fbdev/core/fbcon.c > > @@ -3005,6 +3005,15 @@ static void fbcon_set_all_vcs(struct fb_info *info) > > fbcon_modechanged(info); > > } > > > > + > > +void fbcon_update_vcs(struct fb_info *info, bool all) > > +{ > > + if (all) > > + fbcon_set_all_vcs(info); > > + else > > + fbcon_modechanged(info); > > +} > > + > > int fbcon_mode_deleted(struct fb_info *info, > > struct fb_videomode *mode) > > { > > @@ -3314,12 +3323,6 @@ static int fbcon_event_notify(struct notifier_block *self, > > int idx, ret = 0; > > > > switch(action) { > > - case FB_EVENT_MODE_CHANGE: > > - fbcon_modechanged(info); > > - break; > > - case FB_EVENT_MODE_CHANGE_ALL: > > - fbcon_set_all_vcs(info); > > - break; > > case FB_EVENT_SET_CONSOLE_MAP: > > /* called with console lock held */ > > con2fb = event->data; > > diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c > > index cbd58ba8a59d..55b88163edc2 100644 > > --- a/drivers/video/fbdev/core/fbmem.c > > +++ b/drivers/video/fbdev/core/fbmem.c > > @@ -1039,17 +1039,8 @@ fb_set_var(struct fb_info *info, struct fb_var_screeninfo *var) > > !list_empty(&info->modelist)) > > ret = fb_add_videomode(&mode, &info->modelist); > > > > - if (!ret && (flags & FBINFO_MISC_USEREVENT)) { > > - struct fb_event event; > > - int evnt = (activate & FB_ACTIVATE_ALL) ? > > - FB_EVENT_MODE_CHANGE_ALL : > > - FB_EVENT_MODE_CHANGE; > > - > > - info->flags &= ~FBINFO_MISC_USEREVENT; > > - event.info = info; > > - event.data = &mode; > > - fb_notifier_call_chain(evnt, &event); > > - } > > + if (!ret && (flags & FBINFO_MISC_USEREVENT)) > > + fbcon_update_vcs(info, activate & FB_ACTIVATE_ALL); > > > > return ret; > > } > > diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c > > index 0d7a044852d7..bb1a610d0363 100644 > > --- a/drivers/video/fbdev/sh_mobile_lcdcfb.c > > +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.c > > @@ -1776,8 +1776,6 @@ static void sh_mobile_fb_reconfig(struct fb_info *info) > > struct sh_mobile_lcdc_chan *ch = info->par; > > struct fb_var_screeninfo var; > > struct fb_videomode mode; > > - struct fb_event event; > > - int evnt = FB_EVENT_MODE_CHANGE_ALL; > > > > if (ch->use_count > 1 || (ch->use_count == 1 && !info->fbcon_par)) > > /* More framebuffer users are active */ > > @@ -1799,14 +1797,7 @@ static void sh_mobile_fb_reconfig(struct fb_info *info) > > /* Couldn't reconfigure, hopefully, can continue as before */ > > return; > > > > - /* > > - * fb_set_var() calls the notifier change internally, only if > > - * FBINFO_MISC_USEREVENT flag is set. Since we do not want to fake a > > - * user event, we have to call the chain ourselves. > > - */ > > - event.info = info; > > - event.data = &ch->display.mode; > > - fb_notifier_call_chain(evnt, &event); > > + fbcon_update_vcs(info, true); > > } > > > > /* > > diff --git a/include/linux/fb.h b/include/linux/fb.h > > index 4b9b882f8f52..54d6bee09121 100644 > > --- a/include/linux/fb.h > > +++ b/include/linux/fb.h > > @@ -124,16 +124,12 @@ struct fb_cursor_user { > > * Register/unregister for framebuffer events > > */ > > > > -/* The resolution of the passed in fb_info about to change */ > > -#define FB_EVENT_MODE_CHANGE 0x01 > > /* CONSOLE-SPECIFIC: get console to framebuffer mapping */ > > #define FB_EVENT_GET_CONSOLE_MAP 0x07 > > /* CONSOLE-SPECIFIC: set console to framebuffer mapping */ > > #define FB_EVENT_SET_CONSOLE_MAP 0x08 > > /* A display blank is requested */ > > #define FB_EVENT_BLANK 0x09 > > -/* Private modelist is to be replaced */ > > -#define FB_EVENT_MODE_CHANGE_ALL 0x0B > > /* CONSOLE-SPECIFIC: remap all consoles to new fb - for vga_switcheroo */ > > #define FB_EVENT_REMAP_ALL_CONSOLE 0x0F > > /* A hardware display blank early change occurred */ > > diff --git a/include/linux/fbcon.h b/include/linux/fbcon.h > > index 90e196c835dd..daaa97b0c9e6 100644 > > --- a/include/linux/fbcon.h > > +++ b/include/linux/fbcon.h > > @@ -15,6 +15,7 @@ void fbcon_new_modelist(struct fb_info *info); > > void fbcon_get_requirement(struct fb_info *info, > > struct fb_blit_caps *caps); > > void fbcon_fb_blanked(struct fb_info *info, int blank); > > +void fbcon_update_vcs(struct fb_info *info, bool all); > > #else > > static inline void fb_console_init(void) {} > > static inline void fb_console_exit(void) {} > > @@ -29,6 +30,7 @@ void fbcon_new_modelist(struct fb_info *info) {} > > void fbcon_get_requirement(struct fb_info *info, > > struct fb_blit_caps *caps) {} > > void fbcon_fb_blanked(struct fb_info *info, int blank) {} > > +void fbcon_update_vcs(struct fb_info *info, bool all) {} > > #endif > > > > #endif /* _LINUX_FBCON_H */ > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Date: Tue, 21 May 2019 12:42:59 +0000 Subject: Re: [PATCH 29/33] fbcon: replace FB_EVENT_MODE_CHANGE/_ALL with direct calls Message-Id: <20190521124259.GN21222@phenom.ffwll.local> List-Id: References: <20190520082216.26273-1-daniel.vetter@ffwll.ch> <20190520082216.26273-30-daniel.vetter@ffwll.ch> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Maarten Lankhorst Cc: Daniel Thompson , Bartlomiej Zolnierkiewicz , Daniel Vetter , Intel Graphics Development , LKML , DRI Development , =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Yisheng Xie , Hans de Goede , Mikulas Patocka , linux-fbdev@vger.kernel.org, Jingoo Han , Daniel Vetter , Lee Jones , Peter Rosin On Tue, May 21, 2019 at 12:56:30PM +0200, Maarten Lankhorst wrote: > Op 20-05-2019 om 10:22 schreef Daniel Vetter: > > Create a new wrapper function for this, feels like there's some > > refactoring room here between the two modes. > > > > Signed-off-by: Daniel Vetter > > Cc: Lee Jones > > Cc: Daniel Thompson > > Cc: Jingoo Han > > Cc: Bartlomiej Zolnierkiewicz > > Cc: Daniel Vetter > > Cc: Hans de Goede > > Cc: Yisheng Xie > > Cc: "Michał Mirosław" > > Cc: Peter Rosin > > Cc: Mikulas Patocka > > Cc: linux-fbdev@vger.kernel.org > > --- > > drivers/video/backlight/lcd.c | 2 -- > > drivers/video/fbdev/core/fbcon.c | 15 +++++++++------ > > drivers/video/fbdev/core/fbmem.c | 13 ++----------- > > drivers/video/fbdev/sh_mobile_lcdcfb.c | 11 +---------- > > include/linux/fb.h | 4 ---- > > include/linux/fbcon.h | 2 ++ > > 6 files changed, 14 insertions(+), 33 deletions(-) > > > > diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c > > index 4b40c6a4d441..16298041b141 100644 > > --- a/drivers/video/backlight/lcd.c > > +++ b/drivers/video/backlight/lcd.c > > @@ -32,8 +32,6 @@ static int fb_notifier_callback(struct notifier_block *self, > > /* If we aren't interested in this event, skip it immediately ... */ > > switch (event) { > > case FB_EVENT_BLANK: > > - case FB_EVENT_MODE_CHANGE: > > - case FB_EVENT_MODE_CHANGE_ALL: > > case FB_EARLY_EVENT_BLANK: > > case FB_R_EARLY_EVENT_BLANK: > > break; > > Below it performs a call to set_mode() if it's none of the blanking events; it can be removed. :) Oops ... I think I'll insert a patch here to create a new MODESET event for backlights. We still need this one I think (maybe not for kms, but for old fbdev drivers). And maybe a wrapper for fb_backlight_notify on top ... Thanks for spotting this. -Daniel > > > diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c > > index c1a7476e980f..8cc62d340387 100644 > > --- a/drivers/video/fbdev/core/fbcon.c > > +++ b/drivers/video/fbdev/core/fbcon.c > > @@ -3005,6 +3005,15 @@ static void fbcon_set_all_vcs(struct fb_info *info) > > fbcon_modechanged(info); > > } > > > > + > > +void fbcon_update_vcs(struct fb_info *info, bool all) > > +{ > > + if (all) > > + fbcon_set_all_vcs(info); > > + else > > + fbcon_modechanged(info); > > +} > > + > > int fbcon_mode_deleted(struct fb_info *info, > > struct fb_videomode *mode) > > { > > @@ -3314,12 +3323,6 @@ static int fbcon_event_notify(struct notifier_block *self, > > int idx, ret = 0; > > > > switch(action) { > > - case FB_EVENT_MODE_CHANGE: > > - fbcon_modechanged(info); > > - break; > > - case FB_EVENT_MODE_CHANGE_ALL: > > - fbcon_set_all_vcs(info); > > - break; > > case FB_EVENT_SET_CONSOLE_MAP: > > /* called with console lock held */ > > con2fb = event->data; > > diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c > > index cbd58ba8a59d..55b88163edc2 100644 > > --- a/drivers/video/fbdev/core/fbmem.c > > +++ b/drivers/video/fbdev/core/fbmem.c > > @@ -1039,17 +1039,8 @@ fb_set_var(struct fb_info *info, struct fb_var_screeninfo *var) > > !list_empty(&info->modelist)) > > ret = fb_add_videomode(&mode, &info->modelist); > > > > - if (!ret && (flags & FBINFO_MISC_USEREVENT)) { > > - struct fb_event event; > > - int evnt = (activate & FB_ACTIVATE_ALL) ? > > - FB_EVENT_MODE_CHANGE_ALL : > > - FB_EVENT_MODE_CHANGE; > > - > > - info->flags &= ~FBINFO_MISC_USEREVENT; > > - event.info = info; > > - event.data = &mode; > > - fb_notifier_call_chain(evnt, &event); > > - } > > + if (!ret && (flags & FBINFO_MISC_USEREVENT)) > > + fbcon_update_vcs(info, activate & FB_ACTIVATE_ALL); > > > > return ret; > > } > > diff --git a/drivers/video/fbdev/sh_mobile_lcdcfb.c b/drivers/video/fbdev/sh_mobile_lcdcfb.c > > index 0d7a044852d7..bb1a610d0363 100644 > > --- a/drivers/video/fbdev/sh_mobile_lcdcfb.c > > +++ b/drivers/video/fbdev/sh_mobile_lcdcfb.c > > @@ -1776,8 +1776,6 @@ static void sh_mobile_fb_reconfig(struct fb_info *info) > > struct sh_mobile_lcdc_chan *ch = info->par; > > struct fb_var_screeninfo var; > > struct fb_videomode mode; > > - struct fb_event event; > > - int evnt = FB_EVENT_MODE_CHANGE_ALL; > > > > if (ch->use_count > 1 || (ch->use_count = 1 && !info->fbcon_par)) > > /* More framebuffer users are active */ > > @@ -1799,14 +1797,7 @@ static void sh_mobile_fb_reconfig(struct fb_info *info) > > /* Couldn't reconfigure, hopefully, can continue as before */ > > return; > > > > - /* > > - * fb_set_var() calls the notifier change internally, only if > > - * FBINFO_MISC_USEREVENT flag is set. Since we do not want to fake a > > - * user event, we have to call the chain ourselves. > > - */ > > - event.info = info; > > - event.data = &ch->display.mode; > > - fb_notifier_call_chain(evnt, &event); > > + fbcon_update_vcs(info, true); > > } > > > > /* > > diff --git a/include/linux/fb.h b/include/linux/fb.h > > index 4b9b882f8f52..54d6bee09121 100644 > > --- a/include/linux/fb.h > > +++ b/include/linux/fb.h > > @@ -124,16 +124,12 @@ struct fb_cursor_user { > > * Register/unregister for framebuffer events > > */ > > > > -/* The resolution of the passed in fb_info about to change */ > > -#define FB_EVENT_MODE_CHANGE 0x01 > > /* CONSOLE-SPECIFIC: get console to framebuffer mapping */ > > #define FB_EVENT_GET_CONSOLE_MAP 0x07 > > /* CONSOLE-SPECIFIC: set console to framebuffer mapping */ > > #define FB_EVENT_SET_CONSOLE_MAP 0x08 > > /* A display blank is requested */ > > #define FB_EVENT_BLANK 0x09 > > -/* Private modelist is to be replaced */ > > -#define FB_EVENT_MODE_CHANGE_ALL 0x0B > > /* CONSOLE-SPECIFIC: remap all consoles to new fb - for vga_switcheroo */ > > #define FB_EVENT_REMAP_ALL_CONSOLE 0x0F > > /* A hardware display blank early change occurred */ > > diff --git a/include/linux/fbcon.h b/include/linux/fbcon.h > > index 90e196c835dd..daaa97b0c9e6 100644 > > --- a/include/linux/fbcon.h > > +++ b/include/linux/fbcon.h > > @@ -15,6 +15,7 @@ void fbcon_new_modelist(struct fb_info *info); > > void fbcon_get_requirement(struct fb_info *info, > > struct fb_blit_caps *caps); > > void fbcon_fb_blanked(struct fb_info *info, int blank); > > +void fbcon_update_vcs(struct fb_info *info, bool all); > > #else > > static inline void fb_console_init(void) {} > > static inline void fb_console_exit(void) {} > > @@ -29,6 +30,7 @@ void fbcon_new_modelist(struct fb_info *info) {} > > void fbcon_get_requirement(struct fb_info *info, > > struct fb_blit_caps *caps) {} > > void fbcon_fb_blanked(struct fb_info *info, int blank) {} > > +void fbcon_update_vcs(struct fb_info *info, bool all) {} > > #endif > > > > #endif /* _LINUX_FBCON_H */ > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 29/33] fbcon: replace FB_EVENT_MODE_CHANGE/_ALL with direct calls Date: Tue, 21 May 2019 14:42:59 +0200 Message-ID: <20190521124259.GN21222@phenom.ffwll.local> References: <20190520082216.26273-1-daniel.vetter@ffwll.ch> <20190520082216.26273-30-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-ed1-x541.google.com (mail-ed1-x541.google.com [IPv6:2a00:1450:4864:20::541]) by gabe.freedesktop.org (Postfix) with ESMTPS id 058EC89192 for ; Tue, 21 May 2019 12:43:04 +0000 (UTC) Received: by mail-ed1-x541.google.com with SMTP id w11so29156725edl.5 for ; Tue, 21 May 2019 05:43:03 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Maarten Lankhorst Cc: Daniel Thompson , Bartlomiej Zolnierkiewicz , Daniel Vetter , Intel Graphics Development , LKML , DRI Development , =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Yisheng Xie , Hans de Goede , Mikulas Patocka , linux-fbdev@vger.kernel.org, Jingoo Han , Daniel Vetter , Lee Jones , Peter Rosin List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBNYXkgMjEsIDIwMTkgYXQgMTI6NTY6MzBQTSArMDIwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gT3AgMjAtMDUtMjAxOSBvbSAxMDoyMiBzY2hyZWVmIERhbmllbCBWZXR0ZXI6 Cj4gPiBDcmVhdGUgYSBuZXcgd3JhcHBlciBmdW5jdGlvbiBmb3IgdGhpcywgZmVlbHMgbGlrZSB0 aGVyZSdzIHNvbWUKPiA+IHJlZmFjdG9yaW5nIHJvb20gaGVyZSBiZXR3ZWVuIHRoZSB0d28gbW9k ZXMuCj4gPgo+ID4gU2lnbmVkLW9mZi1ieTogRGFuaWVsIFZldHRlciA8ZGFuaWVsLnZldHRlckBp bnRlbC5jb20+Cj4gPiBDYzogTGVlIEpvbmVzIDxsZWUuam9uZXNAbGluYXJvLm9yZz4KPiA+IENj OiBEYW5pZWwgVGhvbXBzb24gPGRhbmllbC50aG9tcHNvbkBsaW5hcm8ub3JnPgo+ID4gQ2M6IEpp bmdvbyBIYW4gPGppbmdvb2hhbjFAZ21haWwuY29tPgo+ID4gQ2M6IEJhcnRsb21pZWogWm9sbmll cmtpZXdpY3ogPGIuem9sbmllcmtpZUBzYW1zdW5nLmNvbT4KPiA+IENjOiBEYW5pZWwgVmV0dGVy IDxkYW5pZWwudmV0dGVyQGZmd2xsLmNoPgo+ID4gQ2M6IEhhbnMgZGUgR29lZGUgPGhkZWdvZWRl QHJlZGhhdC5jb20+Cj4gPiBDYzogWWlzaGVuZyBYaWUgPHlzeGllQGZveG1haWwuY29tPgo+ID4g Q2M6ICJNaWNoYcWCIE1pcm9zxYJhdyIgPG1pcnEtbGludXhAcmVyZS5xbXFtLnBsPgo+ID4gQ2M6 IFBldGVyIFJvc2luIDxwZWRhQGF4ZW50aWEuc2U+Cj4gPiBDYzogTWlrdWxhcyBQYXRvY2thIDxt cGF0b2NrYUByZWRoYXQuY29tPgo+ID4gQ2M6IGxpbnV4LWZiZGV2QHZnZXIua2VybmVsLm9yZwo+ ID4gLS0tCj4gPiAgZHJpdmVycy92aWRlby9iYWNrbGlnaHQvbGNkLmMgICAgICAgICAgfCAgMiAt LQo+ID4gIGRyaXZlcnMvdmlkZW8vZmJkZXYvY29yZS9mYmNvbi5jICAgICAgIHwgMTUgKysrKysr KysrLS0tLS0tCj4gPiAgZHJpdmVycy92aWRlby9mYmRldi9jb3JlL2ZibWVtLmMgICAgICAgfCAx MyArKy0tLS0tLS0tLS0tCj4gPiAgZHJpdmVycy92aWRlby9mYmRldi9zaF9tb2JpbGVfbGNkY2Zi LmMgfCAxMSArLS0tLS0tLS0tLQo+ID4gIGluY2x1ZGUvbGludXgvZmIuaCAgICAgICAgICAgICAg ICAgICAgIHwgIDQgLS0tLQo+ID4gIGluY2x1ZGUvbGludXgvZmJjb24uaCAgICAgICAgICAgICAg ICAgIHwgIDIgKysKPiA+ICA2IGZpbGVzIGNoYW5nZWQsIDE0IGluc2VydGlvbnMoKyksIDMzIGRl bGV0aW9ucygtKQo+ID4KPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3ZpZGVvL2JhY2tsaWdodC9s Y2QuYyBiL2RyaXZlcnMvdmlkZW8vYmFja2xpZ2h0L2xjZC5jCj4gPiBpbmRleCA0YjQwYzZhNGQ0 NDEuLjE2Mjk4MDQxYjE0MSAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvdmlkZW8vYmFja2xpZ2h0 L2xjZC5jCj4gPiArKysgYi9kcml2ZXJzL3ZpZGVvL2JhY2tsaWdodC9sY2QuYwo+ID4gQEAgLTMy LDggKzMyLDYgQEAgc3RhdGljIGludCBmYl9ub3RpZmllcl9jYWxsYmFjayhzdHJ1Y3Qgbm90aWZp ZXJfYmxvY2sgKnNlbGYsCj4gPiAgCS8qIElmIHdlIGFyZW4ndCBpbnRlcmVzdGVkIGluIHRoaXMg ZXZlbnQsIHNraXAgaXQgaW1tZWRpYXRlbHkgLi4uICovCj4gPiAgCXN3aXRjaCAoZXZlbnQpIHsK PiA+ICAJY2FzZSBGQl9FVkVOVF9CTEFOSzoKPiA+IC0JY2FzZSBGQl9FVkVOVF9NT0RFX0NIQU5H RToKPiA+IC0JY2FzZSBGQl9FVkVOVF9NT0RFX0NIQU5HRV9BTEw6Cj4gPiAgCWNhc2UgRkJfRUFS TFlfRVZFTlRfQkxBTks6Cj4gPiAgCWNhc2UgRkJfUl9FQVJMWV9FVkVOVF9CTEFOSzoKPiA+ICAJ CWJyZWFrOwo+IAo+IEJlbG93IGl0IHBlcmZvcm1zIGEgY2FsbCB0byBzZXRfbW9kZSgpIGlmIGl0 J3Mgbm9uZSBvZiB0aGUgYmxhbmtpbmcgZXZlbnRzOyBpdCBjYW4gYmUgcmVtb3ZlZC4gOikKCk9v cHMgLi4uIEkgdGhpbmsgSSdsbCBpbnNlcnQgYSBwYXRjaCBoZXJlIHRvIGNyZWF0ZSBhIG5ldyBN T0RFU0VUIGV2ZW50CmZvciBiYWNrbGlnaHRzLiBXZSBzdGlsbCBuZWVkIHRoaXMgb25lIEkgdGhp bmsgKG1heWJlIG5vdCBmb3Iga21zLCBidXQgZm9yCm9sZCBmYmRldiBkcml2ZXJzKS4gQW5kIG1h eWJlIGEgd3JhcHBlciBmb3IgZmJfYmFja2xpZ2h0X25vdGlmeSBvbiB0b3AgLi4uCgpUaGFua3Mg Zm9yIHNwb3R0aW5nIHRoaXMuCi1EYW5pZWwKCj4gCj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy92 aWRlby9mYmRldi9jb3JlL2ZiY29uLmMgYi9kcml2ZXJzL3ZpZGVvL2ZiZGV2L2NvcmUvZmJjb24u Ywo+ID4gaW5kZXggYzFhNzQ3NmU5ODBmLi44Y2M2MmQzNDAzODcgMTAwNjQ0Cj4gPiAtLS0gYS9k cml2ZXJzL3ZpZGVvL2ZiZGV2L2NvcmUvZmJjb24uYwo+ID4gKysrIGIvZHJpdmVycy92aWRlby9m YmRldi9jb3JlL2ZiY29uLmMKPiA+IEBAIC0zMDA1LDYgKzMwMDUsMTUgQEAgc3RhdGljIHZvaWQg ZmJjb25fc2V0X2FsbF92Y3Moc3RydWN0IGZiX2luZm8gKmluZm8pCj4gPiAgCQlmYmNvbl9tb2Rl Y2hhbmdlZChpbmZvKTsKPiA+ICB9Cj4gPiAgCj4gPiArCj4gPiArdm9pZCBmYmNvbl91cGRhdGVf dmNzKHN0cnVjdCBmYl9pbmZvICppbmZvLCBib29sIGFsbCkKPiA+ICt7Cj4gPiArCWlmIChhbGwp Cj4gPiArCQlmYmNvbl9zZXRfYWxsX3ZjcyhpbmZvKTsKPiA+ICsJZWxzZQo+ID4gKwkJZmJjb25f bW9kZWNoYW5nZWQoaW5mbyk7Cj4gPiArfQo+ID4gKwo+ID4gIGludCBmYmNvbl9tb2RlX2RlbGV0 ZWQoc3RydWN0IGZiX2luZm8gKmluZm8sCj4gPiAgCQkgICAgICAgc3RydWN0IGZiX3ZpZGVvbW9k ZSAqbW9kZSkKPiA+ICB7Cj4gPiBAQCAtMzMxNCwxMiArMzMyMyw2IEBAIHN0YXRpYyBpbnQgZmJj b25fZXZlbnRfbm90aWZ5KHN0cnVjdCBub3RpZmllcl9ibG9jayAqc2VsZiwKPiA+ICAJaW50IGlk eCwgcmV0ID0gMDsKPiA+ICAKPiA+ICAJc3dpdGNoKGFjdGlvbikgewo+ID4gLQljYXNlIEZCX0VW RU5UX01PREVfQ0hBTkdFOgo+ID4gLQkJZmJjb25fbW9kZWNoYW5nZWQoaW5mbyk7Cj4gPiAtCQli cmVhazsKPiA+IC0JY2FzZSBGQl9FVkVOVF9NT0RFX0NIQU5HRV9BTEw6Cj4gPiAtCQlmYmNvbl9z ZXRfYWxsX3ZjcyhpbmZvKTsKPiA+IC0JCWJyZWFrOwo+ID4gIAljYXNlIEZCX0VWRU5UX1NFVF9D T05TT0xFX01BUDoKPiA+ICAJCS8qIGNhbGxlZCB3aXRoIGNvbnNvbGUgbG9jayBoZWxkICovCj4g PiAgCQljb24yZmIgPSBldmVudC0+ZGF0YTsKPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3ZpZGVv L2ZiZGV2L2NvcmUvZmJtZW0uYyBiL2RyaXZlcnMvdmlkZW8vZmJkZXYvY29yZS9mYm1lbS5jCj4g PiBpbmRleCBjYmQ1OGJhOGE1OWQuLjU1Yjg4MTYzZWRjMiAxMDA2NDQKPiA+IC0tLSBhL2RyaXZl cnMvdmlkZW8vZmJkZXYvY29yZS9mYm1lbS5jCj4gPiArKysgYi9kcml2ZXJzL3ZpZGVvL2ZiZGV2 L2NvcmUvZmJtZW0uYwo+ID4gQEAgLTEwMzksMTcgKzEwMzksOCBAQCBmYl9zZXRfdmFyKHN0cnVj dCBmYl9pbmZvICppbmZvLCBzdHJ1Y3QgZmJfdmFyX3NjcmVlbmluZm8gKnZhcikKPiA+ICAJICAg ICFsaXN0X2VtcHR5KCZpbmZvLT5tb2RlbGlzdCkpCj4gPiAgCQlyZXQgPSBmYl9hZGRfdmlkZW9t b2RlKCZtb2RlLCAmaW5mby0+bW9kZWxpc3QpOwo+ID4gIAo+ID4gLQlpZiAoIXJldCAmJiAoZmxh Z3MgJiBGQklORk9fTUlTQ19VU0VSRVZFTlQpKSB7Cj4gPiAtCQlzdHJ1Y3QgZmJfZXZlbnQgZXZl bnQ7Cj4gPiAtCQlpbnQgZXZudCA9IChhY3RpdmF0ZSAmIEZCX0FDVElWQVRFX0FMTCkgPwo+ID4g LQkJCUZCX0VWRU5UX01PREVfQ0hBTkdFX0FMTCA6Cj4gPiAtCQkJRkJfRVZFTlRfTU9ERV9DSEFO R0U7Cj4gPiAtCj4gPiAtCQlpbmZvLT5mbGFncyAmPSB+RkJJTkZPX01JU0NfVVNFUkVWRU5UOwo+ ID4gLQkJZXZlbnQuaW5mbyA9IGluZm87Cj4gPiAtCQlldmVudC5kYXRhID0gJm1vZGU7Cj4gPiAt CQlmYl9ub3RpZmllcl9jYWxsX2NoYWluKGV2bnQsICZldmVudCk7Cj4gPiAtCX0KPiA+ICsJaWYg KCFyZXQgJiYgKGZsYWdzICYgRkJJTkZPX01JU0NfVVNFUkVWRU5UKSkKPiA+ICsJCWZiY29uX3Vw ZGF0ZV92Y3MoaW5mbywgYWN0aXZhdGUgJiBGQl9BQ1RJVkFURV9BTEwpOwo+ID4gIAo+ID4gIAly ZXR1cm4gcmV0Owo+ID4gIH0KPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3ZpZGVvL2ZiZGV2L3No X21vYmlsZV9sY2RjZmIuYyBiL2RyaXZlcnMvdmlkZW8vZmJkZXYvc2hfbW9iaWxlX2xjZGNmYi5j Cj4gPiBpbmRleCAwZDdhMDQ0ODUyZDcuLmJiMWE2MTBkMDM2MyAxMDA2NDQKPiA+IC0tLSBhL2Ry aXZlcnMvdmlkZW8vZmJkZXYvc2hfbW9iaWxlX2xjZGNmYi5jCj4gPiArKysgYi9kcml2ZXJzL3Zp ZGVvL2ZiZGV2L3NoX21vYmlsZV9sY2RjZmIuYwo+ID4gQEAgLTE3NzYsOCArMTc3Niw2IEBAIHN0 YXRpYyB2b2lkIHNoX21vYmlsZV9mYl9yZWNvbmZpZyhzdHJ1Y3QgZmJfaW5mbyAqaW5mbykKPiA+ ICAJc3RydWN0IHNoX21vYmlsZV9sY2RjX2NoYW4gKmNoID0gaW5mby0+cGFyOwo+ID4gIAlzdHJ1 Y3QgZmJfdmFyX3NjcmVlbmluZm8gdmFyOwo+ID4gIAlzdHJ1Y3QgZmJfdmlkZW9tb2RlIG1vZGU7 Cj4gPiAtCXN0cnVjdCBmYl9ldmVudCBldmVudDsKPiA+IC0JaW50IGV2bnQgPSBGQl9FVkVOVF9N T0RFX0NIQU5HRV9BTEw7Cj4gPiAgCj4gPiAgCWlmIChjaC0+dXNlX2NvdW50ID4gMSB8fCAoY2gt PnVzZV9jb3VudCA9PSAxICYmICFpbmZvLT5mYmNvbl9wYXIpKQo+ID4gIAkJLyogTW9yZSBmcmFt ZWJ1ZmZlciB1c2VycyBhcmUgYWN0aXZlICovCj4gPiBAQCAtMTc5OSwxNCArMTc5Nyw3IEBAIHN0 YXRpYyB2b2lkIHNoX21vYmlsZV9mYl9yZWNvbmZpZyhzdHJ1Y3QgZmJfaW5mbyAqaW5mbykKPiA+ ICAJCS8qIENvdWxkbid0IHJlY29uZmlndXJlLCBob3BlZnVsbHksIGNhbiBjb250aW51ZSBhcyBi ZWZvcmUgKi8KPiA+ICAJCXJldHVybjsKPiA+ICAKPiA+IC0JLyoKPiA+IC0JICogZmJfc2V0X3Zh cigpIGNhbGxzIHRoZSBub3RpZmllciBjaGFuZ2UgaW50ZXJuYWxseSwgb25seSBpZgo+ID4gLQkg KiBGQklORk9fTUlTQ19VU0VSRVZFTlQgZmxhZyBpcyBzZXQuIFNpbmNlIHdlIGRvIG5vdCB3YW50 IHRvIGZha2UgYQo+ID4gLQkgKiB1c2VyIGV2ZW50LCB3ZSBoYXZlIHRvIGNhbGwgdGhlIGNoYWlu IG91cnNlbHZlcy4KPiA+IC0JICovCj4gPiAtCWV2ZW50LmluZm8gPSBpbmZvOwo+ID4gLQlldmVu dC5kYXRhID0gJmNoLT5kaXNwbGF5Lm1vZGU7Cj4gPiAtCWZiX25vdGlmaWVyX2NhbGxfY2hhaW4o ZXZudCwgJmV2ZW50KTsKPiA+ICsJZmJjb25fdXBkYXRlX3ZjcyhpbmZvLCB0cnVlKTsKPiA+ICB9 Cj4gPiAgCj4gPiAgLyoKPiA+IGRpZmYgLS1naXQgYS9pbmNsdWRlL2xpbnV4L2ZiLmggYi9pbmNs dWRlL2xpbnV4L2ZiLmgKPiA+IGluZGV4IDRiOWI4ODJmOGY1Mi4uNTRkNmJlZTA5MTIxIDEwMDY0 NAo+ID4gLS0tIGEvaW5jbHVkZS9saW51eC9mYi5oCj4gPiArKysgYi9pbmNsdWRlL2xpbnV4L2Zi LmgKPiA+IEBAIC0xMjQsMTYgKzEyNCwxMiBAQCBzdHJ1Y3QgZmJfY3Vyc29yX3VzZXIgewo+ID4g ICAqIFJlZ2lzdGVyL3VucmVnaXN0ZXIgZm9yIGZyYW1lYnVmZmVyIGV2ZW50cwo+ID4gICAqLwo+ ID4gIAo+ID4gLS8qCVRoZSByZXNvbHV0aW9uIG9mIHRoZSBwYXNzZWQgaW4gZmJfaW5mbyBhYm91 dCB0byBjaGFuZ2UgKi8gCj4gPiAtI2RlZmluZSBGQl9FVkVOVF9NT0RFX0NIQU5HRQkJMHgwMQo+ ID4gIC8qICAgICAgQ09OU09MRS1TUEVDSUZJQzogZ2V0IGNvbnNvbGUgdG8gZnJhbWVidWZmZXIg bWFwcGluZyAqLwo+ID4gICNkZWZpbmUgRkJfRVZFTlRfR0VUX0NPTlNPTEVfTUFQICAgICAgICAw eDA3Cj4gPiAgLyogICAgICBDT05TT0xFLVNQRUNJRklDOiBzZXQgY29uc29sZSB0byBmcmFtZWJ1 ZmZlciBtYXBwaW5nICovCj4gPiAgI2RlZmluZSBGQl9FVkVOVF9TRVRfQ09OU09MRV9NQVAgICAg ICAgIDB4MDgKPiA+ICAvKiAgICAgIEEgZGlzcGxheSBibGFuayBpcyByZXF1ZXN0ZWQgICAgICAg Ki8KPiA+ICAjZGVmaW5lIEZCX0VWRU5UX0JMQU5LICAgICAgICAgICAgICAgICAgMHgwOQo+ID4g LS8qICAgICAgUHJpdmF0ZSBtb2RlbGlzdCBpcyB0byBiZSByZXBsYWNlZCAqLwo+ID4gLSNkZWZp bmUgRkJfRVZFTlRfTU9ERV9DSEFOR0VfQUxMCTB4MEIKPiA+ICAvKiAgICAgIENPTlNPTEUtU1BF Q0lGSUM6IHJlbWFwIGFsbCBjb25zb2xlcyB0byBuZXcgZmIgLSBmb3IgdmdhX3N3aXRjaGVyb28g Ki8KPiA+ICAjZGVmaW5lIEZCX0VWRU5UX1JFTUFQX0FMTF9DT05TT0xFICAgICAgMHgwRgo+ID4g IC8qICAgICAgQSBoYXJkd2FyZSBkaXNwbGF5IGJsYW5rIGVhcmx5IGNoYW5nZSBvY2N1cnJlZCAq Lwo+ID4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvbGludXgvZmJjb24uaCBiL2luY2x1ZGUvbGludXgv ZmJjb24uaAo+ID4gaW5kZXggOTBlMTk2YzgzNWRkLi5kYWFhOTdiMGM5ZTYgMTAwNjQ0Cj4gPiAt LS0gYS9pbmNsdWRlL2xpbnV4L2ZiY29uLmgKPiA+ICsrKyBiL2luY2x1ZGUvbGludXgvZmJjb24u aAo+ID4gQEAgLTE1LDYgKzE1LDcgQEAgdm9pZCBmYmNvbl9uZXdfbW9kZWxpc3Qoc3RydWN0IGZi X2luZm8gKmluZm8pOwo+ID4gIHZvaWQgZmJjb25fZ2V0X3JlcXVpcmVtZW50KHN0cnVjdCBmYl9p bmZvICppbmZvLAo+ID4gIAkJCSAgIHN0cnVjdCBmYl9ibGl0X2NhcHMgKmNhcHMpOwo+ID4gIHZv aWQgZmJjb25fZmJfYmxhbmtlZChzdHJ1Y3QgZmJfaW5mbyAqaW5mbywgaW50IGJsYW5rKTsKPiA+ ICt2b2lkIGZiY29uX3VwZGF0ZV92Y3Moc3RydWN0IGZiX2luZm8gKmluZm8sIGJvb2wgYWxsKTsK PiA+ICAjZWxzZQo+ID4gIHN0YXRpYyBpbmxpbmUgdm9pZCBmYl9jb25zb2xlX2luaXQodm9pZCkg e30KPiA+ICBzdGF0aWMgaW5saW5lIHZvaWQgZmJfY29uc29sZV9leGl0KHZvaWQpIHt9Cj4gPiBA QCAtMjksNiArMzAsNyBAQCB2b2lkIGZiY29uX25ld19tb2RlbGlzdChzdHJ1Y3QgZmJfaW5mbyAq aW5mbykge30KPiA+ICB2b2lkIGZiY29uX2dldF9yZXF1aXJlbWVudChzdHJ1Y3QgZmJfaW5mbyAq aW5mbywKPiA+ICAJCQkgICBzdHJ1Y3QgZmJfYmxpdF9jYXBzICpjYXBzKSB7fQo+ID4gIHZvaWQg ZmJjb25fZmJfYmxhbmtlZChzdHJ1Y3QgZmJfaW5mbyAqaW5mbywgaW50IGJsYW5rKSB7fQo+ID4g K3ZvaWQgZmJjb25fdXBkYXRlX3ZjcyhzdHJ1Y3QgZmJfaW5mbyAqaW5mbywgYm9vbCBhbGwpIHt9 Cj4gPiAgI2VuZGlmCj4gPiAgCj4gPiAgI2VuZGlmIC8qIF9MSU5VWF9GQkNPTl9IICovCj4gCj4g CgotLSAKRGFuaWVsIFZldHRlcgpTb2Z0d2FyZSBFbmdpbmVlciwgSW50ZWwgQ29ycG9yYXRpb24K aHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRl c2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v ZHJpLWRldmVs