Export to GitHub

procfw - issue #281

Add PSP_SYSTEM_VER to virtual PBP implementation


Posted on Sep 11, 2011 by Swift Kangaroo

What is the expected output? Minimun firmware version shown in xmb plugins who read games.

What do you see instead? Shows 1.00 by default.

What version of the product are you using? On what operating system? 6.60 Pro B9

Please provide any additional information below. The attached patch adds the correct PSP_SYSTEM_VER to the virtual pbp implementation as seen in other CFW/HEN who implements this feature.

Attachments

Comment #1

Posted on Jun 15, 2012 by Quick Camel

Thanks for the patch.

Comment #2

Posted on Jun 16, 2012 by Grumpy Lion

I dont understand how this patch is ment to change anything...

If you look into vshmain.prx disassembly you will see that the XMB auto-sets the required firmware version to 1.0 if the tag isn't found... which is also the reason why PROCFW doesn't fake the tag.

This issue will remain open for another week or so... to give the OP a chance to reason why this patch should be accepted non the less.

If no response is given within a week I will close this issue.

Comment #3

Posted on Jun 17, 2012 by Swift Kangaroo

The patch is only a cosmetic fix for plugins that hook the virtual pbp funcs to read the iso atrributes. The only goal of the patch was to get consistent info from the virtual pbp between different implementations of CFW (for example the ME CFW provides PSP_SYSTEM_VER for virtual pbp).

Of course the "plugins" list that i mentioned is reduced to one that i ported (i haven't followed the psp scene lately so i dunno if other plugins that read need to read PSP_SYSTEM_VER). The PSP_SYSTEM_VER is purely informative and serves no purpose in current CFW implementations other that show the base FW that a game needs to run.

At the time of writting the patch i was the only affected and the only one reading the value so feel free to implement the patch or reject it again :), the patch isn't gonna dissapear and anybody who need this can patch the source since is almost a one line patch (highly unlikely that is gonna get out of sync with the current source tree).

tl;dr: the patch is a purely cosmetic fix for plugin makers that for whatever reason needs to hook the virtual pbp funcs to get the info.

Comment #4

Posted on Aug 26, 2013 by Quick Camel

I think it's a good idea to add this patch because it would allow users to see which version is supposed to be required. Knowing that the Pro Online client doesn't work the same for all FWs (version 0.09 is advised for > 1.00 games whereas version 0.10 is needed for 1.00 games), it would allow users to better recognize the firmware originally required by the game (even though this information is not 100% reliable)

Status: Accepted

Labels:
Type-Defect Priority-Low