Re: [PATCH v2 2/3] mtd: spi-nor: support vcc-supply regulator
From: Michael Walle
Date: Mon Oct 07 2024 - 03:52:20 EST
Hi,
> > > + ret = devm_regulator_get_enable(dev, "vcc");
> > > + if (ret)
> > > + return ret;
> > > +
> >
> > What happens if the SPI-NOR doesn't have a "vcc" regulator?
>
> ...the SPI-NOR will use the dummy regulator.
Which then prints a warning "using dummy regulator". Is that the
usual way to go?
I mean the regulator is actually mandatory because it is the main
voltage rail for the flash. To get rid of the warning one can add a
fixed-regulator (which is correct anyway). But OTOH, the device tree
lists it as optional (marking it as required isn't an option either
because virtually all device trees won't have that property).
-michael
Attachment:
signature.asc
Description: PGP signature