[U-Boot] [PATCH v2 1/7] video: Kconfig: Add VIDEO_IPV3 entry

[Jagan Teki]

Hi Stefano,

On Tue, Nov 29, 2016 at 11:22 PM, Stefano Babic <sbabic at denx.de> wrote:
> Hi Jagan,
>
> On 28/10/2016 15:57, Jagan Teki wrote:
>> From: Jagan Teki <jagan at amarulasolutions.com>
>>
>> Added kconfig entry for CONFIG_VIDEO_IPV3 driver.
>>
>> Cc: Anatolij Gustschin <agust at denx.de>
>> Cc: Stefano Babic <sbabic at denx.de>
>> Cc: Matteo Lisi <matteo.lisi at engicam.com>
>> Cc: Michael Trimarchi <michael at amarulasolutions.com>
>> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
>> ---
>>  drivers/video/Kconfig | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
>> index 577e6d7..134f1c8 100644
>> --- a/drivers/video/Kconfig
>> +++ b/drivers/video/Kconfig
>> @@ -443,6 +443,14 @@ config VIDEO
>>         model. Video drivers typically provide a colour text console and
>>         cursor.
>>
>> +config VIDEO_IPUV3
>> +     bool "i.MX IPUv3 Core video support"
>> +     depends on MX6
>> +     default y if VIDEO
>> +     help
>> +       This enables framebuffer driver for i.MX processors working
>> +       on the IPUv3(Image Processing Unit) internal graphic processor.
>> +
>>  config CFB_CONSOLE
>>       bool "Enable colour frame buffer console"
>>       depends on VIDEO
>>
>
> I have difficulties with your patches. They have updated and moved
> configuration to Kconfig, but they do not care of the existing boards.
>
> This series breaks a lot of boards except yours - I cannot merge it.
>
> The same thing happens to the series I have already merged (but not
> pushed to the server) with "Use CONFIG_DM_ETH support". I have merged it
> and pushed to u-boot-imx, -next, so you can easier see what happens.
> Both series break existing board and I cannot take them, and even if
> they are improving i.MX code with new features.
>
> Can you take a look at fix them in the order:
>
> - first the FEC series ==> see -next branch

Found the issue and made a changes according to previous version
patches which eventually works on all targets.

> - then the IPU (this series).

Fixed and will send the

And also I've one more series with engicam boards [1], shall I send
all these 3 series into one on top of u-boot-imx/master?

[1] [PATCH v3 00/11] imx6: Add Engicam GEAM6UL/i.CoreM6 RQS board support

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.