From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Date: Thu, 16 Aug 2018 17:50:59 +0200 Subject: [U-Boot] [PATCH v6 22/31] cmd: ubi: delete useless and misleading definitions In-Reply-To: <20180816153029.15521-23-miquel.raynal@bootlin.com> References: <20180816153029.15521-1-miquel.raynal@bootlin.com> <20180816153029.15521-23-miquel.raynal@bootlin.com> Message-ID: <20180816175059.75e782c7@bbrezillon> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, 16 Aug 2018 17:30:20 +0200 Miquel Raynal wrote: > These definitions are simply not used and are misleading because similar > definitions exist in jffs2/load_kernel.h and are used widely to define > MTD device types (which is, by the way, totally redundant with what the > MTD core does). Remove these definitions. > > Signed-off-by: Miquel Raynal > Acked-by: Jagan Teki Reviewed-by: Boris Brezillon > --- > cmd/ubi.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/cmd/ubi.c b/cmd/ubi.c > index 913f0f71fd..0a3405a3b1 100644 > --- a/cmd/ubi.c > +++ b/cmd/ubi.c > @@ -27,11 +27,6 @@ > #undef ubi_msg > #define ubi_msg(fmt, ...) printf("UBI: " fmt "\n", ##__VA_ARGS__) > > -#define DEV_TYPE_NONE 0 > -#define DEV_TYPE_NAND 1 > -#define DEV_TYPE_ONENAND 2 > -#define DEV_TYPE_NOR 3 > - > /* Private own data */ > static struct ubi_device *ubi; > static char buffer[80];