Подстажите в чем ошибка, а то я походу туплю уже, но простая подстановка приводит к ошибке.

Определил я пин и порт для светодиода:


Затем значит определил функцию:


И пытаюсь ее вызывать так:

Keil выдает ошибку #54-D: too few arguments in invocation of macro "on"
и ошибку #29: expected an expression

Если вместо LED2_PIN написать LED2,MDR_PORTB, т.е по сути то что определяет дефайн "LED2_PIN", то ошибки нет.
Т.е. вот так работает:
Хотелось бы выкинуть этот порт, чтобы каждый раз его не писать при вызове функции on(), но как обойти ошибку?

Но так нельзя будет вызвать on(LED2,MDR_PORTB);
Вот так - можно, но требует некоторой поддержки со стороны компилятора (хотя бы С99)

Да, понял свою ошибку. Сначала компилятор в макрос on(arg1, arg2) подставляет первый аргумент, который в моем случае есть "LED2,MDR_PORTB", а во второй аргумент ему подставить уже нечего. По сути логично.
Вообщем сделал так:


Вызов on(LED1_PIN); работает.
Теперь можно вызывать on(); если PIN "расписан" как положено - с битовой маской и именем порта, а если нет, то можно вызвать _on();, указав битовую маску и имя порта.

Спасибо, вопрос снят!

Компилятор не подставляет. Подставляет препроцессор.
Пока вы не разберётесь с тем, как устроен процесс компиляции в языке Си, будут появляться другие вопросы.