Re: [PATCH v2 0/2] Add macb support for SiFive FU540-C000

From: Paul Walmsley
Date: Mon Jun 17 2019 - 06:03:17 EST


Hi Yash,

On Mon, 17 Jun 2019, Andreas Schwab wrote:

> On Jun 17 2019, Yash Shah <yash.shah@xxxxxxxxxx> wrote:
>
> > - Add "MACB_SIFIVE_FU540" in Kconfig to support SiFive FU540 in macb
> > driver. This is needed because on FU540, the macb driver depends on
> > SiFive GPIO driver.
>
> This of course requires that the GPIO driver is upstreamed first.

What's the impact of enabling CONFIG_MACB_SIFIVE_FU540 when the GPIO
driver isn't present? (After modifying the Kconfig "depends" line
appropriately.)

Looks to me that it shouldn't have an impact unless the DT string is
present, and even then, the impact might simply be that the MACB driver
may not work?


- Paul