All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/3] Fix spaces and typos in vc04_services
@ 2017-09-26 15:28 Keerthi Reddy
  2017-09-26 15:32 ` [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com Keerthi Reddy
                   ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Keerthi Reddy @ 2017-09-26 15:28 UTC (permalink / raw)
  To: eric, stefan.wahren, outreachy-kernel; +Cc: outreachy-kernel

This patchset removes extra newlines within file 
and also fixes smalls typos.

---
Changes in v2:
edit commit message for patch 2/3

Keerthi Reddy (3):
  staging: vc04_services: please do not use multiple blank lines
  staging: vc04_services: fix typos
  staging: vc04_services: add spaces around << operator

 .../vc04_services/bcm2835-audio/bcm2835-pcm.c      | 11 ----------
 .../vc04_services/bcm2835-audio/bcm2835-vchiq.c    | 11 ----------
 .../vc04_services/bcm2835-camera/mmal-msg.h        | 24 +++++++++++-----------
 .../vc04_services/bcm2835-camera/mmal-parameters.h | 12 +++++------
 .../interface/vchi/connections/connection.h        |  3 ---
 .../interface/vchi/message_drivers/message.h       |  5 -----
 .../staging/vc04_services/interface/vchi/vchi.h    | 12 +----------
 .../vc04_services/interface/vchi/vchi_common.h     |  6 ------
 .../vc04_services/interface/vchiq_arm/vchiq_arm.c  |  9 --------
 .../vc04_services/interface/vchiq_arm/vchiq_arm.h  |  5 -----
 .../vc04_services/interface/vchiq_arm/vchiq_core.c | 10 ---------
 .../vc04_services/interface/vchiq_arm/vchiq_core.h |  1 -
 .../interface/vchiq_arm/vchiq_debugfs.c            |  3 ---
 .../vc04_services/interface/vchiq_arm/vchiq_shim.c |  3 ---
 14 files changed, 19 insertions(+), 96 deletions(-)

-- 
2.7.4



^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com
  2017-09-26 15:28 [PATCH v2 0/3] Fix spaces and typos in vc04_services Keerthi Reddy
@ 2017-09-26 15:32 ` Keerthi Reddy
  2017-09-26 15:49   ` [Outreachy kernel] " Julia Lawall
  2017-09-26 15:32 ` [PATCH v2 2/3] staging: vc04_services: fix typos Keerthi Reddy
  2017-09-26 15:33 ` [PATCH v2 3/3] staging: vc04_services: add spaces around << operator Keerthi Reddy
  2 siblings, 1 reply; 8+ messages in thread
From: Keerthi Reddy @ 2017-09-26 15:32 UTC (permalink / raw)
  To: This, patchset, removes, extra, newlines, within, file; +Cc: outreachy-kernel

Blank lines use up extra space in file and makes the file
larger. So do not use multiple blanklines

Signed-off-by: Keerthi Reddy <keerthigd4990@gmail.com>
---
Changes in v2:
* Capitalize starting letter in commit message.

 drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c     | 11 -----------
 drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c   | 11 -----------
 .../vc04_services/interface/vchi/connections/connection.h     |  3 ---
 .../vc04_services/interface/vchi/message_drivers/message.h    |  5 -----
 drivers/staging/vc04_services/interface/vchi/vchi.h           | 10 ----------
 drivers/staging/vc04_services/interface/vchi/vchi_common.h    |  6 ------
 drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c |  9 ---------
 drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h |  5 -----
 .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c    | 10 ----------
 .../staging/vc04_services/interface/vchiq_arm/vchiq_core.h    |  1 -
 .../staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c |  3 ---
 .../staging/vc04_services/interface/vchiq_arm/vchiq_shim.c    |  3 ---
 12 files changed, 77 deletions(-)

diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
index 94654c0..7e68b3e 100644
--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
+++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
@@ -65,7 +65,6 @@ void bcm2835_playback_fifo(struct bcm2835_alsa_stream *alsa_stream)
 	unsigned int consumed = 0;
 	int new_period = 0;
 
-
 	audio_info("alsa_stream=%p substream=%p\n", alsa_stream,
 		alsa_stream ? alsa_stream->substream : 0);
 
@@ -111,7 +110,6 @@ static int snd_bcm2835_playback_open_generic(
 	int idx;
 	int err;
 
-
 	if (mutex_lock_interruptible(&chip->audio_mutex)) {
 		audio_error("Interrupted whilst waiting for lock\n");
 		return -EINTR;
@@ -184,7 +182,6 @@ static int snd_bcm2835_playback_open_generic(
 out:
 	mutex_unlock(&chip->audio_mutex);
 
-
 	return err;
 }
 
@@ -207,7 +204,6 @@ static int snd_bcm2835_playback_close(struct snd_pcm_substream *substream)
 	struct snd_pcm_runtime *runtime;
 	struct bcm2835_alsa_stream *alsa_stream;
 
-
 	chip = snd_pcm_substream_chip(substream);
 	if (mutex_lock_interruptible(&chip->audio_mutex)) {
 		audio_error("Interrupted whilst waiting for lock\n");
@@ -259,7 +255,6 @@ static int snd_bcm2835_pcm_hw_params(struct snd_pcm_substream *substream,
 	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
 	int err;
 
-
 	err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(params));
 	if (err < 0) {
 		audio_error
@@ -289,7 +284,6 @@ static int snd_bcm2835_pcm_prepare(struct snd_pcm_substream *substream)
 	int channels;
 	int err;
 
-
 	if (mutex_lock_interruptible(&chip->audio_mutex))
 		return -EINTR;
 
@@ -307,13 +301,11 @@ static int snd_bcm2835_pcm_prepare(struct snd_pcm_substream *substream)
 	if (err < 0)
 		audio_error(" error setting hw params\n");
 
-
 	bcm2835_audio_setup(alsa_stream);
 
 	/* in preparation of the stream, set the controls (volume level) of the stream */
 	bcm2835_audio_set_ctls(alsa_stream->chip);
 
-
 	memset(&alsa_stream->pcm_indirect, 0, sizeof(alsa_stream->pcm_indirect));
 
 	alsa_stream->pcm_indirect.hw_buffer_size =
@@ -364,7 +356,6 @@ static int snd_bcm2835_pcm_trigger(struct snd_pcm_substream *substream, int cmd)
 	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
 	int err = 0;
 
-
 	switch (cmd) {
 	case SNDRV_PCM_TRIGGER_START:
 		audio_debug("bcm2835_AUDIO_TRIGGER_START running=%d\n",
@@ -416,7 +407,6 @@ snd_bcm2835_pcm_pointer(struct snd_pcm_substream *substream)
 	struct snd_pcm_runtime *runtime = substream->runtime;
 	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
 
-
 	audio_debug("pcm_pointer... (%d) hwptr=%d appl=%d pos=%d\n", 0,
 		frames_to_bytes(runtime, runtime->status->hw_ptr),
 		frames_to_bytes(runtime, runtime->control->appl_ptr),
@@ -493,7 +483,6 @@ int snd_bcm2835_new_pcm(struct bcm2835_chip *chip, u32 numchannels)
 					      snd_bcm2835_playback_hw.buffer_bytes_max,
 					      snd_bcm2835_playback_hw.buffer_bytes_max);
 
-
 out:
 	mutex_unlock(&chip->audio_mutex);
 
diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
index 5f3d8f2..7469c9b 100644
--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
+++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
@@ -337,7 +337,6 @@ static int vc_vchi_audio_deinit(struct bcm2835_audio_instance *instance)
 {
 	unsigned int i;
 
-
 	if (!instance) {
 		LOG_ERR("%s: invalid handle %p\n", __func__, instance);
 
@@ -369,7 +368,6 @@ static int vc_vchi_audio_deinit(struct bcm2835_audio_instance *instance)
 
 	kfree(instance);
 
-
 	return 0;
 }
 
@@ -382,7 +380,6 @@ static int bcm2835_audio_open_connection(struct bcm2835_alsa_stream *alsa_stream
 		(struct bcm2835_audio_instance *)alsa_stream->instance;
 	int ret;
 
-
 	LOG_INFO("%s: start\n", __func__);
 	BUG_ON(instance);
 	if (instance) {
@@ -443,7 +440,6 @@ int bcm2835_audio_open(struct bcm2835_alsa_stream *alsa_stream)
 	int status;
 	int ret;
 
-
 	my_workqueue_init(alsa_stream);
 
 	ret = bcm2835_audio_open_connection(alsa_stream);
@@ -491,7 +487,6 @@ static int bcm2835_audio_set_ctls_chan(struct bcm2835_alsa_stream *alsa_stream,
 	int status;
 	int ret;
 
-
 	LOG_INFO(" Setting ALSA dest(%d), volume(%d)\n",
 		 chip->dest, chip->volume);
 
@@ -575,7 +570,6 @@ int bcm2835_audio_set_params(struct bcm2835_alsa_stream *alsa_stream,
 	int status;
 	int ret;
 
-
 	LOG_INFO(" Setting ALSA channels(%d), samplerate(%d), bits-per-sample(%d)\n",
 		 channels, samplerate, bps);
 
@@ -636,7 +630,6 @@ int bcm2835_audio_set_params(struct bcm2835_alsa_stream *alsa_stream,
 int bcm2835_audio_setup(struct bcm2835_alsa_stream *alsa_stream)
 {
 
-
 	return 0;
 }
 
@@ -647,7 +640,6 @@ static int bcm2835_audio_start_worker(struct bcm2835_alsa_stream *alsa_stream)
 	int status;
 	int ret;
 
-
 	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
 		LOG_DBG("Interrupted whilst waiting for lock on (%d)\n",
 			instance->num_connections);
@@ -684,7 +676,6 @@ static int bcm2835_audio_stop_worker(struct bcm2835_alsa_stream *alsa_stream)
 	int status;
 	int ret;
 
-
 	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
 		LOG_DBG("Interrupted whilst waiting for lock on (%d)\n",
 			instance->num_connections);
@@ -722,7 +713,6 @@ int bcm2835_audio_close(struct bcm2835_alsa_stream *alsa_stream)
 	int status;
 	int ret;
 
-
 	my_workqueue_quit(alsa_stream);
 
 	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
@@ -780,7 +770,6 @@ static int bcm2835_audio_write_worker(struct bcm2835_alsa_stream *alsa_stream,
 	int status;
 	int ret;
 
-
 	LOG_INFO(" Writing %d bytes from %p\n", count, src);
 
 	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
diff --git a/drivers/staging/vc04_services/interface/vchi/connections/connection.h b/drivers/staging/vc04_services/interface/vchi/connections/connection.h
index e793cdf..67c8438 100644
--- a/drivers/staging/vc04_services/interface/vchi/connections/connection.h
+++ b/drivers/staging/vc04_services/interface/vchi/connections/connection.h
@@ -54,7 +54,6 @@ typedef struct opaque_vchi_connection_info_t VCHI_CONNECTION_STATE_T;
 
 typedef struct vchi_connection_t VCHI_CONNECTION_T;
 
-
 /******************************************************************************
  API
  *****************************************************************************/
@@ -212,7 +211,6 @@ typedef void * (*VCHI_BUFFER_ALLOCATE)(VCHI_CONNECTION_SERVICE_HANDLE_T service_
 // free memory allocated by buffer_allocate
 typedef void   (*VCHI_BUFFER_FREE)(VCHI_CONNECTION_SERVICE_HANDLE_T service_handle, void * address);
 
-
 /******************************************************************************
  System driver struct
  *****************************************************************************/
@@ -321,7 +319,6 @@ struct vchi_connection_t {
 #endif
 };
 
-
 #endif /* CONNECTION_H_ */
 
 /****************************** End of file **********************************/
diff --git a/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h b/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
index a7740a4..834263f 100644
--- a/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
+++ b/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
@@ -41,7 +41,6 @@
 #include "interface/vchi/vchi_cfg_internal.h"
 #include "interface/vchi/vchi_common.h"
 
-
 typedef enum message_event_type {
    MESSAGE_EVENT_NONE,
    MESSAGE_EVENT_NOP,
@@ -111,7 +110,6 @@ typedef struct rx_bulk_slotinfo_t {
    VCHI_FLAGS_T flags;
 } RX_BULK_SLOTINFO_T;
 
-
 /* ----------------------------------------------------------------------
  * each connection driver will have a pool of the following struct.
  *
@@ -155,7 +153,6 @@ typedef struct {
 
 } MESSAGE_EVENT_T;
 
-
 // callbacks
 typedef void VCHI_MESSAGE_DRIVER_EVENT_CALLBACK_T( void *state );
 
@@ -163,7 +160,6 @@ typedef struct {
    VCHI_MESSAGE_DRIVER_EVENT_CALLBACK_T *event_callback;
 } VCHI_MESSAGE_DRIVER_OPEN_T;
 
-
 // handle to this instance of message driver (as returned by ->open)
 typedef struct opaque_mhandle_t *VCHI_MDRIVER_HANDLE_T;
 
@@ -195,7 +191,6 @@ struct opaque_vchi_message_driver_t {
    void    (*debug)( VCHI_MDRIVER_HANDLE_T *handle );
 };
 
-
 #endif // _VCHI_MESSAGE_H_
 
 /****************************** End of file ***********************************/
diff --git a/drivers/staging/vc04_services/interface/vchi/vchi.h b/drivers/staging/vc04_services/interface/vchi/vchi.h
index addb7b0..ee92189 100644
--- a/drivers/staging/vc04_services/interface/vchi/vchi.h
+++ b/drivers/staging/vc04_services/interface/vchi/vchi.h
@@ -39,7 +39,6 @@
 #include "interface/vchi/connections/connection.h"
 #include "vchi_mh.h"
 
-
 /******************************************************************************
  Global defs
  *****************************************************************************/
@@ -92,7 +91,6 @@ typedef struct vchi_msg_vector_ex {
    } u;
 } VCHI_MSG_VECTOR_EX_T;
 
-
 // Construct an entry in a msg vector for a pointer (p) of length (l)
 #define VCHI_VEC_POINTER(p,l)  VCHI_VEC_POINTER, { { (VCHI_MEM_HANDLE_T)(p), (l) } }
 
@@ -103,7 +101,6 @@ typedef struct vchi_msg_vector_ex {
 #define MAKE_FOURCC(x) ((int32_t)( (x[0] << 24) | (x[1] << 16) | (x[2] << 8) | x[3] ))
 #define FOURCC_TO_CHAR(x) (x >> 24) & 0xFF,(x >> 16) & 0xFF,(x >> 8) & 0xFF, x & 0xFF
 
-
 // Opaque service information
 struct opaque_vchi_service_t;
 
@@ -114,8 +111,6 @@ typedef struct {
    void *message;
 } VCHI_HELD_MSG_T;
 
-
-
 // structure used to provide the information needed to open a server or a client
 typedef struct {
 	struct vchi_version version;
@@ -162,7 +157,6 @@ extern "C" {
 extern /*@observer@*/ VCHI_CONNECTION_T * vchi_create_connection( const VCHI_CONNECTION_API_T * function_table,
                                                    const VCHI_MESSAGE_DRIVER_T * low_level);
 
-
 // Routine used to initialise the vchi on both local + remote connections
 extern int32_t vchi_initialise( VCHI_INSTANCE_T *instance_handle );
 
@@ -185,7 +179,6 @@ extern void * vchi_allocate_buffer(VCHI_SERVICE_HANDLE_T handle, uint32_t *lengt
 extern void vchi_free_buffer(VCHI_SERVICE_HANDLE_T handle, void *address);
 extern uint32_t vchi_current_time(VCHI_INSTANCE_T instance_handle);
 
-
 /******************************************************************************
  Global service API
  *****************************************************************************/
@@ -307,7 +300,6 @@ extern int32_t vchi_msg_iter_hold_next( VCHI_MSG_ITER_T *iter,
                                         uint32_t *msg_size, // }
                                         VCHI_HELD_MSG_T *message );
 
-
 /******************************************************************************
  Global bulk API
  *****************************************************************************/
@@ -319,7 +311,6 @@ extern int32_t vchi_bulk_queue_receive( VCHI_SERVICE_HANDLE_T handle,
                                         VCHI_FLAGS_T flags,
                                         void *transfer_handle );
 
-
 // Prepare interface for a transfer from the other side into relocatable memory.
 int32_t vchi_bulk_queue_receive_reloc( const VCHI_SERVICE_HANDLE_T handle,
                                        VCHI_MEM_HANDLE_T h_dst,
@@ -335,7 +326,6 @@ extern int32_t vchi_bulk_queue_transmit( VCHI_SERVICE_HANDLE_T handle,
                                          VCHI_FLAGS_T flags,
                                          void *transfer_handle );
 
-
 /******************************************************************************
  Configuration plumbing
  *****************************************************************************/
diff --git a/drivers/staging/vc04_services/interface/vchi/vchi_common.h b/drivers/staging/vc04_services/interface/vchi/vchi_common.h
index 45c2070..76e10fe 100644
--- a/drivers/staging/vc04_services/interface/vchi/vchi_common.h
+++ b/drivers/staging/vc04_services/interface/vchi/vchi_common.h
@@ -34,7 +34,6 @@
 #ifndef VCHI_COMMON_H_
 #define VCHI_COMMON_H_
 
-
 //flags used when sending messages (must be bitmapped)
 typedef enum {
    VCHI_FLAGS_NONE                      = 0x0,
@@ -118,14 +117,11 @@ typedef enum {
    VCHI_SERVICE_OPTION_MAX
 } VCHI_SERVICE_OPTION_T;
 
-
 //Callback used by all services / bulk transfers
 typedef void (*VCHI_CALLBACK_T)(void *callback_param, //my service local param
 				 VCHI_CALLBACK_REASON_T reason,
 				 void *handle); //for transmitting msg's only
 
-
-
 /*
  * Define vector struct for scatter-gather (vector) operations
  * Vectors can be nested - if a vector element has negative length, then
@@ -154,7 +150,6 @@ typedef struct opaque_vchi_connection_api_t VCHI_CONNECTION_API_T;
 // Opaque type for a message driver
 typedef struct opaque_vchi_message_driver_t VCHI_MESSAGE_DRIVER_T;
 
-
 // Iterator structure for reading ahead through received message queue. Allocated by client,
 // initialised by vchi_msg_look_ahead. Fields are for internal VCHI use only.
 // Iterates over messages in queue at the instant of the call to vchi_msg_lookahead -
@@ -168,5 +163,4 @@ typedef struct {
    void *remove;
 } VCHI_MSG_ITER_T;
 
-
 #endif // VCHI_COMMON_H_
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
index d23152b..6fcc626 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
@@ -109,10 +109,8 @@ static const char *const resume_state_names[] = {
  * requested */
 #define FORCE_SUSPEND_TIMEOUT_MS 200
 
-
 static void suspend_timer_callback(unsigned long context);
 
-
 typedef struct user_service_struct {
 	VCHIQ_SERVICE_T *service;
 	void *userdata;
@@ -2156,8 +2154,6 @@ vchiq_keepalive_thread_func(void *v)
 	return 0;
 }
 
-
-
 VCHIQ_STATUS_T
 vchiq_arm_init_state(VCHIQ_STATE_T *state, VCHIQ_ARM_STATE_T *arm_state)
 {
@@ -2318,7 +2314,6 @@ set_resume_state(VCHIQ_ARM_STATE_T *arm_state,
 	}
 }
 
-
 /* should be called with the write lock held */
 inline void
 start_suspend_timer(VCHIQ_ARM_STATE_T *arm_state)
@@ -2438,7 +2433,6 @@ vchiq_arm_vcsuspend(VCHIQ_STATE_T *state)
 	vchiq_log_trace(vchiq_susp_log_level, "%s", __func__);
 	status = VCHIQ_SUCCESS;
 
-
 	switch (arm_state->vc_suspend_state) {
 	case VC_SUSPEND_REQUESTED:
 		vchiq_log_info(vchiq_susp_log_level, "%s: suspend already "
@@ -2503,7 +2497,6 @@ vchiq_platform_check_suspend(VCHIQ_STATE_T *state)
 	return;
 }
 
-
 static void
 output_timeout_error(VCHIQ_STATE_T *state)
 {
@@ -2683,7 +2676,6 @@ vchiq_check_suspend(VCHIQ_STATE_T *state)
 	return;
 }
 
-
 int
 vchiq_arm_allow_resume(VCHIQ_STATE_T *state)
 {
@@ -2845,7 +2837,6 @@ vchiq_use_internal(VCHIQ_STATE_T *state, VCHIQ_SERVICE_T *service,
 			"%s %s count %d, state count %d",
 			__func__, entity, *entity_uc, local_uc);
 
-
 	write_unlock_bh(&arm_state->susp_res_lock);
 
 	/* Completion is in a done state when we're not suspended, so this won't
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
index bfbd81d..a4cc0db 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
@@ -42,7 +42,6 @@
 #include "vchiq_core.h"
 #include "vchiq_debugfs.h"
 
-
 enum vc_suspend_status {
 	VC_SUSPEND_FORCE_CANCELED = -3, /* Force suspend canceled, too busy */
 	VC_SUSPEND_REJECTED = -2,  /* Videocore rejected suspend request */
@@ -61,15 +60,12 @@ enum vc_resume_status {
 	VC_RESUME_RESUMED      /* Videocore resumed successfully (active) */
 };
 
-
 enum USE_TYPE_E {
 	USE_TYPE_SERVICE,
 	USE_TYPE_SERVICE_NO_RESUME,
 	USE_TYPE_VCHIQ
 };
 
-
-
 typedef struct vchiq_arm_state_struct {
 	/* Keepalive-related data */
 	struct task_struct *ka_thread;
@@ -216,5 +212,4 @@ set_resume_state(VCHIQ_ARM_STATE_T *arm_state,
 extern void
 start_suspend_timer(VCHIQ_ARM_STATE_T *arm_state);
 
-
 #endif /* VCHIQ_ARM_H */
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
index 486be99..c3bb8e0 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
@@ -130,7 +130,6 @@ static const char *const conn_state_names[] = {
 	"RESUME_TIMEOUT"
 };
 
-
 static void
 release_message_sync(VCHIQ_STATE_T *state, VCHIQ_HEADER_T *header);
 
@@ -2168,7 +2167,6 @@ slot_handler_func(void *v)
 				break;
 			}
 
-
 		}
 
 		DEBUG_TRACE(SLOT_HANDLER_LINE);
@@ -2177,7 +2175,6 @@ slot_handler_func(void *v)
 	return 0;
 }
 
-
 /* Called by the recycle thread */
 static int
 recycle_func(void *v)
@@ -2193,7 +2190,6 @@ recycle_func(void *v)
 	return 0;
 }
 
-
 /* Called by the sync thread */
 static int
 sync_func(void *v)
@@ -2301,7 +2297,6 @@ sync_func(void *v)
 	return 0;
 }
 
-
 static void
 init_bulk_queue(VCHIQ_BULK_QUEUE_T *queue)
 {
@@ -2312,14 +2307,12 @@ init_bulk_queue(VCHIQ_BULK_QUEUE_T *queue)
 	queue->remove = 0;
 }
 
-
 inline const char *
 get_conn_state_name(VCHIQ_CONNSTATE_T conn_state)
 {
 	return conn_state_names[conn_state];
 }
 
-
 VCHIQ_SLOT_ZERO_T *
 vchiq_init_slots(void *mem_base, int mem_size)
 {
@@ -3296,7 +3289,6 @@ vchiq_remove_service(VCHIQ_SERVICE_HANDLE_T handle)
 	return status;
 }
 
-
 /* This function may be called by kernel threads or user threads.
  * User threads may receive VCHIQ_RETRY to indicate that a signal has been
  * received and the call should be retried after being returned to user
@@ -3876,7 +3868,6 @@ vchiq_dump_service_state(void *dump_context, VCHIQ_SERVICE_T *service)
 		vchiq_dump_platform_service_state(dump_context, service);
 }
 
-
 void
 vchiq_loud_error_header(void)
 {
@@ -3901,7 +3892,6 @@ vchiq_loud_error_footer(void)
 		"================");
 }
 
-
 VCHIQ_STATUS_T vchiq_send_remote_use(VCHIQ_STATE_T *state)
 {
 	VCHIQ_STATUS_T status = VCHIQ_RETRY;
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
index 9e16465..afc1d81 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
@@ -702,7 +702,6 @@ vchiq_platform_handle_timeout(VCHIQ_STATE_T *state);
 extern void
 vchiq_set_conn_state(VCHIQ_STATE_T *state, VCHIQ_CONNSTATE_T newstate);
 
-
 extern void
 vchiq_log_dump_mem(const char *label, uint32_t addr, const void *voidMem,
 	size_t numBytes);
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
index 9367a9a..766b4fe 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
@@ -32,7 +32,6 @@
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-
 #include <linux/debugfs.h>
 #include "vchiq_core.h"
 #include "vchiq_arm.h"
@@ -52,7 +51,6 @@
 #define VCHIQ_LOG_INFO_STR    "info"
 #define VCHIQ_LOG_TRACE_STR   "trace"
 
-
 /* Top-level debug info */
 struct vchiq_debugfs_info {
 	/* Global 'vchiq' debugfs entry used by all instances */
@@ -316,7 +314,6 @@ void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance)
 	debugfs_remove_recursive(node->dentry);
 }
 
-
 int vchiq_debugfs_init(void)
 {
 	BUG_ON(debugfs_info.vchiq_cfg_dir != NULL);
diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
index 631ff73..d465e1c 100644
--- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
+++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
@@ -84,7 +84,6 @@ VCHI_CONNECTION_T *vchi_create_connection(
  *             void **data,
  *             uint32_t *msg_size,
 
-
  *             VCHI_FLAGS_T flags
  *
  * Description: Routine to return a pointer to the current message (to allow in
@@ -541,7 +540,6 @@ int32_t vchi_connect(VCHI_CONNECTION_T **connections,
 }
 EXPORT_SYMBOL(vchi_connect);
 
-
 /***********************************************************
  * Name: vchi_disconnect
  *
@@ -561,7 +559,6 @@ int32_t vchi_disconnect(VCHI_INSTANCE_T instance_handle)
 }
 EXPORT_SYMBOL(vchi_disconnect);
 
-
 /***********************************************************
  * Name: vchi_service_open
  * Name: vchi_service_create
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH v2 2/3] staging: vc04_services: fix typos
  2017-09-26 15:28 [PATCH v2 0/3] Fix spaces and typos in vc04_services Keerthi Reddy
  2017-09-26 15:32 ` [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com Keerthi Reddy
@ 2017-09-26 15:32 ` Keerthi Reddy
  2017-09-26 15:33 ` [PATCH v2 3/3] staging: vc04_services: add spaces around << operator Keerthi Reddy
  2 siblings, 0 replies; 8+ messages in thread
From: Keerthi Reddy @ 2017-09-26 15:32 UTC (permalink / raw)
  To: eric, stefan.wahren, outreachy-kernel; +Cc: outreachy-kernel

Some words are misspelled. Fix typos.

Signed-off-by: Keerthi Reddy <keerthigd4990@gmail.com>
---
changes in v2:
Capitalize starting letter in commit message

 drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h | 2 +-
 drivers/staging/vc04_services/interface/vchi/vchi.h     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
index 52cdf4d..1144d9d 100644
--- a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
+++ b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
@@ -70,7 +70,7 @@ enum mmal_msg_type {
 
 /* port action request messages differ depending on the action type */
 enum mmal_msg_port_action_type {
-	MMAL_MSG_PORT_ACTION_TYPE_UNKNOWN = 0,      /* Unkown action */
+	MMAL_MSG_PORT_ACTION_TYPE_UNKNOWN = 0,      /* Unknown action */
 	MMAL_MSG_PORT_ACTION_TYPE_ENABLE,           /* Enable a port */
 	MMAL_MSG_PORT_ACTION_TYPE_DISABLE,          /* Disable a port */
 	MMAL_MSG_PORT_ACTION_TYPE_FLUSH,            /* Flush a port */
diff --git a/drivers/staging/vc04_services/interface/vchi/vchi.h b/drivers/staging/vc04_services/interface/vchi/vchi.h
index ee92189..66a3a06 100644
--- a/drivers/staging/vc04_services/interface/vchi/vchi.h
+++ b/drivers/staging/vc04_services/interface/vchi/vchi.h
@@ -187,7 +187,7 @@ extern int32_t vchi_service_create( VCHI_INSTANCE_T instance_handle,
                                     SERVICE_CREATION_T *setup,
                                     VCHI_SERVICE_HANDLE_T *handle );
 
-// Routine to destory a service
+// Routine to destroy a service
 extern int32_t vchi_service_destroy( const VCHI_SERVICE_HANDLE_T handle );
 
 // Routine to open a named service
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH v2 3/3] staging: vc04_services: add spaces around << operator
  2017-09-26 15:28 [PATCH v2 0/3] Fix spaces and typos in vc04_services Keerthi Reddy
  2017-09-26 15:32 ` [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com Keerthi Reddy
  2017-09-26 15:32 ` [PATCH v2 2/3] staging: vc04_services: fix typos Keerthi Reddy
@ 2017-09-26 15:33 ` Keerthi Reddy
  2017-09-26 16:44   ` Stefan Wahren
  2 siblings, 1 reply; 8+ messages in thread
From: Keerthi Reddy @ 2017-09-26 15:33 UTC (permalink / raw)
  To: eric, stefan.wahren, outreachy-kernel; +Cc: outreachy-kernel

Follow coding convention by adding providing space
around << operators

Signed-off-by: Keerthi Reddy <keerthigd4990@gmail.com>
---
 changes in v2:
 * No changes

 .../vc04_services/bcm2835-camera/mmal-msg.h        | 22 +++++++++++-----------
 .../vc04_services/bcm2835-camera/mmal-parameters.h | 12 ++++++------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
index 1144d9d..0dfad29 100644
--- a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
+++ b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
@@ -217,36 +217,36 @@ struct mmal_msg_port_action_reply {
 #define MMAL_VC_SHORT_DATA 128
 
 /** Signals that the current payload is the end of the stream of data */
-#define MMAL_BUFFER_HEADER_FLAG_EOS                    (1<<0)
+#define MMAL_BUFFER_HEADER_FLAG_EOS                    (1 << 0)
 /** Signals that the start of the current payload starts a frame */
-#define MMAL_BUFFER_HEADER_FLAG_FRAME_START            (1<<1)
+#define MMAL_BUFFER_HEADER_FLAG_FRAME_START            (1 << 1)
 /** Signals that the end of the current payload ends a frame */
-#define MMAL_BUFFER_HEADER_FLAG_FRAME_END              (1<<2)
+#define MMAL_BUFFER_HEADER_FLAG_FRAME_END              (1 << 2)
 /** Signals that the current payload contains only complete frames (>1) */
 #define MMAL_BUFFER_HEADER_FLAG_FRAME                  \
 	(MMAL_BUFFER_HEADER_FLAG_FRAME_START|MMAL_BUFFER_HEADER_FLAG_FRAME_END)
 /** Signals that the current payload is a keyframe (i.e. self decodable) */
-#define MMAL_BUFFER_HEADER_FLAG_KEYFRAME               (1<<3)
+#define MMAL_BUFFER_HEADER_FLAG_KEYFRAME               (1 << 3)
 /** Signals a discontinuity in the stream of data (e.g. after a seek).
  * Can be used for instance by a decoder to reset its state
  */
-#define MMAL_BUFFER_HEADER_FLAG_DISCONTINUITY          (1<<4)
+#define MMAL_BUFFER_HEADER_FLAG_DISCONTINUITY          (1 << 4)
 /** Signals a buffer containing some kind of config data for the component
  * (e.g. codec config data)
  */
-#define MMAL_BUFFER_HEADER_FLAG_CONFIG                 (1<<5)
+#define MMAL_BUFFER_HEADER_FLAG_CONFIG                 (1 << 5)
 /** Signals an encrypted payload */
-#define MMAL_BUFFER_HEADER_FLAG_ENCRYPTED              (1<<6)
+#define MMAL_BUFFER_HEADER_FLAG_ENCRYPTED              (1 << 6)
 /** Signals a buffer containing side information */
-#define MMAL_BUFFER_HEADER_FLAG_CODECSIDEINFO          (1<<7)
+#define MMAL_BUFFER_HEADER_FLAG_CODECSIDEINFO          (1 << 7)
 /** Signals a buffer which is the snapshot/postview image from a stills
  * capture
  */
-#define MMAL_BUFFER_HEADER_FLAGS_SNAPSHOT              (1<<8)
+#define MMAL_BUFFER_HEADER_FLAGS_SNAPSHOT              (1 << 8)
 /** Signals a buffer which contains data known to be corrupted */
-#define MMAL_BUFFER_HEADER_FLAG_CORRUPTED              (1<<9)
+#define MMAL_BUFFER_HEADER_FLAG_CORRUPTED              (1 << 9)
 /** Signals that a buffer failed to be transmitted */
-#define MMAL_BUFFER_HEADER_FLAG_TRANSMISSION_FAILED    (1<<10)
+#define MMAL_BUFFER_HEADER_FLAG_TRANSMISSION_FAILED    (1 << 10)
 
 struct mmal_driver_buffer {
 	u32 magic;
diff --git a/drivers/staging/vc04_services/bcm2835-camera/mmal-parameters.h b/drivers/staging/vc04_services/bcm2835-camera/mmal-parameters.h
index e730022..c4a2380 100644
--- a/drivers/staging/vc04_services/bcm2835-camera/mmal-parameters.h
+++ b/drivers/staging/vc04_services/bcm2835-camera/mmal-parameters.h
@@ -22,17 +22,17 @@
  */
 
 /** Common parameter ID group, used with many types of component. */
-#define MMAL_PARAMETER_GROUP_COMMON            (0<<16)
+#define MMAL_PARAMETER_GROUP_COMMON            (0 << 16)
 /** Camera-specific parameter ID group. */
-#define MMAL_PARAMETER_GROUP_CAMERA            (1<<16)
+#define MMAL_PARAMETER_GROUP_CAMERA            (1 << 16)
 /** Video-specific parameter ID group. */
-#define MMAL_PARAMETER_GROUP_VIDEO             (2<<16)
+#define MMAL_PARAMETER_GROUP_VIDEO             (2 << 16)
 /** Audio-specific parameter ID group. */
-#define MMAL_PARAMETER_GROUP_AUDIO             (3<<16)
+#define MMAL_PARAMETER_GROUP_AUDIO             (3 << 16)
 /** Clock-specific parameter ID group. */
-#define MMAL_PARAMETER_GROUP_CLOCK             (4<<16)
+#define MMAL_PARAMETER_GROUP_CLOCK             (4 << 16)
 /** Miracast-specific parameter ID group. */
-#define MMAL_PARAMETER_GROUP_MIRACAST       (5<<16)
+#define MMAL_PARAMETER_GROUP_MIRACAST       (5 << 16)
 
 /* Common parameters */
 enum mmal_parameter_common_type {
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [Outreachy kernel] [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com
  2017-09-26 15:32 ` [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com Keerthi Reddy
@ 2017-09-26 15:49   ` Julia Lawall
  2017-09-26 16:30     ` [Outreachy kernel] [PATCH v2 1/3] er...@anholt.net, stefan...@i2se.com, outreach...@googlegroups.com Keerthi Reddy
  0 siblings, 1 reply; 8+ messages in thread
From: Julia Lawall @ 2017-09-26 15:49 UTC (permalink / raw)
  To: Keerthi Reddy
  Cc: This, patchset, removes, extra, newlines, within, file,
	outreachy-kernel

Something is wrong with the subject line here.

julia

On Tue, 26 Sep 2017, Keerthi Reddy wrote:

> Blank lines use up extra space in file and makes the file
> larger. So do not use multiple blanklines
>
> Signed-off-by: Keerthi Reddy <keerthigd4990@gmail.com>
> ---
> Changes in v2:
> * Capitalize starting letter in commit message.
>
>  drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c     | 11 -----------
>  drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c   | 11 -----------
>  .../vc04_services/interface/vchi/connections/connection.h     |  3 ---
>  .../vc04_services/interface/vchi/message_drivers/message.h    |  5 -----
>  drivers/staging/vc04_services/interface/vchi/vchi.h           | 10 ----------
>  drivers/staging/vc04_services/interface/vchi/vchi_common.h    |  6 ------
>  drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c |  9 ---------
>  drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h |  5 -----
>  .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c    | 10 ----------
>  .../staging/vc04_services/interface/vchiq_arm/vchiq_core.h    |  1 -
>  .../staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c |  3 ---
>  .../staging/vc04_services/interface/vchiq_arm/vchiq_shim.c    |  3 ---
>  12 files changed, 77 deletions(-)
>
> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
> index 94654c0..7e68b3e 100644
> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c
> @@ -65,7 +65,6 @@ void bcm2835_playback_fifo(struct bcm2835_alsa_stream *alsa_stream)
>  	unsigned int consumed = 0;
>  	int new_period = 0;
>
> -
>  	audio_info("alsa_stream=%p substream=%p\n", alsa_stream,
>  		alsa_stream ? alsa_stream->substream : 0);
>
> @@ -111,7 +110,6 @@ static int snd_bcm2835_playback_open_generic(
>  	int idx;
>  	int err;
>
> -
>  	if (mutex_lock_interruptible(&chip->audio_mutex)) {
>  		audio_error("Interrupted whilst waiting for lock\n");
>  		return -EINTR;
> @@ -184,7 +182,6 @@ static int snd_bcm2835_playback_open_generic(
>  out:
>  	mutex_unlock(&chip->audio_mutex);
>
> -
>  	return err;
>  }
>
> @@ -207,7 +204,6 @@ static int snd_bcm2835_playback_close(struct snd_pcm_substream *substream)
>  	struct snd_pcm_runtime *runtime;
>  	struct bcm2835_alsa_stream *alsa_stream;
>
> -
>  	chip = snd_pcm_substream_chip(substream);
>  	if (mutex_lock_interruptible(&chip->audio_mutex)) {
>  		audio_error("Interrupted whilst waiting for lock\n");
> @@ -259,7 +255,6 @@ static int snd_bcm2835_pcm_hw_params(struct snd_pcm_substream *substream,
>  	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
>  	int err;
>
> -
>  	err = snd_pcm_lib_malloc_pages(substream, params_buffer_bytes(params));
>  	if (err < 0) {
>  		audio_error
> @@ -289,7 +284,6 @@ static int snd_bcm2835_pcm_prepare(struct snd_pcm_substream *substream)
>  	int channels;
>  	int err;
>
> -
>  	if (mutex_lock_interruptible(&chip->audio_mutex))
>  		return -EINTR;
>
> @@ -307,13 +301,11 @@ static int snd_bcm2835_pcm_prepare(struct snd_pcm_substream *substream)
>  	if (err < 0)
>  		audio_error(" error setting hw params\n");
>
> -
>  	bcm2835_audio_setup(alsa_stream);
>
>  	/* in preparation of the stream, set the controls (volume level) of the stream */
>  	bcm2835_audio_set_ctls(alsa_stream->chip);
>
> -
>  	memset(&alsa_stream->pcm_indirect, 0, sizeof(alsa_stream->pcm_indirect));
>
>  	alsa_stream->pcm_indirect.hw_buffer_size =
> @@ -364,7 +356,6 @@ static int snd_bcm2835_pcm_trigger(struct snd_pcm_substream *substream, int cmd)
>  	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
>  	int err = 0;
>
> -
>  	switch (cmd) {
>  	case SNDRV_PCM_TRIGGER_START:
>  		audio_debug("bcm2835_AUDIO_TRIGGER_START running=%d\n",
> @@ -416,7 +407,6 @@ snd_bcm2835_pcm_pointer(struct snd_pcm_substream *substream)
>  	struct snd_pcm_runtime *runtime = substream->runtime;
>  	struct bcm2835_alsa_stream *alsa_stream = runtime->private_data;
>
> -
>  	audio_debug("pcm_pointer... (%d) hwptr=%d appl=%d pos=%d\n", 0,
>  		frames_to_bytes(runtime, runtime->status->hw_ptr),
>  		frames_to_bytes(runtime, runtime->control->appl_ptr),
> @@ -493,7 +483,6 @@ int snd_bcm2835_new_pcm(struct bcm2835_chip *chip, u32 numchannels)
>  					      snd_bcm2835_playback_hw.buffer_bytes_max,
>  					      snd_bcm2835_playback_hw.buffer_bytes_max);
>
> -
>  out:
>  	mutex_unlock(&chip->audio_mutex);
>
> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
> index 5f3d8f2..7469c9b 100644
> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c
> @@ -337,7 +337,6 @@ static int vc_vchi_audio_deinit(struct bcm2835_audio_instance *instance)
>  {
>  	unsigned int i;
>
> -
>  	if (!instance) {
>  		LOG_ERR("%s: invalid handle %p\n", __func__, instance);
>
> @@ -369,7 +368,6 @@ static int vc_vchi_audio_deinit(struct bcm2835_audio_instance *instance)
>
>  	kfree(instance);
>
> -
>  	return 0;
>  }
>
> @@ -382,7 +380,6 @@ static int bcm2835_audio_open_connection(struct bcm2835_alsa_stream *alsa_stream
>  		(struct bcm2835_audio_instance *)alsa_stream->instance;
>  	int ret;
>
> -
>  	LOG_INFO("%s: start\n", __func__);
>  	BUG_ON(instance);
>  	if (instance) {
> @@ -443,7 +440,6 @@ int bcm2835_audio_open(struct bcm2835_alsa_stream *alsa_stream)
>  	int status;
>  	int ret;
>
> -
>  	my_workqueue_init(alsa_stream);
>
>  	ret = bcm2835_audio_open_connection(alsa_stream);
> @@ -491,7 +487,6 @@ static int bcm2835_audio_set_ctls_chan(struct bcm2835_alsa_stream *alsa_stream,
>  	int status;
>  	int ret;
>
> -
>  	LOG_INFO(" Setting ALSA dest(%d), volume(%d)\n",
>  		 chip->dest, chip->volume);
>
> @@ -575,7 +570,6 @@ int bcm2835_audio_set_params(struct bcm2835_alsa_stream *alsa_stream,
>  	int status;
>  	int ret;
>
> -
>  	LOG_INFO(" Setting ALSA channels(%d), samplerate(%d), bits-per-sample(%d)\n",
>  		 channels, samplerate, bps);
>
> @@ -636,7 +630,6 @@ int bcm2835_audio_set_params(struct bcm2835_alsa_stream *alsa_stream,
>  int bcm2835_audio_setup(struct bcm2835_alsa_stream *alsa_stream)
>  {
>
> -
>  	return 0;
>  }
>
> @@ -647,7 +640,6 @@ static int bcm2835_audio_start_worker(struct bcm2835_alsa_stream *alsa_stream)
>  	int status;
>  	int ret;
>
> -
>  	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
>  		LOG_DBG("Interrupted whilst waiting for lock on (%d)\n",
>  			instance->num_connections);
> @@ -684,7 +676,6 @@ static int bcm2835_audio_stop_worker(struct bcm2835_alsa_stream *alsa_stream)
>  	int status;
>  	int ret;
>
> -
>  	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
>  		LOG_DBG("Interrupted whilst waiting for lock on (%d)\n",
>  			instance->num_connections);
> @@ -722,7 +713,6 @@ int bcm2835_audio_close(struct bcm2835_alsa_stream *alsa_stream)
>  	int status;
>  	int ret;
>
> -
>  	my_workqueue_quit(alsa_stream);
>
>  	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
> @@ -780,7 +770,6 @@ static int bcm2835_audio_write_worker(struct bcm2835_alsa_stream *alsa_stream,
>  	int status;
>  	int ret;
>
> -
>  	LOG_INFO(" Writing %d bytes from %p\n", count, src);
>
>  	if (mutex_lock_interruptible(&instance->vchi_mutex)) {
> diff --git a/drivers/staging/vc04_services/interface/vchi/connections/connection.h b/drivers/staging/vc04_services/interface/vchi/connections/connection.h
> index e793cdf..67c8438 100644
> --- a/drivers/staging/vc04_services/interface/vchi/connections/connection.h
> +++ b/drivers/staging/vc04_services/interface/vchi/connections/connection.h
> @@ -54,7 +54,6 @@ typedef struct opaque_vchi_connection_info_t VCHI_CONNECTION_STATE_T;
>
>  typedef struct vchi_connection_t VCHI_CONNECTION_T;
>
> -
>  /******************************************************************************
>   API
>   *****************************************************************************/
> @@ -212,7 +211,6 @@ typedef void * (*VCHI_BUFFER_ALLOCATE)(VCHI_CONNECTION_SERVICE_HANDLE_T service_
>  // free memory allocated by buffer_allocate
>  typedef void   (*VCHI_BUFFER_FREE)(VCHI_CONNECTION_SERVICE_HANDLE_T service_handle, void * address);
>
> -
>  /******************************************************************************
>   System driver struct
>   *****************************************************************************/
> @@ -321,7 +319,6 @@ struct vchi_connection_t {
>  #endif
>  };
>
> -
>  #endif /* CONNECTION_H_ */
>
>  /****************************** End of file **********************************/
> diff --git a/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h b/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
> index a7740a4..834263f 100644
> --- a/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
> +++ b/drivers/staging/vc04_services/interface/vchi/message_drivers/message.h
> @@ -41,7 +41,6 @@
>  #include "interface/vchi/vchi_cfg_internal.h"
>  #include "interface/vchi/vchi_common.h"
>
> -
>  typedef enum message_event_type {
>     MESSAGE_EVENT_NONE,
>     MESSAGE_EVENT_NOP,
> @@ -111,7 +110,6 @@ typedef struct rx_bulk_slotinfo_t {
>     VCHI_FLAGS_T flags;
>  } RX_BULK_SLOTINFO_T;
>
> -
>  /* ----------------------------------------------------------------------
>   * each connection driver will have a pool of the following struct.
>   *
> @@ -155,7 +153,6 @@ typedef struct {
>
>  } MESSAGE_EVENT_T;
>
> -
>  // callbacks
>  typedef void VCHI_MESSAGE_DRIVER_EVENT_CALLBACK_T( void *state );
>
> @@ -163,7 +160,6 @@ typedef struct {
>     VCHI_MESSAGE_DRIVER_EVENT_CALLBACK_T *event_callback;
>  } VCHI_MESSAGE_DRIVER_OPEN_T;
>
> -
>  // handle to this instance of message driver (as returned by ->open)
>  typedef struct opaque_mhandle_t *VCHI_MDRIVER_HANDLE_T;
>
> @@ -195,7 +191,6 @@ struct opaque_vchi_message_driver_t {
>     void    (*debug)( VCHI_MDRIVER_HANDLE_T *handle );
>  };
>
> -
>  #endif // _VCHI_MESSAGE_H_
>
>  /****************************** End of file ***********************************/
> diff --git a/drivers/staging/vc04_services/interface/vchi/vchi.h b/drivers/staging/vc04_services/interface/vchi/vchi.h
> index addb7b0..ee92189 100644
> --- a/drivers/staging/vc04_services/interface/vchi/vchi.h
> +++ b/drivers/staging/vc04_services/interface/vchi/vchi.h
> @@ -39,7 +39,6 @@
>  #include "interface/vchi/connections/connection.h"
>  #include "vchi_mh.h"
>
> -
>  /******************************************************************************
>   Global defs
>   *****************************************************************************/
> @@ -92,7 +91,6 @@ typedef struct vchi_msg_vector_ex {
>     } u;
>  } VCHI_MSG_VECTOR_EX_T;
>
> -
>  // Construct an entry in a msg vector for a pointer (p) of length (l)
>  #define VCHI_VEC_POINTER(p,l)  VCHI_VEC_POINTER, { { (VCHI_MEM_HANDLE_T)(p), (l) } }
>
> @@ -103,7 +101,6 @@ typedef struct vchi_msg_vector_ex {
>  #define MAKE_FOURCC(x) ((int32_t)( (x[0] << 24) | (x[1] << 16) | (x[2] << 8) | x[3] ))
>  #define FOURCC_TO_CHAR(x) (x >> 24) & 0xFF,(x >> 16) & 0xFF,(x >> 8) & 0xFF, x & 0xFF
>
> -
>  // Opaque service information
>  struct opaque_vchi_service_t;
>
> @@ -114,8 +111,6 @@ typedef struct {
>     void *message;
>  } VCHI_HELD_MSG_T;
>
> -
> -
>  // structure used to provide the information needed to open a server or a client
>  typedef struct {
>  	struct vchi_version version;
> @@ -162,7 +157,6 @@ extern "C" {
>  extern /*@observer@*/ VCHI_CONNECTION_T * vchi_create_connection( const VCHI_CONNECTION_API_T * function_table,
>                                                     const VCHI_MESSAGE_DRIVER_T * low_level);
>
> -
>  // Routine used to initialise the vchi on both local + remote connections
>  extern int32_t vchi_initialise( VCHI_INSTANCE_T *instance_handle );
>
> @@ -185,7 +179,6 @@ extern void * vchi_allocate_buffer(VCHI_SERVICE_HANDLE_T handle, uint32_t *lengt
>  extern void vchi_free_buffer(VCHI_SERVICE_HANDLE_T handle, void *address);
>  extern uint32_t vchi_current_time(VCHI_INSTANCE_T instance_handle);
>
> -
>  /******************************************************************************
>   Global service API
>   *****************************************************************************/
> @@ -307,7 +300,6 @@ extern int32_t vchi_msg_iter_hold_next( VCHI_MSG_ITER_T *iter,
>                                          uint32_t *msg_size, // }
>                                          VCHI_HELD_MSG_T *message );
>
> -
>  /******************************************************************************
>   Global bulk API
>   *****************************************************************************/
> @@ -319,7 +311,6 @@ extern int32_t vchi_bulk_queue_receive( VCHI_SERVICE_HANDLE_T handle,
>                                          VCHI_FLAGS_T flags,
>                                          void *transfer_handle );
>
> -
>  // Prepare interface for a transfer from the other side into relocatable memory.
>  int32_t vchi_bulk_queue_receive_reloc( const VCHI_SERVICE_HANDLE_T handle,
>                                         VCHI_MEM_HANDLE_T h_dst,
> @@ -335,7 +326,6 @@ extern int32_t vchi_bulk_queue_transmit( VCHI_SERVICE_HANDLE_T handle,
>                                           VCHI_FLAGS_T flags,
>                                           void *transfer_handle );
>
> -
>  /******************************************************************************
>   Configuration plumbing
>   *****************************************************************************/
> diff --git a/drivers/staging/vc04_services/interface/vchi/vchi_common.h b/drivers/staging/vc04_services/interface/vchi/vchi_common.h
> index 45c2070..76e10fe 100644
> --- a/drivers/staging/vc04_services/interface/vchi/vchi_common.h
> +++ b/drivers/staging/vc04_services/interface/vchi/vchi_common.h
> @@ -34,7 +34,6 @@
>  #ifndef VCHI_COMMON_H_
>  #define VCHI_COMMON_H_
>
> -
>  //flags used when sending messages (must be bitmapped)
>  typedef enum {
>     VCHI_FLAGS_NONE                      = 0x0,
> @@ -118,14 +117,11 @@ typedef enum {
>     VCHI_SERVICE_OPTION_MAX
>  } VCHI_SERVICE_OPTION_T;
>
> -
>  //Callback used by all services / bulk transfers
>  typedef void (*VCHI_CALLBACK_T)(void *callback_param, //my service local param
>  				 VCHI_CALLBACK_REASON_T reason,
>  				 void *handle); //for transmitting msg's only
>
> -
> -
>  /*
>   * Define vector struct for scatter-gather (vector) operations
>   * Vectors can be nested - if a vector element has negative length, then
> @@ -154,7 +150,6 @@ typedef struct opaque_vchi_connection_api_t VCHI_CONNECTION_API_T;
>  // Opaque type for a message driver
>  typedef struct opaque_vchi_message_driver_t VCHI_MESSAGE_DRIVER_T;
>
> -
>  // Iterator structure for reading ahead through received message queue. Allocated by client,
>  // initialised by vchi_msg_look_ahead. Fields are for internal VCHI use only.
>  // Iterates over messages in queue at the instant of the call to vchi_msg_lookahead -
> @@ -168,5 +163,4 @@ typedef struct {
>     void *remove;
>  } VCHI_MSG_ITER_T;
>
> -
>  #endif // VCHI_COMMON_H_
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
> index d23152b..6fcc626 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
> @@ -109,10 +109,8 @@ static const char *const resume_state_names[] = {
>   * requested */
>  #define FORCE_SUSPEND_TIMEOUT_MS 200
>
> -
>  static void suspend_timer_callback(unsigned long context);
>
> -
>  typedef struct user_service_struct {
>  	VCHIQ_SERVICE_T *service;
>  	void *userdata;
> @@ -2156,8 +2154,6 @@ vchiq_keepalive_thread_func(void *v)
>  	return 0;
>  }
>
> -
> -
>  VCHIQ_STATUS_T
>  vchiq_arm_init_state(VCHIQ_STATE_T *state, VCHIQ_ARM_STATE_T *arm_state)
>  {
> @@ -2318,7 +2314,6 @@ set_resume_state(VCHIQ_ARM_STATE_T *arm_state,
>  	}
>  }
>
> -
>  /* should be called with the write lock held */
>  inline void
>  start_suspend_timer(VCHIQ_ARM_STATE_T *arm_state)
> @@ -2438,7 +2433,6 @@ vchiq_arm_vcsuspend(VCHIQ_STATE_T *state)
>  	vchiq_log_trace(vchiq_susp_log_level, "%s", __func__);
>  	status = VCHIQ_SUCCESS;
>
> -
>  	switch (arm_state->vc_suspend_state) {
>  	case VC_SUSPEND_REQUESTED:
>  		vchiq_log_info(vchiq_susp_log_level, "%s: suspend already "
> @@ -2503,7 +2497,6 @@ vchiq_platform_check_suspend(VCHIQ_STATE_T *state)
>  	return;
>  }
>
> -
>  static void
>  output_timeout_error(VCHIQ_STATE_T *state)
>  {
> @@ -2683,7 +2676,6 @@ vchiq_check_suspend(VCHIQ_STATE_T *state)
>  	return;
>  }
>
> -
>  int
>  vchiq_arm_allow_resume(VCHIQ_STATE_T *state)
>  {
> @@ -2845,7 +2837,6 @@ vchiq_use_internal(VCHIQ_STATE_T *state, VCHIQ_SERVICE_T *service,
>  			"%s %s count %d, state count %d",
>  			__func__, entity, *entity_uc, local_uc);
>
> -
>  	write_unlock_bh(&arm_state->susp_res_lock);
>
>  	/* Completion is in a done state when we're not suspended, so this won't
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
> index bfbd81d..a4cc0db 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h
> @@ -42,7 +42,6 @@
>  #include "vchiq_core.h"
>  #include "vchiq_debugfs.h"
>
> -
>  enum vc_suspend_status {
>  	VC_SUSPEND_FORCE_CANCELED = -3, /* Force suspend canceled, too busy */
>  	VC_SUSPEND_REJECTED = -2,  /* Videocore rejected suspend request */
> @@ -61,15 +60,12 @@ enum vc_resume_status {
>  	VC_RESUME_RESUMED      /* Videocore resumed successfully (active) */
>  };
>
> -
>  enum USE_TYPE_E {
>  	USE_TYPE_SERVICE,
>  	USE_TYPE_SERVICE_NO_RESUME,
>  	USE_TYPE_VCHIQ
>  };
>
> -
> -
>  typedef struct vchiq_arm_state_struct {
>  	/* Keepalive-related data */
>  	struct task_struct *ka_thread;
> @@ -216,5 +212,4 @@ set_resume_state(VCHIQ_ARM_STATE_T *arm_state,
>  extern void
>  start_suspend_timer(VCHIQ_ARM_STATE_T *arm_state);
>
> -
>  #endif /* VCHIQ_ARM_H */
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> index 486be99..c3bb8e0 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c
> @@ -130,7 +130,6 @@ static const char *const conn_state_names[] = {
>  	"RESUME_TIMEOUT"
>  };
>
> -
>  static void
>  release_message_sync(VCHIQ_STATE_T *state, VCHIQ_HEADER_T *header);
>
> @@ -2168,7 +2167,6 @@ slot_handler_func(void *v)
>  				break;
>  			}
>
> -
>  		}
>
>  		DEBUG_TRACE(SLOT_HANDLER_LINE);
> @@ -2177,7 +2175,6 @@ slot_handler_func(void *v)
>  	return 0;
>  }
>
> -
>  /* Called by the recycle thread */
>  static int
>  recycle_func(void *v)
> @@ -2193,7 +2190,6 @@ recycle_func(void *v)
>  	return 0;
>  }
>
> -
>  /* Called by the sync thread */
>  static int
>  sync_func(void *v)
> @@ -2301,7 +2297,6 @@ sync_func(void *v)
>  	return 0;
>  }
>
> -
>  static void
>  init_bulk_queue(VCHIQ_BULK_QUEUE_T *queue)
>  {
> @@ -2312,14 +2307,12 @@ init_bulk_queue(VCHIQ_BULK_QUEUE_T *queue)
>  	queue->remove = 0;
>  }
>
> -
>  inline const char *
>  get_conn_state_name(VCHIQ_CONNSTATE_T conn_state)
>  {
>  	return conn_state_names[conn_state];
>  }
>
> -
>  VCHIQ_SLOT_ZERO_T *
>  vchiq_init_slots(void *mem_base, int mem_size)
>  {
> @@ -3296,7 +3289,6 @@ vchiq_remove_service(VCHIQ_SERVICE_HANDLE_T handle)
>  	return status;
>  }
>
> -
>  /* This function may be called by kernel threads or user threads.
>   * User threads may receive VCHIQ_RETRY to indicate that a signal has been
>   * received and the call should be retried after being returned to user
> @@ -3876,7 +3868,6 @@ vchiq_dump_service_state(void *dump_context, VCHIQ_SERVICE_T *service)
>  		vchiq_dump_platform_service_state(dump_context, service);
>  }
>
> -
>  void
>  vchiq_loud_error_header(void)
>  {
> @@ -3901,7 +3892,6 @@ vchiq_loud_error_footer(void)
>  		"================");
>  }
>
> -
>  VCHIQ_STATUS_T vchiq_send_remote_use(VCHIQ_STATE_T *state)
>  {
>  	VCHIQ_STATUS_T status = VCHIQ_RETRY;
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> index 9e16465..afc1d81 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.h
> @@ -702,7 +702,6 @@ vchiq_platform_handle_timeout(VCHIQ_STATE_T *state);
>  extern void
>  vchiq_set_conn_state(VCHIQ_STATE_T *state, VCHIQ_CONNSTATE_T newstate);
>
> -
>  extern void
>  vchiq_log_dump_mem(const char *label, uint32_t addr, const void *voidMem,
>  	size_t numBytes);
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
> index 9367a9a..766b4fe 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_debugfs.c
> @@ -32,7 +32,6 @@
>   * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>   */
>
> -
>  #include <linux/debugfs.h>
>  #include "vchiq_core.h"
>  #include "vchiq_arm.h"
> @@ -52,7 +51,6 @@
>  #define VCHIQ_LOG_INFO_STR    "info"
>  #define VCHIQ_LOG_TRACE_STR   "trace"
>
> -
>  /* Top-level debug info */
>  struct vchiq_debugfs_info {
>  	/* Global 'vchiq' debugfs entry used by all instances */
> @@ -316,7 +314,6 @@ void vchiq_debugfs_remove_instance(VCHIQ_INSTANCE_T instance)
>  	debugfs_remove_recursive(node->dentry);
>  }
>
> -
>  int vchiq_debugfs_init(void)
>  {
>  	BUG_ON(debugfs_info.vchiq_cfg_dir != NULL);
> diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
> index 631ff73..d465e1c 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
> @@ -84,7 +84,6 @@ VCHI_CONNECTION_T *vchi_create_connection(
>   *             void **data,
>   *             uint32_t *msg_size,
>
> -
>   *             VCHI_FLAGS_T flags
>   *
>   * Description: Routine to return a pointer to the current message (to allow in
> @@ -541,7 +540,6 @@ int32_t vchi_connect(VCHI_CONNECTION_T **connections,
>  }
>  EXPORT_SYMBOL(vchi_connect);
>
> -
>  /***********************************************************
>   * Name: vchi_disconnect
>   *
> @@ -561,7 +559,6 @@ int32_t vchi_disconnect(VCHI_INSTANCE_T instance_handle)
>  }
>  EXPORT_SYMBOL(vchi_disconnect);
>
> -
>  /***********************************************************
>   * Name: vchi_service_open
>   * Name: vchi_service_create
> --
> 2.7.4
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/960f1d14d6ccc60acd7abb8e5ae9d6198031ca2b.1506439214.git.keerthigd4990%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>


^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [Outreachy kernel] [PATCH v2 1/3] er...@anholt.net, stefan...@i2se.com, outreach...@googlegroups.com
  2017-09-26 15:49   ` [Outreachy kernel] " Julia Lawall
@ 2017-09-26 16:30     ` Keerthi Reddy
  2017-09-26 16:38       ` Julia Lawall
  0 siblings, 1 reply; 8+ messages in thread
From: Keerthi Reddy @ 2017-09-26 16:30 UTC (permalink / raw)
  To: outreachy-kernel


[-- Attachment #1.1: Type: text/plain, Size: 414 bytes --]


> Something is wrong with the subject line here.
Sorry, after writing the send recipients, I have accidentally pasted same 
the recipients in
the subject line too.

 I guess the commit messages didn't change. To avoid confusion, should I 
resend the
patches? 

--Keerthi

On Tuesday, September 26, 2017 at 9:19:48 PM UTC+5:30, Julia Lawall wrote:
>
> Something is wrong with the subject line here. 
>
> julia 
>
>

[-- Attachment #1.2: Type: text/html, Size: 682 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [Outreachy kernel] [PATCH v2 1/3] er...@anholt.net, stefan...@i2se.com, outreach...@googlegroups.com
  2017-09-26 16:30     ` [Outreachy kernel] [PATCH v2 1/3] er...@anholt.net, stefan...@i2se.com, outreach...@googlegroups.com Keerthi Reddy
@ 2017-09-26 16:38       ` Julia Lawall
  0 siblings, 0 replies; 8+ messages in thread
From: Julia Lawall @ 2017-09-26 16:38 UTC (permalink / raw)
  To: Keerthi Reddy; +Cc: outreachy-kernel

[-- Attachment #1: Type: text/plain, Size: 1215 bytes --]



On Tue, 26 Sep 2017, Keerthi Reddy wrote:

>
> > Something is wrong with the subject line here.Sorry, after writing the
> send recipients, I have accidentally pasted same the recipients in
> the subject line too.
>
> ï¿œI guess the commit messages didn't change. To avoid confusion, should I
> resend the
> patches?ￜ

A patch needs a proper subject line.  So you need to send a v2 (or vn+1).
When I responded to your mail, I got a lot of bounces as well, so check
that the email addresses are correct.

julia

>
> --Keerthi
>
> On Tuesday, September 26, 2017 at 9:19:48 PM UTC+5:30, Julia Lawall wrote:
>       Something is wrong with the subject line here.
>
>       julia
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/a7a1e451-c61d-44e2-a27a-
> 72175db2c61a%40googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>
>

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] staging: vc04_services: add spaces around << operator
  2017-09-26 15:33 ` [PATCH v2 3/3] staging: vc04_services: add spaces around << operator Keerthi Reddy
@ 2017-09-26 16:44   ` Stefan Wahren
  0 siblings, 0 replies; 8+ messages in thread
From: Stefan Wahren @ 2017-09-26 16:44 UTC (permalink / raw)
  To: Keerthi Reddy; +Cc: eric, outreachy-kernel

Hi Keerthi,

> Keerthi Reddy <keerthigd4990@gmail.com> hat am 26. September 2017 um 17:33 geschrieben:
> 
> 
> Follow coding convention by adding providing space
> around << operators
> 
> Signed-off-by: Keerthi Reddy <keerthigd4990@gmail.com>
> ---
>  changes in v2:
>  * No changes
> 
>  .../vc04_services/bcm2835-camera/mmal-msg.h        | 22 +++++++++++-----------
>  .../vc04_services/bcm2835-camera/mmal-parameters.h | 12 ++++++------
>  2 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
> index 1144d9d..0dfad29 100644
> --- a/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
> +++ b/drivers/staging/vc04_services/bcm2835-camera/mmal-msg.h
> @@ -217,36 +217,36 @@ struct mmal_msg_port_action_reply {
>  #define MMAL_VC_SHORT_DATA 128
>  
>  /** Signals that the current payload is the end of the stream of data */
> -#define MMAL_BUFFER_HEADER_FLAG_EOS                    (1<<0)
> +#define MMAL_BUFFER_HEADER_FLAG_EOS                    (1 << 0)
>  /** Signals that the start of the current payload starts a frame */
> -#define MMAL_BUFFER_HEADER_FLAG_FRAME_START            (1<<1)
> +#define MMAL_BUFFER_HEADER_FLAG_FRAME_START            (1 << 1)
>  /** Signals that the end of the current payload ends a frame */
> -#define MMAL_BUFFER_HEADER_FLAG_FRAME_END              (1<<2)
> +#define MMAL_BUFFER_HEADER_FLAG_FRAME_END              (1 << 2)
>  /** Signals that the current payload contains only complete frames (>1) */
>  #define MMAL_BUFFER_HEADER_FLAG_FRAME                  \
>  	(MMAL_BUFFER_HEADER_FLAG_FRAME_START|MMAL_BUFFER_HEADER_FLAG_FRAME_END)
>  /** Signals that the current payload is a keyframe (i.e. self decodable) */
> -#define MMAL_BUFFER_HEADER_FLAG_KEYFRAME               (1<<3)
> +#define MMAL_BUFFER_HEADER_FLAG_KEYFRAME               (1 << 3)
>  /** Signals a discontinuity in the stream of data (e.g. after a seek).
>   * Can be used for instance by a decoder to reset its state
>   */
> -#define MMAL_BUFFER_HEADER_FLAG_DISCONTINUITY          (1<<4)
> +#define MMAL_BUFFER_HEADER_FLAG_DISCONTINUITY          (1 << 4)
>  /** Signals a buffer containing some kind of config data for the component
>   * (e.g. codec config data)
>   */
> -#define MMAL_BUFFER_HEADER_FLAG_CONFIG                 (1<<5)
> +#define MMAL_BUFFER_HEADER_FLAG_CONFIG                 (1 << 5)
>  /** Signals an encrypted payload */
> -#define MMAL_BUFFER_HEADER_FLAG_ENCRYPTED              (1<<6)
> +#define MMAL_BUFFER_HEADER_FLAG_ENCRYPTED              (1 << 6)
>  /** Signals a buffer containing side information */
> -#define MMAL_BUFFER_HEADER_FLAG_CODECSIDEINFO          (1<<7)
> +#define MMAL_BUFFER_HEADER_FLAG_CODECSIDEINFO          (1 << 7)
>  /** Signals a buffer which is the snapshot/postview image from a stills
>   * capture
>   */
> -#define MMAL_BUFFER_HEADER_FLAGS_SNAPSHOT              (1<<8)
> +#define MMAL_BUFFER_HEADER_FLAGS_SNAPSHOT              (1 << 8)
>  /** Signals a buffer which contains data known to be corrupted */
> -#define MMAL_BUFFER_HEADER_FLAG_CORRUPTED              (1<<9)
> +#define MMAL_BUFFER_HEADER_FLAG_CORRUPTED              (1 << 9)
>  /** Signals that a buffer failed to be transmitted */
> -#define MMAL_BUFFER_HEADER_FLAG_TRANSMISSION_FAILED    (1<<10)
> +#define MMAL_BUFFER_HEADER_FLAG_TRANSMISSION_FAILED    (1 << 10)
>  

i agree with Julia to use the BIT macro for those flags. But in this case the patch must be split.

Thanks
Stefan


^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-09-26 16:44 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-26 15:28 [PATCH v2 0/3] Fix spaces and typos in vc04_services Keerthi Reddy
2017-09-26 15:32 ` [PATCH v2 1/3] eric@anholt.net, stefan.wahren@i2se.com, outreachy-kernel@googlegroups.com Keerthi Reddy
2017-09-26 15:49   ` [Outreachy kernel] " Julia Lawall
2017-09-26 16:30     ` [Outreachy kernel] [PATCH v2 1/3] er...@anholt.net, stefan...@i2se.com, outreach...@googlegroups.com Keerthi Reddy
2017-09-26 16:38       ` Julia Lawall
2017-09-26 15:32 ` [PATCH v2 2/3] staging: vc04_services: fix typos Keerthi Reddy
2017-09-26 15:33 ` [PATCH v2 3/3] staging: vc04_services: add spaces around << operator Keerthi Reddy
2017-09-26 16:44   ` Stefan Wahren

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.