[SLOF] [PATCH 4/4] Fix typos in the slof folder
Segher Boessenkool
segher at kernel.crashing.org
Thu Feb 23 03:34:30 AEDT 2023
On Wed, Feb 22, 2023 at 04:40:57PM +0100, Thomas Huth wrote:
> Found with the "codespell" utility. I kept "busses" which codespell
> also complains about since it seems to be an old but still valid
> plural of the word "bus".
For all four:
Reviewed-by: Segher Boessenkool <segher at kernel.crashing.org>
> -\ We corespond memory nodes with their respective memory controllers
> +\ We correspond memory nodes with their respective memory controllers
> \ and use /chosen/memory as default memory node to hold the "available" map
"We pair"? "We correlate"?
> \ NOTE -> /chosen/memory is expected 2B initialized before using claim/release
"to be". (It took me a while to realise this , I thought "0x80000000
earlier" or such!)
> \ check for multifunction and for each function
> -\ (dependig from header type) call device or bridge setup
> +\ (depending from header type) call device or bridge setup
"dependent on header type"?
> -// bool dependend pick
> +// bool dependent pick
> // ?PICK ( v1 v2 bool -- v1|v2 )
: ?pick 1 umin pick ;
or
: ?pick IF swap THEN drop ;
(the latter is the actual implementation).
But maybe just delete this word? It isn't used anywhere afaics, and it
isn't a good name for what it does anyway?
Segher
More information about the SLOF
mailing list