linux-kernel - Re: Kernel touch Kconfig consult

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-next>] [day] [month] [year] [list]

Message-ID: <20190623070249.GD204275@dtor-ws>
Date:   Sun, 23 Jun 2019 00:02:49 -0700
From:   Dmitry Torokhov <dmitry.torokhov@...il.com>
To:     "luhua.xu" <luhua.xu@...iatek.com>
Cc:     Rob Herring <robh@...nel.org>, Marek Vasut <marex@...x.de>,
        Nick Dyer <nick.dyer@...ev.co.uk>,
        Richard Leitner <richard.leitner@...data.com>,
        Martin Kepplinger <martink@...teo.de>,
        linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
        weiqi.fu@...iatek.com, wsd_upstream@...iatek.com
Subject: Re: Kernel touch Kconfig consult

Hi,

On Fri, Jun 14, 2019 at 06:47:19AM -0400, luhua.xu wrote:
> Hi Dmitry,Rob,Marek, Nick,Richard,Martin,
> 
> In our  customer support experience, many smartphone have two or three
> touch vendor mixture , and customer use one load to support all touches.
> For easy to config touch driver  we use kernel config like this down
> below,
>  
> We change the config type from 'bool' to 'string'.
>  
> config TOUCHSCREEN_MTK_TOUCH
>   string "Touch IC name for Mediatek package"
>   help
>     Set touch IC name if you have touch panel.
>     To compile this dirver for used touch IC.
>  
> 
> And we config touch driver like this：
> CONFIG_TOUCHSCREEN_MTK_TOUCH="GT9886 GT1151 TD4320"
>  
> I only use one config to support  3 touches, while we have to use 3
> config to support 3  touch drivers if we set the config as 'bool'.
> 
> So can I use Kconfig like this?
> I do look forward to receiving your reply at your convenience .
> 

I really do not see why having a sting is easier to have than 3 bools,
especially if they pertain to different touch controllers. You must also
have some custom processing of the config above as I am pretty sure our
standard build tools would not work for it.

Thanks.

-- 
Dmitry