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)