Add Power Service #82

Felix-Dev · 2021-02-17T18:06:10Z

Description

This PR adds the fully reverse-engineered Power Service (power_0x.prx) to uofw.

Except for the PSP Go (PSP N-1000), all PSP series should be supported. We currently don't have support for the PSP Go model due to a missing reverse engineering of the PSP Go's exclusive hibernation feature. I don't have any plans right now to add that support, so if anyone wants to take a look at that, feel free to go ahead :)

Highlights

Power appears to work! (Check below for more info on how it was tested)
A full public API documentation!
Plenty of additional comments throughout the codebase explaining the power service implemention.

Motivation and Context

Closes #39.

How Has This Been Tested?

The module has been compiled and tested successfully on a PSP-1000. Testing included the following:

Verify that the PSP still boots and runs correctly (XMB & game)
Verify that the PSP suspends and resumes correctly
Verify that power saving features like dimming/turning off the PSP display work
Verify that battery data is displayed correctly in the system

Specific APIs have not been tested and as such might contain bugs. However, the overall impression after testing is that our power module can be run on a PSP device without causing any major issues.

Additional Context

Power service can be compiled using the archaic PSPSDK 0.11 release as well as the most recent PSPSDK as part of the PSP toolchain. Once we've updated the toolchain version used in our CI, support for PSPSDK 0.11 can be removed from power.

Note that if you want to run the power module on your PSP using the UOFW installer, make sure to comment out its patching of the power service to make sure no issue is introduced by running power through the UOFW installer.

Final Notes

I had started working on power all the way back in early 2013 and then stopped soon after that...until starting to work on it again since the beginning of this year. If it weren't for the amazing PSP community on discord, I doubt I would have ever gathered the necessary motivation and drive to finally bring power over the finish line. Many thanks to the PSP community 👍

* Added bunch of missing functions left to reverse engineer.

…ure areas in greater detail.

…] struct. Moved some macros to syscon.h as a result. The size of the struct was also incorrect previously (too large), this has been fixed as part of this change.

…e CI.

John-K

This is fantastic work, @Felix-Dev! I can imagine some of it must have been quite tedious!

John-K · 2021-03-03T04:26:13Z

include/power_error.h

+#define	POWER_ERROR_H
+
+/* This constant represents an unknown power service error (related to the PSP system chip's temperature). */
+#define SCE_POWER_ERROR_0010					0x802B0010


Should this file be a part of / be included from include/common/errors.h?

John-K · 2021-03-03T04:30:57Z

include/power_user.h

+ * This constant specifies that a search for a slot number from among the empty slots is to be automatically performed
+ * when a callback is registered with the ::scePowerRegisterCallback() function.
+ */
+#define SCE_POWER_CALLBACKSLOT_AUTO					(-1)


It seems like there is a lot of duplication with power_kernel.h can we re-factor this out into power_common.h?

John-K · 2021-03-03T04:38:06Z

src/power/battery.c

+ * voltage, charge cycle,...). Internally, we set up a worker thread which periodically polls the battery to update
+ * the collected battery properties.
+ * 
+ * Starting with the PSP N-1000 (PSP-Go) serie, the battery equipped by default no longer has hardware included to


Suggested change

* Starting with the PSP N-1000 (PSP-Go) serie, the battery equipped by default no longer has hardware included to

* Starting with the PSP N-1000 (PSP-Go) series, the battery equipped by default no longer has hardware included to

John-K · 2021-03-03T04:39:37Z

src/power/battery.c

+     * At the time of power service termination, there might be an active [permit battery charging] request
+     * in our battery manager. This request might currently be lined up in our battery worker thread for
+     * execution or scheduled to happen after some delay time. However, as we are in the process of shutting
+     * down the kernel (anf thus our battery worker thread) we will removed any scheduled/lined up work here 


Suggested change

* down the kernel (anf thus our battery worker thread) we will removed any scheduled/lined up work here

* down the kernel (and thus our battery worker thread) we will remove any scheduled/lined up work here

John-K · 2021-04-13T19:49:18Z

include/syscon.h

+#define SCE_SYSCON_BARYON_STATUS_ALARM              (1 << 3)
+#define SCE_SYSCON_BARYON_STATUS_POWER_SW_ON        (1 << 4)
+#define SCE_SYSCON_BARYON_STATUS_IS_LOW_BATTERY_01G (1 << 5)
+#define SCE_SYSCON_BARYON_STATUS_HDD_POWER          (1 << 18)


