AARCH64: Adding Saphira core detection using MIDR.
Siddhesh Poyarekar
siddhesh@gotplt.org
Tue Nov 20 11:32:00 GMT 2018
On 20/11/18 4:18 PM, Sameera Deshpande wrote:
> Please find attached the patch for CPU detection using MIDR code for
> Qualcomm's Saphira processor, and use Falkor specific memset routine
> for Saphira.
The patch looks technically correct, so this only needs a more precise
subject line and detailed description for the patch. We use the email
subject and content for the git commit log so those need to be clear and
precise.
For example "aarch64: Use falkor memset for saphira" would be a more
precise subject since that's the most important functional change in
this patch. The email description should then describe the rationale of
the patch (i.e., the introduction of the new midr for saphira,
consolidation of the checks, etc.) as well as a description of what
saphira is.
That and the ChangeLog should be good enough. Please send a v2 with
these changes.
Thanks,
Siddhesh
More information about the Libc-alpha
mailing list