[PATCH 1/3] net: emaclite: fix broken build
Ramon Fried
rfried.dev at gmail.com
Sat Aug 6 19:31:22 CEST 2022
On Wed, Jul 13, 2022 at 5:02 PM Samuel Obuch <samuel.obuch at codasip.com> wrote:
>
> Function ioremap_nocache seems to be defined only for mips and microblaze
> architectures. Therefore, the function call in the emaclite driver causes
> this driver to be unusable with other architectures, for example riscv.
>
> Signed-off-by: Samuel Obuch <samuel.obuch at codasip.com>
> ---
> drivers/net/xilinx_emaclite.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/net/xilinx_emaclite.c b/drivers/net/xilinx_emaclite.c
> index 6c9f1f7c27..5cd88e04fe 100644
> --- a/drivers/net/xilinx_emaclite.c
> +++ b/drivers/net/xilinx_emaclite.c
> @@ -615,8 +615,12 @@ static int emaclite_of_to_plat(struct udevice *dev)
> int offset = 0;
>
> pdata->iobase = dev_read_addr(dev);
> +#if defined(CONFIG_MICROBLAZE) || defined(CONFIG_MIPS)
> emaclite->regs = (struct emaclite_regs *)ioremap_nocache(pdata->iobase,
> 0x10000);
> +#else
> + emaclite->regs = (struct emaclite_regs *)pdata->iobase;
> +#endif
>
> emaclite->phyaddr = -1;
>
> --
> 2.31.1
>
Hm...
Well, this isn't right,The right solution is to replace
ioremap_nocache() with ioremap().
This way it will work both for MIPS and other architectures.
I can do it myself, you can fix your patch. let me know.
Thanks,
Ramon.
More information about the U-Boot
mailing list