diff options
| author | Himangi Saraogi <himangi774@gmail.com> | 2014-08-05 09:26:47 -0400 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2014-08-05 14:08:00 -0400 |
| commit | 81cb6b6be54d160ac8feb61f1510f33a3cece1d3 (patch) | |
| tree | d05e94452b9e4ee84e18c9ccae98f66cdcd45259 | |
| parent | 8b0276d281c9c9e62669736703c87385436a1265 (diff) | |
sound/oss/pss: Remove typedefs pss_mixerdata and pss_confdata
The Linux kernel coding style guidelines suggest not using typedefs
for structure types. This patch gets rid of the typedefs for pss_mixerdata
and pss_confdata.
The following Coccinelle semantic patch is used to make the change.
@tn@
identifier i;
type td;
@@
-typedef
struct i { ... }
-td
;
@@
type tn.td;
identifier tn.i;
@@
-td
+ struct i
Signed-off-by: Himangi Saraogi <himangi774@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
| -rw-r--r-- | sound/oss/pss.c | 46 |
1 files changed, 24 insertions, 22 deletions
diff --git a/sound/oss/pss.c b/sound/oss/pss.c index 145e36b2cfd0..ca0d6e9f49f5 100644 --- a/sound/oss/pss.c +++ b/sound/oss/pss.c | |||
| @@ -123,25 +123,25 @@ static bool pss_mixer; | |||
| 123 | #endif | 123 | #endif |
| 124 | 124 | ||
| 125 | 125 | ||
| 126 | typedef struct pss_mixerdata { | 126 | struct pss_mixerdata { |
| 127 | unsigned int volume_l; | 127 | unsigned int volume_l; |
| 128 | unsigned int volume_r; | 128 | unsigned int volume_r; |
| 129 | unsigned int bass; | 129 | unsigned int bass; |
| 130 | unsigned int treble; | 130 | unsigned int treble; |
| 131 | unsigned int synth; | 131 | unsigned int synth; |
| 132 | } pss_mixerdata; | 132 | }; |
| 133 | 133 | ||
| 134 | typedef struct pss_confdata { | 134 | struct pss_confdata { |
| 135 | int base; | 135 | int base; |
| 136 | int irq; | 136 | int irq; |
| 137 | int dma; | 137 | int dma; |
| 138 | int *osp; | 138 | int *osp; |
| 139 | pss_mixerdata mixer; | 139 | struct pss_mixerdata mixer; |
| 140 | int ad_mixer_dev; | 140 | int ad_mixer_dev; |
| 141 | } pss_confdata; | 141 | }; |
| 142 | 142 | ||
| 143 | static pss_confdata pss_data; | 143 | static struct pss_confdata pss_data; |
| 144 | static pss_confdata *devc = &pss_data; | 144 | static struct pss_confdata *devc = &pss_data; |
| 145 | static DEFINE_SPINLOCK(lock); | 145 | static DEFINE_SPINLOCK(lock); |
| 146 | 146 | ||
| 147 | static int pss_initialized; | 147 | static int pss_initialized; |
| @@ -150,7 +150,7 @@ static int pss_cdrom_port = -1; /* Parameter for the PSS cdrom port */ | |||
| 150 | static bool pss_enable_joystick; /* Parameter for enabling the joystick */ | 150 | static bool pss_enable_joystick; /* Parameter for enabling the joystick */ |
| 151 | static coproc_operations pss_coproc_operations; | 151 | static coproc_operations pss_coproc_operations; |
| 152 | 152 | ||
| 153 | static void pss_write(pss_confdata *devc, int data) | 153 | static void pss_write(struct pss_confdata *devc, int data) |
| 154 | { | 154 | { |
| 155 | unsigned long i, limit; | 155 | unsigned long i, limit; |
| 156 | 156 | ||
| @@ -206,7 +206,7 @@ static int __init probe_pss(struct address_info *hw_config) | |||
| 206 | return 1; | 206 | return 1; |
| 207 | } | 207 | } |
| 208 | 208 | ||
| 209 | static int set_irq(pss_confdata * devc, int dev, int irq) | 209 | static int set_irq(struct pss_confdata *devc, int dev, int irq) |
| 210 | { | 210 | { |
| 211 | static unsigned short irq_bits[16] = | 211 | static unsigned short irq_bits[16] = |
| 212 | { | 212 | { |
| @@ -232,7 +232,7 @@ static int set_irq(pss_confdata * devc, int dev, int irq) | |||
| 232 | return 1; | 232 | return 1; |
| 233 | } | 233 | } |
| 234 | 234 | ||
| 235 | static void set_io_base(pss_confdata * devc, int dev, int base) | 235 | static void set_io_base(struct pss_confdata *devc, int dev, int base) |
| 236 | { | 236 | { |
| 237 | unsigned short tmp = inw(REG(dev)) & 0x003f; | 237 | unsigned short tmp = inw(REG(dev)) & 0x003f; |
| 238 | unsigned short bits = (base & 0x0ffc) << 4; | 238 | unsigned short bits = (base & 0x0ffc) << 4; |
| @@ -240,7 +240,7 @@ static void set_io_base(pss_confdata * devc, int dev, int base) | |||
| 240 | outw(bits | tmp, REG(dev)); | 240 | outw(bits | tmp, REG(dev)); |
| 241 | } | 241 | } |
| 242 | 242 | ||
| 243 | static int set_dma(pss_confdata * devc, int dev, int dma) | 243 | static int set_dma(struct pss_confdata *devc, int dev, int dma) |
| 244 | { | 244 | { |
| 245 | static unsigned short dma_bits[8] = | 245 | static unsigned short dma_bits[8] = |
| 246 | { | 246 | { |
| @@ -264,7 +264,7 @@ static int set_dma(pss_confdata * devc, int dev, int dma) | |||
| 264 | return 1; | 264 | return 1; |
| 265 | } | 265 | } |
| 266 | 266 | ||
| 267 | static int pss_reset_dsp(pss_confdata * devc) | 267 | static int pss_reset_dsp(struct pss_confdata *devc) |
| 268 | { | 268 | { |
| 269 | unsigned long i, limit = jiffies + HZ/10; | 269 | unsigned long i, limit = jiffies + HZ/10; |
| 270 | 270 | ||
| @@ -275,7 +275,7 @@ static int pss_reset_dsp(pss_confdata * devc) | |||
| 275 | return 1; | 275 | return 1; |
| 276 | } | 276 | } |
| 277 | 277 | ||
| 278 | static int pss_put_dspword(pss_confdata * devc, unsigned short word) | 278 | static int pss_put_dspword(struct pss_confdata *devc, unsigned short word) |
| 279 | { | 279 | { |
| 280 | int i, val; | 280 | int i, val; |
| 281 | 281 | ||
| @@ -291,7 +291,7 @@ static int pss_put_dspword(pss_confdata * devc, unsigned short word) | |||
| 291 | return 0; | 291 | return 0; |
| 292 | } | 292 | } |
| 293 | 293 | ||
| 294 | static int pss_get_dspword(pss_confdata * devc, unsigned short *word) | 294 | static int pss_get_dspword(struct pss_confdata *devc, unsigned short *word) |
| 295 | { | 295 | { |
| 296 | int i, val; | 296 | int i, val; |
| 297 | 297 | ||
| @@ -307,7 +307,8 @@ static int pss_get_dspword(pss_confdata * devc, unsigned short *word) | |||
| 307 | return 0; | 307 | return 0; |
| 308 | } | 308 | } |
| 309 | 309 | ||
| 310 | static int pss_download_boot(pss_confdata * devc, unsigned char *block, int size, int flags) | 310 | static int pss_download_boot(struct pss_confdata *devc, unsigned char *block, |
| 311 | int size, int flags) | ||
| 311 | { | 312 | { |
| 312 | int i, val, count; | 313 | int i, val, count; |
| 313 | unsigned long limit; | 314 | unsigned long limit; |
| @@ -397,7 +398,7 @@ static int pss_download_boot(pss_confdata * devc, unsigned char *block, int size | |||
| 397 | } | 398 | } |
| 398 | 399 | ||
| 399 | /* Mixer */ | 400 | /* Mixer */ |
| 400 | static void set_master_volume(pss_confdata *devc, int left, int right) | 401 | static void set_master_volume(struct pss_confdata *devc, int left, int right) |
| 401 | { | 402 | { |
| 402 | static unsigned char log_scale[101] = { | 403 | static unsigned char log_scale[101] = { |
| 403 | 0xdb, 0xe0, 0xe3, 0xe5, 0xe7, 0xe9, 0xea, 0xeb, 0xec, 0xed, 0xed, 0xee, | 404 | 0xdb, 0xe0, 0xe3, 0xe5, 0xe7, 0xe9, 0xea, 0xeb, 0xec, 0xed, 0xed, 0xee, |
| @@ -416,7 +417,7 @@ static void set_master_volume(pss_confdata *devc, int left, int right) | |||
| 416 | pss_write(devc, log_scale[right] | 0x0100); | 417 | pss_write(devc, log_scale[right] | 0x0100); |
| 417 | } | 418 | } |
| 418 | 419 | ||
| 419 | static void set_synth_volume(pss_confdata *devc, int volume) | 420 | static void set_synth_volume(struct pss_confdata *devc, int volume) |
| 420 | { | 421 | { |
| 421 | int vol = ((0x8000*volume)/100L); | 422 | int vol = ((0x8000*volume)/100L); |
| 422 | pss_write(devc, 0x0080); | 423 | pss_write(devc, 0x0080); |
| @@ -425,21 +426,21 @@ static void set_synth_volume(pss_confdata *devc, int volume) | |||
| 425 | pss_write(devc, vol); | 426 | pss_write(devc, vol); |
| 426 | } | 427 | } |
| 427 | 428 | ||
| 428 | static void set_bass(pss_confdata *devc, int level) | 429 | static void set_bass(struct pss_confdata *devc, int level) |
| 429 | { | 430 | { |
| 430 | int vol = (int)(((0xfd - 0xf0) * level)/100L) + 0xf0; | 431 | int vol = (int)(((0xfd - 0xf0) * level)/100L) + 0xf0; |
| 431 | pss_write(devc, 0x0010); | 432 | pss_write(devc, 0x0010); |
| 432 | pss_write(devc, vol | 0x0200); | 433 | pss_write(devc, vol | 0x0200); |
| 433 | }; | 434 | }; |
| 434 | 435 | ||
| 435 | static void set_treble(pss_confdata *devc, int level) | 436 | static void set_treble(struct pss_confdata *devc, int level) |
| 436 | { | 437 | { |
| 437 | int vol = (((0xfd - 0xf0) * level)/100L) + 0xf0; | 438 | int vol = (((0xfd - 0xf0) * level)/100L) + 0xf0; |
| 438 | pss_write(devc, 0x0010); | 439 | pss_write(devc, 0x0010); |
| 439 | pss_write(devc, vol | 0x0300); | 440 | pss_write(devc, vol | 0x0300); |
| 440 | }; | 441 | }; |
| 441 | 442 | ||
| 442 | static void pss_mixer_reset(pss_confdata *devc) | 443 | static void pss_mixer_reset(struct pss_confdata *devc) |
| 443 | { | 444 | { |
| 444 | set_master_volume(devc, 33, 33); | 445 | set_master_volume(devc, 33, 33); |
| 445 | set_bass(devc, 50); | 446 | set_bass(devc, 50); |
| @@ -499,7 +500,8 @@ static int ret_vol_stereo(int left, int right) | |||
| 499 | return ((right << 8) | left); | 500 | return ((right << 8) | left); |
| 500 | } | 501 | } |
| 501 | 502 | ||
| 502 | static int call_ad_mixer(pss_confdata *devc,unsigned int cmd, void __user *arg) | 503 | static int call_ad_mixer(struct pss_confdata *devc, unsigned int cmd, |
| 504 | void __user *arg) | ||
| 503 | { | 505 | { |
| 504 | if (devc->ad_mixer_dev != NO_WSS_MIXER) | 506 | if (devc->ad_mixer_dev != NO_WSS_MIXER) |
| 505 | return mixer_devs[devc->ad_mixer_dev]->ioctl(devc->ad_mixer_dev, cmd, arg); | 507 | return mixer_devs[devc->ad_mixer_dev]->ioctl(devc->ad_mixer_dev, cmd, arg); |
| @@ -509,7 +511,7 @@ static int call_ad_mixer(pss_confdata *devc,unsigned int cmd, void __user *arg) | |||
| 509 | 511 | ||
| 510 | static int pss_mixer_ioctl (int dev, unsigned int cmd, void __user *arg) | 512 | static int pss_mixer_ioctl (int dev, unsigned int cmd, void __user *arg) |
| 511 | { | 513 | { |
| 512 | pss_confdata *devc = mixer_devs[dev]->devc; | 514 | struct pss_confdata *devc = mixer_devs[dev]->devc; |
| 513 | int cmdf = cmd & 0xff; | 515 | int cmdf = cmd & 0xff; |
| 514 | 516 | ||
| 515 | if ((cmdf != SOUND_MIXER_VOLUME) && (cmdf != SOUND_MIXER_BASS) && | 517 | if ((cmdf != SOUND_MIXER_VOLUME) && (cmdf != SOUND_MIXER_BASS) && |
