Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=740eb8358f2bf337fd4ec62abb5b8cca7c652e5a
Commit:     740eb8358f2bf337fd4ec62abb5b8cca7c652e5a
Parent:     93521d274b7fb4e6da5772768683e4984783d9e7
Author:     Clemens Ladisch <[EMAIL PROTECTED]>
AuthorDate: Fri Jan 4 09:22:20 2008 +0100
Committer:  Jaroslav Kysela <[EMAIL PROTECTED]>
CommitDate: Thu Jan 31 17:29:44 2008 +0100

    [ALSA] oxygen: use uintptr_t in pointer casts
    
    When we store the DMA channel number in the substream's private_data
    pointer, use uintptr_t as an intermediate step when casting from/to
    unsigned int to prevent the compiler from whining when the pointer size
    is different.
    
    Signed-off-by: Clemens Ladisch <[EMAIL PROTECTED]>
    Signed-off-by: Jaroslav Kysela <[EMAIL PROTECTED]>
---
 sound/pci/oxygen/oxygen_pcm.c |   20 +++++++++++++-------
 1 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/sound/pci/oxygen/oxygen_pcm.c b/sound/pci/oxygen/oxygen_pcm.c
index 941399b..ff46ba5 100644
--- a/sound/pci/oxygen/oxygen_pcm.c
+++ b/sound/pci/oxygen/oxygen_pcm.c
@@ -168,6 +168,12 @@ static struct snd_pcm_hardware oxygen_hardware[PCM_COUNT] 
= {
        },
 };
 
+static inline unsigned int
+oxygen_substream_channel(struct snd_pcm_substream *substream)
+{
+       return (unsigned int)(uintptr_t)substream->runtime->private_data;
+}
+
 static int oxygen_open(struct snd_pcm_substream *substream,
                       unsigned int channel)
 {
@@ -175,7 +181,7 @@ static int oxygen_open(struct snd_pcm_substream *substream,
        struct snd_pcm_runtime *runtime = substream->runtime;
        int err;
 
-       runtime->private_data = (void *)channel;
+       runtime->private_data = (void *)(uintptr_t)channel;
        runtime->hw = oxygen_hardware[channel];
        err = snd_pcm_hw_constraint_step(runtime, 0,
                                         SNDRV_PCM_HW_PARAM_PERIOD_BYTES, 32);
@@ -248,7 +254,7 @@ static int oxygen_ac97_open(struct snd_pcm_substream 
*substream)
 static int oxygen_close(struct snd_pcm_substream *substream)
 {
        struct oxygen *chip = snd_pcm_substream_chip(substream);
-       unsigned int channel = (unsigned int)substream->runtime->private_data;
+       unsigned int channel = oxygen_substream_channel(substream);
 
        mutex_lock(&chip->mutex);
        chip->pcm_active &= ~(1 << channel);
@@ -337,7 +343,7 @@ static int oxygen_hw_params(struct snd_pcm_substream 
*substream,
                            struct snd_pcm_hw_params *hw_params)
 {
        struct oxygen *chip = snd_pcm_substream_chip(substream);
-       unsigned int channel = (unsigned int)substream->runtime->private_data;
+       unsigned int channel = oxygen_substream_channel(substream);
        int err;
 
        err = snd_pcm_lib_malloc_pages(substream,
@@ -516,7 +522,7 @@ static int oxygen_ac97_hw_params(struct snd_pcm_substream 
*substream,
 static int oxygen_hw_free(struct snd_pcm_substream *substream)
 {
        struct oxygen *chip = snd_pcm_substream_chip(substream);
-       unsigned int channel = (unsigned int)substream->runtime->private_data;
+       unsigned int channel = oxygen_substream_channel(substream);
 
        spin_lock_irq(&chip->reg_lock);
        chip->interrupt_mask &= ~(1 << channel);
@@ -540,7 +546,7 @@ static int oxygen_spdif_hw_free(struct snd_pcm_substream 
*substream)
 static int oxygen_prepare(struct snd_pcm_substream *substream)
 {
        struct oxygen *chip = snd_pcm_substream_chip(substream);
-       unsigned int channel = (unsigned int)substream->runtime->private_data;
+       unsigned int channel = oxygen_substream_channel(substream);
        unsigned int channel_mask = 1 << channel;
 
        spin_lock_irq(&chip->reg_lock);
@@ -575,7 +581,7 @@ static int oxygen_trigger(struct snd_pcm_substream 
*substream, int cmd)
 
        snd_pcm_group_for_each_entry(s, substream) {
                if (snd_pcm_substream_chip(s) == chip) {
-                       mask |= 1 << (unsigned int)s->runtime->private_data;
+                       mask |= 1 << oxygen_substream_channel(s);
                        snd_pcm_trigger_done(s, substream);
                }
        }
@@ -594,7 +600,7 @@ static snd_pcm_uframes_t oxygen_pointer(struct 
snd_pcm_substream *substream)
 {
        struct oxygen *chip = snd_pcm_substream_chip(substream);
        struct snd_pcm_runtime *runtime = substream->runtime;
-       unsigned int channel = (unsigned int)runtime->private_data;
+       unsigned int channel = oxygen_substream_channel(substream);
        u32 curr_addr;
 
        /* no spinlock, this read should be atomic */
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to