RTEMS | Complete the u-boot support for qoriq BSP (!1046)

Wed Feb 25 13:20:35 UTC 2026

Sebastian Huber started a new discussion on bsps/powerpc/qoriq/start/uboot_support.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1046#note_143481

 > + * 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.
 > + */
 > +
 > +#include <stdint.h>
 > +
 > +#include <bsp.h>
 > +
 > +#if defined(HAS_UBOOT)

I would do it like this:

The usage HAVE_UBOOT define should be removed from uboot_getenv.c. The uboot_support.c should be removed for all BSPs. Definitions of uboot_environment and uboot_environment_size should move to uboot_getenv.c with values provided by two new BSP options shared by the powerpc BSPs. If these options are not defined, then both should be 0 initialized, so that bsp_uboot_getenv() always returns NULL.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1046#note_143481
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/20260225/62ac8757/attachment-0001.htm>