This looks fine to me. To follow convention laid out by other controllers in the Camera app, if the controller has any initial configuration to do, it's usually done in a separate function like:

```js
SoundController.prototype.configure = function() {
  // Tell audio channel manager that we want to adjust the notification
  // channel if the user press the volumeup/volumedown buttons in Camera.
  if (navigator.mozAudioChannelManager) {
    navigator.mozAudioChannelManager.volumeControlChannel = 'notification';
  }
};
```
And then called from the constructor.

```js
function SoundsController(app) {
  debug('initializing');
  var list = app.settings.sounds.get('list');
  this.sounds = new Sounds(list);
  this.app = app;
  this.configure();
  this.bindEvents();
  debug('initialized');
};
```