[PATCH linux-next] staging: vchiq_arm: coding style

[Stefan Wahren]

Hi Sergiy,

Am 03.03.2017 um 09:05 schrieb Sergiy Redko:
> fix coding style: move brace to the line above

please make the subject of your patch more distinct. We get a lot of
coding style fixes for vchiq.

Btw it looks you missed to add Greg as staging maintainer.

>
> Signed-off-by: Sergiy Redko <sergredko at gmail.com>
> ---
>  drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> 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 48984abc3854..ca896a518e15 100644
> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_shim.c
> @@ -797,8 +797,7 @@ int32_t vchi_get_peer_version(const VCHI_SERVICE_HANDLE_T handle, short *peer_ve
>  {
>  	int32_t ret = -1;
>  	SHIM_SERVICE_T *service = (SHIM_SERVICE_T *)handle;
> -	if (service)
> -	{
> +	if (service) {

Since you want to fix coding stlye i think it's better to leave an empty
line between the declaration and the if statement.

Stefan

>  		VCHIQ_STATUS_T status;
>  
>  		status = vchiq_get_peer_version(service->handle, peer_version);