Re: [PATCH] mtdblock: warn if opened on NAND

From: Miquel Raynal <miquel.raynal@bootlin.com>
To: "Bjørn Mork" <bjorn@mork.no>
Cc: <Tudor.Ambarus@microchip.com>, <ezequiel@vanguardiasur.com.ar>,
	<richard@nod.at>, <vigneshr@ti.com>,
	<linux-mtd@lists.infradead.org>
Subject: Re: [PATCH] mtdblock: warn if opened on NAND
Date: Fri, 22 Apr 2022 11:03:51 +0200	[thread overview]
Message-ID: <20220422110351.6b5cd370@xps13> (raw)
In-Reply-To: <87wnfhmxlo.fsf@miraculix.mork.no>

Hi folks,

bjorn@mork.no wrote on Fri, 22 Apr 2022 08:38:27 +0200:

> <Tudor.Ambarus@microchip.com> writes:
> 
> >> Commit e07403a8c6be is based on two false assumptions:
> >> 1) the presence of the mtdblock driver is a problem  
> >
> > It is a problem. Using it will wear out parts of your raw flash relatively
> > fast.  
> 
> There is a significant difference between PRESENCE and USING.
> 
> I'll just stop this now. You're not arguing with me but woith some
> straw man you created.

Right now there is a message for _every_ mtd device that gets created
only if mtdblock support is built-in. Of course enabling mtdblock if
you don't use it is a waste of space, but it is far less an issue than
people actually using it and I think that is our main target.

The warnings from Ezequiel were a good idea but right now I think we
all agree that they are too frequent. A single warning would be an
improvement, but otherwise a warning only when ->open() gets called
seems a good enough step forward for me.

Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/