On 1/28/25 10:11 AM, Mattijs Korpershoek wrote:
Hi Michael,
Thank you for the patch.
On mar., janv. 28, 2025 at 02:49, Marek Vasut marex@denx.de wrote:
On 1/28/25 1:30 AM, Michael Ferolito wrote:
The current behaviour of this function will dereference a null pointer if the serial# environment variable is unset. This was discovered on a board where U-Boot did not have access to the first 256MB of ram, resulting in a board crash. In the event that U-Boot has full access to memory, it will still read from address 0, which is probably not optimal. This simple check is enough to fix it
Signed-off-by: Michael Ferolito michaelsunn101@gmail.com Cc: Marek Vasut marex@denx.de Cc: Heiko Schocher hs@denx.de Cc: Kyungmin Park kyungmin.park@samsung.com
The subject tag is "usb: gadget: g_dnl:" instead of "USB:" , with that fixed:
Reviewed-by: Marek Vasut marex@denx.de
Agreed with Marek on this. Please add:
Reviewed-by: Mattijs Korpershoek mkorpershoek@baylibre.com
As well.
Marek, since this is gadget, can this go through my tree?
Yes of course.