Is this a typo? sceSysconGetBaryonStatus returns a u8, so (1 << 7) would be the max possible

John-K · 2021-04-14T02:46:11Z

src/power/clockFrequency.c

+ *      - Scratchpad memory
+ *      - Graphic Engine and the Graphic Engine's eDRAM
+ *      - DMA transfers for general-purpose memory copying
+ *      - DMA transfers for the Memory Stic, WLAN, USB, LCD, or audio


Suggested change

* - DMA transfers for the Memory Stic, WLAN, USB, LCD, or audio

* - DMA transfers for the Memory Stick, WLAN, USB, LCD, or audio

John-K · 2021-04-14T02:46:30Z

src/power/clockFrequency.c

+ *      - Graphic Engine and the Graphic Engine's eDRAM
+ *      - DMA transfers for general-purpose memory copying
+ *      - DMA transfers for the Memory Stic, WLAN, USB, LCD, or audio
+ *      - DMA transfers for transfers to the MediaEngine


Suggested change

* - DMA transfers for transfers to the MediaEngine

* - DMA transfers to the MediaEngine

John-K · 2021-04-14T03:02:07Z

src/power/power.c

+
+    u8 baryonVersionMajor = PSP_SYSCON_BARYON_GET_VERSION_MAJOR(g_Power.baryonVersion);
+    u8 baryonVersionMinor = PSP_SYSCON_BARYON_GET_VERSION_MINOR(g_Power.baryonVersion);
+    if (baryonVersionMajor == 0x2 


This could be greatly simplified, but it seems like we're trying to match Sony's implementation which must have had the optimizer turned off or something.

John-K · 2021-04-14T03:05:17Z

src/power/power.c

+}
+
+// scePower_DB9D28DD
+s32 scePowerUnregitserCallback(s32 slot) __attribute__((alias("scePowerUnregisterCallback")));


Nicely handled

John-K · 2021-04-14T03:09:36Z

src/power/powerSwitch.c

+ * used to contain a copy of the on-board eDRAM content as the 4MB eDRAM memory is volatile when the system is
+ * suspended. When the PSP resumes, this saved content is then written back to the on-board eDRAM. 


Suggested change

* used to contain a copy of the on-board eDRAM content as the 4MB eDRAM memory is volatile when the system is

* suspended. When the PSP resumes, this saved content is then written back to the on-board eDRAM.

* used to contain a copy of the on-board eDRAM content as the 4MB eDRAM memory is volatile and lost when

* the system is suspended. When the PSP resumes, this saved content is then written back to the on-board eDRAM.

fjtrujy · 2022-05-19T05:52:24Z

I saw some .a files added to the PR, is that right?

joel16 · 2022-05-19T13:20:12Z

I saw some .a files added to the PR, is that right?

Yes these are stub libraries required to build the module.

Linblow · 2024-01-27T21:34:51Z

Why is this not merged yet?

joel16 · 2024-01-28T15:16:00Z

Why is this not merged yet?

Well, it can't now because it has merge conflicts. Aything in this PR under the "src" directory needs to be moved to "src/kd/".

artart78 · 2024-01-28T15:23:17Z

Why is this not merged yet?

The stuff asked by Hedge a few years ago has not been fixed yet. Feel free to do it if you want. It could be ignored and merged anyway but I feel it'd be a waste to discard the remarks.

Well, it can't now because it has merge conflicts. Aything in this PR under the "src" directory needs to be moved to "src/kd/".

This is trivial to rebase anyway, that's not an issue.

Felix-Dev added 30 commits March 21, 2013 23:34

Added scePowerSetClock** functions and _scePowerBatteryEnd().

e99eb72

fix merge conflicts

b518c27

Add exports file (not yet complete).

6aa13e0

Fix error in _scePowerBatteryEnd().

27c428b

Added _scePowerBatterySuspend().

07b86b8

Add _scePowerBatteryUpdatePhase().

b8068ed

* Fix error in _scePowerBatteryUpdatePhase().

809abae

* Added bunch of missing functions left to reverse engineer.

Add two more missing functions.

85c2fc7

Add scePowerGetInnerTemp().

e6dbdb3

Now using error definitions in include power_error.h.

a8c0bb3

Add _scePowerBatterySysconCmdIntr().

321f947

Add _scePowerBatteryDelayedPermitCharging().

