[alsa-devel] [PATCH 0/6] Misc ALSA changes, documentation, style

Antonio Ospite ospite at studenti.unina.it
Mon Jan 28 23:42:26 CET 2013

From: Antonio Ospite <ao2 at amarulasolutions.com>


here are some fixes for things I noticed while studying ALSA.

I'd like some feedback especially on patch 1.
Patches 2 and 3 are documentation fixes.
Patches 4, 5 and 6, are basically style fixes.

Feel free to discard patch 6 if you find it too invasive, it's just
cosmetics but some maintainers don't like big style changes even if they
are in the "right" direction. I don't know alsa devs' position about

I am sending the whole series to alsa-devel and CCing topic specific
mailing lists or developers on a per-patch basis.


Antonio Ospite (6):
  Force a cast to silence a warning from "sparse"
  Documentation/DocBook/writing-an-alsa-driver.tmpl: fix some thinkos
  Documentation/DocBook/writing-an-alsa-driver.tmpl: fix some typos
  sound/usb/caiaq/device.c: fix use of MODULE_SUPPORTED_DEVICES()
  sound/usb/card.c: cosmetics, remove a leading space
  sound/core/pcm_native.c: cosmetics, fix use of spaces

 Documentation/DocBook/writing-an-alsa-driver.tmpl |   11 +-
 include/sound/memalloc.h                          |    2 +-
 sound/core/pcm_native.c                           |  120 ++++++++++-----------
 sound/usb/caiaq/device.c                          |    8 +-
 sound/usb/card.c                                  |    2 +-
 5 files changed, 71 insertions(+), 72 deletions(-)

Antonio Ospite

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?

More information about the Alsa-devel mailing list