Open Source & Linux Lab

It's better when it's simple

User Tools

Site Tools


etc:users:jcmvbkbc:linux-xtensa:patch-review-checklist

This is an old revision of the document!


Patch review check list

  • If it has inline asm
    • Does it use specific instructions, does it correctly handle cores w/o such instructions
    • Does it correctly specify argument constraints (e.g. earlyclobber (=&) arguments)
    • Does it use possibly reserved registers (e.g. a7 with CONFIG_FRAME_POINTER)
etc/users/jcmvbkbc/linux-xtensa/patch-review-checklist.1397084660.txt.gz · Last modified: 2014/04/10 03:04 by jcmvbkbc