n Thu, Dec 29, 2016 at 5:29 PM, Sebastian Reichel <sre@xxxxxxxxxx> wrote: > On Wed, Dec 28, 2016 at 01:50:17PM +0100, Linus Walleij wrote: >> On Sun, Dec 11, 2016 at 11:21 PM, Bartosz Golaszewski >> <bgolaszewski@xxxxxxxxxxxx> wrote: >> >> > Some power-measuring ADCs work together with power load switches which >> > allow to power-cycle measured devices. >> > >> > An example use case would be measuring the power consumption of a >> > development board during boot using a power monitor such as TI INA226 >> > and power-cycling the board remotely using a TPS229* power switch. >> > >> > Add an iio driver for simple GPIO power switches and expose a sysfs >> > attribute allowing to toggle their state. >> > >> > Signed-off-by: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> >> >> I don't get this, isn't this doing the same as >> drivers/power/reset/gpio-poweroff.c >> ? >> >> With the only difference that the latter uses the standard syscall >> from pm_power_off to reboot the system instead of some random >> sysfs file. > > As far as I understand it, the TPS229 is used by Barzosz to poweroff > a remote system. The gpio-poweroff driver is used to poweroff the > local system. Thanks yeah I understood this from the context of later patches. Well if such a property is used it should be the property of the remote system per se, and the remote system should then also be desribed in DT, not half-described by dangling references at random nodes. So this needs to be re-architected to either describe the remote system in DT and handle it in the kernel, or handle it all from userspace if it is a one-off non-product thing. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html