Revert "PCI: brcmstb: Split brcm_pcie_setup() into two funcs"

Revert "PCI: brcmstb: Split brcm_pcie_setup() into two funcs"

This reverts commit 1c2e8d65dc68f9ea2941a586aa2f2f3b685ebbb4.

This is part of a revert of the following commits:

  7cfa1839264c ("PCI: brcmstb: Do not turn off WOL regulators on suspend")
  b933a4eb3cd3 ("PCI: brcmstb: Add control of subdevice voltage regulators")
  79159d14a73e ("PCI: brcmstb: Add mechanism to turn on subdev regulators")
  1c2e8d65dc68 ("PCI: brcmstb: Split brcm_pcie_setup() into two funcs")

Cyril reported that 1c2e8d65dc68 ("PCI: brcmstb: Split brcm_pcie_setup()
into two funcs"), which appeared in v5.17-rc1, broke booting on the
Raspberry Pi Compute Module 4.  Apparently 1c2e8d65dc68 panics with an
Asynchronous SError Interrupt, and after further commits here is a black
screen on HDMI and no output on the serial console.

This does not seem to affect the Raspberry Pi 4 B.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=215925
Link: https://lore.kernel.org/r/20220511201856.808690-5-helgaas@kernel.org
Reported-by: Cyril Brulebois <kibi@debian.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>