[coreboot-gerrit] [S] Change in coreboot[main]: arch/arm64: Extend cache helper functions

[David Milosevic (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Benjamin Doron, Julius Werner, Marvin Drees, Paul Menzel.

David Milosevic has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79106?usp=email )

Change subject: arch/arm64: Extend cache helper functions
......................................................................

Patch Set 6:

(3 comments)

File src/arch/arm64/armv8/cache.c:

https://review.coreboot.org/c/coreboot/+/79106/comment/b41f5134_c7f89f35 :
PS5, Line 55: static uint64_t cache_numsets(const enum cache_level level, const enum cache_type type)
            : {
            : 	cache_select(level, type);
            : 
            : 	/* (Number of sets in cache) - 1 */
            : 	return get_ccsidr_el1_numsets(raw_read_ccsidr_el1()) + 1;
            : }
> I agree. […]

Done

https://review.coreboot.org/c/coreboot/+/79106/comment/a61d0855_22d7b0e5 :
PS5, Line 87: (struct cache_info) {
> This will also memcpy 0 to .size first before updating it. […]

Done

https://review.coreboot.org/c/coreboot/+/79106/comment/96ebdc7c_463cb4ad :
PS5, Line 88:
> No newline after {

Done