That would work better, but to me it seems odd to have one define with a prefix and the others without. I could change them all, but that seems like excessive churn for the very minor 'issue' that it fixes. After rereadig rsi_load_9113_firmware(), it seems like just dropping the CONFIG_ would be a reasonable change that doesn't affect readability. Cheers, Peter