[PATCH] arm: spl: Use separate fault handlers instead of a single common one

Tom Rini trini at konsulko.com
Fri Dec 15 14:17:35 CET 2023


On Fri, Dec 15, 2023 at 01:10:45PM +0100, Csókás Bence wrote:
> 
> 
> 2023. 12. 14. 18:53 keltezéssel, Tom Rini írta:
> > On Thu, Dec 14, 2023 at 05:29:31PM +0100, Csókás Bence wrote:
> > 
> > > It may be necessary to set breakpoints etc. on a specific fault handler in SPL.
> > > Add a Kconfig option to separate the different handlers into their own individual infinite loops.
> > > 
> > > Signed-off-by: Csókás Bence <csokas.bence at prolan.hu>
> > > ---
> > >   arch/arm/Kconfig       | 10 ++++++++++
> > >   arch/arm/lib/vectors.S | 18 ++++++++++++++++++
> > >   2 files changed, 28 insertions(+)
> > > 
> > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > > index 328e2ddc33..47494cb72b 100644
> > > --- a/arch/arm/Kconfig
> > > +++ b/arch/arm/Kconfig
> > > @@ -79,6 +79,16 @@ config SPL_SYS_NO_VECTOR_TABLE
> > >   	depends on SPL
> > >   	bool
> > > +config SPL_USE_SEPARATE_FAULT_HANDLERS
> > > +	bool "Use separate fault handlers instead of a single common one"
> > > +	depends on !SPL_SYS_NO_VECTOR_TABLE
> > 
> > Will this dep catch that this is only valid for !ARM64 && !CPU_V7M ?
> 
> Is it? How so?
> So should I put `depends on !SPL_SYS_NO_VECTOR_TABLE && !ARM64 && !CPU_V7M`
> instead?

Well, looking at arch/arm/lib/Makefile, vector.S is only used in the
case I said, so you would need to modify the other files as well.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231215/4a196eed/attachment.sig>


More information about the U-Boot mailing list