Re: [PATCH v2] i2c: designware: Introduce semaphore reservation timer to AMDPSP driver

From: Jan Dąbroś
Date: Mon Aug 22 2022 - 06:36:09 EST


sob., 20 sie 2022 o 08:30 Wolfram Sang <wsa@xxxxxxxxxx> napisał(a):
>
> On Fri, Aug 12, 2022 at 09:15:26AM +0200, Jan Dabros wrote:
> > In order to optimize performance, limit amount of back and forth
> > transactions between x86 and PSP. This is done by introduction of
> > semaphore reservation period - that is window in which x86 isn't
> > releasing the bus immediately after each I2C transaction.
> >
> > In order to protect PSP from being starved while waiting for
> > arbitration, after a programmed time bus is automatically released by a
> > deferred function.
> >
> > Signed-off-by: Jan Dabros <jsd@xxxxxxxxxxxx>
>
> Fixed this checkpatch check:
>
> CHECK: Blank lines aren't necessary after an open brace '{'
> #60: FILE: drivers/i2c/busses/i2c-designware-amdpsp.c:265:

Are you using the default checkpatch.pl script available on the top of
tree kernel baseline? For some reason my checkpatch.pl hasn't reported
this, but (due to operational error) I used the version of script from
5.10 stabilize branch..

> and applied to for-next, thanks!

Thanks!

Best Regards,
Jan