Kinsey Moore commented on a discussion on 
bsps/aarch64/raspberrypi/start/mailbox.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/915#note_138816

 > +
 > +    memset(buffer, 0, size);
 > +    return RTEMS_SUCCESSFUL;
 > +}
 > +
 > +
 > +
 > +void rpi_mbox_write(uint8_t channel, uint32_t data){
 > +    while(BCM2835_REG(BCM2711_MBOX_STATUS) & BCM2711_MBOX_FULL){
 > +    }
 > +
 > +    BCM2835_REG(BCM2711_MBOX_WRITE) = (data & MBOX_MAIL_DATA_MASK) | 
 > (channel & MBOX_MAIL_CHANNEL_MASK);
 > +
 > +}
 > +
 > +uint32_t rpi_mbox_read(uint8_t channel){

All of these functions should also use RTEMS-style braces with function opening 
and closing braces on their own lines.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/915#note_138816
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