Skip to content

Commit

Permalink
powerpc: Move FW feature probing out of pseries probe()
Browse files Browse the repository at this point in the history
We move the function itself to pseries/firmware.c and call it along
with almost all other flat device-tree parsers from early_init_devtree()

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
[mpe: Move #ifdefs into the header by providing pseries_probe_fw_features()]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
  • Loading branch information
Benjamin Herrenschmidt authored and Michael Ellerman committed Jul 21, 2016
1 parent c40785a commit 3808a88
Show file tree
Hide file tree
Showing 5 changed files with 56 additions and 48 deletions.
6 changes: 6 additions & 0 deletions arch/powerpc/include/asm/firmware.h
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,12 @@ extern int fwnmi_active;

extern unsigned int __start___fw_ftr_fixup, __stop___fw_ftr_fixup;

#ifdef CONFIG_PPC_PSERIES
void pseries_probe_fw_features(void);
#else
static inline void pseries_probe_fw_features(void) { };
#endif

#endif /* __ASSEMBLY__ */
#endif /* __KERNEL__ */
#endif /* __ASM_POWERPC_FIRMWARE_H */
4 changes: 4 additions & 0 deletions arch/powerpc/kernel/prom.c
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@
#include <asm/fadump.h>
#include <asm/debug.h>
#include <asm/epapr_hcalls.h>
#include <asm/firmware.h>

#include <mm/mmu_decl.h>

Expand Down Expand Up @@ -755,6 +756,9 @@ void __init early_init_devtree(void *params)
#endif
epapr_paravirt_early_init();

/* Now try to figure out if we are running on LPAR and so on */
pseries_probe_fw_features();

DBG(" <- early_init_devtree()\n");
}

Expand Down
48 changes: 46 additions & 2 deletions arch/powerpc/platforms/pseries/firmware.c
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
*/


#include <linux/of_fdt.h>
#include <asm/firmware.h>
#include <asm/prom.h>
#include <asm/udbg.h>
Expand Down Expand Up @@ -69,7 +70,8 @@ hypertas_fw_features_table[] = {
* device-tree/ibm,hypertas-functions. Ultimately this functionality may
* be moved into prom.c prom_init().
*/
void __init fw_hypertas_feature_init(const char *hypertas, unsigned long len)
static void __init fw_hypertas_feature_init(const char *hypertas,
unsigned long len)
{
const char *s;
int i;
Expand Down Expand Up @@ -113,7 +115,7 @@ vec5_fw_features_table[] = {
{FW_FEATURE_PRRN, OV5_PRRN},
};

void __init fw_vec5_feature_init(const char *vec5, unsigned long len)
static void __init fw_vec5_feature_init(const char *vec5, unsigned long len)
{
unsigned int index, feat;
int i;
Expand All @@ -131,3 +133,45 @@ void __init fw_vec5_feature_init(const char *vec5, unsigned long len)

pr_debug(" <- fw_vec5_feature_init()\n");
}

/*
* Called very early, MMU is off, device-tree isn't unflattened
*/
static int __init probe_fw_features(unsigned long node, const char *uname, int
depth, void *data)
{
const char *prop;
int len;
static int hypertas_found;
static int vec5_found;

if (depth != 1)
return 0;

if (!strcmp(uname, "rtas") || !strcmp(uname, "rtas@0")) {
prop = of_get_flat_dt_prop(node, "ibm,hypertas-functions",
&len);
if (prop) {
powerpc_firmware_features |= FW_FEATURE_LPAR;
fw_hypertas_feature_init(prop, len);
}

hypertas_found = 1;
}

if (!strcmp(uname, "chosen")) {
prop = of_get_flat_dt_prop(node, "ibm,architecture-vec-5",
&len);
if (prop)
fw_vec5_feature_init(prop, len);

vec5_found = 1;
}

return hypertas_found && vec5_found;
}

void __init pseries_probe_fw_features(void)
{
of_scan_flat_dt(probe_fw_features, NULL);
}
5 changes: 0 additions & 5 deletions arch/powerpc/platforms/pseries/pseries.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,6 @@ extern void request_event_sources_irqs(struct device_node *np,

#include <linux/of.h>

extern void __init fw_hypertas_feature_init(const char *hypertas,
unsigned long len);
extern void __init fw_vec5_feature_init(const char *hypertas,
unsigned long len);

struct pt_regs;

extern int pSeries_system_reset_exception(struct pt_regs *regs);
Expand Down
41 changes: 0 additions & 41 deletions arch/powerpc/platforms/pseries/setup.c
Original file line number Diff line number Diff line change
Expand Up @@ -659,45 +659,6 @@ static void pseries_power_off(void)
for (;;);
}

/*
* Called very early, MMU is off, device-tree isn't unflattened
*/

static int __init pseries_probe_fw_features(unsigned long node,
const char *uname, int depth,
void *data)
{
const char *prop;
int len;
static int hypertas_found;
static int vec5_found;

if (depth != 1)
return 0;

if (!strcmp(uname, "rtas") || !strcmp(uname, "rtas@0")) {
prop = of_get_flat_dt_prop(node, "ibm,hypertas-functions",
&len);
if (prop) {
powerpc_firmware_features |= FW_FEATURE_LPAR;
fw_hypertas_feature_init(prop, len);
}

hypertas_found = 1;
}

if (!strcmp(uname, "chosen")) {
prop = of_get_flat_dt_prop(node, "ibm,architecture-vec-5",
&len);
if (prop)
fw_vec5_feature_init(prop, len);

vec5_found = 1;
}

return hypertas_found && vec5_found;
}

static int __init pSeries_probe(void)
{
unsigned long root = of_get_flat_dt_root();
Expand All @@ -717,8 +678,6 @@ static int __init pSeries_probe(void)

pr_debug("pSeries detected, looking for LPAR capability...\n");

/* Now try to figure out if we are running on LPAR */
of_scan_flat_dt(pseries_probe_fw_features, NULL);

#ifdef __LITTLE_ENDIAN__
if (firmware_has_feature(FW_FEATURE_SET_MODE)) {
Expand Down

0 comments on commit 3808a88

Please sign in to comment.