[PATCH] staging: ion: Align cases with switch

Laura Abbott labbott at redhat.com
Fri Sep 23 22:11:23 UTC 2016


On 09/23/2016 01:40 PM, Christopher Pezley wrote:
> The preferred indentation for cases and switches has the cases at
> the same level as the switch.
>

Please remember to tag your patches with v2 when sending the
next version to tell them apart (e.g. [PATCHv2])

Acked-by: Laura Abbott <labbott at redhat.com>

> Signed-off-by: Christopher H. Pezley <chris at pezley.net>
> ---
>  drivers/staging/android/ion/ion_of.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c
> index de0899a..3c050b0 100644
> --- a/drivers/staging/android/ion/ion_of.c
> +++ b/drivers/staging/android/ion/ion_of.c
> @@ -58,15 +58,15 @@ int ion_setup_heap_common(struct platform_device *parent,
>  	int ret = 0;
>
>  	switch (heap->type) {
> -		case ION_HEAP_TYPE_CARVEOUT:
> -		case ION_HEAP_TYPE_CHUNK:
> -			if (heap->base && heap->size)
> -				return 0;
> -
> -			ret = of_reserved_mem_device_init(heap->priv);
> -			break;
> -		default:
> -			break;
> +	case ION_HEAP_TYPE_CARVEOUT:
> +	case ION_HEAP_TYPE_CHUNK:
> +		if (heap->base && heap->size)
> +			return 0;
> +
> +		ret = of_reserved_mem_device_init(heap->priv);
> +		break;
> +	default:
> +		break;
>  	}
>
>  	return ret;
>



More information about the devel mailing list