2023-11-20 08:44:47

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH 14/14] arm: multi_v7_defconfig: Enable CONFIG_RAVB

On Mon, Nov 20, 2023, at 08:00, Claudiu wrote:
> From: Claudiu Beznea <[email protected]>
>
> ravb driver is used by RZ/G1H. Enable it in multi_v7_defconfig.
>
> Signed-off-by: Claudiu Beznea <[email protected]>

We have a mix of =y and =m for ethernet drivers, and usually
only have drivers built-in when they are frequently tested
with NFS root booting.

Do you need this as well, or could it be =m instead?

Arnd


2023-11-20 08:59:16

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH 14/14] arm: multi_v7_defconfig: Enable CONFIG_RAVB

On Mon, Nov 20, 2023 at 9:44 AM Arnd Bergmann <[email protected]> wrote:
> On Mon, Nov 20, 2023, at 08:00, Claudiu wrote:
> > From: Claudiu Beznea <[email protected]>
> >
> > ravb driver is used by RZ/G1H. Enable it in multi_v7_defconfig.

Used by:
- iWave-RZ/G1M/G1N Qseven carrier board,
- iWave-RZ/G1H Qseven board,
- iWave-RZG1E SODIMM carrier board,
- iWave-RZ/G1C single board computer.

So I'd write "used by various iWave RZ/G1 development boards".

> > Signed-off-by: Claudiu Beznea <[email protected]>
>
> We have a mix of =y and =m for ethernet drivers, and usually
> only have drivers built-in when they are frequently tested
> with NFS root booting.
>
> Do you need this as well, or could it be =m instead?

As the default chosen/bootargs for the iWave-RZ/G1M/G1N Qseven carrier
board contains root=/dev/nfs, builtin is appropriate.
The iWave-RZ/G1H Qseven board defaults to root=/dev/mmcblk0p1.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2023-11-21 06:15:47

by claudiu beznea

[permalink] [raw]
Subject: Re: [PATCH 14/14] arm: multi_v7_defconfig: Enable CONFIG_RAVB



On 20.11.2023 10:44, Arnd Bergmann wrote:
> On Mon, Nov 20, 2023, at 08:00, Claudiu wrote:
>> From: Claudiu Beznea <[email protected]>
>>
>> ravb driver is used by RZ/G1H. Enable it in multi_v7_defconfig.
>>
>> Signed-off-by: Claudiu Beznea <[email protected]>
>
> We have a mix of =y and =m for ethernet drivers, and usually
> only have drivers built-in when they are frequently tested
> with NFS root booting.
>
> Do you need this as well, or could it be =m instead?

I would prefer to have it =y as internal testing infrastructure is using NFS.

Thank you,
Claudiu Beznea

>
> Arnd

2023-11-27 10:08:35

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH 14/14] arm: multi_v7_defconfig: Enable CONFIG_RAVB

On Mon, Nov 20, 2023 at 9:58 AM Geert Uytterhoeven <[email protected]> wrote:
> On Mon, Nov 20, 2023 at 9:44 AM Arnd Bergmann <[email protected]> wrote:
> > On Mon, Nov 20, 2023, at 08:00, Claudiu wrote:
> > > From: Claudiu Beznea <[email protected]>
> > >
> > > ravb driver is used by RZ/G1H. Enable it in multi_v7_defconfig.
>
> Used by:
> - iWave-RZ/G1M/G1N Qseven carrier board,
> - iWave-RZ/G1H Qseven board,
> - iWave-RZG1E SODIMM carrier board,
> - iWave-RZ/G1C single board computer.
>
> So I'd write "used by various iWave RZ/G1 development boards".

Reviewed-by: Geert Uytterhoeven <[email protected]>
i.e. will queue in renesas-devel for v6.8, with the above updated.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds