From: Eddie Huang <eddie.huang@mediatek.com> To: Peter Hurley <peter@hurleysoftware.com> Cc: Mark Rutland <mark.rutland@arm.com>, linux-doc@vger.kernel.org, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will.deacon@arm.com>, Jiri Slaby <jslaby@suse.cz>, Jonathan Corbet <corbet@lwn.net>, yh.chen@mediatek.com, Howard Chen <ibanezchen@gmail.com>, linux-serial@vger.kernel.org, "Joe.C" <yingjoe.chen@mediatek.com>, devicetree@vger.kernel.org, Vladimir Murzin <vladimir.murzin@arm.com>, Pawel Moll <pawel.moll@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Rob Herring <robh+dt@kernel.org>, Matthias Brugger <matthias.bgg@gmail.com>, linux-arm-kernel@lists.infradead.org, Alan Cox <alan@linux.intel.com>, srv_heupstream@mediatek.com, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, linux-kernel@vger.kernel.org, Ashwin Chaugule <ashwin.chaugule@linaro.org>, Kumar Gala <galak@codeaurora.org> Subject: Re: [PATCH v2 1/4] tty: serial: Add 8250 earlycon to support noinit option Date: Mon, 2 Feb 2015 11:16:34 +0800 [thread overview] Message-ID: <1422846994.5512.25.camel@mtksdaap41> (raw) In-Reply-To: <54CE6FD9.5030606@hurleysoftware.com> Hi Peter, On Sun, 2015-02-01 at 13:26 -0500, Peter Hurley wrote: > On 02/01/2015 11:27 AM, Peter Hurley wrote: > > Hi Eddie, > > > > On 01/12/2015 08:08 AM, Eddie Huang wrote: > >> Add earlycon support not only baudrate option, but also add noinit option. > >> If use noinit option, 8250 earlycon will not init serial hardware and use > >> loader setting. > > > > I see this went into Greg's tty-testing branch. > > > > The only point of this is to not program the divisor, right? > > > > I ask because early_serial8250_setup() could already handle this without > > extra options by simply not doing divisor programming if no baud option is > > present. > > Does the patch below work for your use-case? > > [ Note: the patch applies to 3.19-rcX. To test, your noinit patches need to be > reverted first. > ] > > --- >% --- > From: Peter Hurley <peter@hurleysoftware.com> > Subject: [PATCH] serial: 8250_early: Assume uart already initialized if no > baud option > > The <baud><parity><bit> option string is not supplied if the earlycon > is started via devicetree and OF_EARLYCON_DECLARE(). The option string > is also not required if started via kernel command line parameters of > the form: > earlycon=uart,mmio,<addr> > console=uart,mmio,<addr> > > If earlycon_device->baud is 0, then an option string was not supplied. > In this case, assume the uart has already been initialized by the > bootloader or firmware. > > Signed-off-by: Peter Hurley <peter@hurleysoftware.com> > --- > drivers/tty/serial/8250/8250_early.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/tty/serial/8250/8250_early.c b/drivers/tty/serial/8250/8250_early.c > index d7b831b..1701d00 100644 > --- a/drivers/tty/serial/8250/8250_early.c > +++ b/drivers/tty/serial/8250/8250_early.c > @@ -149,12 +149,18 @@ static int __init early_serial8250_setup(struct earlycon_device *device, > return 0; > > if (!device->baud) { > + struct uart_port *port = &device->port; > + unsigned int ier; > + > device->baud = probe_baud(&device->port); > snprintf(device->options, sizeof(device->options), "%u", > device->baud); > - } > > - init_port(device); > + /* assume the device was initialized, only mask interrupts */ > + ier = serial8250_early_in(port, UART_IER); > + serial8250_early_out(port, UART_IER, ier & UART_IER_UUE); > + } else > + init_port(device); Should add brace in else. Where is original line here. early_device = device; > device->con->write = early_serial8250_write; > return 0; With above comment, I test ok on my platform. Eddie Best Regards.
WARNING: multiple messages have this Message-ID (diff)
From: eddie.huang@mediatek.com (Eddie Huang) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 1/4] tty: serial: Add 8250 earlycon to support noinit option Date: Mon, 2 Feb 2015 11:16:34 +0800 [thread overview] Message-ID: <1422846994.5512.25.camel@mtksdaap41> (raw) In-Reply-To: <54CE6FD9.5030606@hurleysoftware.com> Hi Peter, On Sun, 2015-02-01 at 13:26 -0500, Peter Hurley wrote: > On 02/01/2015 11:27 AM, Peter Hurley wrote: > > Hi Eddie, > > > > On 01/12/2015 08:08 AM, Eddie Huang wrote: > >> Add earlycon support not only baudrate option, but also add noinit option. > >> If use noinit option, 8250 earlycon will not init serial hardware and use > >> loader setting. > > > > I see this went into Greg's tty-testing branch. > > > > The only point of this is to not program the divisor, right? > > > > I ask because early_serial8250_setup() could already handle this without > > extra options by simply not doing divisor programming if no baud option is > > present. > > Does the patch below work for your use-case? > > [ Note: the patch applies to 3.19-rcX. To test, your noinit patches need to be > reverted first. > ] > > --- >% --- > From: Peter Hurley <peter@hurleysoftware.com> > Subject: [PATCH] serial: 8250_early: Assume uart already initialized if no > baud option > > The <baud><parity><bit> option string is not supplied if the earlycon > is started via devicetree and OF_EARLYCON_DECLARE(). The option string > is also not required if started via kernel command line parameters of > the form: > earlycon=uart,mmio,<addr> > console=uart,mmio,<addr> > > If earlycon_device->baud is 0, then an option string was not supplied. > In this case, assume the uart has already been initialized by the > bootloader or firmware. > > Signed-off-by: Peter Hurley <peter@hurleysoftware.com> > --- > drivers/tty/serial/8250/8250_early.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/tty/serial/8250/8250_early.c b/drivers/tty/serial/8250/8250_early.c > index d7b831b..1701d00 100644 > --- a/drivers/tty/serial/8250/8250_early.c > +++ b/drivers/tty/serial/8250/8250_early.c > @@ -149,12 +149,18 @@ static int __init early_serial8250_setup(struct earlycon_device *device, > return 0; > > if (!device->baud) { > + struct uart_port *port = &device->port; > + unsigned int ier; > + > device->baud = probe_baud(&device->port); > snprintf(device->options, sizeof(device->options), "%u", > device->baud); > - } > > - init_port(device); > + /* assume the device was initialized, only mask interrupts */ > + ier = serial8250_early_in(port, UART_IER); > + serial8250_early_out(port, UART_IER, ier & UART_IER_UUE); > + } else > + init_port(device); Should add brace in else. Where is original line here. early_device = device; > device->con->write = early_serial8250_write; > return 0; With above comment, I test ok on my platform. Eddie Best Regards.
next prev parent reply other threads:[~2015-02-02 3:16 UTC|newest] Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-12 13:08 [PATCH v2 0/4] tty: serial: Add earlycon support to MT8173 SoC Eddie Huang 2015-01-12 13:08 ` Eddie Huang 2015-01-12 13:08 ` [PATCH v2 1/4] tty: serial: Add 8250 earlycon to support noinit option Eddie Huang 2015-01-12 13:08 ` Eddie Huang 2015-01-12 15:35 ` Arnd Bergmann 2015-01-12 15:35 ` Arnd Bergmann 2015-01-12 16:08 ` Alan Cox 2015-01-12 16:08 ` Alan Cox 2015-01-13 1:05 ` Eddie Huang 2015-01-13 1:05 ` Eddie Huang 2015-02-01 16:27 ` Peter Hurley 2015-02-01 16:27 ` Peter Hurley 2015-02-01 16:27 ` Peter Hurley 2015-02-01 18:26 ` Peter Hurley 2015-02-01 18:26 ` Peter Hurley 2015-02-01 18:26 ` Peter Hurley 2015-02-02 3:16 ` Eddie Huang [this message] 2015-02-02 3:16 ` Eddie Huang 2015-02-02 3:45 ` Peter Hurley 2015-02-02 3:45 ` Peter Hurley 2015-02-02 4:28 ` Greg Kroah-Hartman 2015-02-02 4:28 ` Greg Kroah-Hartman 2015-02-02 5:15 ` Peter Hurley 2015-02-02 5:15 ` Peter Hurley 2015-02-02 5:24 ` Greg Kroah-Hartman 2015-02-02 5:24 ` Greg Kroah-Hartman 2015-02-02 5:33 ` Eddie Huang 2015-02-02 5:33 ` Eddie Huang 2015-02-02 5:43 ` Peter Hurley 2015-02-02 5:43 ` Peter Hurley 2015-02-02 5:43 ` Peter Hurley 2015-02-02 18:46 ` Greg Kroah-Hartman 2015-02-02 18:46 ` Greg Kroah-Hartman 2015-02-02 2:45 ` Eddie Huang 2015-02-02 2:45 ` Eddie Huang 2015-02-02 3:27 ` Peter Hurley 2015-02-02 3:27 ` Peter Hurley 2015-01-12 13:08 ` [PATCH v2 2/4] tty: serial: 8250_mtk: Add earlycon Eddie Huang 2015-01-12 13:08 ` Eddie Huang 2015-01-12 13:08 ` [PATCH v2 3/4] arm64: dts: Add mediatek MT8173 earlycon support Eddie Huang 2015-01-12 13:08 ` Eddie Huang 2015-01-12 13:08 ` [PATCH v2 4/4] Document: Modify 8250 earlycon kernel parameters Eddie Huang 2015-01-12 13:08 ` Eddie Huang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1422846994.5512.25.camel@mtksdaap41 \ --to=eddie.huang@mediatek.com \ --cc=alan@linux.intel.com \ --cc=ashwin.chaugule@linaro.org \ --cc=catalin.marinas@arm.com \ --cc=corbet@lwn.net \ --cc=devicetree@vger.kernel.org \ --cc=galak@codeaurora.org \ --cc=gregkh@linuxfoundation.org \ --cc=ibanezchen@gmail.com \ --cc=ijc+devicetree@hellion.org.uk \ --cc=jslaby@suse.cz \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-serial@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=matthias.bgg@gmail.com \ --cc=pawel.moll@arm.com \ --cc=peter@hurleysoftware.com \ --cc=robh+dt@kernel.org \ --cc=srv_heupstream@mediatek.com \ --cc=vladimir.murzin@arm.com \ --cc=will.deacon@arm.com \ --cc=yh.chen@mediatek.com \ --cc=yingjoe.chen@mediatek.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.