RE: [PATCH v4 29/34] Intel tsens i2c slave driver.

[Gross, Mark]

> -----Original Message-----
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Sent: Sunday, January 31, 2021 6:14 PM
> To: mgross@xxxxxxxxxxxxxxx; markgross@xxxxxxxxxx; arnd@xxxxxxxx; bp@xxxxxxx;
> damien.lemoal@xxxxxxx; dragan.cvetic@xxxxxxxxxx;
> gregkh@xxxxxxxxxxxxxxxxxxx; corbet@xxxxxxx; palmerdabbelt@xxxxxxxxxx;
> paul.walmsley@xxxxxxxxxx; peng.fan@xxxxxxx; robh+dt@xxxxxxxxxx;
> shawnguo@xxxxxxxxxx; jassisinghbrar@xxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx; C, Udhayakumar <udhayakumar.c@xxxxxxxxx>
> Subject: Re: [PATCH v4 29/34] Intel tsens i2c slave driver.
> 
> On 1/29/21 6:21 PM, mgross@xxxxxxxxxxxxxxx wrote:
> > diff --git a/drivers/misc/intel_tsens/Kconfig b/drivers/misc/intel_tsens/Kconfig
> > index 8b263fdd80c3..be8d27e81864 100644
> > --- a/drivers/misc/intel_tsens/Kconfig
> > +++ b/drivers/misc/intel_tsens/Kconfig
> > @@ -14,6 +14,20 @@ config INTEL_TSENS_LOCAL_HOST
> >  	  Say Y if using a processor that includes the Intel VPU such as
> >  	  Keem Bay.  If unsure, say N.
> >
> > +config INTEL_TSENS_I2C_SLAVE
> > +	bool "I2C slave driver for intel tsens"
> > +	depends on INTEL_TSENS_LOCAL_HOST
> > +	depends on I2C=y && I2C_SLAVE
> > +	help
> > +	  This option enables tsens I2C slave driver.
> 
> Good, it's OK now.
> The repeat v3 and then v4 confused me.

Hmm, I'm confused too.  I should have only sent v4 on Friday.  I'll look over my history and logs to see how I could have sent to blasts one or V3 and then V4.   Ugh.  Sorry about mixing the V3 and V4 patches in the same batch.

FWIW I make an outgoing directory and forgot to clean out the v3 patches that where there from the earlier posting. Resulting in a mixing of the old and current patch set. 

I'll write a script to automate the creation of the outgoing directory for next time that should correct future screw ups.

I'm very sorry for the noise.

--mark