diff options
| author | Enrico Weigelt <info@metux.net> | 2020-12-15 17:35:31 +0100 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2020-12-15 21:47:45 -0800 |
| commit | 5ae96d779ff3eeb2977919ff311a6c8849943c2d (patch) | |
| tree | 02b49934c388664bd97bab1f47261fe728bdaaf9 /drivers/nvdimm/btt.h | |
| parent | 9a7e3d7f056831a6193d6d737fb7a26dfdceb04b (diff) | |
| download | linux-5ae96d779ff3eeb2977919ff311a6c8849943c2d.tar.gz linux-5ae96d779ff3eeb2977919ff311a6c8849943c2d.tar.bz2 linux-5ae96d779ff3eeb2977919ff311a6c8849943c2d.zip | |
libnvdimm: Cleanup include of badblocks.h
* drivers/nvdimm/core.c doesn't use anything from badblocks.h on its own,
thus including it isn't needed. There's indeed indirect use, via funcs
in nd.h, but this one already includes badblocks.h.
* drivers/nvdimm/claim.c calls stuff from badblocks.h and therefore should
include it on its own (instead of relying any other header doing that)
* drivers/nvdimm/btt.h doesn't really need anything from badblocks.h and
can easily live with a forward declaration of struct badblocks (just
having pointers to it, but not dereferencing it anywhere)
Signed-off-by: Enrico Weigelt <info@metux.net>
Link: https://lore.kernel.org/r/20201215163531.21446-1-info@metux.net
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/nvdimm/btt.h')
| -rw-r--r-- | drivers/nvdimm/btt.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/nvdimm/btt.h b/drivers/nvdimm/btt.h index 2e258bee7db2..aa53e0b769bd 100644 --- a/drivers/nvdimm/btt.h +++ b/drivers/nvdimm/btt.h @@ -7,7 +7,6 @@ #ifndef _LINUX_BTT_H #define _LINUX_BTT_H -#include <linux/badblocks.h> #include <linux/types.h> #define BTT_SIG_LEN 16 @@ -197,6 +196,8 @@ struct arena_info { int log_index[2]; }; +struct badblocks; + /** * struct btt - handle for a BTT instance * @btt_disk: Pointer to the gendisk for BTT device |