From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753517AbcC1FOM (ORCPT ); Mon, 28 Mar 2016 01:14:12 -0400 Received: from lists.s-osg.org ([54.187.51.154]:58286 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752536AbcC1FOI (ORCPT ); Mon, 28 Mar 2016 01:14:08 -0400 Subject: Re: [PATCH v2] ARM: exynos_defconfig: Enable Samsung media platform drivers as modules To: Krzysztof Kozlowski , linux-kernel@vger.kernel.org References: <1458779208-6144-1-git-send-email-javier@osg.samsung.com> <56F33C46.8060501@samsung.com> <56F4AD53.2020307@osg.samsung.com> <56F8B2D0.9010609@samsung.com> Cc: linux-samsung-soc@vger.kernel.org, Tobias Jakobi , Kukjin Kim From: Javier Martinez Canillas Message-ID: <56F8BD99.9050907@osg.samsung.com> Date: Mon, 28 Mar 2016 01:14:01 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <56F8B2D0.9010609@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Krzysztof, On 03/28/2016 12:28 AM, Krzysztof Kozlowski wrote: > On 25.03.2016 12:15, Javier Martinez Canillas wrote: >>> >>> How about doing the same for multi_v7? >>> >> >> I didn't consider multi_v7 because media drivers aren't necessary for booting >> the boards and so it could increase build times for not real benefits in most >> machines. But I can enable it in multi_v7 as a follow-up if you think that it >> makes sense there too. > > I consider use cases of multi_v7 the same as exynos (except the > difference in policy: use always modules). This means that if it makes > sense to enable some media drivers on exynos (for build coverage, boot > testing and real usage) then it makes sense to do the same for multi_v7. > Ok, thanks a lot for your suggestion. I'll post a patch for multi_v7 then. > In ideal future we would move entirely from exynos-specific image to one > common image for testing: the multi_v7. It would make things simpler. > > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America