diff mbox

[2/2] ARM: multi_v7_defconfig: Enable Exynos MFC driver as module

Message ID 1462914448-24480-2-git-send-email-javier@osg.samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Javier Martinez Canillas May 10, 2016, 9:07 p.m. UTC
Exynos SoCs have a Multi Format Video Hardware Codec (MFC) IP block that
can be used to {en,de}code video streams by hardware. Enable support for
its driver as a module so the MFC can be tested.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

This patch depends on https://patchwork.kernel.org/patch/8970711/

 arch/arm/configs/multi_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski May 11, 2016, 6:27 a.m. UTC | #1
On 05/10/2016 11:07 PM, Javier Martinez Canillas wrote:
> Exynos SoCs have a Multi Format Video Hardware Codec (MFC) IP block that
> can be used to {en,de}code video streams by hardware. Enable support for
> its driver as a module so the MFC can be tested.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> 
> ---
> 
> This patch depends on https://patchwork.kernel.org/patch/8970711/
> 
>  arch/arm/configs/multi_v7_defconfig | 1 +
>  1 file changed, 1 insertion(+)


Both patches look good but only exynos_defconfig ended in Patchwork.
This one did not... because you forgot to Cc the samsung-soc list.

Best regards,
Krzysztod
Javier Martinez Canillas May 11, 2016, 12:14 p.m. UTC | #2
Hello Krzysztof,

On 05/11/2016 02:27 AM, Krzysztof Kozlowski wrote:
> On 05/10/2016 11:07 PM, Javier Martinez Canillas wrote:
>> Exynos SoCs have a Multi Format Video Hardware Codec (MFC) IP block that
>> can be used to {en,de}code video streams by hardware. Enable support for
>> its driver as a module so the MFC can be tested.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>
>> ---
>>
>> This patch depends on https://patchwork.kernel.org/patch/8970711/
>>
>>  arch/arm/configs/multi_v7_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
> 
> 
> Both patches look good but only exynos_defconfig ended in Patchwork.
> This one did not... because you forgot to Cc the samsung-soc list.
>

Sigh, this time I used the cc list suggested by get_maintainers.pl and
forgot to double check if was complete. I'll resend and cc samsung-soc.
 
> Best regards,
> Krzysztod
> 

Best regards,
Krzysztof Kozlowski May 11, 2016, 12:17 p.m. UTC | #3
On 05/11/2016 02:14 PM, Javier Martinez Canillas wrote:
> Hello Krzysztof,
> 
> On 05/11/2016 02:27 AM, Krzysztof Kozlowski wrote:
>> On 05/10/2016 11:07 PM, Javier Martinez Canillas wrote:
>>> Exynos SoCs have a Multi Format Video Hardware Codec (MFC) IP block that
>>> can be used to {en,de}code video streams by hardware. Enable support for
>>> its driver as a module so the MFC can be tested.
>>>
>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>>
>>> ---
>>>
>>> This patch depends on https://patchwork.kernel.org/patch/8970711/
>>>
>>>  arch/arm/configs/multi_v7_defconfig | 1 +
>>>  1 file changed, 1 insertion(+)
>>
>>
>> Both patches look good but only exynos_defconfig ended in Patchwork.
>> This one did not... because you forgot to Cc the samsung-soc list.
>>
> 
> Sigh, this time I used the cc list suggested by get_maintainers.pl and
> forgot to double check if was complete. I'll resend and cc samsung-soc.

No need. I found it on arm-kernel patchwork and I will apply it from there.

Best regards,
Krzysztof
Javier Martinez Canillas May 11, 2016, 12:18 p.m. UTC | #4
Hello Krzysztof,

On 05/11/2016 08:17 AM, Krzysztof Kozlowski wrote:
> On 05/11/2016 02:14 PM, Javier Martinez Canillas wrote:
>> Hello Krzysztof,
>>
>> On 05/11/2016 02:27 AM, Krzysztof Kozlowski wrote:
>>> On 05/10/2016 11:07 PM, Javier Martinez Canillas wrote:
>>>> Exynos SoCs have a Multi Format Video Hardware Codec (MFC) IP block that
>>>> can be used to {en,de}code video streams by hardware. Enable support for
>>>> its driver as a module so the MFC can be tested.
>>>>
>>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>>>
>>>> ---
>>>>
>>>> This patch depends on https://patchwork.kernel.org/patch/8970711/
>>>>
>>>>  arch/arm/configs/multi_v7_defconfig | 1 +
>>>>  1 file changed, 1 insertion(+)
>>>
>>>
>>> Both patches look good but only exynos_defconfig ended in Patchwork.
>>> This one did not... because you forgot to Cc the samsung-soc list.
>>>
>>
>> Sigh, this time I used the cc list suggested by get_maintainers.pl and
>> forgot to double check if was complete. I'll resend and cc samsung-soc.
> 
> No need. I found it on arm-kernel patchwork and I will apply it from there.
>

Great, thanks a lot for your help.
 
> Best regards,
> Krzysztof
> 

Best regards,
diff mbox

Patch

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 2ba3f0cd2261..8086e9e13ff6 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -534,6 +534,7 @@  CONFIG_VIDEO_EXYNOS_FIMC_LITE=m
 CONFIG_VIDEO_EXYNOS4_FIMC_IS=m
 CONFIG_V4L_MEM2MEM_DRIVERS=y
 CONFIG_VIDEO_SAMSUNG_S5P_JPEG=m
+CONFIG_VIDEO_SAMSUNG_S5P_MFC=m
 CONFIG_VIDEO_RENESAS_JPU=m
 CONFIG_VIDEO_RENESAS_VSP1=m
 # CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set