RTEMS | aarch64: add rk3399_rockpro64 BSP (!894)
Joel Sherrill (@joel)
gitlab at rtems.org
Sat Jan 24 18:36:33 UTC 2026
Joel Sherrill started a new discussion on bsps/aarch64/rk3399/include/bsp.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/894#note_139929
> + * documentation and/or other materials provided with the distribution.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
> + * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
> + * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
> + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
> + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
> + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
> + * POSSIBILITY OF SUCH DAMAGE.
> + */
> +
> +#ifndef LIBBSP_AARCH64_RK3399_BSP_H
bsp.h is a public header file and even though there are many counter examples, should not include private information with names that are BSP specific
These address constants should be in another header file not included from bsp.h. Every application should not see them.
@opticron may have a suggestion for an alternative specific file name. It would be in bsp/
--
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/894#note_139929
You're receiving this email because of your account on gitlab.rtems.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20260124/cea2c28d/attachment.htm>
More information about the bugs
mailing list