Hi Guenter, I've spoken with Derek, and we are in agreement this should be moved. Which part of the patch series do you want me to put the move in? Or is it something you want to do yourself prior to/after the patch series. lmk and I will coordinate for V2 As for @Joaquin, can you ack the move since you are the module author? Antheas On Fri, 27 Dec 2024 at 18:12, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > > On Thu, Dec 26, 2024 at 01:16:15PM -0800, Derek J. Clark wrote: > > > > > > On December 26, 2024 1:08:02 PM PST, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > > >On Thu, Dec 26, 2024 at 12:27:30PM +0100, Antheas Kapenekakis wrote: > > >> This three part series updates the oxpsensors module to bring it in line > > >> with its Windows OneXPlayer counterpart. First, it adds support for all > > >> 2024 OneXPlayer handhelds and their special variants. > > >> > > >> Then, it adds the new charge limiting and bypass features that were first > > >> introduced in the X1 and retrofit to older OneXFly variants and for > > >> controlling the turbo led found in the X1 models. For Bypass, it adds a new > > >> bypass variant BypassS0 that is only active while the device is in the S0 > > >> state. > > >> > > > > > >This is a hardware monitoring driver. It is not a charge controller driver, > > >and it is not a LED controller driver. If such control is wanted/needed for > > >this system, it should be implemented either as mfd device with client drivers, > > >or the entire driver should be moved to platform drivers if there is a desire > > >to keep it as single driver. > > > > > >Guenter > > > > I think moving this to x86 platform makes a lot of sense to ensure two separate drivers can't do async writes to the EC. We probably should have done that when adding the turbo button toggle anyway. I'll coordinate that effort with Tobias and Antheas directly before moving forward. > > > > Fine with me, but "ensure two drivers can't do asynchronous writes" is not > really an argument. mfd supports that as well for pretty much all mfd > drivers. > > Guenter
