linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry
@ 2013-06-17 16:42 Sylwester Nawrocki
  2013-06-18  3:39 ` Sachin Kamat
  0 siblings, 1 reply; 4+ messages in thread
From: Sylwester Nawrocki @ 2013-06-17 16:42 UTC (permalink / raw)
  To: linux-media; +Cc: Sylwester Nawrocki, Kyungmin Park

This change is mainly to update the driver's path changed from
drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
While at it, remove non-existent files rule, move the whole entry to
the Samsung drivers section and add the patch tracking system URL.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 MAINTAINERS |   17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3d7782b..d2c5618 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1153,15 +1153,6 @@ L:	linux-media@vger.kernel.org
 S:	Maintained
 F:	drivers/media/platform/s5p-g2d/
 
-ARM/SAMSUNG S5P SERIES FIMC SUPPORT
-M:	Kyungmin Park <kyungmin.park@samsung.com>
-M:	Sylwester Nawrocki <s.nawrocki@samsung.com>
-L:	linux-arm-kernel@lists.infradead.org
-L:	linux-media@vger.kernel.org
-S:	Maintained
-F:	arch/arm/plat-samsung/include/plat/*fimc*
-F:	drivers/media/platform/s5p-fimc/
-
 ARM/SAMSUNG S5P SERIES Multi Format Codec (MFC) SUPPORT
 M:	Kyungmin Park <kyungmin.park@samsung.com>
 M:	Kamil Debski <k.debski@samsung.com>
@@ -6930,6 +6921,14 @@ F:	drivers/regulator/s5m*.c
 F:	drivers/rtc/rtc-sec.c
 F:	include/linux/mfd/samsung/
 
+SAMSUNG S5P/EXYNOS4 SOC SERIES CAMERA SUBSYSTEM DRIVERS
+M:	Kyungmin Park <kyungmin.park@samsung.com>
+M:	Sylwester Nawrocki <s.nawrocki@samsung.com>
+L:	linux-media@vger.kernel.org
+Q:	https://patchwork.linuxtv.org/project/linux-media/list/
+S:	Supported
+F:	drivers/media/platform/exynos4-is/
+
 SAMSUNG S3C24XX/S3C64XX SOC SERIES CAMIF DRIVER
 M:	Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
 L:	linux-media@vger.kernel.org
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry
  2013-06-17 16:42 [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry Sylwester Nawrocki
@ 2013-06-18  3:39 ` Sachin Kamat
  2013-06-18 21:38   ` Sylwester Nawrocki
  0 siblings, 1 reply; 4+ messages in thread
From: Sachin Kamat @ 2013-06-18  3:39 UTC (permalink / raw)
  To: Sylwester Nawrocki; +Cc: linux-media, Kyungmin Park

Hi Sylwester,

Just a couple of nits inline.

On 17 June 2013 22:12, Sylwester Nawrocki <s.nawrocki@samsung.com> wrote:
> This change is mainly to update the driver's path changed from
> drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
> While at it, remove non-existent files rule, move the whole entry to
> the Samsung drivers section and add the patch tracking system URL.

How about adding git URL too (of your repo)?

>
> Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> ---
>  MAINTAINERS |   17 ++++++++---------
>  1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3d7782b..d2c5618 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1153,15 +1153,6 @@ L:       linux-media@vger.kernel.org
>  S:     Maintained
>  F:     drivers/media/platform/s5p-g2d/
>
> -ARM/SAMSUNG S5P SERIES FIMC SUPPORT
> -M:     Kyungmin Park <kyungmin.park@samsung.com>
> -M:     Sylwester Nawrocki <s.nawrocki@samsung.com>
> -L:     linux-arm-kernel@lists.infradead.org
> -L:     linux-media@vger.kernel.org
> -S:     Maintained
> -F:     arch/arm/plat-samsung/include/plat/*fimc*
> -F:     drivers/media/platform/s5p-fimc/
> -
>  ARM/SAMSUNG S5P SERIES Multi Format Codec (MFC) SUPPORT

Probably ARM could be removed from here too and may be other places if
they exist just like below entry.

>  M:     Kyungmin Park <kyungmin.park@samsung.com>
>  M:     Kamil Debski <k.debski@samsung.com>
> @@ -6930,6 +6921,14 @@ F:       drivers/regulator/s5m*.c
>  F:     drivers/rtc/rtc-sec.c
>  F:     include/linux/mfd/samsung/
>
> +SAMSUNG S5P/EXYNOS4 SOC SERIES CAMERA SUBSYSTEM DRIVERS
> +M:     Kyungmin Park <kyungmin.park@samsung.com>
> +M:     Sylwester Nawrocki <s.nawrocki@samsung.com>
> +L:     linux-media@vger.kernel.org
> +Q:     https://patchwork.linuxtv.org/project/linux-media/list/
> +S:     Supported
> +F:     drivers/media/platform/exynos4-is/
> +

Considering alphabetical order (now that ARM is removed), this block
should come after
SAMSUNG S3C24XX/S3C64XX...

>  SAMSUNG S3C24XX/S3C64XX SOC SERIES CAMIF DRIVER
>  M:     Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
>  L:     linux-media@vger.kernel.org
> --


-- 
With warm regards,
Sachin

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry
  2013-06-18  3:39 ` Sachin Kamat
