Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1487976

[PATCH v2 32/46] mtd: nandsim: Remove NS_IS_INITIALIZED

From Daniel Walter <dwalter@sigma-star.at>
Newsgroups linux.kernel
Subject [PATCH v2 32/46] mtd: nandsim: Remove NS_IS_INITIALIZED
Date 2016-09-21 12:00 +0200
Message-ID <sjMtP-7lX-25@gated-at.bofh.it> (permalink)
References <sjMk9-7il-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Richard Weinberger <richard@nod.at>

init_nandsim() has exactly one caller, the construction
function. So, checking for this condition makes no sense.

Signed-off-by: Richard Weinberger <richard@nod.at>
---
 drivers/mtd/nand/nandsim.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/drivers/mtd/nand/nandsim.c b/drivers/mtd/nand/nandsim.c
index 915e3bb..971f7b4 100644
--- a/drivers/mtd/nand/nandsim.c
+++ b/drivers/mtd/nand/nandsim.c
@@ -185,9 +185,6 @@ MODULE_PARM_DESC(defaults,	 "Register a MTD during module load using default val
 #define NS_MDELAY(ns, us) \
 	do { if (ns->do_delays) mdelay(us); } while (0)
 
-/* Is the nandsim structure initialized ? */
-#define NS_IS_INITIALIZED(ns) ((ns)->geom.totsz != 0)
-
 /* Good operation completion status */
 #define NS_STATUS_OK(ns) (NAND_STATUS_READY | (NAND_STATUS_WP * ((ns)->lines.wp == 0)))
 
@@ -839,11 +836,6 @@ static int init_nandsim(struct mtd_info *mtd, struct nandsim_params *nsparam)
 	uint64_t remains;
 	uint64_t next_offset;
 
-	if (NS_IS_INITIALIZED(ns)) {
-		pr_err("init_nandsim: nandsim is already initialized\n");
-		return -EIO;
-	}
-
 	/* Force mtd to not do delays */
 	chip->chip_delay = 0;
 
-- 
2.8.3

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 00/46] Nandsim facelift (part I of II) Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 13/46] mtd: nandsim: Add final logic for multiple instances Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 12/46] mtd: nandsim: Make debugfs logic multi instance capable Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 09/46] mtd: nandsim: Don't directly use module parameters Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 08/46] mtd: nandsim: Kill global nsmtd Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 04/46] mtd: Don't unconditionally execute remove notifiers Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 05/46] mtd: Don't print a scary message when trying to remove a busy MTD Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to free NAND related resources Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
    Re: [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to  free NAND related resources Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-21 12:00 +0200
    Re: [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to  free NAND related resources kbuild test robot <lkp@intel.com> - 2016-09-21 14:50 +0200
    Re: [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to  free NAND related resources Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-21 16:30 +0200
      Re: [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to  free NAND related resources Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 16:50 +0200
        Re: [PATCH v2 02/46] mtd: nand: Provide nand_cleanup() function to  free NAND related resources Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-21 16:50 +0200
  [PATCH v2 01/46] mtdpart: Propagate _get/put_device() Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
    Re: [PATCH v2 01/46] mtdpart: Propagate _get/put_device() Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-21 12:20 +0200
  [PATCH v2 14/46] mtd: nandsim: Add simulator id to MTD parition name Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 07/46] mtd: nandsim: Begin with removal of global state Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 15/46] mtd: nandsim: Introduce backend operations Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 10/46] mtd: nandsim: Add helper functions for pointer magic Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 11/46] mtd: nandsim: Factor out nandsim parameters Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
  [PATCH v2 03/46] mtd: Don't unconditionally unregister reboot notifier Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 11:50 +0200
    Re: [PATCH v2 03/46] mtd: Don't unconditionally unregister reboot  notifier Boris Brezillon <boris.brezillon@free-electrons.com> - 2016-09-21 16:40 +0200
      Re: [PATCH v2 03/46] mtd: Don't unconditionally unregister reboot  notifier Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 16:40 +0200
  [PATCH v2 24/46] mtd: nandsim: Wire up NANDSIM_MODE_CACHEFILE ioctl mode Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 38/46] mtd: nandsim: Expose partition creation logic to user space Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 31/46] mtd: nandsim: Remove NS_RAW_OFFSET_OOB Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 35/46] mtd: nandsim: Make NANDSIM_MAX_DEVICES part of uapi Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 40/46] mtd: nandsim: Expose BBT, delays, etc.. to userspace Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 22/46] mtd: nandsim: Cleanup destroy handlers Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 33/46] mtd: nandsim: Relax page size restrictions Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 32/46] mtd: nandsim: Remove NS_IS_INITIALIZED Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 45/46] mtd: nandsim: Always answer all 8 bytes from NAND_CMD_READID Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 17/46] mtd: nandsim: Allow external backends Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 44/46] mtd: nandsim: Wire up NANDSIM_IOC_DESTROY_INSTANCE Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 21/46] mtd: nandsim: Implement preliminary destructor function Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
    Re: [PATCH v2 21/46] mtd: nandsim: Implement preliminary destructor  function kbuild test robot <lkp@intel.com> - 2016-09-21 15:00 +0200
  [PATCH v2 29/46] um: Add nandsim backend driver Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 39/46] mtd: nandsim: Rework init error paths Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 28/46] mtd: nandsim: Refine exports Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 26/46] mtd: nandsim: use the existing output macros Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 36/46] mtd: nandsim: Cleanup constants Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 16/46] mtd: nandsim: Print error when backend init failed Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 25/46] mtd: nandsim: Print backend name Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 23/46] mtd: nandsim: Unify file backend init logic Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 34/46] mtd: nandsim: Support bitflip and read error emulation in file backend Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 43/46] mtd: nandsim: Wire up NANDSIM_IOC_NEW_INSTANCE Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 41/46] mtd: nandsim: Expose support for weakpages/blocks to userspace Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 46/46] mtd/nandsim: Add ioctl for info Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 19/46] mtd: nandsim: UAPI v1 Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 20/46] mtd: nandsim: Implement preliminary constructor function Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 18/46] mtd: nandsim: Add basic support for a file backend Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 37/46] mtd: nandsim: Turn parts[] into a integer Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 27/46] mtd: nandsim: Add no_oob mode Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200
  [PATCH v2 42/46] mtd: nandsim: Don't printk on ENOMEM Daniel Walter <dwalter@sigma-star.at> - 2016-09-21 12:00 +0200

csiph-web