Re: [PATCH] [media] media: rename media unregister function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>, Linux Media Mailing List <linux-media@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH] [media] media: rename media unregister function
From: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
Date: Fri, 18 Mar 2016 10:48:13 -0300
Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>, Kyungmin Park <kyungmin.park@xxxxxxxxxxx>, Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>, Kukjin Kim <kgene@xxxxxxxxxx>, Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>, Hyun Kwon <hyun.kwon@xxxxxxxxxx>, Michal Simek <michal.simek@xxxxxxxxxx>, Sören Brinkmann <soren.brinkmann@xxxxxxxxxx>, Antti Palosaari <crope@xxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Jaroslav Kysela <perex@xxxxxxxx>, Takashi Iwai <tiwai@xxxxxxxx>, Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>, Hans Verkuil <hans.verkuil@xxxxxxxxx>, Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>, Junghak Sung <jh1009.sung@xxxxxxxxxxx>, Inki Dae <inki.dae@xxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Geunyoung Kim <nenggun.kim@xxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Shuah Khan <shuahkh@xxxxxxxxxxxxxxx>, Rafael Lourenço de Lima Chehab <chehabrafael@xxxxxxxxx>, Tommi Rantala <tt.rantala@xxxxxxxxx>, Matthias Schwarzott <zzam@xxxxxxxxxx>, Patrick Boettcher <patrick.boettcher@xxxxxxxxx>, Luis de Bethencourt <luis@xxxxxxxxxxxxxxxxx>, Amitoj Kaur Chawla <amitoj1606@xxxxxxxxx>, Julia Lawall <Julia.Lawall@xxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-samsung-soc@xxxxxxxxxxxxxxx, linux-renesas-soc@xxxxxxxxxxxxxxx, devel@xxxxxxxxxxxxxxxxxxxx, alsa-devel@xxxxxxxxxxxxxxxx
In-reply-to: <2ffc02c944068b2c8655727238d1542f8328385d.1458306276.git.mchehab@osg.samsung.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0

Hello Mauro,

On 03/18/2016 10:05 AM, Mauro Carvalho Chehab wrote:
> Now that media_device_unregister() also does a cleanup, rename it
> to media_device_unregister_cleanup().
>

I believe there should be a Suggested-by Sakari Ailus tag here.
 
> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>

The patch looks good and I agree that makes things more clear.

Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

References:
- [PATCH] [media] media: rename media unregister function
  - From: Mauro Carvalho Chehab

Prev by Date: [PATCH] [media] media: rename media unregister function
Next by Date: Re: [PATCH] [media] media: rename media unregister function
Previous by thread: [PATCH] [media] media: rename media unregister function
Next by thread: Re: [PATCH] [media] media: rename media unregister function
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SOC] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Samba] [Device Mapper]