blackfin: Kconfig symbol VIDEO_MT9M114?

blackfin: Kconfig symbol VIDEO_MT9M114?

[Paul Bolle]

Steven, Sonic,

Your commit 54e4ff4d4024 ("blackfin: adi gpio driver and pinctrl driver
support") added a check for the Kconfig macro CONFIG_VIDEO_MT9M114. That
commit was added during the v3.13 release cycle.

There's no Kconfig symbol VIDEO_MT9M114 in mainline. And it doesn't look
like this was a typo. So why was that check added?

Regards,


Paul Bolle

Re: blackfin: Kconfig symbol VIDEO_MT9M114?

[Steven Miao]

Hi Paul,

On Wed, Feb 12, 2014 at 6:18 PM, Paul Bolle <pebolle@tiscali.nl> wrote:
> Steven, Sonic,
>
> Your commit 54e4ff4d4024 ("blackfin: adi gpio driver and pinctrl driver
> support") added a check for the Kconfig macro CONFIG_VIDEO_MT9M114. That
> commit was added during the v3.13 release cycle.
>
> There's no Kconfig symbol VIDEO_MT9M114 in mainline. And it doesn't look
> like this was a typo. So why was that check added?
VIDEO_MT9M114 is from patch "Aptina mt9m114 HD digital image sensor
driver", but unfortunately this patch hasn't been accepted yet.
https://patchwork.linuxtv.org/patch/16308/
This Kconfig symbol should be removed until the mt9m114 driver is
accepted by mainline.

>
> Regards,
>
>
> Paul Bolle
>

-steven

Re: blackfin: Kconfig symbol VIDEO_MT9M114?

[Paul Bolle]

Hi Steven,

On Thu, 2014-02-13 at 17:00 +0800, Steven Miao wrote:
> VIDEO_MT9M114 is from patch "Aptina mt9m114 HD digital image sensor
> driver", but unfortunately this patch hasn't been accepted yet.
> https://patchwork.linuxtv.org/patch/16308/
> This Kconfig symbol should be removed until the mt9m114 driver is
> accepted by mainline.

Thanks for looking into this.

It's easy for me to submit a trivial patch to remove the one line
depending on these Kconfig macros. But it seems that line should be
added to the queue for that driver at the same time, otherwise we risk
losing it. So it's safer if you or the people working on that driver
coordinate that, isn't it?


Paul Bolle

RE: blackfin: Kconfig symbol VIDEO_MT9M114?

[Zhang, Sonic]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1330 bytes --]

Hi Paul,

I will send out a patch to remove it.

Regards,

Sonic

>-----Original Message-----
>From: Paul Bolle [mailto:pebolle@tiscali.nl]
>Sent: Thursday, February 13, 2014 5:30 PM
>To: Steven Miao
>Cc: Zhang, Sonic; Richard Weinberger; adi-buildroot-devel@lists.sourceforge.net;
>open list:CAN NETWORK DRIVERS <linux-can@vger.kernel.org>, open
>list:NETWORKING DRIVERS <netdev@vger.kernel.org>, open list
>Subject: Re: blackfin: Kconfig symbol VIDEO_MT9M114?
>
>Hi Steven,
>
>On Thu, 2014-02-13 at 17:00 +0800, Steven Miao wrote:
>> VIDEO_MT9M114 is from patch "Aptina mt9m114 HD digital image sensor
>> driver", but unfortunately this patch hasn't been accepted yet.
>> https://patchwork.linuxtv.org/patch/16308/
>> This Kconfig symbol should be removed until the mt9m114 driver is
>> accepted by mainline.
>
>Thanks for looking into this.
>
>It's easy for me to submit a trivial patch to remove the one line depending on these
>Kconfig macros. But it seems that line should be added to the queue for that driver at
>the same time, otherwise we risk losing it. So it's safer if you or the people working on
>that driver coordinate that, isn't it?
>
>
>Paul Bolle
>

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