linux-kernel - Re: [PATCH 0/5] hwmon: 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]

Message-ID: <b876bed9-38dc-233f-053a-4b0d4ad990b2@roeck-us.net>
Date:   Sun, 21 May 2017 20:02:07 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     Wolfram Sang <wsa@...-dreams.de>, linux-i2c@...r.kernel.org
Cc:     linux-doc@...r.kernel.org, linux-hwmon@...r.kernel.org,
        linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/5] hwmon: move include files out of include/linux/i2c

On 05/21/2017 01:34 PM, 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 th > 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

While I understand where you are coming from, I am not typically that aggressive.
Such removals force vendors who are not really forthcoming with upstreaming to
deviate even further from upstream. It makes them even less likely to submit their
code upstream, and it may result in enforcing their belief that upstream doesn't
really care about vendors struggling to release boards and systems to their
customers.

> that now, but I did it for one i2c master driver recently. So, it may be
> possible. pmbus.h got moved just one layer upwards, see the patch description
> there.
> 
> I prefer the series to go upstream via the subsystem tree; if you prefer that I
> take it via I2C, just let me know.
> 
Series applied to hwmon-next.

Thanks,
Guenter

> 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
> 
> Thanks and kind regards,
> 
>     Wolfram
> 
> 
> Wolfram Sang (5):
>    hwmon: ads1015: move header file out of I2C realm
>    hwmon: ds620: move header file out of I2C realm
>    hwmon: ltc4245: move header file out of I2C realm
>    hwmon: max6639: move header file out of I2C realm
>    hwmon: pmbus: move header file out of I2C realm
> 
>   Documentation/hwmon/ads1015                    | 2 +-
>   Documentation/hwmon/ltc4245                    | 2 +-
>   Documentation/hwmon/pmbus-core                 | 2 +-
>   MAINTAINERS                                    | 4 ++--
>   drivers/hwmon/ads1015.c                        | 2 +-
>   drivers/hwmon/ds620.c                          | 2 +-
>   drivers/hwmon/ltc4245.c                        | 2 +-
>   drivers/hwmon/max6639.c                        | 2 +-
>   drivers/hwmon/pmbus/pmbus.c                    | 2 +-
>   drivers/hwmon/pmbus/pmbus_core.c               | 2 +-
>   drivers/hwmon/pmbus/ucd9000.c                  | 2 +-
>   drivers/hwmon/pmbus/ucd9200.c                  | 2 +-
>   drivers/iio/adc/ti-ads1015.c                   | 2 +-
>   include/linux/{i2c => platform_data}/ads1015.h | 0
>   include/linux/{i2c => platform_data}/ds620.h   | 0
>   include/linux/{i2c => platform_data}/ltc4245.h | 0
>   include/linux/{i2c => platform_data}/max6639.h | 0
>   include/linux/{i2c => }/pmbus.h                | 0
>   18 files changed, 14 insertions(+), 14 deletions(-)
>   rename include/linux/{i2c => platform_data}/ads1015.h (100%)
>   rename include/linux/{i2c => platform_data}/ds620.h (100%)
>   rename include/linux/{i2c => platform_data}/ltc4245.h (100%)
>   rename include/linux/{i2c => platform_data}/max6639.h (100%)
>   rename include/linux/{i2c => }/pmbus.h (100%)
>