On 07/27/2017 05:28 PM, Rob Clark wrote:
> Signed-off-by: Rob Clark <robdcl...@gmail.com>
> ---
> Just noticed that there didn't seem to be quite enough break's. Pretty
> sure the intention wasn't to fall-thru
>
> lib/efi_loader/efi_device_path_to_text.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lib/efi_loader/efi_device_path_to_text.c
> b/lib/efi_loader/efi_device_path_to_text.c
> index 612e380617..4b2f43f0c8 100644
> --- a/lib/efi_loader/efi_device_path_to_text.c
> +++ b/lib/efi_loader/efi_device_path_to_text.c
> @@ -52,6 +52,7 @@ static uint16_t *efi_convert_device_node_to_text(
> break;
> }
> }
> + break;
> case DEVICE_PATH_TYPE_MEDIA_DEVICE:
> switch (device_node->sub_type) {
> case DEVICE_PATH_SUB_TYPE_FILE_PATH:
> @@ -63,6 +64,7 @@ static uint16_t *efi_convert_device_node_to_text(
> memcpy(buffer, device_node->data, buffer_size);
> break;
> }
> + break;
> }
>
> /*
>
Thank you for catching this.
Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
