| Message ID | 20241026080050.125214-1-ilias.apalodimas@linaro.org |
|---|---|
| State | Accepted |
| Commit | 84b95e918942d0e29e3634d265694030f76faf5a |
| Headers | show |
| Series | efi_loader: Remove unused diskid | expand |
On 10/26/24 10:00, Ilias Apalodimas wrote: > That variable is defined and assigned a value in two functions > but it's never used. > > Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> > --- > lib/efi_loader/efi_disk.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/lib/efi_loader/efi_disk.c b/lib/efi_loader/efi_disk.c > index 93a9a5ac0250..1f3de0a23399 100644 > --- a/lib/efi_loader/efi_disk.c > +++ b/lib/efi_loader/efi_disk.c > @@ -561,11 +561,9 @@ static int efi_disk_create_raw(struct udevice *dev, efi_handle_t agent_handle) > { > struct efi_disk_obj *disk; > struct blk_desc *desc; > - int diskid; > efi_status_t ret; > > desc = dev_get_uclass_plat(dev); > - diskid = desc->devnum; > > ret = efi_disk_add_dev(NULL, NULL, desc, > NULL, 0, &disk, agent_handle); > @@ -608,7 +606,6 @@ static int efi_disk_create_part(struct udevice *dev, efi_handle_t agent_handle) > struct disk_part *part_data; > struct disk_partition *info; > unsigned int part; > - int diskid; > struct efi_handler *handler; > struct efi_device_path *dp_parent; > struct efi_disk_obj *disk; > @@ -618,7 +615,6 @@ static int efi_disk_create_part(struct udevice *dev, efi_handle_t agent_handle) > return -1; > > desc = dev_get_uclass_plat(dev_get_parent(dev)); > - diskid = desc->devnum; > > part_data = dev_get_uclass_plat(dev); > part = part_data->partnum;
diff --git a/lib/efi_loader/efi_disk.c b/lib/efi_loader/efi_disk.c index 93a9a5ac0250..1f3de0a23399 100644 --- a/lib/efi_loader/efi_disk.c +++ b/lib/efi_loader/efi_disk.c @@ -561,11 +561,9 @@ static int efi_disk_create_raw(struct udevice *dev, efi_handle_t agent_handle) { struct efi_disk_obj *disk; struct blk_desc *desc; - int diskid; efi_status_t ret; desc = dev_get_uclass_plat(dev); - diskid = desc->devnum; ret = efi_disk_add_dev(NULL, NULL, desc, NULL, 0, &disk, agent_handle); @@ -608,7 +606,6 @@ static int efi_disk_create_part(struct udevice *dev, efi_handle_t agent_handle) struct disk_part *part_data; struct disk_partition *info; unsigned int part; - int diskid; struct efi_handler *handler; struct efi_device_path *dp_parent; struct efi_disk_obj *disk; @@ -618,7 +615,6 @@ static int efi_disk_create_part(struct udevice *dev, efi_handle_t agent_handle) return -1; desc = dev_get_uclass_plat(dev_get_parent(dev)); - diskid = desc->devnum; part_data = dev_get_uclass_plat(dev); part = part_data->partnum;
That variable is defined and assigned a value in two functions but it's never used. Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> --- lib/efi_loader/efi_disk.c | 4 ---- 1 file changed, 4 deletions(-)