[PATCH 2/5] char: lp: detach the device when parallel port is removed

2018-12-07 Thread Sudip Mukherjee
-by: Sudip Mukherjee --- drivers/char/lp.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 7e207ff0f2fe..e21ed4007d0f 100644 --- a/drivers/char/lp.c +++ b/drivers/char/lp.c @@ -976,6 +976,8 @@ static void lp_attach

[PATCH 3/5] char: lp: use first unused lp number while registering

2018-12-07 Thread Sudip Mukherjee
number array to find the first unused lp number and use that to register the lp device with the parallel port. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index e21ed4007d0f

[PATCH 5/5] char: lp: use new parport device model

2018-12-07 Thread Sudip Mukherjee
Modify lp driver to use the new parallel port device model. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index cef2ea386f5e..5c8d780637bd 100644 --- a/drivers/char

[PATCH 1/5] char: lp: introduce list to save port number

2018-12-07 Thread Sudip Mukherjee
When we are registering lp in LP_PARPORT_AUTO mode, we are not keeping any record of the parallel port number to which lp is connecting. Add an array to save the port number to it. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 4/5] char: lp: properly count the lp devices

2018-12-07 Thread Sudip Mukherjee
device and dmesg shows: lp: ignoring parallel port (max. 8) Decrement the lp_count when the device detaches as this represents the number of lp devices connected to the system. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers

Re: [PATCH 4.14 054/146] f2fs: fix to do sanity check with block address in main area

2018-12-04 Thread Sudip Mukherjee
Hi Greg, On Tue, Dec 4, 2018 at 11:05 AM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. > > -- > > commit c9b60788fc760d136211853f10ce73dc152d1f4a upstream. There is another upstream commit which fixes this.

Re: [PATCH] ubi: fastmap: Check each mapping only once

2018-12-02 Thread Sudip Mukherjee
On Sun, Dec 2, 2018 at 2:35 PM Sasha Levin wrote: > > On Sun, Dec 02, 2018 at 11:50:33AM +, Sudip Mukherjee wrote: > >Hi Greg, > > > >On Sun, Dec 2, 2018 at 8:51 AM Richard Weinberger wrote: > >> > >> Greg, > >> > >> Am

Re: [PATCH] ubi: fastmap: Check each mapping only once

