Skip to content

Commit

Permalink
phb3/4: Move IO VPD preload out to a common place
Browse files Browse the repository at this point in the history
The code is duplicated between phb3 and phb4 for no reason

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Michael Neuling <mikey@neuling.org>

# Conflicts:
#	core/init.c
#	hw/phb3.c
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
  • Loading branch information
ozbenh authored and stewartsmith committed Jun 26, 2017
1 parent ead3caf commit 077fdef
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 24 deletions.
2 changes: 1 addition & 1 deletion core/init.c
Original file line number Diff line number Diff line change
Expand Up @@ -992,7 +992,7 @@ void __noreturn __nomcount main_cpu_entry(const void *fdt)

pci_nvram_init();

phb3_preload_vpd();
preload_io_vpd();
preload_capp_ucode();
start_preload_kernel();

Expand Down
11 changes: 11 additions & 0 deletions core/vpd.c
Original file line number Diff line number Diff line change
Expand Up @@ -272,3 +272,14 @@ void vpd_preload(struct dt_node *hub_node)

vpd_lid = vpd_lid_preload(lxr);
}

void preload_io_vpd(void)
{
const struct dt_property *prop;

prop = dt_find_property(dt_root, "ibm,io-vpd");
if (!prop) {
/* LX VPD Lid not already loaded */
vpd_preload(dt_root);
}
}
10 changes: 0 additions & 10 deletions hw/phb3.c
Original file line number Diff line number Diff line change
Expand Up @@ -4934,16 +4934,6 @@ static void phb3_probe_pbcq(struct dt_node *pbcq)
add_chip_dev_associativity(np);
}

void phb3_preload_vpd(void)
{
const struct dt_property *prop;

prop = dt_find_property(dt_root, "ibm,io-vpd");
if (!prop) {
/* LX VPD Lid not already loaded */
vpd_preload(dt_root);
}
}

void probe_phb3(void)
{
Expand Down
11 changes: 0 additions & 11 deletions hw/phb4.c
Original file line number Diff line number Diff line change
Expand Up @@ -4356,17 +4356,6 @@ static void phb4_probe_pbcq(struct dt_node *pbcq)
}
}

void phb4_preload_vpd(void)
{
const struct dt_property *prop;

prop = dt_find_property(dt_root, "ibm,io-vpd");
if (!prop) {
/* LX VPD Lid not already loaded */
vpd_preload(dt_root);
}
}

void probe_phb4(void)
{
struct dt_node *np;
Expand Down
3 changes: 1 addition & 2 deletions include/skiboot.h
Original file line number Diff line number Diff line change
Expand Up @@ -214,9 +214,8 @@ extern void setup_reset_vector(void);
extern void probe_p7ioc(void);
extern void probe_phb3(void);
extern void probe_phb4(void);
extern void phb3_preload_vpd(void);
extern int preload_capp_ucode(void);
extern void phb4_preload_vpd(void);
extern void preload_io_vpd(void);
extern void probe_npu(void);
extern void probe_npu2(void);
extern void uart_init(void);
Expand Down

0 comments on commit 077fdef

Please sign in to comment.