From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753558AbcALWmd (ORCPT ); Tue, 12 Jan 2016 17:42:33 -0500 Received: from mail-pf0-f179.google.com ([209.85.192.179]:35725 "EHLO mail-pf0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751615AbcALWma (ORCPT ); Tue, 12 Jan 2016 17:42:30 -0500 Subject: Re: [RFC PATCH] always probe UART HW when options are not specified To: Mason References: <5672D18E.8000301@laposte.net> <5672E2CF.6080705@hurleysoftware.com> <5672E76A.3010506@laposte.net> <5672F588.8070503@hurleysoftware.com> <5672FD35.5070609@laposte.net> <56731689.70702@hurleysoftware.com> <56740FC3.50302@laposte.net> <5674204C.1000907@hurleysoftware.com> <56782DE5.7090708@laposte.net> <56798EC7.1030804@laposte.net> <5693C52B.90003@laposte.net> <5693D41E.8050601@hurleysoftware.com> <5693ECBA.50003@laposte.net> <5693FD3C.5080601@hurleysoftware.com> <5694C94B.3090405@free.fr> <56950C21.3010805@laposte.net> <5695584E.3040902@hurleysoftware.com> <56957D8B.3080906@free.fr> Cc: Sebastian Frias , Greg Kroah-Hartman , linux-serial@vger.kernel.org, LKML , Mans Rullgard From: Peter Hurley Message-ID: <56958154.5090003@hurleysoftware.com> Date: Tue, 12 Jan 2016 14:42:28 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <56957D8B.3080906@free.fr> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/12/2016 02:26 PM, Mason wrote: > On 12/01/2016 20:47, Peter Hurley wrote: > >> I expect that patch to go into -next sometime during the 4.5-rc cycle. > > Do you mean the "Add Au1x00/RT288x earlycon support" patch? No. This one http://www.spinics.net/lists/linux-serial/msg20328.html > I find it strange that you have rt288x_putc while also > using AU1x00_FOO. Seems like mixing naming styles. I wrote it in 10 mins. > Did you see my post about the underlying IP block? > Might be appropriate to use that name there? I did, and thanks for reminding me. I did follow that email thread a while back about the provenance of this register layout. I'd rather not go back through the source and rename the symbols. Afaict, the Au1x00 landed first in source and then the RT288x afterwards. So I'd like to stick with those. I named the register layout on the basis of the original usage which started in the 8250 port driver with the au_* accessor functions. The earlycon functions I named that way because that's the name of the earlycon (which originates from the DT compatible key which can't be changed). I do want people to be able to find other relevant code by grepping these names, so I was thinking of adding a follow-on patch that uses the AU1x00_ register names in the 8250 port driver LUT. Regards, Peter Hurley