Actually, I'm thinking you might as well eliminate the #ifdef stuff altogether. The tsec is only used on 83xx, 85xx, and 86xx, and any future part will surely use this definition: #define TSEC_BASE_ADDR (CFG_IMMR + CFG_TSEC1_OFFSET)