Skip to content

Commit

Permalink
[POWERPC] PS3: Split device setup for static vs. dynamic devices
Browse files Browse the repository at this point in the history
Split the device setup code in ps3_register_repository_device() in two
routines:
  1. ps3_setup_static_device(), to handle the setup of static devices in the
     PS3 repository, which can be __init,
  2. ps3_setup_dynamic_device(), to handle the setup of storage devices that
     may appear later in the PS3 repository.

This fixes a few section mismatch warnings.

Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
Geert Uytterhoeven authored and Paul Mackerras committed Apr 1, 2008
1 parent fa90f70 commit b163a25
Showing 1 changed file with 46 additions and 32 deletions.
78 changes: 46 additions & 32 deletions arch/powerpc/platforms/ps3/device-init.c
Original file line number Diff line number Diff line change
Expand Up @@ -499,41 +499,14 @@ static int __init ps3_register_graphics_devices(void)
}

/**
* ps3_register_repository_device - Register a device from the repositiory info.
*
* ps3_setup_dynamic_device - Setup a dynamic device from the repository
*/

static int ps3_register_repository_device(
const struct ps3_repository_device *repo)
static int ps3_setup_dynamic_device(const struct ps3_repository_device *repo)
{
int result;

switch (repo->dev_type) {
case PS3_DEV_TYPE_SB_GELIC:
result = ps3_setup_gelic_device(repo);
if (result) {
pr_debug("%s:%d ps3_setup_gelic_device failed\n",
__func__, __LINE__);
}
break;
case PS3_DEV_TYPE_SB_USB:

/* Each USB device has both an EHCI and an OHCI HC */

result = ps3_setup_ehci_device(repo);

if (result) {
pr_debug("%s:%d ps3_setup_ehci_device failed\n",
__func__, __LINE__);
}

result = ps3_setup_ohci_device(repo);

if (result) {
pr_debug("%s:%d ps3_setup_ohci_device failed\n",
__func__, __LINE__);
}
break;
case PS3_DEV_TYPE_STOR_DISK:
result = ps3_setup_storage_dev(repo, PS3_MATCH_ID_STOR_DISK);

Expand Down Expand Up @@ -572,6 +545,48 @@ static int ps3_register_repository_device(
return result;
}

/**
* ps3_setup_static_device - Setup a static device from the repository
*/

static int __init ps3_setup_static_device(const struct ps3_repository_device *repo)
{
int result;

switch (repo->dev_type) {
case PS3_DEV_TYPE_SB_GELIC:
result = ps3_setup_gelic_device(repo);
if (result) {
pr_debug("%s:%d ps3_setup_gelic_device failed\n",
__func__, __LINE__);
}
break;
case PS3_DEV_TYPE_SB_USB:

/* Each USB device has both an EHCI and an OHCI HC */

result = ps3_setup_ehci_device(repo);

if (result) {
pr_debug("%s:%d ps3_setup_ehci_device failed\n",
__func__, __LINE__);
}

result = ps3_setup_ohci_device(repo);

if (result) {
pr_debug("%s:%d ps3_setup_ohci_device failed\n",
__func__, __LINE__);
}
break;

default:
return ps3_setup_dynamic_device(repo);
}

return result;
}

static void ps3_find_and_add_device(u64 bus_id, u64 dev_id)
{
struct ps3_repository_device repo;
Expand Down Expand Up @@ -601,7 +616,7 @@ static void ps3_find_and_add_device(u64 bus_id, u64 dev_id)
pr_debug("%s:%u: device %lu:%lu found after %u retries\n",
__func__, __LINE__, bus_id, dev_id, retries);

ps3_register_repository_device(&repo);
ps3_setup_dynamic_device(&repo);
return;
}

Expand Down Expand Up @@ -905,8 +920,7 @@ static int __init ps3_register_devices(void)

ps3_register_graphics_devices();

ps3_repository_find_devices(PS3_BUS_TYPE_SB,
ps3_register_repository_device);
ps3_repository_find_devices(PS3_BUS_TYPE_SB, ps3_setup_static_device);

ps3_register_sound_devices();

Expand Down

0 comments on commit b163a25

Please sign in to comment.