Re: [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex

[PATCH v2 0/9] i2c: i801: Series with improvements · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
[PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Wolfram Sang <wsa@kernel.org> · 2021-08-10
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-11
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Andy Shevchenko <hidden> · 2021-08-12
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Wolfram Sang <wsa@kernel.org> · 2021-08-17
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Jean Delvare <hidden> · 2021-08-26
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Andy Shevchenko <hidden> · 2021-08-26
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-31
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Jean Delvare <hidden> · 2021-08-31
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-31
Re: [PATCH v2 1/9] i2c: i801: Improve disabling runtime pm · Heiner Kallweit <hkallweit1@gmail.com> · 2021-09-01
[PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex · Wolfram Sang <wsa@kernel.org> · 2021-08-10
Re: [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-11
Re: [PATCH v2 2/9] i2c: i801: make p2sb_spinlock a mutex · Andy Shevchenko <hidden> · 2021-08-12
[PATCH v2 3/9] i2c: i801: Remove not needed debug message · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 3/9] i2c: i801: Remove not needed debug message · Wolfram Sang <wsa@kernel.org> · 2021-08-10
[PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-11
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Andy Shevchenko <hidden> · 2021-08-12
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Jean Delvare <hidden> · 2021-08-26
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Jean Delvare <hidden> · 2021-08-26
Re: [PATCH v2 4/9] i2c: i801: Improve is_dell_system_with_lis3lv02d · Wolfram Sang <wsa@kernel.org> · 2021-09-29
[PATCH v2 5/9] i2c: i801: Remove not needed check for PCI_COMMAND_INTX_DISABLE · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 5/9] i2c: i801: Remove not needed check for PCI_COMMAND_INTX_DISABLE · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 5/9] i2c: i801: Remove not needed check for PCI_COMMAND_INTX_DISABLE · Jean Delvare <hidden> · 2021-08-26
Re: [PATCH v2 5/9] i2c: i801: Remove not needed check for PCI_COMMAND_INTX_DISABLE · Wolfram Sang <wsa@kernel.org> · 2021-09-29
[PATCH v2 6/9] i2c: i801: Improve i801_acpi_probe/remove functions · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 6/9] i2c: i801: Improve i801_acpi_probe/remove functions · Wolfram Sang <wsa@kernel.org> · 2021-09-29
[PATCH v2 7/9] i2c: i801: Improve i801_add_mux · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 7/9] i2c: i801: Improve i801_add_mux · Wolfram Sang <wsa@kernel.org> · 2021-09-29
[PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-11
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Andy Shevchenko <hidden> · 2021-08-12
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Jean Delvare <hidden> · 2021-08-27
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Wolfram Sang <wsa@kernel.org> · 2021-09-29
Re: [PATCH v2 8/9] i2c: i801: Improve register_dell_lis3lv02d_i2c_device · Jean Delvare <hidden> · 2021-10-01
[PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device · Heiner Kallweit <hkallweit1@gmail.com> · 2021-08-06
Re: [PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device · Andy Shevchenko <hidden> · 2021-08-11
Re: [PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device · Wolfram Sang <wsa@kernel.org> · 2021-10-02
Re: [PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device · Wolfram Sang <wsa@kernel.org> · 2021-11-29
Re: [PATCH v2 9/9] i2c: i801: Improve handling platform data for tco device · Heiner Kallweit <hkallweit1@gmail.com> · 2021-11-29
[PATCH v3] i2c: i801: Improve handling platform data for tco device · Heiner Kallweit <hkallweit1@gmail.com> · 2021-11-29

From: Andy Shevchenko <hidden>
Date: 2021-08-11 15:43:54

On Fri, Aug 06, 2021 at 11:13:29PM +0200, Heiner Kallweit wrote:

p2sb_spinlock is used in i801_add_tco_spt() only, and in process context
only. Therefore a mutex is sufficient, and we can make the definition
local to i801_add_tco_spt().

The problem with either AFAICT is that it should actually hold PCI rescan lock.
See the discussion with Message-ID
20210308122020.57071-1-andriy.shevchenko@linux.intel.com for the details.

-- 
With Best Regards,
Andy Shevchenko

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help