diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2017-09-22 05:46:45 +0200 |
---|---|---|
committer | Boris Brezillon <boris.brezillon@free-electrons.com> | 2017-09-22 09:04:32 +0200 |
commit | 8582a03e028f666d15acc651e0491c02941d13e7 (patch) | |
tree | 2a2b0a0413053437aeabc8912870c1947b8f4d82 /drivers/mtd/nand/denali_pci.c | |
parent | mtd: nand: denali: use more FIELD_PREP / FIELD_GET where appropriate (diff) | |
download | linux-8582a03e028f666d15acc651e0491c02941d13e7.tar.xz linux-8582a03e028f666d15acc651e0491c02941d13e7.zip |
mtd: nand: denali: clean up comments
This driver explains too much about what is apparent from the code.
Comments around basic APIs such as init_completion(), spin_lock_init(),
etc. seem unneeded lessons to kernel developers.
(With those comments dropped, denali_drv_init() is small enough,
so it has been merged into the probe function.)
Also, NAND driver developers should know the NAND init procedure, so
there is no need to explain nand_scan_ident/tail.
I removed FSF's address from the license blocks, and added simple
comments to struct members.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Diffstat (limited to 'drivers/mtd/nand/denali_pci.c')
-rw-r--r-- | drivers/mtd/nand/denali_pci.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/mtd/nand/denali_pci.c b/drivers/mtd/nand/denali_pci.c index 7d5600b6d07c..57fb7ae31412 100644 --- a/drivers/mtd/nand/denali_pci.c +++ b/drivers/mtd/nand/denali_pci.c @@ -109,7 +109,6 @@ failed_remap_reg: return ret; } -/* driver exit point */ static void denali_pci_remove(struct pci_dev *dev) { struct denali_nand_info *denali = pci_get_drvdata(dev); @@ -125,5 +124,4 @@ static struct pci_driver denali_pci_driver = { .probe = denali_pci_probe, .remove = denali_pci_remove, }; - module_pci_driver(denali_pci_driver); |