linux-kernel - Re: [PATCH 0/2] i2c: move include files out of include/linux/i2c

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

Date:   Fri, 14 Jul 2017 07:10:02 +0200
From:   Peter Rosin <peda@...ntia.se>
To:     Wolfram Sang <wsa@...-dreams.de>, linux-i2c@...r.kernel.org
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/2] i2c: move include files out of include/linux/i2c

On 2017-07-13 11:46, Wolfram Sang wrote:
> Hi peda,
> 
> On Sun, May 21, 2017 at 10:37:41PM +0200, Wolfram Sang wrote:
>> It doesn't make sense to use include/linux/i2c for client drivers which may in
>> fact rather be hwmon or input or whatever devices. As a result, I want to
>> deprecate include/linux/i2c for good. This series moves the include files to a
>> better location, largely include/platform_data because that is what most of the
>> moved include files contain. Note that some files don't seem to have upstream
>> users in board code, so they maybe could even be removed? I didn't check for
>> that now, but I did it for one i2c master driver recently. So, it may be
>> possible.
>>
>> And while i2c muxes might be a corner case, I still want to deprecate
>> include/linux/i2c for good ;)
>>
>> No runtime testing because of no HW, but buildbot is happy with this series at
>> least. A branch can be found here:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/platform_data
> 
> Since there were no other i2c-mux patches this merge window and thus no
> pull request, do you mind if I pick these two patches for my second pull
> request this merge window to have them included in 4.13 already?

I don't seem to have this series in my inbox and not in my i2c folder either?
(and I don't remember having seen the patches, but this was a while ago so I
could easily have forgotten everything about it)

Anyway, I had to lookup the branch to see what this was about. And in that
branch, I find b7f1e84715a1 ("i2c: pca954x: move header file out of I2C realm"),
but I initially didn't find the mlxcpld patch. That was because it had an
"x86: " prefix that I didn't react to. I first found the mlxcpld patch in
patchwork and then I found it in the branch too, but in patchwork it has the
expected "i2c: mux: " prefix so I'm not sure what you are talking about
*exactly*.

But I'm sure you'll do something sensible, just go ahead.

Cheers,
Peter

PS. Please Cc me if you want a faster reaction, I didn't notice this
yesterday.