Re: [PATCH 2/5] powerpc/microwatt: Device-tree updates

[Segher Boessenkool <segher@kernel.crashing.org>]

On Wed, Jan 29, 2025 at 09:52:09AM +1100, Paul Mackerras wrote:
> -			isa = <3000>;
> +			isa = <3010>;

Does this mean 3.1, or 3.01?  If the former, can this also encode 3.1C?
Should uwatt say to support that?

>  			little-endian {
> -				isa = <2050>;
> -				usable-privilege = <3>;
> +				isa = <0>;
> +				usable-privilege = <7>;
> +				os-support = <0>;
>  				hwcap-bit-nr = <1>;
>  			};

What does "isa" in this node mean?  Why is it changed to 0?  (I don't
know this node at all, I only know a property with the same name).

>  			cache-inhibited-large-page {
> -				isa = <2040>;
> -				usable-privilege = <2>;
> +				isa = <0>;
> +				usable-privilege = <6>;
> +				os-support = <0>;
>  			};

Similar question here.

> -				isa = <2010>;
> +				isa = <0x00>;
>  				usable-privilege = <2>;
> +				os-support = <0>;
>  			};

And here.  That's a quite woolly way to say "0" btw ;-)


Segher