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

From: Peter Rosin
Date: Fri Jul 14 2017 - 01:10:31 EST

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:// 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

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


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