| Message ID | 20240913101445.16513-1-kfting@nuvoton.com |
|---|---|
| Headers | show |
| Series | i2c: npcm: Bug fixes read/write operation, checkpatch | expand |
On Fri, Sep 13, 2024 at 06:14:40PM +0800, warp5tw@gmail.com wrote: > From: Tyrone Ting <kfting@nuvoton.com> > > This patchset includes the following fixes: > > - Enable the target functionality in the interrupt handling routine > when the i2c transfer is about to finish. > - Correct the read/write operation procedure. > - Introduce a software flag to handle the bus error (BER) condition > which is not caused by the i2c transfer. > - Modify timeout calculation. > - Assign the client address earlier logically. > - Use an i2c frequency table for the frequency parameters assignment. > - Coding style fix. > > The NPCM I2C driver is tested on NPCM750 and NPCM845 evaluation boards. Somehow your 6th patch becomes independent from the email thread. (Initially I thought it was a separate fix) Please, check what's going on with email settings on your side.
On Fri, Sep 13, 2024 at 06:14:40PM +0800, warp5tw@gmail.com wrote: > From: Tyrone Ting <kfting@nuvoton.com> > > This patchset includes the following fixes: > > - Enable the target functionality in the interrupt handling routine > when the i2c transfer is about to finish. > - Correct the read/write operation procedure. > - Introduce a software flag to handle the bus error (BER) condition > which is not caused by the i2c transfer. > - Modify timeout calculation. > - Assign the client address earlier logically. > - Use an i2c frequency table for the frequency parameters assignment. > - Coding style fix. > > The NPCM I2C driver is tested on NPCM750 and NPCM845 evaluation boards. The series titled as "Bug fixes...", however I haven't noticed many Fixes tags in it.
Hi Andy: Thank you for your feedback. Andy Shevchenko <andriy.shevchenko@linux.intel.com> 於 2024年9月13日 週五 下午6:31寫道： > > On Fri, Sep 13, 2024 at 06:14:40PM +0800, warp5tw@gmail.com wrote: > > From: Tyrone Ting <kfting@nuvoton.com> > > > > This patchset includes the following fixes: > > > > - Enable the target functionality in the interrupt handling routine > > when the i2c transfer is about to finish. > > - Correct the read/write operation procedure. > > - Introduce a software flag to handle the bus error (BER) condition > > which is not caused by the i2c transfer. > > - Modify timeout calculation. > > - Assign the client address earlier logically. > > - Use an i2c frequency table for the frequency parameters assignment. > > - Coding style fix. > > > > The NPCM I2C driver is tested on NPCM750 and NPCM845 evaluation boards. > > The series titled as "Bug fixes...", however I haven't noticed many > Fixes tags in it. Understood. I'll remove the "Bug fixes" in the title in the next patch set. > > -- > With Best Regards, > Andy Shevchenko > > Thank you. Regards, Tyrone
Hi Andy: Thank you for your feedback. Andy Shevchenko <andriy.shevchenko@linux.intel.com> 於 2024年9月13日 週五 下午6:30寫道： > > On Fri, Sep 13, 2024 at 06:14:40PM +0800, warp5tw@gmail.com wrote: > > From: Tyrone Ting <kfting@nuvoton.com> > > > > This patchset includes the following fixes: > > > > - Enable the target functionality in the interrupt handling routine > > when the i2c transfer is about to finish. > > - Correct the read/write operation procedure. > > - Introduce a software flag to handle the bus error (BER) condition > > which is not caused by the i2c transfer. > > - Modify timeout calculation. > > - Assign the client address earlier logically. > > - Use an i2c frequency table for the frequency parameters assignment. > > - Coding style fix. > > > > The NPCM I2C driver is tested on NPCM750 and NPCM845 evaluation boards. > > Somehow your 6th patch becomes independent from the email thread. > (Initially I thought it was a separate fix) > > Please, check what's going on with email settings on your side. > Understood. I'll check it. > -- > With Best Regards, > Andy Shevchenko > > Thank you. Regards, Tyrone
From: Tyrone Ting <kfting@nuvoton.com> This patchset includes the following fixes: - Enable the target functionality in the interrupt handling routine when the i2c transfer is about to finish. - Correct the read/write operation procedure. - Introduce a software flag to handle the bus error (BER) condition which is not caused by the i2c transfer. - Modify timeout calculation. - Assign the client address earlier logically. - Use an i2c frequency table for the frequency parameters assignment. - Coding style fix. The NPCM I2C driver is tested on NPCM750 and NPCM845 evaluation boards. Addressed comments from: - Andi Shyti : https://lore.kernel.org/lkml/ cfdfldh5tuhb4r5pdpgolcr2roeewsobedet2uvmpbnqlw5yh4@c4a2szsbs2r2/ - Andi Shyti : https://lore.kernel.org/lkml/ stnyjmnqdobzq2f2ntq32tu4kq6ohsxyevjn5rgz3uu2qncuzl@nt4ifscgokgj/ - Andy Shevchenko : https://lore.kernel.org/lkml/ZtWnd8bmiu-M4fQg @smile.fi.intel.com/ - Andy Shevchenko : https://lore.kernel.org/lkml/Zt7Nn9uJSeHFUZZF @smile.fi.intel.com/ - Andi Shyti : https://lore.kernel.org/lkml/ 2kqhf2ad3omx3dsjucrqhtnonnox7ghtp7vkogrwrdfh3dgg2o@4cpa4gfg6c3f/ Changes since version 2: - Add more explanations in the commit message and code modification. - Use lower character names for declarations. - Remove Fixes tags in commits which are not to fix bugs. Changes since version 1: - Restore the npcm_i2caddr array length to fix the smatch warning. - Remove unused variables. - Handle the condition where scl_table_cnt reaches to the maximum value. - Fix the checkpatch warning. Charles Boyer (1): i2c: npcm: Enable slave in eob interrupt Tyrone Ting (5): i2c: npcm: correct the read/write operation procedure i2c: npcm: use a software flag to indicate a BER condition i2c: npcm: Modify timeout evaluation mechanism i2c: npcm: Modify the client address assignment i2c: npcm: use i2c frequency table drivers/i2c/busses/i2c-npcm7xx.c | 281 ++++++++++++++++++++----------- 1 file changed, 179 insertions(+), 102 deletions(-) base-commit: 48b83f5f68edb4d19771d5ecc54bbbc37166f753