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