spapr, spapr_nvdimm: fold NVDIMM validation in the same place
commitbeb6073fe7f53526bc49f9cb1a3a830c2cdaf3e7
authorDaniel Henrique Barboza <danielhb413@gmail.com>
Tue, 25 Aug 2020 21:57:48 +0000 (25 18:57 -0300)
committerDavid Gibson <david@gibson.dropbear.id.au>
Tue, 8 Sep 2020 00:08:42 +0000 (8 10:08 +1000)
tree78136e2dcbe5cfceac1d0eda636a3dd726c02ade
parent90d282d0858cf5a38f3e8a7e201aeab2a0ccbe88
spapr, spapr_nvdimm: fold NVDIMM validation in the same place

NVDIMM has different contraints and conditions than the regular
DIMM and we'll need to add at least one more.

Instead of relying on 'if (nvdimm)' conditionals in the body of
spapr_memory_pre_plug(), use the existing spapr_nvdimm_validate_opts()
and put all NVDIMM handling code there. Rename it to
spapr_nvdimm_validate() to reflect that the function is now checking
more than the nvdimm device options. This makes spapr_memory_pre_plug()
a bit easier to follow, and we can tune in NVDIMM parameters
and validation in the same place.

Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Message-Id: <20200825215749.213536-3-danielhb413@gmail.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
hw/ppc/spapr.c
hw/ppc/spapr_nvdimm.c
include/hw/ppc/spapr_nvdimm.h