- Mar 12, 2020
-
-
Ajay Singh authored
Modified the 'clock-names' property by removing '_clk' from its name and remove '_spi/sdio' from 'compatible' string. Signed-off-by:
Ajay Singh <ajay.kathat@microchip.com> Link: https://lore.kernel.org/r/20200307085523.7320-3-ajay.kathat@microchip.com Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Ajay Singh authored
Make use of 'interrupts' property instead of using gpio for handling the interrupt as suggested in [1]. [1]. https://lore.kernel.org/linux-wireless/20200303015558.GA6876@bogus Signed-off-by:
-
- Mar 11, 2020
-
-
Carlos Henrique Lima Melara authored
Fixed WARNING: Missing a blank line after declarations in qlge_mpi.c to the following lines: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:94: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:240: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:258: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:356: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:915: WARNING: Missing a blank line after declarations FILE: drivers/staging/qlge/qlge_mpi.c:1099: Signed-off-by:
Carlos Henrique Lima Melara <charlesmelara@outlook.com> Link: https://lore.kernel.org/r/CP2PR80MB434086A44A1ED5BB662B9D70A8FC0@CP2PR80MB4340.lamprd80.prod.outlook.com Signed-off-by:
-
Andre Pinto authored
Fix checkpatch check: 'informations' may be misspelled - perhaps 'information'? in rtw_mlme_ext.c:1151. Signed-off-by:
Andre Pinto <andrealmeidap1996@gmail.com> Link: https://lore.kernel.org/r/20200311012638.18889-1-andrealmeidap1996@gmail.com Signed-off-by:
-
Lourdes Pedrajas authored
Improve line grouping for better understanding the course of action and where every comment belongs to. Signed-off-by:
Lourdes Pedrajas <lu@pplo.net> Reviewed-by:
Stefano Brivio <sbrivio@redhat.com> Link: https://lore.kernel.org/r/3fada7fe76047ecf06d0345c0ab9216424a126f2.1583848946.git.lu@pplo.net Signed-off-by:
-
Lourdes Pedrajas authored
Every time priv->op_mode is not NL80211_IFTYPE_ADHOC it sets ATIM Window, but this feature it is marked TODO in 2014 and did go untouched since 2015. Signed-off-by:
-
Sergio Paracuellos authored
The clock related code concerns me from the very beginning because there are some set ups got from legacy driver that are not documented anywhere. According to the programming guide 0x7c is 'CPE_ROSC_SEL1' register and 0x80 is 'CPU_CPE_CN'. I do think this set up is not needed at all and the proper thing to do is just enable the clock bit for each pcie port. Hence remove useless code and do the right thing which is setting up the clock bit for each port enabled. Signed-off-by:
Sergio Paracuellos <sergio.paracuellos@gmail.com> Link: https://lore.kernel.org/r/20200310113459.30539-1-sergio.paracuellos@gmail.com Signed-off-by:
-
Deepak R Varma authored
There are two #if 0 blocks that have no recent history of any change. Remove those code blocks for improved readability of the code. Signed-off-by:
Deepak R Varma <mh12gx2825@gmail.com> Link: https://lore.kernel.org/r/20200310212545.GA8914@deeUbuntu Signed-off-by:
-
Kaaira Gupta authored
struct hif_cnf_reset is not used anywhere in wfx. It's parent file hif_api_cmd.h is imported form some other driver, hence some functions in it are not used. This is one of them. Hence remove it. Signed-off-by:
Kaaira Gupta <kgupta@es.iitr.ac.in> Link: https://lore.kernel.org/r/20200310142509.25632-7-kgupta@es.iitr.ac.in Signed-off-by:
-
Kaaira Gupta authored
int ret is uneccessarily declared and 0 assigned to it. Return o directly instead. Signed-off-by:
-
Kaaira Gupta authored
remove extra space after a cast in file data_tx.h Signed-off-by:
-
Kaaira Gupta authored
remove extra spaces after casts in file data_tx.c Signed-off-by:
-
Kaaira Gupta authored
policies[i].uploaded is a bool. 1 is assigned to it. Change it to bool. Signed-off-by:
-
Kaaira Gupta authored
remove an unesseccary space after a cast in file data_rx.c Signed-off-by:
-
- Mar 10, 2020
-
-
Greg Kroah-Hartman authored
Now that there is a "real" solution for exfat in the vfs tree queued up to be merged in 5.7-rc1 the "old" exfat code in staging can be removed. Many thanks to Valdis for doing the work to get this into the tree in the first place, it was greatly appreciated. Acked-by:
Valdis Kletnieks <valdis.kletnieks@vt.edu> Cc: Pali Rohár <pali@kernel.org> Cc: Stephen Rothwell <sfr@canb.auug.org.au> Cc: Al Viro <viro@ZenIV.linux.org.uk> Cc: Namjae Jeon <namjae.jeon@samsung.com> Cc: Sungjong Seo <sj1557.seo@samsung.com> Cc: Christoph Hellwig <hch@lst.de> Link: https://lore.kernel.org/r/20200310105421.GA2810679@kroah.com Signed-off-by:
-
Sergio Paracuellos authored
Function 'mt7621_pcie_init_virtual_bridges' is a bit mess and can be refactorized properly in a cleaner way. Introduce new 'pcie_rmw' inline function helper to do clear and set the correct bits this function needs to work. Signed-off-by:
-
Zhenzhong Duan authored
When softsynthu device fails the register, "/dev/softsynthu" should be printed instead of "/dev/softsynth". Signed-off-by:
Zhenzhong Duan <zhenzhong.duan@gmail.com> Reviewed-by:
Samuel Thibault <samuel.thibault@ens-lyon.org> Cc: William Hubbs <w.d.hubbs@gmail.com> Cc: Chris Brannon <chris@the-brannons.com> Cc: Kirk Reiser <kirk@reisers.ca> Link: https://lore.kernel.org/r/20200305072151.403-1-zhenzhong.duan@gmail.com Signed-off-by:
-
Malcolm Priestley authored
The variable merely toggles true to false and is unused. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Link: https://lore.kernel.org/r/130a4078-2502-a381-46c4-b473815e153b@gmail.com Signed-off-by:
-
Malcolm Priestley authored
mac80211 is the only user of in_use to start it and should not be true when so. So internal toggling of this variable is not relevant. Signed-off-by:
-
Sergiu Cuciurean authored
In a recent change to the SPI subsystem in commit <bebcfd27 > ("spi: introduce `delay` field for `spi_transfer` + spi_transfer_delay_exec()"), a new `delay` struct was added to replace the `delay_usecs`. This change replaces the current `delay_usecs` with `delay` for this driver. The `spi_transfer_delay_exec()` function [in the SPI framework] makes sure that both `delay_usecs` & `delay` are used (in this order to preserve backwards compatibility). Signed-off-by:
Sergiu Cuciurean <sergiu.cuciurean@analog.com> Link: https://lore.kernel.org/r/20200304074319.22107-1-sergiu.cuciurean@analog.com Signed-off-by:
-
Sergiu Cuciurean authored
In a recent change to the SPI subsystem in commit <bebcfd27 > ("spi: introduce `delay` field for `spi_transfer` + spi_transfer_delay_exec()"), a new `delay` struct was added to replace the `delay_usecs`. This change replaces the current `delay_usecs` with `delay` for this driver. The `spi_transfer_delay_exec()` function [in the SPI framework] makes sure that both `delay_usecs` & `delay` are used (in this order to preserve backwards compatibility). Signed-off-by:
-
Shreeya Patel authored
Add space around & operator for improving the code readability. Signed-off-by:
Shreeya Patel <shreeya.patel23498@gmail.com> Acked-by:
Julia Lawall <julia.lawall@inria.fr> Link: https://lore.kernel.org/r/20200308192152.26403-1-shreeya.patel23498@gmail.com Signed-off-by:
-
Lourdes Pedrajas authored
Remove unused code. Signed-off-by:
-
Payal Kshirsagar authored
Align code to match open parenthesis. Signed-off-by:
Payal Kshirsagar <payalskshirsagar1234@gmail.com> Link: https://lore.kernel.org/r/20200309165138.5313-1-payalskshirsagar1234@gmail.com Signed-off-by:
-
Lourdes Pedrajas authored
Remove function ffsSetAttr() which is unused. Signed-off-by:
-
Shreeya Patel authored
Remove comparison to "true" from if statement to maintain the kernel coding style. Reported by checkpatch.pl Signed-off-by:
-
Payal Kshirsagar authored
Remove line over 80 characters. Signed-off-by:
-
Payal Kshirsagar authored
Blank line is not necessary before a close brace '}', remove it. Signed-off-by:
Jérôme Pouiller <jerome.pouiller@silabs.com> Link: https://lore.kernel.org/r/20200309165528.5721-1-payalskshirsagar1234@gmail.com Signed-off-by:
-
Payal Kshirsagar authored
Align code to open parenthesis. Signed-off-by:
-
Thomas Gleixner authored
Nothing outside of low level architecture code is supposed to look up interrupt descriptors and fiddle with them. Replace the open coded abuse by calling generic_handle_irq(). This still does not explain why and in which context this connection magic is injecting interrupts in the first place and why this is correct and safe, but at least the API abuse is gone. Fixes: 036aad9d ("greybus: gpio: add interrupt handling support") Fixes: 2611ebef ("greybus: gpio: don't call irq-flow handler directly") Signed-off-by:
Thomas Gleixner <tglx@linutronix.de> Acked-by:
Johan Hovold <johan@kernel.org> Link: https://lore.kernel.org/r/87o8t9boqq.fsf@nanos.tec.linutronix.de Signed-off-by:
-
Lourdes Pedrajas authored
Remove function gb_i2c_smbus_xfer() which is unused. Signed-off-by:
-
Payal Kshirsagar authored
Blank lines aren't necessary after an open brace '{'. Signed-off-by: -
Payal Kshirsagar authored
Add spaces around operators cleanup reported by checkpatch. Signed-off-by:
-
Payal Kshirsagar authored
Remove white spaces before tabs and align code properly for readability. Signed-off-by:
-
Payal Kshirsagar authored
Remove unnecessary multiple blank lines. Signed-off-by:
-
Payal Kshirsagar authored
Align code to the open parenthesis to fix the alignment issue. Signed-off-by:
-
Payal Kshirsagar authored
To avoid style issues, remove multiple blank lines. Signed-off-by:
-
- Mar 04, 2020
-
-
Ajay Singh authored
Use flexible-array member introduced in C99 instead of zero-length array. Most of zero-length array was already taken care in previous patch [1]. Now modified few more cases which were not handled earlier. [1]. https://patchwork.kernel.org/patch/11394197/ Signed-off-by:
-
Ajay Singh authored
Remove the unnecessary always true 'if' conditions and simiplifed the logic as suggested in [1]. [1]. https://lore.kernel.org/linux-wireless/20200302092346.GA24308@kadam/ Suggested-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Ajay Singh authored
Use correct data for memcpy in wilc_hif_pack_sta_param(). Its reported by Smatch static code analyser tool as discussed in [1]. [1]. https://lore.kernel.org/linux-wireless/20200302092346.GA24308@kadam/ Suggested-by:
-
