Page MenuHomeFreeBSD

Introduce platform CPU info container for ARM. Use it for ACTLR.
AbandonedPublic

Authored by zbb on May 11 2017, 3:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 11:14 AM
Unknown Object (File)
Tue, Apr 16, 7:43 AM
Unknown Object (File)
Sat, Apr 13, 11:36 AM
Unknown Object (File)
Sat, Apr 13, 12:49 AM
Unknown Object (File)
Tue, Apr 9, 10:51 AM
Unknown Object (File)
Tue, Apr 9, 3:01 AM
Unknown Object (File)
Dec 20 2023, 5:55 AM
Unknown Object (File)
Oct 31 2023, 2:47 AM
Subscribers

Details

Summary

This change allows for additional, platform-specific quirks
in the CPU configuration based on CPU info.
Add plat_cpuinfo to store any quirks or additional settings
to be applied by the platform code.
Use it to modify default ACTLR set-up if needed.

Submitted by: Zbigniew Bodek <zbb@semihalf.com>
Obtained from: Semihalf
Sponsored by: Stormshield

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 9204
Build 9651: CI src buildJenkins

Event Timeline

Otherwise, I'm fine with this.

sys/arm/arm/cpuinfo.c
248

Please, follow logic used in this function and use one variable for 'mask' and other for 'set'.

sys/arm/include/cpuinfo.h
129

Only minor nit - plat_cpuinfo doesn't looks like best name - what about:

struct cpuinfo_quirks {
        uint32_t actlr_mask;
        uint32_t actlr_set;
};

Will use environment variable.