Received-SPF: pass (google.com: domain of linux-kernel+bounces-189738-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sun, 26 May 2024 13:44:07 +0100 (BST)
From: "Maciej W. Rozycki" <macro@orcam.me.uk>
To: Jiaxun Yang <jiaxun.yang@flygoat.com>
cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>, 
    "linux-mips@vger.kernel.org" <linux-mips@vger.kernel.org>, 
    linux-kernel@vger.kernel.org, 
    =?UTF-8?Q?Philippe_Mathieu-Daud=C3=A9?= <philmd@linaro.org>
Subject: Re: [PATCH v3 0/9] MIPS: Unify low-level debugging functionalities
In-Reply-To: <a62c1c0b-0b2a-4b3f-85df-da586e103fcb@app.fastmail.com>
Message-ID: <alpine.DEB.2.21.2405261326260.1257@angie.orcam.me.uk>
References: <20240502-mips_debug_ll-v3-0-3b61f30e484c@flygoat.com> <3dcf3ac1-5494-482a-a80a-df4126e6ae59@app.fastmail.com> <3d6883ed-f8f4-44e5-a184-e5499c44f0f7@app.fastmail.com> <Zk2mt/FsgltvhVzf@alpha.franken.de> <13aa508f-6830-4d52-87fd-5063f737c990@app.fastmail.com>
 <a62c1c0b-0b2a-4b3f-85df-da586e103fcb@app.fastmail.com>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII

On Wed, 22 May 2024, Jiaxun Yang wrote:

> That being said, have you noticed that prom_putchar and early_printk is
> a non-extant on generic mach, ingenic, ralink etc? That's because we
> really don't want to introduce any platform specific UART code for
> early debugging on new platforms. With DEBUG_LL introduced by Arm it's
> only a Kconfig option to do the trick.

 IMHO that is however the logical thing to do.  And then you need no magic 
options to fiddle with and say a distribution kernel will dump whatever it 
has to say if something wrong has happened early on.

 IOW just wire `prom_putchar' as required, using C code preferably.  NB 
YAMON does have a `print' entry point for console output, so for the Malta 
platform you can trivially use just that, no need for messy ad hoc 8250 
code.

 As to intercepting exceptions, it depends.  Again YAMON does handle that 
and dumps the register state, so with the Malta you get the information 
required.  For less capable ones it might make sense, but it ISTM like a 
candidate for an independent change, and then again I fail to see why the 
handler has to be written in the assembly language rather than C.

  Maciej