linux-kernel - RE: [PATCH] i2c: slave: support I2C_SLAVE

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 31 Mar 2020 08:48:03 +0000
From:   "Biwen Li (OSS)" <biwen.li@....nxp.com>
To:     Oleksij Rempel <o.rempel@...gutronix.de>
CC:     "Biwen Li (OSS)" <biwen.li@....nxp.com>,
        Leo Li <leoyang.li@....com>,
        "linux@...pel-privat.de" <linux@...pel-privat.de>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "wsa@...-dreams.de" <wsa@...-dreams.de>,
        Jiafei Pan <jiafei.pan@....com>,
        "linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] i2c: slave: support I2C_SLAVE_STOP event for the read
 transactions

> Hi,
> 
> On Mon, Mar 30, 2020 at 10:56:28AM +0000, Biwen Li wrote:
> > Hi All,
> >
> > The patch based on https://patchwork.ozlabs.org/patch/1203640/
> > and it is not accepted, so it(https://patchwork.ozlabs.org/patch/1203640/)
> is needed before apply the new patch.
> 
> I'm not sure what should I do with this information. This links are to the patch
> which needed to be update by you, before it can be reviewed and then
> hopefully accepted. Why do you sending one more patch for review for a patch
> which is not accepted?
Hi Oleksij,
Okay, got it. I will resend the patch after the old patch is accepted by upstream. Thank you
for your reply.
> 
> > Best Regards,
> > Biwen Li
> > > Support I2C_SLAVE_STOP event for the read transactions(master read
> > > from
> > > slave)
> > >
> > > Signed-off-by: Biwen Li <biwen.li@....com>
> > > ---
> > >  drivers/i2c/busses/i2c-imx.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/i2c/busses/i2c-imx.c
> > > b/drivers/i2c/busses/i2c-imx.c index
> > > 62517a41b32d..1fd0d87885d5 100644
> > > --- a/drivers/i2c/busses/i2c-imx.c
> > > +++ b/drivers/i2c/busses/i2c-imx.c
> > > @@ -1464,6 +1464,7 @@ static irqreturn_t i2c_imx_slave_isr(struct
> > > imx_i2c_struct *i2c_imx)
> > >  		ctl &= ~I2CR_MTX;
> > >  		imx_i2c_write_reg(ctl, i2c_imx, IMX_I2C_I2CR);
> > >  		imx_i2c_read_reg(i2c_imx, IMX_I2C_I2DR);
> > > +		i2c_slave_event(i2c_imx->slave, I2C_SLAVE_STOP, &value);
> > >  	}
> > >  	return IRQ_HANDLED;
> > >  }
> > > --
> > > 2.17.1
> 
> Regards,
> Oleksij
> --
> Pengutronix e.K.                           |
> |
> Steuerwalder Str. 21                       | http://www.pengutronix.de/
> |
> 31137 Hildesheim, Germany                  | Phone:
> +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:
> +49-5121-206917-5555 |