Re: [PATCH v3 02/35] efi: Enable DM_ETH for the app

2021-11-03 Thread Simon Glass
Hi Heinrich,

On Wed, 27 Oct 2021 at 23:40, Heinrich Schuchardt  wrote:
>
> On 10/26/21 5:30 AM, Simon Glass wrote:
> > There is no need to avoid driver model for networking. Drop this.
> >
> > Signed-off-by: Simon Glass 
>
> This seems to be an unrelated problem in building efi-x86_app_defconfig:
>
>LD  u-boot
> ld.bfd: arch/x86/lib/bios_asm.o: warning: relocation against `idt_ptr'
> in read-only section `.text'
> ld.bfd: warning: creating DT_TEXTREL in a shared object
>
> I have been using
> binutils 2.35.2-2
> gcc 4:10.2.1-1
> as provided with Debian Bulleyes.

Hmm can't repeat that. Can you please give me the build command, commit?

Mine is:

gcc version 10.2.1 20210110 (Debian 10.2.1-6)
GNU ld (GNU Binutils for Debian) 2.35.2

Regards,
Simon

>
> Reviewed-by: Heinrich Schuchardt 
>
> > ---
> >
> > (no changes since v2)
> >
> > Changes in v2:
> > - Add new patch to enable DM_ETH for the app
> >
> >   configs/efi-x86_app_defconfig | 1 -
> >   1 file changed, 1 deletion(-)
> >
> > diff --git a/configs/efi-x86_app_defconfig b/configs/efi-x86_app_defconfig
> > index e9ee66250cf..3f7c9ba9008 100644
> > --- a/configs/efi-x86_app_defconfig
> > +++ b/configs/efi-x86_app_defconfig
> > @@ -32,7 +32,6 @@ CONFIG_ENV_OVERWRITE=y
> >   CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> >   CONFIG_REGMAP=y
> >   CONFIG_SYSCON=y
> > -# CONFIG_DM_ETH is not set
> >   # CONFIG_REGEX is not set
> >   # CONFIG_GZIP is not set
> >   CONFIG_EFI=y
> >
>


Re: [PATCH v3 02/35] efi: Enable DM_ETH for the app

2021-10-27 Thread Heinrich Schuchardt

On 10/26/21 5:30 AM, Simon Glass wrote:

There is no need to avoid driver model for networking. Drop this.

Signed-off-by: Simon Glass 


This seems to be an unrelated problem in building efi-x86_app_defconfig:

  LD  u-boot
ld.bfd: arch/x86/lib/bios_asm.o: warning: relocation against `idt_ptr'
in read-only section `.text'
ld.bfd: warning: creating DT_TEXTREL in a shared object

I have been using
binutils 2.35.2-2
gcc 4:10.2.1-1
as provided with Debian Bulleyes.

Reviewed-by: Heinrich Schuchardt 


---

(no changes since v2)

Changes in v2:
- Add new patch to enable DM_ETH for the app

  configs/efi-x86_app_defconfig | 1 -
  1 file changed, 1 deletion(-)

diff --git a/configs/efi-x86_app_defconfig b/configs/efi-x86_app_defconfig
index e9ee66250cf..3f7c9ba9008 100644
--- a/configs/efi-x86_app_defconfig
+++ b/configs/efi-x86_app_defconfig
@@ -32,7 +32,6 @@ CONFIG_ENV_OVERWRITE=y
  CONFIG_SYS_RELOC_GD_ENV_ADDR=y
  CONFIG_REGMAP=y
  CONFIG_SYSCON=y
-# CONFIG_DM_ETH is not set
  # CONFIG_REGEX is not set
  # CONFIG_GZIP is not set
  CONFIG_EFI=y





[PATCH v3 02/35] efi: Enable DM_ETH for the app

2021-10-25 Thread Simon Glass
There is no need to avoid driver model for networking. Drop this.

Signed-off-by: Simon Glass 
---

(no changes since v2)

Changes in v2:
- Add new patch to enable DM_ETH for the app

 configs/efi-x86_app_defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/configs/efi-x86_app_defconfig b/configs/efi-x86_app_defconfig
index e9ee66250cf..3f7c9ba9008 100644
--- a/configs/efi-x86_app_defconfig
+++ b/configs/efi-x86_app_defconfig
@@ -32,7 +32,6 @@ CONFIG_ENV_OVERWRITE=y
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
 CONFIG_REGMAP=y
 CONFIG_SYSCON=y
-# CONFIG_DM_ETH is not set
 # CONFIG_REGEX is not set
 # CONFIG_GZIP is not set
 CONFIG_EFI=y
-- 
2.33.0.1079.g6e70778dc9-goog