[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.2.21.2004051817310.4156324@eddie.linux-mips.org>
Date: Sun, 5 Apr 2020 18:23:03 +0100 (BST)
From: "Maciej W. Rozycki" <macro@...ux-mips.org>
To: Jiaxun Yang <jiaxun.yang@...goat.com>
cc: linux-mips@...r.kernel.org, Fangrui Song <maskray@...gle.com>,
Nathan Chancellor <natechancellor@...il.com>,
Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] MIPS: malta: Set load address for 32bit kernel
correctly
On Mon, 6 Apr 2020, Jiaxun Yang wrote:
> > Given the description above I think it should be done uniformly and
> >automatically across all platforms by trimming the address supplied
> >with
> >$(load-y) to low 8 digits in a single place, that is at the place where
> >
> >the variable is consumed. This will reduce clutter across Makefile
> >fragments, avoid inconsistencies and extra work to handle individual
> >platforms as the problem is triggered over and over again, and limit
> >the
> >risk of mistakes.
>
> I was intended to do like this but failed to find a proper way.
>
> Makefile isn't designed for any kind of calculation.
> And shell variables are 64-bit signed so it can't hold such a huge variable.
>
> Just wish somebody can give me a way to do like:
>
> ifndef CONFIG_64BIT
> load-y = $(load-y) & 0xffffffff
> endif
Use the usual shell tools like `sed', `cut', `awk', or whatever we use in
the kernel build already for other purposes. There's no need to do any
actual calculation here to extract the last 8 characters (and the leading
`0x' prefix). At worst you can write a small C program, compile it with
the build system compiler and run, as we already do for some stuff.
Maciej
Powered by blists - more mailing lists