Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751148AbcKYIao (ORCPT ); Fri, 25 Nov 2016 03:30:44 -0500 Received: from mout.kundenserver.de ([212.227.126.131]:65129 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750799AbcKYIah (ORCPT ); Fri, 25 Nov 2016 03:30:37 -0500 From: Arnd Bergmann To: Robin van der Gracht Cc: Greg Kroah-Hartman , Rob Herring , Linus Walleij , Miguel Ojeda Sandonis , linux-kernel@vger.kernel.org Subject: Re: [PATCH] auxdisplay: ht16k33: select CONFIG_FB_SYS_FOPS Date: Fri, 25 Nov 2016 09:30:07 +0100 Message-ID: <3275301.7nJ4tt8fXD@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <20161125085004.74c9acb5@erd979> References: <20161123130702.432411-1-arnd@arndb.de> <20161125085004.74c9acb5@erd979> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:FuIb5uLP2WYx1lTqIcx0ZqYOdigMPjyxpD3PxO9zj5jQxhpszul m1fECVzt8mQG4qVh0v7wOf6HWUUvT4XDfNKjf2yUpEtrV1XBPq521JHDLfXrNT4T4j5g1WM YZWmc57D9RQOFQvMDGiK3gF+uRdzXd266nrmJQtHRoIM5Plu1vyXj2IWUej4BlC2J+MGZCe obcERQ19G00bO67Z0/oeg== X-UI-Out-Filterresults: notjunk:1;V01:K0:xIR1AF2Siro=:awFZp7e8gjHp9WPogoBNW3 e7KTSbcvC9UO2jamxGQNkW8GAn0xxu2+PRajR6ORKlYTEuIpcYRON6aw2eOhXtzVTaIva6lRp cVfSRyHsnHNfqvyp1AQSqCpUgF13A7bjxT/RXQxSh1e1iSLXLJfcjC4RiZFMj0qhkwPyaxrgi jB/OUCo7+o16dMrxqXgFMYsLw5OTMyNqLyViACbUBRtu7A0kXfeCkmXD6YdVI1KB9uRk+39N4 iQagrf/bTmXmQTFsj7R6woEtesvSgumgESE02oW1OsOE73DeetYorgb/F3DxvYSU2T7FSAVJ/ jgNe5LObqFlCQWZFDUZ0h80fsICa7Wjn2j99cxnW/o7Gb0D9CYtw/9vm9YHpUNqzQTG4do7Dk xMHgJ9n+VnUXi6dLSyGG+/YMlYphQtK/Qk+/c2f//ux16yszmZmJZphR6soNiQViSxOOQfLIF K4cfmRbmVp73Woc011U974uQQbQJAV+xbjWld1NdTLbCdKIuJlHkgx65wh8hZFb3LyZh2XeH4 n9M8IUnFwfWpo9RHe6p5UQXH0dIwdXx58U55oaf9YMjgpkvc5+9VE0rDO73HixIC9FMN8pbdY j0AcArrT4cboD4vUNybpsKVcN9PajZ2CkvJtzNE+fO+4845ylY/BfpeR9KD8n1LZSRXH3TEZj VDpxKP9xEDA/+v3Q6ujllHfujAHd+NUqwEjFmhhU5H3/6blodC9z5nqOduTbEYFmPthlyRj5z lQB0zDd1lyq8Sw7uWkASAqVDBXjTw0V2S9KaIQ== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 825 Lines: 28 On Friday, November 25, 2016 8:50:04 AM CET Robin van der Gracht wrote: > > Thanks for reporting this. You are right about the missing helper. > However, the fb_ops struct uses several helpers which are all missing. > > static struct fb_ops ht16k33_fb_ops = { > .owner = THIS_MODULE, > .fb_read = fb_sys_read, > .fb_write = fb_sys_write, > .fb_fillrect = sys_fillrect, > .fb_copyarea = sys_copyarea, > .fb_imageblit = sys_imageblit, > .fb_mmap = ht16k33_mmap, > }; > > HT16K33 should also select: > FB_CFB_FILLRECT > FB_CFB_COPYAREA > FB_CFB_IMAGEBLIT > Ah right. I had not run into those so far during randconfig testing, probably because there is usually at least one other framebuffer enabled that selects them. Can you submit a patch to add all those? Arnd