2018-12-02 Thread Sudip Mukherjee
Hi Greg, On Sun, Dec 2, 2018 at 8:51 AM Richard Weinberger wrote: > > Greg, > > Am Donnerstag, 29. November 2018, 09:09:23 CET schrieb Greg KH: > > On Mon, Nov 26, 2018 at 11:38:42AM +0100, Martin Kepplinger wrote: > > > From: Richard Weinberger > > > > > > [ Upstream commit

[PATCH] parport: parport_pc: Mark expected switch fall-through

2018-11-25 Thread Sudip Mukherjee
From: "Gustavo A. R. Silva" In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Addresses-Coverity-ID: 114730 ("Missing break in switch") Signed-off-by: Gustavo A. R. Silva Signed-off-by: Sudip Mukherjee

[PATCH 2/6] char: lp: move trailing statement to next line

2018-11-25 Thread Sudip Mukherjee
Fix checkpatch errors for trailing if else statements. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 7b15272c0510..65bf32244f48 100644 --- a/drivers/char/lp.c +++ b

[PATCH 1/6] char: lp: remove trailing whitespace

2018-11-25 Thread Sudip Mukherjee
Fix checkpatch error for trailing whitespace. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 8c4dd1a3bb6a..7b15272c0510 100644 --- a/drivers

[PATCH 6/6] char: lp: fix spacing style before open parenthesis

2018-11-25 Thread Sudip Mukherjee
Fixes the checkpatch warning: "space prohibited between function name and open parenthesis" Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 184 +++--- 1 file changed, 92 insertions(+), 92 deletions(-) diff --git a/drivers/c

[PATCH 3/6] char: lp: do not use return as a function

2018-11-25 Thread Sudip Mukherjee
return is not a function, parentheses are not required. Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 65bf32244f48..4153c6f5683a 100644 --- a/drivers/char/lp.c +++ b

[PATCH 4/6] char: lp: use tabs instead of spaces

2018-11-25 Thread Sudip Mukherjee
Fixes the checkpatch error: ERROR: code indent should use tabs where possible Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 4153c6f5683a..15d03aa55a1a 100644

[PATCH 5/6] char: lp: fix whitespace with pointers

2018-11-25 Thread Sudip Mukherjee
Fixes checkpatch warning: "foo * bar should be foo *bar" Signed-off-by: Sudip Mukherjee --- drivers/char/lp.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/char/lp.c b/drivers/char/lp.c index 15d03aa55a1a..017ea457a9d7 100644 --- a/drivers

Re: [PATCH] udf: fix dvd mounting error

2018-11-16 Thread Sudip Mukherjee
On Fri, Nov 16, 2018 at 01:56:33PM +0100, Jan Kara wrote: > On Thu 15-11-18 12:26:00, Sudip Mukherjee wrote: > > Some DVDs are failing to mount with the error: > > > > [ 632.305370] UDF-fs: warning (device loop0): udf_load_vrs: No anchor found > > [ 632.305372] UDF-

[PATCH] udf: fix dvd mounting error

2018-11-15 Thread Sudip Mukherjee
ing volume 'TOUR_2017_THANKSGIVING_25_DISC ', timestamp 2018/01/30 05:57 (1000) The DVD is available at: https://www.amazon.co.jp/Live-%E3%80%8CMr-Children-DOME-STADIUM-Thanksgiving/dp/B079B5W6KC Cc: Andrew Gabbasov Signed-off-by: Sudip Mukherjee --- I dont't think this is the c

Re: [PATCH 4.14 053/143] x86/power: Fix some ordering bugs in __restore_processor_context()

2018-11-02 Thread Sudip Mukherjee
Hi Greg, On Fri, Nov 2, 2018 at 6:51 PM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. This has been fixed by another upstream commit: 7ee18d677989 ("x86/power: Make restore_processor_context() sane") You can not directly apply it,

Re: [PATCH 4.14 060/143] net/mlx5: Fix mlx5_get_vector_affinity function

2018-11-02 Thread Sudip Mukherjee
Hi Greg, On Fri, Nov 2, 2018 at 6:52 PM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. There is another upstream patch which fixes this one. e3ca34880652 ("net/mlx5: Fix build break when CONFIG_SMP=n") -- Regards Sudip

Re: [PATCH 4.14 069/143] ARM: tegra: Fix ULPI regression on Tegra20

2018-11-02 Thread Sudip Mukherjee
Hi Greg, On Fri, Nov 2, 2018 at 6:53 PM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. It seems this has already been reverted upstream: 9bf4e370048d ("ARM: dts: tegra20: Revert "Fix ULPI regression on Tegra20"") -- Regards Sudip

Re: [PATCH 4.14 29/45] cgroup/cpuset: remove circular dependency deadlock

2018-10-11 Thread Sudip Mukherjee
Hi Greg, On Thu, Oct 11, 2018 at 4:54 PM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Prateek Sood > > commit aa24163b2ee5c92120e32e99b5a93143a0f4258e upstream. This has been already reverted in

Re: [PATCH 4.14 66/94] dm thin metadata: try to avoid ever aborting transactions

2018-10-08 Thread Sudip Mukherjee
HI Greg, On Mon, Oct 8, 2018 at 8:06 PM Greg Kroah-Hartman wrote: > > 4.14-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Joe Thornber > > [ Upstream commit 3ab91828166895600efd9cdc3a0eb32001f7204a ] This has been fixed by a later

Re: [PATCH 4.14 00/64] 4.14.73-stable review

2018-09-27 Thread Sudip Mukherjee
Hi Greg, On Thu, Sep 27, 2018 at 9:56 PM, Sudip Mukherjee wrote: > Hi Greg, > > On Thu, Sep 27, 2018 at 10:03 AM, Greg Kroah-Hartman > wrote: >> This is the start of the stable review cycle for the 4.14.73 release. >> There are 64 patches in this series, all will

Re: [PATCH 4.14 00/64] 4.14.73-stable review

2018-09-27 Thread Sudip Mukherjee
Hi Greg, On Thu, Sep 27, 2018 at 10:03 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.73 release. > There are 64 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me

Re: [PATCH 4.14 31/64] spi: fix IDR collision on systems with both fixed and dynamic SPI bus numbers

2018-09-27 Thread Sudip Mukherjee
Hi Greg, On Thu, Sep 27, 2018 at 10:03 AM, Greg Kroah-Hartman wrote: > 4.14-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Kirill Kapranov > > commit 1a4327fbf4554d5b78d75b19a13d40d6de220159 upstream. > > On systems where some

Re: [PATCH 4.14 64/64] crypto: x86/aegis,morus - Do not require OSXSAVE for SSE2

2018-09-27 Thread Sudip Mukherjee
Hi Greg, On Thu, Sep 27, 2018 at 10:04 AM, Greg Kroah-Hartman wrote: > 4.14-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Ondrej Mosnacek > > commit 24568b47d48ec8c906fd0f589489a08b17e1edca upstream. > > It turns out OSXSAVE needs to be

[PATCH 1/2] parport: sunbpp: fix error return code

2018-07-12 Thread Sudip Mukherjee
From: Julia Lawall Return an error code on failure. Change leading spaces to tab on the first if. Problem found using Coccinelle. Signed-off-by: Julia Lawall Signed-off-by: Sudip Mukherjee --- drivers/parport/parport_sunbpp.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions

[PATCH 2/2] parport: ieee1284: mark expected switch fall-throughs

2018-07-12 Thread Sudip Mukherjee
From: "Gustavo A. R. Silva" In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva Signed-off-by: Sudip Mukherjee --- drivers/parport/ieee1284.c | 3 ++- 1 file changed, 2 insertions(+),

BUG: KASAN: use-after-free in ex_handler_refcount

2018-07-10 Thread Sudip Mukherjee
Hi All, I was running a KASAN enabled kernel and noticed the following: [ 916.786725] == [ 916.786746] BUG: KASAN: use-after-free in ex_handler_refcount+0x5b/0x127 [ 916.786753] Write of size 4 at addr 880105144bc0 by task

Re: [PATCH] parport: Add support for the WCH384 4S multi-IO card

2018-07-05 Thread Sudip Mukherjee
Hi Matt, On Sat, May 26, 2018 at 12:35:23PM -0700, Matt Turner wrote: > This Multi-IO card has one serial 16550-like serial connectors. Here's > the lspci output, after this commit is applied: > > 01:00.0 Serial controller [0700]: Device [1c00:3470] (rev 10) (prog-if 05 > [16850]) >

[PATCH resend] libata: blacklist Micron 500IT SSD with MU01 firmware

2018-05-19 Thread Sudip Mukherjee
From: Sudip Mukherjee <sudipm.mukher...@gmail.com> While whitelisting Micron M500DC drives, the tweaked blacklist entry enabled queued TRIM from M500IT variants also. But these do not support queued TRIM. And while using those SSDs with the latest kernel we have seen errors and even the par

Re: [PATCH v2] libata: blacklist Micron SSD

2018-05-02 Thread Sudip Mukherjee
Hi Martin, On Mon, Apr 30, 2018 at 11:24:49PM -0400, Martin K. Petersen wrote: > > Sudip, > > > v1: Only M500IT MU01 was blacklisted. > > > > v2: Whitelist M500IT BG02 and M500DC and then blacklist all other Micron. > > I think my preference would be to blacklist M500IT with the MU01 >

[PATCH] staging: sm750fb: add drm development branch details

2018-04-29 Thread Sudip Mukherjee
...@lists.freedesktop.org Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/staging/sm750fb/TODO | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/TODO b/drivers/staging/sm750fb/TODO index f710ab1..aeb2df3 100644 --- a/drivers/staging/s

[PATCH v2] libata: blacklist Micron SSD

2018-04-29 Thread Sudip Mukherjee
From: Sudip Mukherjee <sudipm.mukher...@gmail.com> While whitelisting Micron M500DC drives, the tweaked blacklist entry enabled queued TRIM for M500IT variants also. But these do not support queued TRIM. And while using those SSDs with the latest kernel we have seen errors and even the par

Re: [PATCH] libata: blacklist Micron 500IT SSD with MU01 firmware

2018-04-05 Thread Sudip Mukherjee
Hi Martin, On Fri, Mar 16, 2018 at 05:49:25PM -0400, Martin K. Petersen wrote: > > Sudip, > > >> Any reason to believe this deficiency is limited to MU01 firmware? > > > > Not really. But we also use MG02 firmware and afaik, Micron has > > confirmed that MG02 is a fixed one. I have only

Re: [PATCH] libata: blacklist Micron 500IT SSD with MU01 firmware

2018-03-16 Thread Sudip Mukherjee
Hi Martin, On Fri, Mar 16, 2018 at 11:54:30AM -0400, Martin K. Petersen wrote: > > Sudip, > > > While whitelisting Micron M500DC drives, the tweaked blacklist entry > > enabled queued TRIM from M500IT variants also. But these do not support > > queued TRIM. And while using those SSDs with the

[PATCH] libata: blacklist Micron 500IT SSD with MU01 firmware

2018-03-16 Thread Sudip Mukherjee
;) Cc: sta...@vger.kernel.org Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/ata/libata-core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index cb789f8..1fb96c6 100644 --- a/drivers/ata/libata-core.c +++ b/dr

