From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aleksey Makarov Subject: Re: [PATCH 1/3] printk: make preferred_console local static bool Date: Mon, 25 Jan 2016 20:55:16 +0600 Message-ID: <56A63754.7030108@linaro.org> References: <1453722324-22407-1-git-send-email-aleksey.makarov@linaro.org> <1453722324-22407-2-git-send-email-aleksey.makarov@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org To: Andy Shevchenko Cc: "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-arm Mailing List , "linux-serial@vger.kernel.org" , Graeme Gregory , Russell King , Greg Kroah-Hartman , "Rafael J . Wysocki" , Leif Lindholm List-Id: linux-serial@vger.kernel.org On 25.01.2016 20:24, Andy Shevchenko wrote: > On Mon, Jan 25, 2016 at 1:45 PM, Aleksey Makarov > wrote: >> The variable preferred_console is used only inside register_console() >> and it's semantics is boolean. Make it clear. > > However the patch looks okay it brings imbalance to understanding how > exactly the preferred console is chosen. On the contrary, I would say it makes things more clear. Also please consider this patch preparatory for the further changes. See my replies to Joe Perches.