unfortunately I still need to re-test with that commit reverted
On Thu, Mar 26, 2020 at 03:52:58PM -0500, Matt DeVillier wrote:
> On Thu, Mar 26, 2020 at 2:33 PM Paul Menzel <pmenzel@molgen.mpg.de> wrote:
> > Am 26.03.20 um 20:29 schrieb Matt DeVillier:
> > > as requested I went ahead and retested a few different Chromebook
> > > devices/platforms using upstream coreboot + SeaBIOS master, with the
> > > stock EC firmware, to asses the current state of the PS2 keyboard
> > > function:
> > >
> > > google/link (IvyBridge): bad (good w/EC update)
> > > google/wolf (Haswell): good
> > > google/clapper (Baytrail): good (with HARDWARE_IRQ unset)
> > > google/lulu (Broadwell): good
> > > google/chell (Skylake): good (with HARDWARE_IRQ unset)
> > > google/cyan (Braswell): good (with HARDWARE_IRQ unset)
> > >
> > > so I think the patch is no longer needed, since LINK is the only board
> > > which is problematic, and I'm not even sure the patch addresses that;
> > > pretty sure the fix needs to be on the EC end.
> >
> > Thank you so much going through this tedious QA work.
> >
> > Just to be sure, by *master* you mean without your merged commit
> > bfdb3f86 (ps2port: adjust init routine to fix PS/2 keyboard issues),
> > that mean a commit before or with the commit reverted?
> >
>
> I apologize, I didn't realize this commit had been merged already, so
> was testing
> against current master (de88a96) which includes this patch. I will
> need to re-test
> against 29ee1fb or earlier to see if it's actually needed
Any further updates on this?
Thanks,
-Kevin