Re: [1/2] pps: client: use new parport device model

2018-03-05 Thread Sudip Mukherjee
On Sun, Jan 28, 2018 at 11:22:08PM +, Sudip Mukherjee wrote: > Modify pps client driver to use the new parallel port device model. > In that process, added an index to mention the device number when we > have more than one parallel port. > > Signed-off-by: Sudip Mukherjee

Re: [PATCH 01/12] parport: ax88796: Delete an error message for a failed memory allocation in parport_ax88796_probe()

2018-03-03 Thread Sudip Mukherjee
Hi Greg, On Thu, Mar 01, 2018 at 01:06:09PM +0100, Greg Kroah-Hartman wrote: > On Thu, Mar 01, 2018 at 12:46:07PM +0200, Andy Shevchenko wrote: > > On Wed, Feb 7, 2018 at 9:47 PM, Sudip Mukherjee > > <sudipm.mukher...@gmail.com> wrote: > > > From: Markus Elfring

[PATCH 02/12] parport: ax88796: Improve a size determination in parport_ax88796_probe()

2018-02-07 Thread Sudip Mukherjee
sue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_ax88796.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/

[PATCH 04/12] parport: Add support for BrainBoxes PX272/PX306 MIO card

2018-02-07 Thread Sudip Mukherjee
d-by: Nikola Ciprich <nikola.cipr...@linuxbox.cz> Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 12 1 file changed, 12 insertions(+) diff --git a/driv

