All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Brandt <Chris.Brandt@renesas.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	"uli@fpond.eu" <uli@fpond.eu>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Greg KH <gregkh@linuxfoundation.org>,
	Jiri Slaby <jslaby@suse.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	Linux-sh list <linux-sh@vger.kernel.org>,
	"open list:SERIAL DRIVERS" <linux-serial@vger.kernel.org>
Subject: RE: [PATCH/RFC 4/4] sh-sci: Derive regshift value from DT compatible value
Date: Tue, 07 Aug 2018 21:10:13 +0000	[thread overview]
Message-ID: <TY1PR01MB1562ED9B51B87FBE0F2C1A828A270@TY1PR01MB1562.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <CAMuHMdWgJrPRh8TRTrEjWsaRAhEQX0T7M5eBGP_z3Ft=T6FfDg@mail.gmail.com>

SGkgR2VlcnQsDQoNCk9uIFR1ZXNkYXksIEF1Z3VzdCAwNywgMjAxOCwgR2VlcnQgVXl0dGVyaG9l
dmVuIHdyb3RlOg0KPiA+IEJ1dCwgd2hlbiBJIHJ1biB0aGUgY29kZSwgSSBlbmQgdXAgaW4gZnVu
Y3Rpb24gc2NpZl9lYXJseV9jb25zb2xlX3NldHVwLA0KPiA+IG5vdCByemEyX2Vhcmx5X2NvbnNv
bGVfc2V0dXANCj4gDQo+IEhtbSwgcGVyaGFwcyBpdCBkb2Vzbid0IHBpY2sgdGhlIGZpcnN0L21v
c3Qgc3BlY2lhbGl6ZWQgbWF0Y2guDQo+IA0KPiA+IEknbSBhc3N1bWluZyBJJ20ganVzdCBzdXBw
b3NlZCB0byB1c2UgdGhpcyBvbiBteSBib290YXJnczoNCj4gPiAgICBlYXJseWNvbj1zY2lmLDB4
RTgwMDkwMDANCj4gDQo+IEp1c3QgImVhcmx5Y29uIiBzaG91bGQgYmUgc3VmZmljaWVudC4gSXQn
bGwgZmluZCB0aGUgcmlnaHQgcG9ydCBmcm9tDQo+IGNob3Nlbi9zdGRvdXQtcGF0aA0KPiBpbiBE
VC4gQ2FuIHlvdSBwbGVhc2UgcmV0cnkgdXNpbmcgdGhhdD8NCg0KV2l0aCBqdXN0IHVzaW5nICJl
YXJseWNvbiIsIEkgZ2V0IHRoZSBzYW1lIHJlc3VsdC4NCg0KSSB0aGluayB0aGUgY29kZSB0aGF0
IGRvZXMgdGhlIG1hdGNoaW5nIGlzIGluIGRyaXZlcnMvb2YvZmR0LmMNCkZ1bmN0aW9uICJlYXJs
eV9pbml0X2R0X3NjYW5fY2hvc2VuX3N0ZG91dCIuDQoNCkknbGwgc3RlcCB0aHJvdWdoIHRoZSBj
b2RlIGEgYml0IHRvIHNlZSBpZiBJIGNhbiBmaWd1cmUgb3V0IHdoeS4NCg0KQ2hyaXMNCg0KDQo

WARNING: multiple messages have this Message-ID (diff)
From: Chris Brandt <Chris.Brandt@renesas.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	"uli@fpond.eu" <uli@fpond.eu>,
	Yoshinori Sato <ysato@users.sourceforge.jp>,
	Greg KH <gregkh@linuxfoundation.org>,
	Jiri Slaby <jslaby@suse.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	Linux-sh list <linux-sh@vger.kernel.org>,
	"open list:SERIAL DRIVERS" <linux-serial@vger.kernel.org>
Subject: RE: [PATCH/RFC 4/4] sh-sci: Derive regshift value from DT compatible value
Date: Tue, 7 Aug 2018 21:10:13 +0000	[thread overview]
Message-ID: <TY1PR01MB1562ED9B51B87FBE0F2C1A828A270@TY1PR01MB1562.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <CAMuHMdWgJrPRh8TRTrEjWsaRAhEQX0T7M5eBGP_z3Ft=T6FfDg@mail.gmail.com>

Hi Geert,

