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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E55E1C001DF for ; Tue, 25 Jul 2023 23:09:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231265AbjGYXJz (ORCPT ); Tue, 25 Jul 2023 19:09:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229740AbjGYXJw (ORCPT ); Tue, 25 Jul 2023 19:09:52 -0400 Received: from mx.treblig.org (unknown [IPv6:2a00:1098:5b::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 35961E0; Tue, 25 Jul 2023 16:09:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=treblig.org ; s=bytemarkmx; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID :Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID :Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To: Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe :List-Post:List-Owner:List-Archive; bh=tpcY/KRZk5hiP87JmAb/M+fcCqvEACvi7wT7XhSE4Vg=; b=c0MMfSLPkjphEMgyJhSFY51/jz FMArs3JcyyCm6OJNQZzNH0BOKVo+3b1Z+7Dl3fYzAtQNRXH1o1f9YAmVGRkC9WX3uzjbIhkvUAZ0Q aAeNt1dpyNqz4JxmzhoG1Js+6P4jpighifXpfIOUhKYs0TduKPwStyljD3pD6FGLxQ+zZXOR3n+TX xbxtFeyFMj+L1Nd1OoZBntBWnYF5tG268ztEw1UYIxwRB+dPI/NHXr3ngmyA748pm88U5dBo7cett d6MJad9KvaIzQvgxq3GwxCOmtTAnGZCE/sblpRjrjjlLFLHU20mQ0XWIWbZS+nI47ZHDld36FV/8r oDMhcdtg==; Received: from dg by mx.treblig.org with local (Exim 4.94.2) (envelope-from ) id 1qOR9x-003Kui-43; Tue, 25 Jul 2023 23:09:41 +0000 Date: Tue, 25 Jul 2023 23:09:41 +0000 From: "Dr. David Alan Gilbert" To: John Paul Adrian Glaubitz , akpm@linux-foundation.org Cc: Sam Ravnborg , davem@davemloft.net, benh@kernel.crashing.org, sparclinux@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] sparc: Use shared font data Message-ID: References: <20230724235851.165871-1-linux@treblig.org> <20230725161040.GA832394@ravnborg.org> <605d12e8a4fdcb238efc9b18fbd2637474de0049.camel@physik.fu-berlin.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <605d12e8a4fdcb238efc9b18fbd2637474de0049.camel@physik.fu-berlin.de> X-Chocolate: 70 percent or better cocoa solids preferably X-Operating-System: Linux/5.10.0-23-amd64 (x86_64) X-Uptime: 23:08:03 up 19 days, 8:39, 1 user, load average: 0.00, 0.01, 0.00 User-Agent: Mutt/2.0.5 (2021-01-21) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * John Paul Adrian Glaubitz (glaubitz@physik.fu-berlin.de) wrote: > Hi Dave! Hi Adrian, > On Tue, 2023-07-25 at 16:53 +0000, Dr. David Alan Gilbert wrote: > > * Sam Ravnborg (sam@ravnborg.org) wrote: > > > On Tue, Jul 25, 2023 at 12:58:51AM +0100, linux@treblig.org wrote: > > > > From: "Dr. David Alan Gilbert" > > > > > > > > sparc has a 'btext' font used for the console which is almost identical > > > > to the shared font_sun8x16, so use it rather than duplicating the data. > > > > > > > > They were actually identical until about a decade ago when > > > > commit bcfbeecea11c ("drivers: console: font_: Change a glyph from > > > > "broken bar" to "vertical line"") > > > > > > > > which changed the | in the shared font to be a solid > > > > bar rather than a broken bar. That's the only difference. > > > > > > > > This was originally spotted by PMD which noticed that PPC does > > > > the same thing with the same data, and they also share a bunch > > > > of functions to manipulate the data. The PPC code and the functions > > > > I'll look at another time if this patch is OK. > > > > > > > > Tested very lightly with a boot without FS in qemu. > > > > > > > > Signed-off-by: Dr. David Alan Gilbert > > > > > > Looks good, thanks for the fixes. > > > Reviewed-by: Sam Ravnborg > > > > Thanks > > > > > Let's hope someone picks it up... > > > > I was hoping Dave would, but I realise Sparc doesn't get much > > these days. > > Of course if anyone feels guilty about their own patches adding code > > they can take this patch to make ~340 lines of penance. > > You can ask Andrew Morton to pick it up through his tree. He usually does > that when no one else is willing to pick a patch up. Thanks for the suggestion, I've copied Andrew in, although there's obviously no rush since it's just a dedupe. Dave > Adrian > > -- > .''`. John Paul Adrian Glaubitz > : :' : Debian Developer > `. `' Physicist > `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913 -- -----Open up your eyes, open up your mind, open up your code ------- / Dr. David Alan Gilbert | Running GNU/Linux | Happy \ \ dave @ treblig.org | | In Hex / \ _________________________|_____ http://www.treblig.org |_______/