[PATCH 09/12] parport: Switch to use module_pci_driver() macro

2018-02-07 Thread Sudip Mukherjee
;andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 16 +--- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c index

[PATCH 05/12] parport: PCI core handles power state for us

2018-02-07 Thread Sudip Mukherjee
;andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 40 1 file changed, 12 insertions(+), 28 deletions(-) diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parpor

[PATCH 10/12] parport: Sort headers alphabetically

2018-02-07 Thread Sudip Mukherjee
y Shevchenko <andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c index

[PATCH 01/12] parport: ax88796: Delete an error message for a failed memory allocation in parport_ax88796_probe()

2018-02-07 Thread Sudip Mukherjee
From: Markus Elfring <elfr...@users.sourceforge.net> Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net> Signed-off-by: Sudip Mukherjee &l

[PATCH 11/12] parport: Replace short License header by SPDX identifier

2018-02-07 Thread Sudip Mukherjee
From: Andy Shevchenko <andriy.shevche...@linux.intel.com> No functional changes involved. Tested-by: Nikola Ciprich <nikola.cipr...@linuxbox.cz> Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com&

[PATCH 06/12] parport: Convert to use managed functions pcim_* and devm_*

2018-02-07 Thread Sudip Mukherjee
;andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parport_serial.c index

[PATCH 12/12] parport_pc: Add support for WCH CH382L PCI-E single parallel port card.

2018-02-07 Thread Sudip Mukherjee
dlab-i.ru> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_pc.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/parport/parport_pc.c b/drivers/parport/parport_pc.c index 489492b..aa6bb50 100644 --- a/drivers/parport/parport_pc.c +++

[PATCH 08/12] parport: Convert printk(KERN_WARN) to dev_warn()

2018-02-07 Thread Sudip Mukherjee
off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/parport_serial.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/parport/parport_serial.c b/drivers/parport/parpor

[PATCH 07/12] parport: Don't shadow error codes in ->probe()

2018-02-07 Thread Sudip Mukherjee
From: Andy Shevchenko <andriy.shevche...@linux.intel.com> When ->probe() calls helper functions return theirs error codes instead of shadowing them. Tested-by: Nikola Ciprich <nikola.cipr...@linuxbox.cz> Signed-off-by: Andy Shevchenko <andriy.shevche...@linux.intel.com>

[PATCH 03/12] parport: ax88796: Delete an unnecessary variable initialisation in parport_ax88796_probe()

2018-02-07 Thread Sudip Mukherjee
From: Markus Elfring <elfr...@users.sourceforge.net> The local variable "pp" will eventually be set to an appropriate pointer a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net> Signed-o

[PATCH 2/2] pps: generator: use new parport device model

2018-01-28 Thread Sudip Mukherjee
Modify pps generator driver to use the new parallel port device model. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- Tested on qemu with two parallel ports. drivers/pps/generators/pps_gen_parport.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff

[PATCH 1/2] pps: client: use new parport device model

2018-01-28 Thread Sudip Mukherjee
Modify pps client driver to use the new parallel port device model. In that process, added an index to mention the device number when we have more than one parallel port. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- Tested on qemu with two parallel ports. drivers/pps/c

[PATCH] m32r: remove abort()

2018-01-25 Thread Sudip Mukherjee
Commit 7c2c11b208be ("arch: define weak abort()") has introduced a weak abort() which is common for all arch. And, so we will not need arch specific abort which has the same code as the weak abort(). Remove the abort() for m32r. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.

