Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

N900: Integrate xprot PulseAudio module #62

Open
dderby opened this issue Feb 7, 2018 · 10 comments
Assignees

Comments

@dderby
Copy link
Member

@dderby dderby commented Feb 7, 2018

The N900 speakers have a design flaw and are at risk of blowing up. In Fremantle, Nokia implemented the xprot PulseAudio module blob to prevent this from happening. xprot has already been RE'd by @freemangordon so we should integrate this into the N900 Leste build as soon as possible to prevent anyone from accidentally blowing up their speakers. It may need additional porting changes as the PA version in Ascii is much newer than the one in Fremantle.

@MerlijnWajer

This comment has been minimized.

Copy link
Member

@MerlijnWajer MerlijnWajer commented Feb 7, 2018

Would you like to try to make this happen? First step would be to run normal PA on leste, and then see if you can port the module(s).

@dderby

This comment has been minimized.

Copy link
Member Author

@dderby dderby commented Feb 7, 2018

I can have a look but I can't promise when I'll get to this. I'm not that familiar with PA beyond the basics so I would need to do some investigation and I also lack free time. Given the urgency (in my opinion) of this issue, if someone else wants to pick this up, please do so.

@freemangordon

This comment has been minimized.

Copy link

@freemangordon freemangordon commented Feb 7, 2018

I think @pavelmachek is the one to take the task, given he is the one most familiar with PA amongst us (AFAIK), if he wants to do so.

@pavelmachek

This comment has been minimized.

Copy link

@pavelmachek pavelmachek commented Feb 7, 2018

@MerlijnWajer

This comment has been minimized.

Copy link
Member

@MerlijnWajer MerlijnWajer commented Feb 8, 2018

I've assigned myself, but I don't intend to pick this up next week or so. There are a few more pressing issues. The images are meant for developers, so I'd hope that people would not currently fry their speakers. Furthermore, the sound is muted by default. ;-)

@dderby

This comment has been minimized.

Copy link
Member Author

@dderby dderby commented Mar 5, 2018

Something to think about...

Whilst porting xprot to PulseAudio 10 might be the best solution in the short-term, would it be possible to port xprot to somewhere lower in the software stack (e.g. ALSA or the kernel)? That way we won't tie ourselves to single solution (I'm not the biggest fan of PA) and we can start to think about replacing it with an audio server more suitable to mobile devices, like CRAS for example:

https://events.static.linuxfound.org/sites/events/files/slides/CRAS%20ELC%202014%20%281%29.pdf
https://www.chromium.org/chromium-os/chromiumos-design-docs/cras-chromeos-audio-server

@MerlijnWajer

This comment has been minimized.

Copy link
Member

@MerlijnWajer MerlijnWajer commented Mar 5, 2018

I personally don't mind PA much, since it has allowed me to easily stream my laptops audio to my n900, and then to fm using the n900. We could consider proving xprot in a different manner, but really, most devices (other than n900) won't even need it.

@dderby

This comment has been minimized.

Copy link
Member Author

@dderby dderby commented Mar 6, 2018

The N950 and N9 have versions of xprot too (might need REing).

An audio server with a smaller footprint than PA would be appealing, particularly on a memory constrained device like the N900. I realise that that with further REing and all, this potentially would be a lot work so I'm not saying we should do it. I'm just putting this out there for consideration.

@pavelmachek

This comment has been minimized.

Copy link

@pavelmachek pavelmachek commented Mar 8, 2018

@jonwil

This comment has been minimized.

Copy link

@jonwil jonwil commented Jun 25, 2018

This is the most up-to-date repository in terms of the cloning of the PulseAudio blobs.
https://github.com/community-ssu/pulseaudio-nokia/commits/master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.