Joel Sherrill started a new discussion: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1111#note_146391


Earlier you said: I consolidated the APIC definitions by removing the pc386 
header (`bsps/i386/include/bsp/apic.h`) and making pc386 include `<apic.h>`, 
which resolves to `bsps/x86_64/amd64/include/apic.h`.

It is against the RTEMS Coding Conventions to "reach up and over" to another 
BSP especially one that is in another architecture. The proper solution is to 
move this header to bsps/include/.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1111#note_146391
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to