linux-kernel - Re: [PATCH v11 0/7] [media] Add analog mode support for Medion MD95700

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 29 Apr 2019 14:00:37 +0200
From:   Hans Verkuil <hverkuil@...all.nl>
To:     "Maciej S. Szmigiero" <mail@...iej.szmigiero.name>
Cc:     Michael Krufky <mkrufky@...uxtv.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Andy Walls <awalls@...metrocast.net>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        linux-media@...r.kernel.org
Subject: Re: [PATCH v11 0/7] [media] Add analog mode support for Medion
 MD95700

On 4/29/19 1:42 PM, Maciej S. Szmigiero wrote:
> Hi Hans,
> 
> On 29.04.2019 10:17, Hans Verkuil wrote:
>> Hi Maciej,
>>
>> I reviewed v11 and I had only two comments in two sources. Can you fix that
>> and post a v12?
> 
> Thanks for looking at this quickly, will do.
> 
>> Mike, once v12 is posted, can you try and test it on your hardware?
>>
>> The goal is to get this merged for 5.3.
>>
>> Maciej, just FYI: I won't merge the last patch. It's OK to post it, that
>> way it is archived if anyone is ever interested in this in the future, but
>> it's not something that we want to merge. It adds a new public API to enable
>> this, and that's not something we'd like to do unless there is a really
>> good reason. And adding a debug feature for old hardware isn't enough of a
>> reason. Sorry.
> 
> :(
> 
> I was hoping that maybe something like CONFIG_VIDEO_ADV_DEBUG will do the
> trick.
> 
> I'm afraid that if this is out-of-tree it will quickly bitrot since people
> will submit patches to "optimize" the in-kernel code unknowingly breaking
> compatibility with this patch.
> 
> But I respect your decision here.

If analog TV was still a hot new topic, then this would be interesting to add.
But the reality is that we see very little activity in that area. In fact, I'm
impressed by your perseverance with this patch series since you have been
working on this since 2017!

Regards,

	Hans