@ 2013-06-18 21:38   ` Sylwester Nawrocki
  0 siblings, 0 replies; 4+ messages in thread
From: Sylwester Nawrocki @ 2013-06-18 21:38 UTC (permalink / raw)
  To: Sachin Kamat
  Cc: Sylwester Nawrocki, linux-media, Kyungmin Park, linux-samsung-soc

Hi Sachin,

Thanks for your review.

On 06/18/2013 05:39 AM, Sachin Kamat wrote:
> Hi Sylwester,
>
> Just a couple of nits inline.
>
> On 17 June 2013 22:12, Sylwester Nawrocki<s.nawrocki@samsung.com>  wrote:
>> This change is mainly to update the driver's path changed from
>> drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
>> While at it, remove non-existent files rule, move the whole entry to
>> the Samsung drivers section and add the patch tracking system URL.
>
> How about adding git URL too (of your repo)?

Yes, I guess I should add it.

>> Signed-off-by: Sylwester Nawrocki<s.nawrocki@samsung.com>
>> Signed-off-by: Kyungmin Park<kyungmin.park@samsung.com>
>> ---
>>   MAINTAINERS |   17 ++++++++---------
>>   1 file changed, 8 insertions(+), 9 deletions(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 3d7782b..d2c5618 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1153,15 +1153,6 @@ L:       linux-media@vger.kernel.org
>>   S:     Maintained
>>   F:     drivers/media/platform/s5p-g2d/
>>
>> -ARM/SAMSUNG S5P SERIES FIMC SUPPORT
>> -M:     Kyungmin Park<kyungmin.park@samsung.com>
>> -M:     Sylwester Nawrocki<s.nawrocki@samsung.com>
>> -L:     linux-arm-kernel@lists.infradead.org
>> -L:     linux-media@vger.kernel.org
>> -S:     Maintained
>> -F:     arch/arm/plat-samsung/include/plat/*fimc*
>> -F:     drivers/media/platform/s5p-fimc/
>> -
>>   ARM/SAMSUNG S5P SERIES Multi Format Codec (MFC) SUPPORT
>
> Probably ARM could be removed from here too and may be other places if
> they exist just like below entry.

Yes, IMHO those driver entries are misplaced now. I guess the better
place would be at the Samsung entries section.

>>   M:     Kyungmin Park<kyungmin.park@samsung.com>
>>   M:     Kamil Debski<k.debski@samsung.com>
>> @@ -6930,6 +6921,14 @@ F:       drivers/regulator/s5m*.c
>>   F:     drivers/rtc/rtc-sec.c
>>   F:     include/linux/mfd/samsung/
>>
>> +SAMSUNG S5P/EXYNOS4 SOC SERIES CAMERA SUBSYSTEM DRIVERS
>> +M:     Kyungmin Park<kyungmin.park@samsung.com>
>> +M:     Sylwester Nawrocki<s.nawrocki@samsung.com>
>> +L:     linux-media@vger.kernel.org
>> +Q:     https://patchwork.linuxtv.org/project/linux-media/list/
>> +S:     Supported
>> +F:     drivers/media/platform/exynos4-is/
>> +
>
> Considering alphabetical order (now that ARM is removed), this block
> should come after SAMSUNG S3C24XX/S3C64XX...

Oops, right, thanks for spotting this.

>>   SAMSUNG S3C24XX/S3C64XX SOC SERIES CAMIF DRIVER
>>   M:     Sylwester Nawrocki<sylvester.nawrocki@gmail.com>
>>   L:     linux-media@vger.kernel.org

Regards,
Sylwester

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry
@ 2013-06-21 13:11 Sylwester Nawrocki
  0 siblings, 0 replies; 4+ messages in thread
From: Sylwester Nawrocki @ 2013-06-21 13:11 UTC (permalink / raw)
  To: linux-media; +Cc: kyungmin.park, linux-samsung-soc, Sylwester Nawrocki

This change is mainly to update the driver's path changed from
drivers/media/platform/s5p-fimc to drivers/media/platform/exynos4-is/.
While at it, remove non-existent files rule, move the whole entry to
the Samsung drivers section and add the patch tracking system URL
and git repository details.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
Changes since v1:
 - added SCM details, moved to proper location to keep
   alphabetical order
---
 MAINTAINERS |   19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3d7782b..22435e8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1153,15 +1153,6 @@ L:	linux-media@vger.kernel.org
 S:	Maintained
 F:	drivers/media/platform/s5p-g2d/
 
-ARM/SAMSUNG S5P SERIES FIMC SUPPORT
-M:	Kyungmin Park <kyungmin.park@samsung.com>
-M:	Sylwester Nawrocki <s.nawrocki@samsung.com>
-L:	linux-arm-kernel@lists.infradead.org
-L:	linux-media@vger.kernel.org
-S:	Maintained
-F:	arch/arm/plat-samsung/include/plat/*fimc*
-F:	drivers/media/platform/s5p-fimc/
-
 ARM/SAMSUNG S5P SERIES Multi Format Codec (MFC) SUPPORT
 M:	Kyungmin Park <kyungmin.park@samsung.com>
 M:	Kamil Debski <k.debski@samsung.com>
@@ -6945,6 +6936,16 @@ L:	linux-media@vger.kernel.org
 S:	Supported
 F:	drivers/media/i2c/s5c73m3/*
 
+SAMSUNG S5P/EXYNOS4 SOC SERIES CAMERA SUBSYSTEM DRIVERS
+M:	Kyungmin Park <kyungmin.park@samsung.com>
+M:	Sylwester Nawrocki <s.nawrocki@samsung.com>
+L:	linux-media@vger.kernel.org
+Q:	https://patchwork.linuxtv.org/project/linux-media/list/
+T:	git git://linuxtv.org/snawrocki/samsung.git
+S:	Supported
+F:	drivers/media/platform/exynos4-is/
+F:	include/media/s5p_fimc.h
+
 SERIAL DRIVERS
 M:	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 L:	linux-serial@vger.kernel.org
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-06-21 13:11 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-17 16:42 [PATCH] MAINTAINERS: Update S5P/Exynos FIMC driver entry Sylwester Nawrocki
2013-06-18  3:39 ` Sachin Kamat
2013-06-18 21:38   ` Sylwester Nawrocki
  -- strict thread matches above, loose matches on Subject: below --
2013-06-21 13:11 Sylwester Nawrocki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).