Re: [PATCH] arch: drop duplicate exports of abort()

2018-01-02 Thread Sudip Mukherjee
On Tue, Jan 02, 2018 at 11:38:29AM +, Russell King - ARM Linux wrote: > On Tue, Jan 02, 2018 at 11:34:45AM +0000, Sudip Mukherjee wrote: > > Hi Arnd, > > > > On Tue, Jan 02, 2018 at 11:31:12AM +0100, Arnd Bergmann wrote: > > > We now have exports in both archi

Re: [PATCH] arch: drop duplicate exports of abort()

2018-01-02 Thread Sudip Mukherjee
Hi Arnd, On Tue, Jan 02, 2018 at 11:31:12AM +0100, Arnd Bergmann wrote: > We now have exports in both architecture code in in common code, > which causes a link failure when symbol versioning is eanbled, on > four architectures: > > kernel/exit.o: In function `__crc_abort': >

[PATCH] net: sch: api: fix tcf_block_get

2017-12-22 Thread Sudip Mukherjee
CONFIG_NET_CLS is not defined. Fixes: 8d1a77f974ca ("net: sch: api: add extack support in tcf_block_get") Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- The build log of next-20171222 is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/320089922 include/n

Re: [PATCH v2] arch: define weak abort

2017-12-21 Thread Sudip Mukherjee
ths-dereference can generate calls to abort() > from modular code too. > > Reported-by: Vineet Gupta <vineet.gup...@synopsys.com> > Cc: Sudip Mukherjee <sudipm.mukher...@gmail.com> > Cc: Arnd Bergmann <a...@arndb.de> > Cc: "Alexey Brodkin" <alexey.brod

[PATCH v2] arch: define weak abort

2017-12-12 Thread Sudip Mukherjee
ven remove the arch specific 'abort' after this is done. Cc: Alexey Brodkin <alexey.brod...@synopsys.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- Hi Alexey, I was thinking of sending the m32r revert patch in few days. My m32r builds are having a little probl

[PATCH] arch: define weak abort

2017-12-11 Thread Sudip Mukherjee
is common for all arch and fix the problem permanently. We can even remove the arch specific 'abort' after this is done. Cc: Alexey Brodkin <alexey.brod...@synopsys.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- kernel/exit.c | 8 1 file changed, 8 ins

Re: [PATCH] frv: fix build failure

2017-12-03 Thread Sudip Mukherjee
On Mon, Nov 27, 2017 at 10:25:16AM -0800, Vineet Gupta wrote: > +CC linux-arch, Arnd > > On 11/23/2017 09:17 AM, Alexey Brodkin wrote: > >Hi Sudip, > > > >On Tue, 2017-11-21 at 22:10 +, Sudip Mukherjee wrote: > I understand the case/need for adding a weak/c

Re: [PATCH] frv: fix build failure

2017-11-23 Thread Sudip Mukherjee
Hi Alexey, On Thu, Nov 23, 2017 at 05:17:19PM +, Alexey Brodkin wrote: > Hi Sudip, > > On Tue, 2017-11-21 at 22:10 +0000, Sudip Mukherjee wrote: > > The frv defconfig build is failing with the error: > > lib/mpi/mpih-div.o: In function `mpihelp_divrem': > &

[PATCH] frv: fix build failure

2017-11-21 Thread Sudip Mukherjee
'. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- The build log of next-20171121 is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/305076731 arch/frv/kernel/traps.c | 9 + 1 file changed, 9 insertions(+) diff --git a/arch/frv/kernel/traps.c b/arch/frv/kernel/t

[PATCH] xtensa: fix build failure

2017-11-11 Thread Sudip Mukherjee
emove __bio_kmap_atomic") Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- The build log of next-20171110 is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/38449 arch/xtensa/platforms/iss/simdisk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

[PATCH 2/3] sm501fb: unregister framebuffer only if registered

2017-11-04 Thread Sudip Mukherjee
There are cases when panel and crt both are not defined and only one of them is defined and initialized. In such cases, while removing the device, unregister the framebuffer only if it was registered. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/video/fbdev/sm5

[PATCH 3/3] sm501fb: suspend and resume fb if it exists

2017-11-04 Thread Sudip Mukherjee
There are cases when panel and crt both are not defined and only one of them is defined and initialized. In such cases, suspend or resume it only if it is defined. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/video/fbdev/sm501fb.c | 6 ++ 1 file chan

[PATCH 1/3] sm501fb: deallocate colormap only if allocated