4191584

Add scePowerGetBatteryVolt().

31e6f75

Add scePowerGetBatteryLifePercent().

18eee51

Some smaller changes.

7a46e7e

Add scePowerGetBatteryFullCapacity().

1010b70

Add scePowerGetBatteryType().

d494180

Added scePowerIsLowBattery().

5c51297

Add scePowerIsBatteryCharging().

583b9df

Add scePowerIsPowerOnline() and scePowerIsBatteryExist().

0abfda6

Add scePowerGetLowBatteryCapacity().

94382e0

Add scePowerGetForceSuspendCapacity().

b4f78b8

Add scePowerBatteryUpdateInfo().

e83979c

Use updated waitmode defines for sceKernelPollEventFlag() calls.

38d7569

Add scePowerGetBatteryChargeCycle().

c54241a

Add scePowerGetBatteryElec().

2f04aa6

Add scePowerGetBatteryTemp().

80a0c23

Add scePowerGetBatteryLifeTime().

7a5dbf9

Add scePowerGetBatteryRemainCapacity().

7396fd4

Add scePowerIsSuspendRequired().

0f716c9

Felix-Dev added 14 commits February 16, 2021 18:11

CI testing (2)...

3e5f03f

Add comment explaining why two pragma statements are needed for now.

33c614b

Add more comments explaining the power service and its different feat…

e29d91a

…ure areas in greater detail.

Remove duplicate space char.

60fe0f8

Replace [SceSysEventResumePowerState] struct with [ScePowerResumeInfo…

91b6421

…] struct. Moved some macros to syscon.h as a result. The size of the struct was also incorrect previously (too large), this has been fixed as part of this change.

Add missing space characters.

a2d1344

Merge branch 'master' into power

f04be24

Small layout fix.

220a673

Use real names for two imported APIs.

6c0b336

Fix merge conflicts.

3eaa990

Regenerate sysreg static lib after bug was fixed in export.exp file.

541b24a

Add parentheses for macro argument.

1d262b6

Merge branch 'master' into power

bef810b

Remove pragma statement required for older psp-gcc version used in th…

a288a9d

…e CI.

John-K approved these changes Apr 14, 2021

View reviewed changes

artart78 added 3 commits May 21, 2022 17:27

Merge remote-tracking branch 'origin/master' into power

9f9cf6f

power: some fixes for compiling with the latest toolchain

8f68f26

Merge remote-tracking branch 'origin/master' into power

5833022

artart78 and others added 5 commits January 28, 2024 22:14

Merge branch 'master' into power

ded39ee

Move power to src/kd/

22aef02

power: fix Makefile after repo changes

8aa01b7

Merge branch 'master' into power

d7eb93b

Fix compile error.

cce8c88

kotcrab mentioned this pull request May 8, 2025

Fix handling of imported variables #141

Merged

Fix various typos.

f7ed03a

	* Starting with the PSP N-1000 (PSP-Go) serie, the battery equipped by default no longer has hardware included to
	* Starting with the PSP N-1000 (PSP-Go) series, the battery equipped by default no longer has hardware included to

	* down the kernel (anf thus our battery worker thread) we will removed any scheduled/lined up work here
	* down the kernel (and thus our battery worker thread) we will remove any scheduled/lined up work here

	* - DMA transfers for the Memory Stic, WLAN, USB, LCD, or audio
	* - DMA transfers for the Memory Stick, WLAN, USB, LCD, or audio

	* - DMA transfers for transfers to the MediaEngine
	* - DMA transfers to the MediaEngine

		* used to contain a copy of the on-board eDRAM content as the 4MB eDRAM memory is volatile when the system is
		* suspended. When the PSP resumes, this saved content is then written back to the on-board eDRAM.

Add Power Service #82

Are you sure you want to change the base?

Add Power Service #82

Uh oh!

Conversation

Felix-Dev commented Feb 17, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Highlights

Motivation and Context

How Has This Been Tested?

Additional Context

Final Notes

Uh oh!

John-K left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

fjtrujy commented May 19, 2022

Uh oh!

joel16 commented May 19, 2022

Uh oh!

Linblow commented Jan 27, 2024

Uh oh!

joel16 commented Jan 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

artart78 commented Jan 28, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Felix-Dev commented Feb 17, 2021 •

edited

Loading

joel16 commented Jan 28, 2024 •

edited

Loading