On Tuesday, August 07, 2018, Geert Uytterhoeven wrote:
> > But, when I run the code, I end up in function scif_early_console_setup,
> > not rza2_early_console_setup
> 
> Hmm, perhaps it doesn't pick the first/most specialized match.
> 
> > I'm assuming I'm just supposed to use this on my bootargs:
> >    earlycon=scif,0xE8009000
> 
> Just "earlycon" should be sufficient. It'll find the right port from
> chosen/stdout-path
> in DT. Can you please retry using that?

With just using "earlycon", I get the same result.

I think the code that does the matching is in drivers/of/fdt.c
Function "early_init_dt_scan_chosen_stdout".

I'll step through the code a bit to see if I can figure out why.

Chris



  reply	other threads:[~2018-08-07 21:10 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-06 14:07 [PATCH/RFC 0/4] sh-sci : Do not derive regshift from regsize Geert Uytterhoeven
2018-08-06 14:07 ` Geert Uytterhoeven
2018-08-06 14:07 ` [PATCH/RFC 1/4] sh-sci: Use a separate sci_port for earlycon Geert Uytterhoeven
2018-08-06 14:07   ` Geert Uytterhoeven
2018-08-06 14:07 ` [PATCH/RFC 2/4] sh-sci: Take into account regshift to fix earlycon breakage Geert Uytterhoeven
2018-08-06 14:07   ` Geert Uytterhoeven
2018-08-06 14:07 ` [PATCH/RFC 3/4] Revert "serial: sh-sci: Compute the regshift value for SCI ports" Geert Uytterhoeven
2018-08-06 14:07   ` Geert Uytterhoeven
2018-08-06 14:16   ` Laurent Pinchart
2018-08-06 14:16     ` Laurent Pinchart
2018-08-06 14:34     ` Geert Uytterhoeven
2018-08-06 14:34       ` Geert Uytterhoeven
2018-08-06 14:41       ` Laurent Pinchart
2018-08-06 14:41         ` Laurent Pinchart
2018-08-06 14:52         ` Geert Uytterhoeven
2018-08-06 14:52           ` Geert Uytterhoeven
2018-08-06 14:07 ` [PATCH/RFC 4/4] sh-sci: Derive regshift value from DT compatible value Geert Uytterhoeven
2018-08-06 14:07   ` Geert Uytterhoeven
2018-08-06 14:18   ` Chris Brandt
2018-08-06 14:18     ` Chris Brandt
2018-08-06 14:38     ` Geert Uytterhoeven
2018-08-06 14:38       ` Geert Uytterhoeven
2018-08-06 16:10       ` Chris Brandt
2018-08-06 16:10         ` Chris Brandt
2018-08-07 19:24       ` Chris Brandt
2018-08-07 19:24         ` Chris Brandt
2018-08-07 19:37         ` Geert Uytterhoeven
2018-08-07 19:37           ` Geert Uytterhoeven
2018-08-07 21:10           ` Chris Brandt [this message]
2018-08-07 21:10             ` Chris Brandt
2018-08-08  0:16           ` Chris Brandt
2018-08-08  0:16             ` Chris Brandt
2018-08-08 10:11             ` Geert Uytterhoeven
2018-08-08 10:11               ` Geert Uytterhoeven
2018-08-08 10:39               ` Chris Brandt
2018-08-08 10:39                 ` Chris Brandt
2018-08-08 11:05                 ` Geert Uytterhoeven
2018-08-08 11:05                   ` Geert Uytterhoeven
2018-08-06 14:37 ` [PATCH/RFC 0/4] sh-sci : Do not derive regshift from regsize Laurent Pinchart
2018-08-06 14:37   ` Laurent Pinchart
2018-08-06 14:41   ` Laurent Pinchart
2018-08-06 14:41     ` Laurent Pinchart
2018-08-06 14:41   ` Geert Uytterhoeven
2018-08-06 14:41     ` Geert Uytterhoeven
2018-08-08 11:02 ` Geert Uytterhoeven
2018-08-08 11:02   ` Geert Uytterhoeven
2018-08-08 11:32   ` Chris Brandt
2018-08-08 11:32     ` Chris Brandt
2018-08-08 20:46   ` Chris Brandt
2018-08-08 20:46     ` Chris Brandt

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=TY1PR01MB1562ED9B51B87FBE0F2C1A828A270@TY1PR01MB1562.jpnprd01.prod.outlook.com \
    --to=chris.brandt@renesas.com \
    --cc=geert+renesas@glider.be \
    --cc=geert@linux-m68k.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jslaby@suse.com \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=uli@fpond.eu \
    --cc=ysato@users.sourceforge.jp \
    /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: link
Be 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.