Patch "fpga: altera-hps2fpga: fix multiple init of l3_remap_lock" has been added to the 4.12-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
fpga: altera-hps2fpga: fix multiple init of l3_remap_lock
to the 4.12-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
fpga-altera-hps2fpga-fix-multiple-init-of-l3_remap_lock.patch
and it can be found in the queue-4.12 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
4ae2bd4b3ada3dfd80ca8110b4f567752966ca1eMon Sep 17 00:00:00 2001
From: Ian Abbott <[email protected]>
Date: Tue, 1 Aug 2017 21:20:54 -0500
Subject: fpga: altera-hps2fpga: fix multiple init of l3_remap_lock
From: Ian Abbott <[email protected]>
commit 4ae2bd4b3ada3dfd80ca8110b4f567752966ca1e upstream.
The global spinlock l3_remap_lock is reinitialized every time the "probe" function alt_fpga_bridge_probe() is called. It should only be initialized once. Use DEFINE_SPINLOCK() to initialize it statically.
--- drivers/fpga/altera-hps2fpga.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
drivers/fpga/altera-hps2fpga.c +1 -3
@@ -66,7 +66,7 @@ static int alt_hps2fpga_enable_show(stru/* The L3 REMAP register is write only, so keep a cached value. */ static unsigned int l3_remap_shadow; -static spinlock_t l3_remap_lock; +static DEFINE_SPINLOCK (l3_remap_lock);static int _alt_hps2fpga_enable_set (struct altera_hps2fpga_data *priv,bool enable)
@@ -171,8 +171,6 @@ static int alt_fpga_bridge_probe(structreturn -EBUSY ; } -spin_lock_init (&l3_remap_lock); -if (!of_property_read_u32 (dev->of_node,"bridge-enable" , &enable)) {if (enable >1 ) {dev_warn (dev,"invalid bridge-enable %u > 1\n" , enable);
Patches currently in stable-queue which might be from [email protected] are
queue-4.12/fpga-altera-hps2fpga-fix-multiple-init-of-l3_remap_lock.patch