The PR was only submitted to fix Subtitles on S912's anyway so it's no big deal if it gets rejected anyway. Makes for a bit of Dev discussion about various things which is always good.
It really is not needed for other Chipsets.
When i get time I will amend the PR so it's specifically S912 only.
If you are planning on doing S905 builds at some point would it not be better to keep it as is? I'm very interested to see the feedback it get's from other users in your builds. Curious if you have tried r8p1 yet? I have attached the user-space binary to this post.