2017-11-04 Thread Sudip Mukherjee
There are cases when panel and crt both are not defined and only one of them is defined and initialized. In such cases, while removing the device deallocate the colormap only if that particular fb is defined. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/video

[PATCH] parport: make parport_ip32_ops const and __initconst

2017-10-15 Thread Sudip Mukherjee
From: Bhumika Goyal <bhumi...@gmail.com> Make this const as it is only used during a copy operation. This usage is inside init function and the structure is not referenced after initialisation, so make it __initconst too. Signed-off-by: Bhumika Goyal <bhumi...@gmail.com> Signed-o

[PATCH] hamradio: baycom_par: use new parport device model

2017-10-15 Thread Sudip Mukherjee
Modify baycom driver to use the new parallel port device model. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- Not tested on real hardware, only tested on kvm with 32 bit guest and verified that the device is binding to the driver properly in par96_open but then unb

Re: [PATCH] parisc: parport_gsc: constify parisc_device_id

2017-10-14 Thread Sudip Mukherjee
On Sat, Aug 19, 2017 at 10:38:41PM +0530, Arvind Yadav wrote: > parisc_device_id are not supposed to change at runtime. All functions > working with parisc_device_id provided by work > with > const parisc_device_id. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav

[PATCH] m32r: fix build failure

2017-09-29 Thread Sudip Mukherjee
From: Sudip Mukherjee <sudipm.mukher...@gmail.com> The allmodconfig build of m32r is failing with the error: lib/mpi/mpih-div.o: In function 'mpihelp_divrem': mpih-div.c:(.text+0x40): undefined reference to 'abort' mpih-div.c:(.text+0x40): relocation truncated to fit: R_M32R_26_PCRE

Re: [PATCH] alpha: fix build failures

2017-09-27 Thread Sudip Mukherjee
Hi Andrew, On Sun, Sep 17, 2017 at 06:20:10PM +0100, Sudip Mukherjee wrote: > The build of alpha allmodconfig is giving error: > arch/alpha/include/asm/mmu_context.h: In function 'ev5_switch_mm': > arch/alpha/include/asm/mmu_context.h:160:2: error: > implicit declaration

Re: [PATCH 4.12 00/52] 4.12.14-stable review

2017-09-20 Thread Sudip Mukherjee
On Tue, Sep 19, 2017 at 08:34:22AM +0200, Greg Kroah-Hartman wrote: > On Mon, Sep 18, 2017 at 03:22:36PM +0100, Sudip Mukherjee wrote: > > Hi Greg, > > > > On Mon, Sep 18, 2017 at 11:10:54AM +0200, Greg Kroah-Hartman wrote: > > > NOTE: This is going to be the last

Re: [PATCH 4.12 00/52] 4.12.14-stable review

2017-09-18 Thread Sudip Mukherjee
Hi Greg, On Mon, Sep 18, 2017 at 11:10:54AM +0200, Greg Kroah-Hartman wrote: > NOTE: This is going to be the last 4.12.y kernel, after this one, this > branch is end-of-life, please move to the 4.13.y kernel tree at this > point in time. > > This is the start of the stable review cycle for the

[PATCH v2] IB/mlx5: fix debugfs cleanup

2017-09-18 Thread Sudip Mukherjee
From: Sudip Mukherjee <sudipm.mukher...@gmail.com> If delay_drop_debugfs_init() fails in any of the operations to create debugfs, it is calling delay_drop_debugfs_cleanup() as part of its cleanup. But delay_drop_debugfs_cleanup() checks for 'dbg' and since we have not yet pointe

[PATCH] m32r: define CPU_BIG_ENDIAN

2017-09-17 Thread Sudip Mukherjee
Define CPU_BIG_ENDIAN like the way CPU_LITTLE_ENDIAN is defined. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- arch/m32r/Kconfig | 4 1 file changed, 4 insertions(+) diff --git a/arch/m32r/Kconfig b/arch/m32r/Kconfig index 87cde1e..0777f3a 100644 --- a/arch/m32r/Kconfig

[PATCH] alpha: fix build failures

2017-09-17 Thread Sudip Mukherjee
-declaration] The file 'mmu_context.h' needed an extra header file. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- The build log of next-20170915 is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/275726322 arch/alpha/include/asm/mmu_context.h | 1 + 1 file chan

[PATCH] MIPS: MSP71xx: fix build failure

2017-09-17 Thread Sudip Mukherjee
-declaration] The file 'msp_smp.c' was only missing a header file. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- The build logs of next-20170915 is at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/275726387 arch/mips/pmcs-msp71xx/msp_smp.c | 1 + 1 file chan

[PATCH] rtc: sun6i: fix memory leak

