From 2340c646e6a5da658e114be2f721ff90853588d4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=81lvaro=20Fern=C3=A1ndez=20Rojas?= Date: Fri, 9 Aug 2019 19:50:30 +0200 Subject: brcm2708: update to latest patches from RPi foundation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Álvaro Fernández Rojas --- ...2835-audio-reorder-variable-declarations-.patch | 118 +++++++++++++++++++++ 1 file changed, 118 insertions(+) create mode 100644 target/linux/brcm2708/patches-4.19/950-0466-staging-bcm2835-audio-reorder-variable-declarations-.patch (limited to 'target/linux/brcm2708/patches-4.19/950-0466-staging-bcm2835-audio-reorder-variable-declarations-.patch') diff --git a/target/linux/brcm2708/patches-4.19/950-0466-staging-bcm2835-audio-reorder-variable-declarations-.patch b/target/linux/brcm2708/patches-4.19/950-0466-staging-bcm2835-audio-reorder-variable-declarations-.patch new file mode 100644 index 0000000000..21fcdff527 --- /dev/null +++ b/target/linux/brcm2708/patches-4.19/950-0466-staging-bcm2835-audio-reorder-variable-declarations-.patch @@ -0,0 +1,118 @@ +From dd4a89f63d646fb22aa92d84a5f56b0afed4490b Mon Sep 17 00:00:00 2001 +From: Nicolas Saenz Julienne +Date: Wed, 17 Oct 2018 21:01:52 +0200 +Subject: [PATCH 466/773] staging: bcm2835-audio: reorder variable declarations + & remove trivial comments + +commit d048385a070552ae819f99f05bd03ec41072783d upstream. + +When it comes to declaring variables it's preferred, when possible, to +use an inverted tree organization scheme. + +Also, removes some comments that were useless. + +Signed-off-by: Nicolas Saenz Julienne +Reviewed-by: Takashi Iwai +Acked-by: Stefan Wahren +Signed-off-by: Greg Kroah-Hartman +--- + .../vc04_services/bcm2835-audio/bcm2835-pcm.c | 10 ++-------- + .../vc04_services/bcm2835-audio/bcm2835-vchiq.c | 4 ++-- + .../staging/vc04_services/bcm2835-audio/bcm2835.c | 14 +++++++------- + 3 files changed, 11 insertions(+), 17 deletions(-) + +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c +@@ -164,14 +164,11 @@ static int snd_bcm2835_playback_spdif_op + return snd_bcm2835_playback_open_generic(substream, 1); + } + +-/* close callback */ + static int snd_bcm2835_playback_close(struct snd_pcm_substream *substream) + { +- /* the hardware-specific codes will be here */ +- +- struct bcm2835_chip *chip; +- struct snd_pcm_runtime *runtime; + struct bcm2835_alsa_stream *alsa_stream; ++ struct snd_pcm_runtime *runtime; ++ struct bcm2835_chip *chip; + + chip = snd_pcm_substream_chip(substream); + mutex_lock(&chip->audio_mutex); +@@ -195,20 +192,17 @@ static int snd_bcm2835_playback_close(st + return 0; + } + +-/* hw_params callback */ + static int snd_bcm2835_pcm_hw_params(struct snd_pcm_substream *substream, + struct snd_pcm_hw_params *params) + { + return snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(params)); + } + +-/* hw_free callback */ + static int snd_bcm2835_pcm_hw_free(struct snd_pcm_substream *substream) + { + return snd_pcm_lib_free_pages(substream); + } + +-/* prepare callback */ + static int snd_bcm2835_pcm_prepare(struct snd_pcm_substream *substream) + { + struct bcm2835_chip *chip = snd_pcm_substream_chip(substream); +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c +@@ -94,9 +94,9 @@ static void audio_vchi_callback(void *pa + void *msg_handle) + { + struct bcm2835_audio_instance *instance = param; +- int status; +- int msg_len; + struct vc_audio_msg m; ++ int msg_len; ++ int status; + + if (reason != VCHI_CALLBACK_MSG_AVAILABLE) + return; +--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835.c ++++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835.c +@@ -161,8 +161,8 @@ static int snd_add_child_device(struct d + struct bcm2835_audio_driver *audio_driver, + u32 numchans) + { +- struct snd_card *card; + struct bcm2835_chip *chip; ++ struct snd_card *card; + int err; + + err = snd_card_new(dev, -1, NULL, THIS_MODULE, sizeof(*chip), &card); +@@ -225,12 +225,12 @@ static int snd_add_child_device(struct d + + static int snd_add_child_devices(struct device *device, u32 numchans) + { +- int i; +- int count_devices = 0; +- int minchannels = 0; +- int extrachannels = 0; + int extrachannels_per_driver = 0; + int extrachannels_remainder = 0; ++ int count_devices = 0; ++ int extrachannels = 0; ++ int minchannels = 0; ++ int i; + + for (i = 0; i < ARRAY_SIZE(children_devices); i++) + if (*children_devices[i].is_enabled) +@@ -258,9 +258,9 @@ static int snd_add_child_devices(struct + extrachannels_remainder); + + for (i = 0; i < ARRAY_SIZE(children_devices); i++) { +- int err; +- int numchannels_this_device; + struct bcm2835_audio_driver *audio_driver; ++ int numchannels_this_device; ++ int err; + + if (!*children_devices[i].is_enabled) + continue; -- cgit v1.2.3