From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 19 Dec 2014 02:43:22 +0100 (CET) Received: from unicorn.mansr.com ([81.2.72.234]:42988 "EHLO unicorn.mansr.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27008199AbaLSBnUYbGo5 convert rfc822-to-8bit (ORCPT ); Fri, 19 Dec 2014 02:43:20 +0100 Received: by unicorn.mansr.com (Postfix, from userid 51770) id DCF7A1538A; Fri, 19 Dec 2014 01:43:13 +0000 (GMT) From: =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?= To: Joe Perches Cc: Thomas Gleixner , linux-arch , LKML , Geert Uytterhoeven , linux-m68k , Steven Miao , Ralf Baechle , linux-mips Subject: Re: rfc: remove early_printk from a few arches? (blackfin, m68k, mips) References: <1418849927.28384.1.camel@perches.com> <1418951658.28384.20.camel@perches.com> Date: Fri, 19 Dec 2014 01:43:13 +0000 In-Reply-To: <1418951658.28384.20.camel@perches.com> (Joe Perches's message of "Thu, 18 Dec 2014 17:14:18 -0800") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 44824 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: mans@mansr.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips Joe Perches writes: > On Fri, 2014-12-19 at 00:33 +0100, Thomas Gleixner wrote: >> On Wed, 17 Dec 2014, Joe Perches wrote: >> > It seems like early_printk can be configured into >> > a few architectures but also appear not to be used. >> > >> > $ git grep -w "early_printk" >> ... >> > These seem to the only uses: >> ... >> > So blackfin, m68k, and mips seems to have it possible to enable, >> > but also don't appear at first glance to use it, >> >> Hint: CONFIG_EARLY_PRINTK covers far more than early_printk() > > I know this. > > Note also I didn't specify CONFIG_EARLY_PRINTK, > just early_printk. > >> > Is early_printk really used by these architectures? >> > Should it be removed? >> >> Sure, if you have a good reason to remove working functionality. > > Unused. What exactly are you proposing to remove? I see no unused code related to early printk (in any variant) under arch/mips. -- Måns Rullgård mans@mansr.com