2017-09-17 Thread Sudip Mukherjee
If 'clk_data' is not allocated we returned but we failed to free 'rtc'. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/rtc/rtc-sun6i.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/rtc/rtc-sun6i.c b/drivers/rtc/rtc-sun6i.c index 3

[PATCH] IB/mlx5: fix debugfs cleanup

2017-09-17 Thread Sudip Mukherjee
are left with stray debugfs elements and also a memory leak. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/infiniband/hw/mlx5/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c index a

[PATCH] rpmsg: glink: fix memory leak

2017-09-17 Thread Sudip Mukherjee
We missed freeing intent and intent->data on error conditions. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/rpmsg/qcom_glink_native.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/rpmsg/qcom_glink_native.c b/driv

[PATCH] hamradio: baycom: use new parport device model

2017-09-17 Thread Sudip Mukherjee
Modify baycom driver to use the new parallel port device model. Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- Not tested on real hardware, only tested on qemu and verified that the device is binding to the driver properly in epp_open but then unbinding as the

[PATCH 3/3] parport: use release_mem_region instead of release_resource

2017-08-12 Thread Sudip Mukherjee
From: Anton Vasilyev <vasil...@ispras.ru> Use api pair of request_mem_region and release_mem_region instead of release_resource. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Vasilyev <vasil...@ispras.ru> Signed-off-by: Sudip Mukherjee &l

[PATCH 2/3] parport: cleanup statics initialization to NULL or 0

2017-08-12 Thread Sudip Mukherjee
From: Carlos Palminha <carlos.palmi...@synopsys.com> based on checkpatch, cleanup the "do not initialise statics to" 0 or NULL. Signed-off-by: Carlos Palminha <palmi...@synopsys.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- drivers/parport/dai

[PATCH 1/3] parport_pc: use pr_cont

2017-08-12 Thread Sudip Mukherjee
From: Mikulas Patocka <mpato...@redhat.com> The kernel adds newline automatically between printk calls, we must use pr_cont if we want multiple printk strings on the same line. Signed-off-by: Mikulas Patocka <mpato...@redhat.com> Signed-off-by: Sudip Mukherjee <sudipm.mukh

Re: [PATCH 2/5] parport: cleanup prohibited space after functions name

2017-08-07 Thread Sudip Mukherjee
On Mon, Jul 17, 2017 at 05:36:59PM +0100, Carlos Palminha wrote: > based on checkpatch, remove prohibited space between function name and > open parenthesis '('. > > Signed-off-by: Carlos Palminha > --- > drivers/parport/ieee1284.c | 212 >

Re: [PATCH 1/5] parport: add newline to trailing statements

2017-08-07 Thread Sudip Mukherjee
Hi, On Mon, Jul 17, 2017 at 05:36:58PM +0100, Carlos Palminha wrote: > based on checkpatch, add newline to trailing statements > > Signed-off-by: Carlos Palminha > --- > drivers/parport/daisy.c| 7 +-- > drivers/parport/ieee1284.c | 17 +

[PATCH] staging: sm750fb: avoid conflicting vesafb

2017-07-29 Thread Sudip Mukherjee
for other arch.' Cc: <sta...@vger.kernel.org> # v4.4 Signed-off-by: Teddy Wang <teddy.w...@siliconmotion.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> --- drivers/staging/sm750fb/sm750.c | 24 +

Re: [RESEND PATCH] m32r: defconfig: Cleanup from old Kconfig options

2017-07-21 Thread Sudip Mukherjee
l >CONFIG_MTD_PARTITIONS"); > - MTD_CHAR: commit 660685d9d1b4 ("mtd: merge mtdchar module with >mtdcore"); > - MTD_CONCAT: commit f53fdebcc3e1 ("mtd: drop MTD_CONCAT from Kconfig >entirely"); > > Signed-off-by: Krzysztof Kozlowski <k...@kernel.org> Acked-by: Sudip Mukherjee <sudipm.mukher...@gmail.com>

Re: [PATCH RESEND] staging: sm750fb: avoid conflicting vesafb

2017-07-12 Thread Sudip Mukherjee
On Wed, Jul 12, 2017 at 08:06:24AM +0200, Greg Kroah-Hartman wrote: > On Tue, Jul 11, 2017 at 10:03:01PM +0100, Sudip Mukherjee wrote: > > Hi Greg, > > > > On Fri, Jun 30, 2017 at 09:57:43PM +0100, Sudip Mukherjee wrote: > > > From: Teddy Wang

Re: [PATCH RESEND] staging: sm750fb: avoid conflicting vesafb

