On 19 February 2016 at 16:30, Laszlo Ersek <[email protected]> wrote:

> On 02/10/16 15:07, Shivamurthy Shastri wrote:
> > error: pNicDevice variable set but not used
> >
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Shivamurthy Shastri <[email protected]>
> > ---
> >  OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> b/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> > index 9eeb61f..c061a6b 100644
> > --- a/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> > +++ b/OptionRomPkg/Bus/Usb/UsbNetworking/Ax88772b/SimpleNetwork.c
> > @@ -700,10 +700,8 @@ SN_ReceiveFilters (
> >    EFI_SIMPLE_NETWORK_MODE * pMode;
> >    EFI_STATUS Status = EFI_SUCCESS;
> >    EFI_TPL TplPrevious;
> > -  NIC_DEVICE * pNicDevice;
> >
> >    TplPrevious = gBS->RaiseTPL(TPL_CALLBACK);
> > -  pNicDevice = DEV_FROM_SIMPLE_NETWORK ( pSimpleNetwork );
> >    pMode = pSimpleNetwork->Mode;
> >
> >    if (pSimpleNetwork == NULL) {
> >
>
> As I said in the other (similar) thread, the subject should identify the
> top level package. It should go:
>
> OptionRomPkg: Ax88772b: ...
>
> Thanks
> Laszlo
>

​I will send new patch for that.

Thanks,
Shiva​
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to