[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200923060228.GA1515@sultan-box.localdomain>
Date: Tue, 22 Sep 2020 23:02:28 -0700
From: Sultan Alsawaf <sultan@...neltoast.com>
To: Jiri Kosina <jikos@...nel.org>
Cc: Wolfram Sang <wsa@...nel.org>, linux-i2c@...r.kernel.org,
aaron.ma@...onical.com, admin@...ma.net,
andriy.shevchenko@...ux.intel.com, benjamin.tissoires@...hat.com,
hdegoede@...hat.com, hn.chen@...dahitech.com,
jarkko.nikula@...ux.intel.com, kai.heng.feng@...onical.com,
linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
mika.westerberg@...ux.intel.com, vicamo.yang@...onical.com
Subject: Re: [PATCH v2 0/4] i2c-hid: Save power by reducing i2c xfers with
block reads
On Tue, Sep 22, 2020 at 09:59:44PM +0200, Jiri Kosina wrote:
> On Tue, 22 Sep 2020, Wolfram Sang wrote:
>
> > > Hans, Benjamin, could you please give this patchset some smoke-testing? It
> > > looks good to me, but I'd like it to get some testing from your testing
> > > machinery before merging.
> >
> > Please give me some more days. I am not fully convinced yet that this
> > use of I2C_M_RECV_LEN is not broken on some controllers.
> >
> > Plus, I'd favor if this could go via I2C tree. It is within I2C where
> > the non-trivial changes are. The HID part is just the final bit. Can we
> > agree on that?
>
> Absolutely no problem with that. But I'd like to have this ran through
> Benjamin/Hans first too.
>
> Thanks,
>
> --
> Jiri Kosina
> SUSE Labs
>
I suppose the HID part does need to be held off until all the adapters are
updated with functional I2C_M_RECV_LEN bits.
I just got a Ryzen laptop which panics when using I2C_M_RECV_LEN.
So it looks like only the designware changes can be considered for merging now.
Sultan
Powered by blists - more mailing lists