2017-07-11 Thread Sudip Mukherjee
Hi Greg, On Fri, Jun 30, 2017 at 09:57:43PM +0100, Sudip Mukherjee wrote: > From: Teddy Wang <teddy.w...@siliconmotion.com> > > If vesafb is enabled in the config then /dev/fb0 is created by vesa > and this sm750 driver gets fb1, fb2. But we need to be fb0 and fb1 to > effec

[PATCH RESEND] staging: sm750fb: avoid conflicting vesafb

2017-06-30 Thread Sudip Mukherjee
: <sta...@vger.kernel.org> # v4.4+ Signed-off-by: Teddy Wang <teddy.w...@siliconmotion.com> Signed-off-by: Sudip Mukherjee <sudipm.mukher...@gmail.com> --- In the previous send, why #ifdef is used was asked. https://lkml.org/lkml/2017/6/25/57 Answered at: https://lkml.org/lkml/2017/6

Re: userspace regression with hwmon

2017-06-27 Thread Sudip Mukherjee
On Mon, Jun 26, 2017 at 10:06:11AM -0700, Guenter Roeck wrote: > On Mon, Jun 26, 2017 at 03:33:20PM +0100, Sudip Mukherjee wrote: > > Hi Guenter, Jean, > > > > The patch d72d19c26c41 ("hwmon: (coretemp) Convert to use > > devm_hwmon_device_register_with_g

Re: [PATCH] MIPS: perf: fix build failure

2017-06-26 Thread Sudip Mukherjee
On Mon, Jun 26, 2017 at 08:44:48AM +0200, Marcin Nowakowski wrote: > Hi Sudip, > > This patch fixes the build error, but leaves the I6500 handling incorrect. I am failing to understand why I6500 handling is incorrect. I am seeing that after applying my patch the I6500 handling is same as

userspace regression with hwmon

2017-06-26 Thread Sudip Mukherjee
Hi Guenter, Jean, The patch d72d19c26c41 ("hwmon: (coretemp) Convert to use devm_hwmon_device_register_with_groups") has changed the location of the sysfs nodes from /sys/devices/platform/coretemp.0/ to /sys/devices/platform/coretemp.0/hwmon/hwmon0/ and that has broken some of our usespace

Re: [PATCH 1/2] staging: sm750fb: avoid conflicting vesafb

2017-06-25 Thread Sudip Mukherjee
On Sun, Jun 25, 2017 at 04:27:23PM +0200, Greg Kroah-Hartman wrote: > On Sun, Jun 25, 2017 at 02:54:29PM +0100, Sudip Mukherjee wrote: > > On Sun, Jun 25, 2017 at 02:54:51PM +0200, Greg Kroah-Hartman wrote: > > > On Sun, Jun 25, 2017 at 01:43:34PM +0100, Sudip Mukherjee wrote: &

Re: [PATCH 1/2] staging: sm750fb: avoid conflicting vesafb

2017-06-25 Thread Sudip Mukherjee
On Sun, Jun 25, 2017 at 02:54:51PM +0200, Greg Kroah-Hartman wrote: > On Sun, Jun 25, 2017 at 01:43:34PM +0100, Sudip Mukherjee wrote: > > On Mon, Jun 19, 2017 at 09:32:57PM +0100, Sudip Mukherjee wrote: > > > From: Teddy Wang <teddy.w...@siliconmotion.com> > >

Re: [PATCH 1/2] staging: sm750fb: avoid conflicting vesafb

2017-06-25 Thread Sudip Mukherjee
On Mon, Jun 19, 2017 at 09:32:57PM +0100, Sudip Mukherjee wrote: > From: Teddy Wang <teddy.w...@siliconmotion.com> > > If vesafb is enabled in the config then /dev/fb0 is created by vesa > and this sm750 driver gets fb1, fb2. But we need to be fb0 and fb1 to > effectively

[PATCH] MIPS: perf: fix build failure

2017-06-23 Thread Sudip Mukherjee
unter handling for I6400") removed the previous case and added it as a separate case. But patch dd71e57bacb5 ("MIPS: perf: add I6500 handling") added it back to its old location and thus giving us two duplicate case. Fixes: dd71e57bacb5 ("MIPS: perf: add I6500 handling")

Re: [PATCH] sm501fb: Silence a coccinelle script warning.

2017-06-20 Thread Sudip Mukherjee
Hi Bartlomiej, On Wed, Jun 14, 2017 at 12:54:58PM +0200, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Monday, May 08, 2017 08:11:16 AM Christophe JAILLET wrote: > > According to surrounding goto, it is likely that 'goto err_probed_panel' is > > expected here. > > This change is just done in

  1   2   3   4   5   6   7   8   9   10   >