public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Guo Heyi <heyi.guo@linaro.org>
To: "Ni, Ruiyu" <ruiyu.ni@Intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [PATCH 2/2] Hisilicon/D0x: Enable tftp command by default
Date: Mon, 26 Mar 2018 17:03:10 +0800	[thread overview]
Message-ID: <20180326090310.GB67369@SZX1000114654> (raw)
In-Reply-To: <d3e8edcb-23a6-0dfb-ec1f-f058350b25e4@Intel.com>

Thanks Ray.

Does that mean we need build the dynamic command driver separately and store it
in other media instead of UEFI fd image? Right now if I include the driver into
the fd image, it will be automatically added to EFI Shell command list; we don't
need to run the load command.

Hi Leif,

Is the policy to forbid including dynamic command driver into UEFI fd image?

If we need other media to store tftp command driver, then the command will
become less useful, because it is mainly used to download something else. If we
have other media like USB disk, we can use this "other media" instead of network
download to store the final target.

Thanks,
Heyi

On Fri, Mar 23, 2018 at 12:51:45PM +0800, Ni, Ruiyu wrote:
> On 3/20/2018 8:15 PM, Guo Heyi wrote:
> >I've no idea about how to use Driver####; let me spend some time to learn first
> >:)
> 
> Heyi,
> you could use "load xxxDriver.efi" to load the dynamic command in shell.
> After that, you can run "tftp" in shell just as running an internal command.
> 
> >
> >Regards,
> >
> >Heyi
> >
> >On Tue, Mar 20, 2018 at 09:51:32AM +0000, Leif Lindholm wrote:
> >>Ah, apologies.
> >>
> >>I would be reluctant to add commands not covered by the UEFI Shell
> >>Specification by default.
> >>
> >>Since it is now a dynamic command, is there any way of loading this
> >>dynamically (perhaps via DRIVER####) where you feel the need for it?
> >>
> >>/
> >>     Leif
> >>
> >>On Tue, Mar 20, 2018 at 03:54:46PM +0800, Guo Heyi wrote:
> >>>Ping :)
> >>>
> >>>
> >>>On Wed, Mar 07, 2018 at 04:02:30PM +0000, Ard Biesheuvel wrote:
> >>>>On 7 March 2018 at 03:03, Heyi Guo <heyi.guo@linaro.org> wrote:
> >>>>>Since D0x platforms always have network enabled, we would like to
> >>>>>enable tftp command by default so that we can download something in
> >>>>>EFI Shell.
> >>>>>
> >>>>>Contributed-under: TianoCore Contribution Agreement 1.1
> >>>>>Signed-off-by: Heyi Guo <heyi.guo@linaro.org>
> >>>>>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >>>>>Cc: Leif Lindholm <leif.lindholm@linaro.org>
> >>>>
> >>>>The first patch looks fine to me, but I would like to give Leif a
> >>>>chance to comment on the policy side of this patch.
> >>>>
> >>>>Please ping us by the end of next week if we haven't responded by then.
> >>>>
> >>>>>---
> >>>>>  Platform/Hisilicon/D03/D03.dsc | 2 ++
> >>>>>  Platform/Hisilicon/D05/D05.dsc | 1 +
> >>>>>  2 files changed, 3 insertions(+)
> >>>>>
> >>>>>diff --git a/Platform/Hisilicon/D03/D03.dsc b/Platform/Hisilicon/D03/D03.dsc
> >>>>>index cb0669d639d1..fce1e60b1275 100644
> >>>>>--- a/Platform/Hisilicon/D03/D03.dsc
> >>>>>+++ b/Platform/Hisilicon/D03/D03.dsc
> >>>>>@@ -29,6 +29,8 @@ [Defines]
> >>>>>    SKUID_IDENTIFIER               = DEFAULT
> >>>>>    FLASH_DEFINITION               = Platform/Hisilicon/$(PLATFORM_NAME)/$(PLATFORM_NAME).fdf
> >>>>>
> >>>>>+  DEFINE INCLUDE_TFTP_COMMAND    = TRUE
> >>>>>+
> >>>>>  !include Silicon/Hisilicon/Hisilicon.dsc.inc
> >>>>>
> >>>>>  [LibraryClasses.common]
> >>>>>diff --git a/Platform/Hisilicon/D05/D05.dsc b/Platform/Hisilicon/D05/D05.dsc
> >>>>>index 8373a821a496..f007f3d2d7e8 100644
> >>>>>--- a/Platform/Hisilicon/D05/D05.dsc
> >>>>>+++ b/Platform/Hisilicon/D05/D05.dsc
> >>>>>@@ -29,6 +29,7 @@ [Defines]
> >>>>>    SKUID_IDENTIFIER               = DEFAULT
> >>>>>    FLASH_DEFINITION               = Platform/Hisilicon/$(PLATFORM_NAME)/$(PLATFORM_NAME).fdf
> >>>>>    DEFINE EDK2_SKIP_PEICORE=0
> >>>>>+  DEFINE INCLUDE_TFTP_COMMAND    = TRUE
> >>>>>    DEFINE NETWORK_IP6_ENABLE      = FALSE
> >>>>>    DEFINE HTTP_BOOT_ENABLE        = FALSE
> >>>>>
> >>>>>--
> >>>>>2.7.4
> >>>>>
> >_______________________________________________
> >edk2-devel mailing list
> >edk2-devel@lists.01.org
> >https://lists.01.org/mailman/listinfo/edk2-devel
> >
> 
> 
> -- 
> Thanks,
> Ray
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


  reply	other threads:[~2018-03-26  8:56 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-07  3:03 [PATCH 1/2] Hisilicon/D0x: fix tftp command init failure Heyi Guo
2018-03-07  3:03 ` [PATCH 2/2] Hisilicon/D0x: Enable tftp command by default Heyi Guo
2018-03-07 16:02   ` Ard Biesheuvel
2018-03-20  7:54     ` Guo Heyi
2018-03-20  9:51       ` Leif Lindholm
2018-03-20 12:15         ` Guo Heyi
2018-03-23  4:51           ` Ni, Ruiyu
2018-03-26  9:03             ` Guo Heyi [this message]
2018-04-02  2:51               ` Guo Heyi
2018-04-16 12:33                 ` Leif Lindholm
2018-04-17  1:58                   ` Guo Heyi
2018-04-18 10:59 ` [PATCH 1/2] Hisilicon/D0x: fix tftp command init failure Leif Lindholm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180326090310.GB67369@SZX1000114654 \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox