Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
Date:   Tue, 28 Nov 2023 12:04:24 +0100
From:   Miquel Raynal <miquel.raynal@bootlin.com>
To:     Tudor Ambarus <tudor.ambarus@linaro.org>
Cc:     liao jaime <jaimeliao.tw@gmail.com>,
        Michael Walle <michael@walle.cc>, jaimeliao@mxic.com.tw,
        pratyush@kernel.org, richard@nod.at, linux-mtd@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mtd: spi-nor: stop printing superfluous debug info
Message-ID: <20231128120424.79f8f215@xps-13>
In-Reply-To: <756aa94a-3237-4689-885a-1e17504e72b6@linaro.org>
References: <20231127165908.1734951-1-tudor.ambarus@linaro.org>
        <42c96d63d1ea4f7e8f16a3c8eb0a4cf1@walle.cc>
        <20231128100313.3c990f69@xps-13>
        <18ba4126dbd9e49846344b517ad2fbdd@walle.cc>
        <CAAQoYR=ZAEVeoCwkRmky2CHVeKGX55JijM8Q6D5msz_5iW+jtw@mail.gmail.com>
        <d7bb4580-00b9-4206-bbfc-bd203c21a877@linaro.org>
        <756aa94a-3237-4689-885a-1e17504e72b6@linaro.org>
Organization: Bootlin
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hello,

tudor.ambarus@linaro.org wrote on Tue, 28 Nov 2023 09:47:22 +0000:

> On 11/28/23 09:39, Tudor Ambarus wrote:
> >=20
> > cut
> >  =20
> >>>> Maybe we don't need this at all, as long as one message remains about
> >>>> the JEDEC ID, but keep in mind that spi-nors are commonly storing the
> >>>> rootfs and if your spi-nor does not boot you don't have a userspace =
yet
> >>>> and all the debugfs entries are purely useless. =20
> >>>
> >>> Good point.
> >>>
> >>> Just curious, do you know any boards which has the rootfs writable on
> >>> the spi-nor flash? =20
> >> I am also interested.
> >> =20
> >=20
> > Having the rootfs stored on SPI NOR is a poor design decision as you're
> > better of with a NAND, which is cheaper and faster on writes. I tried in
> > the past a ubifs on top of a large (64 and 128MB) SPI NOR flash. But
> > they were plug-able flashes, not something that is always tied to the
> > board. Microchip's sama7g5ek comes with a 128MB macronix SPI NOR flash
> > populated. But there are other vendors that provide large SPI NORs if
> > really needed.
> >  =20
>=20
> pressed sent too soon :). What I wanted to say is that it's not uncommon
> for vendors to populate large SPI NOR flashes, there are others as well,
> thus Miquel's concerns are valid. There may be people out there having
> rootfs on top of SPI NORs.

I don't think it is a too bad approach to store a rootfs on a spi-nor,
as long as your "regularly written" data files are somewhere else
(otherwise it's fine, but slow). It's actually nice to keep your boot
files and rootfs in a "safe" place whereas your app is stored somewhere
else, so users can mess around: they will not break the system.

Thanks,
Miqu=C3=A8l