Skip to contentSkip to conversation

MIPS: ingenic: remove unused platform_data header file

PATCH Applied Sam Ravnborg 3 participants 4 messages Oct 26, 2020 – Oct 27, 2020 MIPS

lore
Share & tools
lore

Sam Ravnborg

author
#0 lore raw

PATCH 1/1 MIPS: ingenic: remove unused platform_data header file

There are no users of this headers file in the kernel. All users are likely migrated to device tree which is a good thing.

Signed-off-by Sam Ravnborg <[email protected]>
Cc Thomas Bogendoerfer <[email protected]>
Cc Paul Cercueil <[email protected]>
Cc Harvey Hunt <[email protected]>
Cc [email protected]

---

I noticed this while surfing around in platform_data for no apperant reason. So no fancy tooling or some such involved.

Sam

 .../linux/platform_data/jz4740/jz4740_nand.h  | 25 -------------------
 1 file changed, 25 deletions(-)
 delete mode 100644 include/linux/platform_data/jz4740/jz4740_nand.h
include/linux/platform_data/jz4740/jz4740_nand.h deleted +0 -25
deleted file mode 100644
index b3f066d63059..000000000000
@@ -1,25 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/*
- *  Copyright (C) 2009-2010, Lars-Peter Clausen <[email protected]>
- *  JZ4740 SoC NAND controller driver
- */
-
-#ifndef __JZ4740_NAND_H__
-#define __JZ4740_NAND_H__
-
-#include <linux/mtd/rawnand.h>
-#include <linux/mtd/partitions.h>
-
-#define JZ_NAND_NUM_BANKS 4
-
-struct jz_nand_platform_data {
-	int			num_partitions;
-	struct mtd_partition	*partitions;
-
-	unsigned char banks[JZ_NAND_NUM_BANKS];
-
-	void (*ident_callback)(struct platform_device *, struct mtd_info *,
-				struct mtd_partition **, int *num_partitions);
-};
-
-#endif
Signature · 1 line
2.27.0
Reviewed-by on v1 1/1
#1 lore raw

Hi,

Quoted 8 lines · Sam Ravnborg

There are no users of this headers file in the kernel. All users are likely migrated to device tree which is a good thing.

Signed-off-by Sam Ravnborg <[email protected]>
Cc Thomas Bogendoerfer <[email protected]>
Cc Paul Cercueil <[email protected]>
Cc Harvey Hunt <[email protected]>
Cc [email protected]
Quoted 4 lines

---

I noticed this while surfing around in platform_data for no apperant reason. So no fancy tooling or some such involved.

The jz4740-nand driver has been removed in Linux 5.3. Therefore:

Reviewed-by Paul Cercueil <[email protected]>

Cheers,
-Paul

Quoted 42 lines

Sam

 .../linux/platform_data/jz4740/jz4740_nand.h  | 25 
-------------------
 1 file changed, 25 deletions(-)
 delete mode 100644 include/linux/platform_data/jz4740/jz4740_nand.h
include/linux/platform_data/jz4740/jz4740_nand.h deleted +0 -26
deleted file mode 100644
index b3f066d63059..000000000000
@@ -1,25 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/*
- *  Copyright (C) 2009-2010, Lars-Peter Clausen <[email protected]>
- *  JZ4740 SoC NAND controller driver
- */
-
-#ifndef __JZ4740_NAND_H__
-#define __JZ4740_NAND_H__
-
-#include <linux/mtd/rawnand.h>
-#include <linux/mtd/partitions.h>
-
-#define JZ_NAND_NUM_BANKS 4
-
-struct jz_nand_platform_data {
-	int			num_partitions;
-	struct mtd_partition	*partitions;
-
-	unsigned char banks[JZ_NAND_NUM_BANKS];
-
-	void (*ident_callback)(struct platform_device *, struct mtd_info *,
-				struct mtd_partition **, int *num_partitions);
-};
-
-#endif
--

2.27.0

maintainer Applied
#2 lore raw
In reply to Sam Ravnborg
Quoted 10 lines · Sam Ravnborg

There are no users of this headers file in the kernel. All users are likely migrated to device tree which is a good thing.

Signed-off-by Sam Ravnborg <[email protected]>
Cc Thomas Bogendoerfer <[email protected]>
Cc Paul Cercueil <[email protected]>
Cc Harvey Hunt <[email protected]>
Cc [email protected]

---

Quoted 8 lines

I noticed this while surfing around in platform_data for no apperant reason. So no fancy tooling or some such involved.

Sam

 .../linux/platform_data/jz4740/jz4740_nand.h  | 25 -------------------
 1 file changed, 25 deletions(-)
 delete mode 100644 include/linux/platform_data/jz4740/jz4740_nand.h

applied to mips-next.

Thomas.

Signature · 2 lines
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]
author
#3 lore raw

Hi Thomas,

Quoted 14 lines · Thomas Bogendoerfer
Quoted 13 lines · Sam Ravnborg

There are no users of this headers file in the kernel. All users are likely migrated to device tree which is a good thing.

Signed-off-by Sam Ravnborg <[email protected]>
Cc Thomas Bogendoerfer <[email protected]>
Cc Paul Cercueil <[email protected]>
Cc Harvey Hunt <[email protected]>
Cc [email protected]

---

I noticed this while surfing around in platform_data for no apperant reason. So no fancy tooling or some such involved.

Quoted 7 lines
Quoted 5 lines

Sam

 .../linux/platform_data/jz4740/jz4740_nand.h  | 25 -------------------
 1 file changed, 25 deletions(-)
 delete mode 100644 include/linux/platform_data/jz4740/jz4740_nand.h

applied to mips-next.

Thanks for the quick response. Hopefully I can contribute something a little more useful next time :-)

But then cleaning out old cruft is never a waste of time.

Sam