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
| ||
|
Message-ID: <4xm4v5gq72klvg4xk4fzpyuui7hmmh63gdoflghwmo7pvid4ub@l22jnobt6n5j> Date: Fri, 6 Sep 2024 18:03:01 +0200 From: Andi Shyti <andi.shyti@...nel.org> To: Kimriver Liu <kimriver.liu@...ngine.com> Cc: jarkko.nikula@...ux.intel.com, andriy.shevchenko@...ux.intel.com, mika.westerberg@...ux.intel.com, jsd@...ihalf.com, linux-i2c@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] i2c: designware: fix master is holding SCL low while ENABLE bit is disabled Hi Kimriver, On Fri, Sep 06, 2024 at 03:47:31PM GMT, Kimriver Liu wrote: > It was observed issuing ABORT bit(IC_ENABLE[1]) will not work when > IC_ENABLE is already disabled. > > Check if ENABLE bit(IC_ENABLE[0]) is disabled when the master is > holding SCL low. If ENABLE bit is disabled, the software need > enable it before trying to issue ABORT bit. otherwise, > the controller ignores any write to ABORT bit. > > Signed-off-by: Kimriver Liu <kimriver.liu@...ngine.com> > > --- > V5->V6: restore i2c_dw_is_master_idling() function checking > V4->V5: delete master idling checking > V3->V4: > 1. update commit messages and add patch version and changelog > 2. move print the error message in i2c_dw_xfer > V2->V3: change (!enable) to (!(enable & DW_IC_ENABLE_ENABLE)) > V1->V2: used standard words in function names and addressed review comments > > link to V1: > https://lore.kernel.org/lkml/20240904064224.2394-1-kimriver.liu@siengine.com/ You've ignored all of Andy's reviews, and this is not acceptable. You've submitted this patch three times today in a short period, and Andy has responded to each one with the same comment, which you continue to disregard. You must follow up on the reviews. I will accept this patch for now, but if you keep ignoring feedback, I will reject future patches until you address all reviews from those taking their time to review your changes. Andi
Powered by blists - more mailing lists