[RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-17 Thread George G. Davis
the posix_same_owner() test within the list_for_each_entry() loop to directly compare caller_fl against current fl entry. Signed-off-by: Armin Kuster [EMAIL PROTECTED] Signed-off-by: George G. Davis [EMAIL PROTECTED] --- Not sure if this is the correct fix but it does resolve the hangs we're observing

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-17 Thread George G. Davis
On Wed, Oct 17, 2007 at 02:51:57PM -0400, George G. Davis wrote: Not sure if this is the correct fix but it does resolve the hangs we're observing in posix_locks_deadlock(). Comments greatly appreciated... Attached is a test case which exhibits the hang on an F7 host with all updates applied

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-18 Thread George G. Davis
On Wed, Oct 17, 2007 at 02:51:57PM -0400, George G. Davis wrote: --- Not sure if this is the correct fix but it does resolve the hangs we're observing in posix_locks_deadlock(). Please disregard the previous patch, it's not quite right - causes occasional segfaults and clearly did not retain

[PATCH] Fix local_irq_* macro definition thinkos for the !TRACE_IRQFLAGS_SUPPORT case

2007-08-03 Thread George G. Davis
From: George G. Davis [EMAIL PROTECTED] Fix local_irq_* macro definition errors for the !TRACE_IRQFLAGS_SUPPORT case in which the macros are errantly redefined rather than defined. Signed-off-by: George G. Davis [EMAIL PROTECTED] diff --git a/include/linux/irqflags.h b/include/linux/irqflags.h

[PATCH] Fix scripts/mkuboot.sh to return status

2005-03-03 Thread George G. Davis
: submitted to linux-kernel@vger.kernel.org Keywords: Description: Fix scripts/mkuboot.sh to return non-zero status to indicate build failure when uImage target is not created. Signed-off-by: George G. Davis [EMAIL PROTECTED] Index: linux-2.6.11-bk/scripts/mkuboot.sh

bk chmod o+r drivers/scsi/a100u2w.c please

2005-01-27 Thread George G. Davis
Greetings, Just a silly request but can someone with http://linux.bkbits.net/linux-2.5 admin priviliges make it so? TIA! -- Regards, George - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [PATCH 1/2] Allow constructor name selection by architecture.

2013-04-01 Thread George G. Davis
On Jun 6, 2012, at 6:12 AM, Russell King - ARM Linux wrote: On Tue, May 29, 2012 at 10:06:14AM +0100, Russell King - ARM Linux wrote: On Mon, May 28, 2012 at 10:30:05PM +0200, Sam Ravnborg wrote: On Mon, May 28, 2012 at 07:33:37PM +0100, Vincent Sanders wrote: From: Vincent Sanders

Re: [PATCH 1/2] Allow constructor name selection by architecture.

2013-04-01 Thread George G. Davis
On Mon, Apr 1, 2013 at 5:58 PM, Sam Ravnborg s...@ravnborg.org wrote: On Mon, Apr 01, 2013 at 05:47:38PM -0400, George G. Davis wrote: On Jun 6, 2012, at 6:12 AM, Russell King - ARM Linux wrote: On Tue, May 29, 2012 at 10:06:14AM +0100, Russell King - ARM Linux wrote: On Mon, May 28, 2012

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-11-02 Thread George G. Davis
On Fri, Oct 26, 2007 at 01:07:50PM -0400, J. Bruce Fields wrote: It may take multiple steps to identify a deadlock. With the above you'll miss deadlocks like process 1 is requesting a lock held by process 2 process 2 is blocking on a lock held by process 3 process 3 is

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: On Fri, Jan 9, 2015 at 8:29 AM, George G. Davis ggdavi...@gmail.com wrote: Export of_reserved_mem_device_{init,release} so that modules can initialize and release their assigned per-device cma_area. I believe the original intent

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 01:33:10PM -0500, George G. Davis wrote: On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: On Fri, Jan 9, 2015 at 8:29 AM, George G. Davis ggdavi...@gmail.com wrote: Export of_reserved_mem_device_{init,release} so that modules can initialize and release

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 04:29:43PM -0600, Rob Herring wrote: On Fri, Jan 9, 2015 at 4:05 PM, George G. Davis ggdavi...@gmail.com wrote: On Fri, Jan 09, 2015 at 01:33:10PM -0500, George G. Davis wrote: On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: On Fri, Jan 9, 2015 at 8:29

[PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
Export of_reserved_mem_device_{init,release} so that modules can initialize and release their assigned per-device cma_area. Signed-off-by: George G. Davis george_da...@mentor.com --- drivers/of/of_reserved_mem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/of/of_reserved_mem.c b

[PATCH] mm: cma: Fix totalcma_pages to include DT defined CMA regions

2015-01-09 Thread George G. Davis
by all CMA regions. Signed-off-by: George G. Davis george_da...@mentor.com --- mm/cma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/cma.c b/mm/cma.c index a85ae28..75016fd 100644 --- a/mm/cma.c +++ b/mm/cma.c @@ -199,6 +199,7 @@ int __init cma_init_reserved_mem(phys_addr_t

Re: lustre_dlm_flags.h GPLv3 license clarification

2015-05-13 Thread George G. Davis
Hello Oleg, On Wed, May 13, 2015 at 01:53:35AM +, Drokin, Oleg wrote: Hello! On May 12, 2015, at 6:43 PM, George G. Davis wrote: Greetings, Resending this using the correct version of get_maintainers.pl. As recommended in the thread SPDX-License-Identifier” [1], I would like

lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Greetings, As recommended in the thread SPDX-License-Identifier” [1], I would like to followup with the maintainers of the file drivers/staging/lustre/lustre/include/lustre_dlm_flags.h regarding the GPLv3 license statement contained in that file [2]. Is there an GPLv3 exception clause which

lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Greetings, Resending this using the correct version of get_maintainers.pl. As recommended in the thread SPDX-License-Identifier” [1], I would like to followup with the maintainers of the file drivers/staging/lustre/lustre/include/lustre_dlm_flags.h regarding the GPLv3 license statement

Re: lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Hello Greg, On Tue, May 12, 2015 at 03:31:17PM -0700, Greg Kroah-Hartman wrote: On Tue, May 12, 2015 at 06:05:24PM -0400, George G. Davis wrote: Greetings, As recommended in the thread SPDX-License-Identifier” [1], I would like to followup with the maintainers of the file drivers

[RFC][PATCH] mmc: core: Disable HPI for certain Hynix eMMC cards

2016-06-09 Thread George G. Davis
would also get disabled if HPI is disabled. Signed-off-by: Pratibhasagar V <prati...@codeaurora.org> Signed-off-by: Subhash Jadavani <subha...@codeaurora.org> [gdavis: Forward port and cleanup https://gitlab.com/k2wl/g2_kernel/commit/84af3731019921a28d595dbf6cbf00539706a42c] Signe

Re: [PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Hello Fabio, On Tue, May 31, 2016 at 02:26:54PM -0300, Fabio Estevam wrote: > On Tue, May 31, 2016 at 2:17 PM, George G. Davis > <george_da...@mentor.com> wrote: > > Convert the i.MX FEC driver and DT files to use the GPIOD API > > which automatically handles GPIO pola

Re: [PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Hello Russell, On Tue, May 31, 2016 at 07:26:42PM +0100, Russell King - ARM Linux wrote: > On Tue, May 31, 2016 at 02:24:50PM -0400, George G. Davis wrote: > > Hello Fabio, > > > > On Tue, May 31, 2016 at 02:26:54PM -0300, Fabio Estevam wrote: > > > On Tue, May

[PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Convert the i.MX FEC driver and DT files to use the GPIOD API which automatically handles GPIO polarity. Signed-off-by: George G. Davis <george_da...@mentor.com> --- Documentation/devicetree/bindings/net/fsl-fec.txt | 7 ++- arch/arm/boot/dts/imx25-karo-tx25.dts | 3 ++-

Re: [PATCH v2 1/2] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Nick, On Tue, Jul 17, 2018 at 10:00:05PM +0100, Nick Dyer wrote: > On Tue, Jul 17, 2018 at 08:16:25PM +0200, Paweł Chmiel wrote: > > This patch adds optional regulators, which can be used to power > > up touchscreen. After enabling regulators, we need to wait 150msec. > > This value is

Re: [PATCH v3 2/3] Input: atmel_mxt_ts: Wait for device be ready for communication

2018-08-22 Thread George G. Davis
Hello Paweł, On Tue, Jul 31, 2018 at 05:18:10PM +0200, Paweł Chmiel wrote: > According to documentation, device isn't ready for communication, > until firmware asserts the CHG line. Add missing wait for this. > > Signed-off-by: Paweł Chmiel > --- > drivers/input/touchscreen/atmel_mxt_ts.c | 11

Re: [PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Paweł, I would suggest dropping sentence punctuation in the patch subject line: Input: atmel_mxt_ts: Add support for optional regulators. More below... On Tue, Jul 31, 2018 at 05:18:09PM +0200, Paweł Chmiel wrote: > This patch adds optional regulators, which can be used to power > up

[PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
since 'ret' will in fact be initialized after exiting the 'while()' loop. Squelch the smatch error since it is a false positive. Fixes: 7ca5f6be7900 ("i2c: recovery: add get_bus_free callback") Signed-off-by: George G. Davis --- drivers/i2c/i2c-core-base.c | 2 +- 1 file changed, 1 inse

Re: [PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
On Fri, Oct 05, 2018 at 06:28:41PM +0200, Wolfram Sang wrote: > On Fri, Oct 05, 2018 at 12:18:25PM -0400, George G. Davis wrote: > > The following smatch error was introduced by commit 7ca5f6be7900 ("i2c: > > recovery: add get_bus_free callback"): > > >

Re: [PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
On Fri, Oct 05, 2018 at 07:43:01PM +0200, Wolfram Sang wrote: > > > Feel free to ignore if this is a topic already discussed in other > > threads. To be honest, I did not search the mailing list to determine if > > this particular issue was already discussed. > > Not this particular issue, but

bk chmod o+r drivers/scsi/a100u2w.c please

2005-01-27 Thread George G. Davis
Greetings, Just a silly request but can someone with http://linux.bkbits.net/linux-2.5 admin priviliges make it so? TIA! -- Regards, George - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at

[PATCH] Fix scripts/mkuboot.sh to return status

2005-03-03 Thread George G. Davis
: submitted to linux-kernel@vger.kernel.org Keywords: Description: Fix scripts/mkuboot.sh to return non-zero status to indicate build failure when uImage target is not created. Signed-off-by: George G. Davis <[EMAIL PROTECTED]> Index: linux-2.6.11-bk/scripts/mkub

[PATCH] Fix local_irq_* macro definition thinkos for the !TRACE_IRQFLAGS_SUPPORT case

2007-08-03 Thread George G. Davis
From: George G. Davis <[EMAIL PROTECTED]> Fix local_irq_* macro definition errors for the !TRACE_IRQFLAGS_SUPPORT case in which the macros are errantly "redefined" rather than defined. Signed-off-by: George G. Davis <[EMAIL PROTECTED]> diff --git a/include/linux/irqf

[RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-17 Thread George G. Davis
und is to change the posix_same_owner() test within the list_for_each_entry() loop to directly compare caller_fl against current fl entry. Signed-off-by: Armin Kuster <[EMAIL PROTECTED]> Signed-off-by: George G. Davis <[EMAIL PROTECTED]> --- Not sure if this is the correct fix but it

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-17 Thread George G. Davis
On Wed, Oct 17, 2007 at 02:51:57PM -0400, George G. Davis wrote: > Not sure if this is the correct fix but it does resolve the hangs we're > observing in posix_locks_deadlock(). Comments greatly appreciated... Attached is a test case which exhibits the hang on an F7 host with all updates a

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-10-18 Thread George G. Davis
On Wed, Oct 17, 2007 at 02:51:57PM -0400, George G. Davis wrote: > --- > Not sure if this is the correct fix but it does resolve the hangs we're > observing in posix_locks_deadlock(). Please disregard the previous patch, it's not quite right - causes occasional segfaults and cl

Re: [RFC][PATCH] Fix hang in posix_locks_deadlock()

2007-11-02 Thread George G. Davis
On Fri, Oct 26, 2007 at 01:07:50PM -0400, J. Bruce Fields wrote: > > It may take multiple steps to identify a deadlock. With the above > you'll miss deadlocks like > > process 1 is requesting a lock held by process 2 > process 2 is blocking on a lock held by process 3 >

Re: [PATCH 1/2] Allow constructor name selection by architecture.

2013-04-01 Thread George G. Davis
On Jun 6, 2012, at 6:12 AM, Russell King - ARM Linux wrote: > On Tue, May 29, 2012 at 10:06:14AM +0100, Russell King - ARM Linux wrote: >> On Mon, May 28, 2012 at 10:30:05PM +0200, Sam Ravnborg wrote: >>> On Mon, May 28, 2012 at 07:33:37PM +0100, Vincent Sanders wrote: From: Vincent Sanders

Re: [PATCH 1/2] Allow constructor name selection by architecture.

2013-04-01 Thread George G. Davis
On Mon, Apr 1, 2013 at 5:58 PM, Sam Ravnborg wrote: > On Mon, Apr 01, 2013 at 05:47:38PM -0400, George G. Davis wrote: >> On Jun 6, 2012, at 6:12 AM, Russell King - ARM Linux wrote: >> > On Tue, May 29, 2012 at 10:06:14AM +0100, Russell King - ARM Linux wrote: >> >>

Re: [PATCH] net: sysctl: cleanup net_sysctl_init error exit paths

2019-07-08 Thread George G. Davis
Hello David, On Fri, May 17, 2019 at 10:43:45AM -0400, George G. Davis wrote: > Hello David, > > On Thu, May 16, 2019 at 02:27:44PM -0700, David Miller wrote: > > From: "George G. Davis" > > Date: Thu, 16 May 2019 11:23:08 -0400 > > > > > Unwind

[RFC][PATCH] Makefile: Fix checkstack.pl arm64 wrong or unknown architecture

2019-05-31 Thread George G. Davis
arm64 case. Signed-off-by: George G. Davis --- Makefile | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 11358153d8f2..3e615e8553c0 100644 --- a/Makefile +++ b/Makefile @@ -1695,7 +1695,11 @@ PHONY += checkstack kernelrelease kernelversion image_

Re: [RFC][PATCH] Makefile: Fix checkstack.pl arm64 wrong or unknown architecture

2019-05-31 Thread George G. Davis
Hello Masahiro, On Sat, Jun 01, 2019 at 01:02:37AM +0900, Masahiro Yamada wrote: > On Sat, Jun 1, 2019 at 12:27 AM George G. Davis > wrote: > > > > The following error occurs for the `make ARCH=arm64 checkstack` case: > > > > aarch64-linux-gnu-objdump -

Re: [RFC][PATCH] Makefile: Fix checkstack.pl arm64 wrong or unknown architecture

2019-05-31 Thread George G. Davis
Hello Masahiro, On Sat, Jun 01, 2019 at 02:22:36AM +0900, Masahiro Yamada wrote: // CUT > As far as I understood, checkstack.pl is supposed to > understand both ARCH= and 'uname -m'. > > > For example, the following commit supports x86, x86_64, i386, > by using regular expression. > > commit

Re: [PATCH] tracing: kmem: convert call_site addresses to user friendly symbols

2019-07-22 Thread George G. Davis
Hello Steve, On Mon, Jul 22, 2019 at 09:43:43AM -0400, Steven Rostedt wrote: > > Looking at the kbuild report... > > On Fri, 19 Jul 2019 22:22:40 -0400 > "George G. Davis" wrote: > > > diff --git a/include/trace/events/kmem.h b/include/trace/events/kmem.h &g

[PATCH v2] tracing: kmem: convert call_site addresses to user friendly symbols

2019-07-22 Thread George G. Davis
friendly symbols which is especially helpful for module callers when you don't have a copy of kallsyms for the running kernel. Reported-by: kbuild test robot Signed-off-by: George G. Davis --- Change history: v1: - First submission v2: - Fix kbuild test robot issues as suggested by Steven

[PATCH v2] selftests: watchdog: Validate optional file argument

2019-09-16 Thread George G. Davis
eniu Rosca Tested-by: Eugeniu Rosca Signed-off-by: George G. Davis Signed-off-by: Eugeniu Rosca --- v1: Applied/tested on commit ce54eab71e210f ("kunit: fix failure to build without printk") of https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git/log/?h=next v2:

Re: [PATCH] net: sysctl: cleanup net_sysctl_init error exit paths

2019-09-17 Thread George G. Davis
Hello David, On Mon, Jul 08, 2019 at 06:47:32PM -0400, George G. Davis wrote: > Hello David, > > On Fri, May 17, 2019 at 10:43:45AM -0400, George G. Davis wrote: > > Hello David, > > > > On Thu, May 16, 2019 at 02:27:44PM -0700, David Miller wrote: > > > From

Re: [PATCH v2] selftests: watchdog: Validate optional file argument

2019-09-17 Thread George G. Davis
Hello Shuah, On Tue, Sep 17, 2019 at 12:05:06PM -0600, shuah wrote: > On 9/17/19 11:50 AM, Eugeniu Rosca wrote: > >On Tue, Sep 17, 2019 at 11:44:45AM -0600, shuah wrote: > >>On 9/17/19 10:54 AM, Eugeniu Rosca wrote: > >>>Shuah, > >>> > >>>On Tue, Sep 17, 2019 at 09:25:31AM -0600, shuah wrote: >

Re: [PATCH v2] selftests: watchdog: Validate optional file argument

2019-09-17 Thread George G. Davis
On Tue, Sep 17, 2019 at 02:32:47PM -0400, George G. Davis wrote: > Hello Shuah, > > On Tue, Sep 17, 2019 at 12:05:06PM -0600, shuah wrote: > > On 9/17/19 11:50 AM, Eugeniu Rosca wrote: > > >On Tue, Sep 17, 2019 at 11:44:45AM -0600, shuah wrote: > > >>On 9

Re: [PATCH] net: sysctl: cleanup net_sysctl_init error exit paths

2019-09-18 Thread George G. Davis
Hello Nicolas, On Wed, Sep 18, 2019 at 11:44:55AM +0200, Nicolas Dichtel wrote: > Le 17/09/2019 à 17:53, George G. Davis a écrit : > [snip] > > Ping, "Linux 5.3" kernel has been released [1] and it appears that the > > 5.4 merge window is open. The patch [2] remain

Re: [PATCH] ARM: Fix null die() string for unhandled data and prefetch abort cases

2019-07-25 Thread George G. Davis
Hello Russell, Thanks for your prompt reply! On Sat, Jul 20, 2019 at 01:30:23PM +0100, Russell King - ARM Linux admin wrote: > On Fri, Jul 19, 2019 at 10:32:55PM -0400, George G. Davis wrote: > > When an unhandled data or prefetch abort occurs, the die() string > > is

Re: [PATCH] ARM: Fix null die() string for unhandled data and prefetch abort cases

2019-07-25 Thread George G. Davis
Hello Russell, On Thu, Jul 25, 2019 at 10:55:40PM +0100, Russell King - ARM Linux admin wrote: > On Thu, Jul 25, 2019 at 05:37:54PM -0400, George G. Davis wrote: > > Hello Russell, > > > > Thanks for your prompt reply! > > > > On Sat, Jul 20, 2019 at 01:30:23P

Re: [PATCH] ARM: Fix null die() string for unhandled data and prefetch abort cases

2019-07-25 Thread George G. Davis
Hello Russell, On Thu, Jul 25, 2019 at 11:32:49PM +0100, Russell King - ARM Linux admin wrote: > On Thu, Jul 25, 2019 at 06:24:01PM -0400, George G. Davis wrote: > > Hello Russell, > > > > On Thu, Jul 25, 2019 at 10:55:40PM +0100, Russell King - ARM Linux admin > >

[PATCH] tracing: kmem: convert call_site addresses to user friendly symbols

2019-07-19 Thread George G. Davis
friendly symbols which is especially helpful for module callers when you don't have a copy of kallsyms for the running kernel. Signed-off-by: George G. Davis --- include/trace/events/kmem.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/trace/events/kmem.h b

[PATCH] ARM: Fix null die() string for unhandled data and prefetch abort cases

2019-07-19 Thread George G. Davis
as to the cause of the fault. Signed-off-by: George G. Davis --- arch/arm/mm/fault.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mm/fault.c b/arch/arm/mm/fault.c index 0048eadd0681..dddea0a21220 100644 --- a/arch/arm/mm/fault.c +++ b/arch/arm/mm/fault.c @@ -557,7 +557,7

Re: [PATCH] selftests: watchdog: Add optional file argument

2019-08-29 Thread George G. Davis
Hello Eugeniu, On Thu, Aug 29, 2019 at 04:38:14PM +0200, Eugeniu Rosca wrote: > Hi George, > > On Thu, Aug 29, 2019 at 12:39:25AM -0400, George G. Davis wrote: > > Some systems have multiple watchdog devices where the first device > > registered is assigned to the /dev

[PATCH v2] selftests: watchdog: Add optional file argument

2019-08-30 Thread George G. Davis
-by: George G. Davis --- v1: - https://lkml.org/lkml/2019/8/29/16 v2: - Update printf for ENOENT case based on report from Eugeniu Rosca --- tools/testing/selftests/watchdog/watchdog-test.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests

Re: [PATCH v2] selftests: watchdog: Add optional file argument

2019-08-30 Thread George G. Davis
Hello Eugeniu, On Fri, Aug 30, 2019 at 03:38:13PM +0200, Eugeniu Rosca wrote: > Hi George, > > On Fri, Aug 30, 2019 at 08:53:16AM -0400, George G. Davis wrote: > > v2: > > - Update printf for ENOENT case based on report from Eugeniu Rosca > > Below interdiff [1] matc

Re: [PATCH v2] selftests: watchdog: Add optional file argument

2019-08-30 Thread George G. Davis
On Fri, Aug 30, 2019 at 09:12:10AM -0600, shuah wrote: > On 8/30/19 6:53 AM, George G. Davis wrote: > >diff --git a/tools/testing/selftests/watchdog/watchdog-test.c > >b/tools/testing/selftests/watchdog/watchdog-test.c > >@@ -69,6 +70,7 @@ static void term(int sig) > &

[PATCH v3] selftests: watchdog: Add optional file argument

2019-08-30 Thread George G. Davis
-by: George G. Davis --- v1: - https://lkml.org/lkml/2019/8/29/16 v2: - Update printf for ENOENT case based on report from Eugeniu Rosca v3: - Split long printf line in two as requested by Shuah Khan --- tools/testing/selftests/watchdog/watchdog-test.c | 20 +--- 1 file changed, 17

Re: [PATCH v2] selftests: watchdog: Add optional file argument

2019-08-30 Thread George G. Davis
On Fri, Aug 30, 2019 at 10:20:23AM -0600, shuah wrote: > On 8/30/19 10:13 AM, George G. Davis wrote: > >On Fri, Aug 30, 2019 at 09:12:10AM -0600, shuah wrote: > >>Can you split this line into two printf's. Checkpatch doesn't like > >>it. > >> > >Su

[PATCH] selftests: watchdog: cleanup whitespace in usage options

2019-08-30 Thread George G. Davis
Convert hard spaces to tabs in usage options. Suggested-by: Shuah Khan Signed-off-by: George G. Davis --- tools/testing/selftests/watchdog/watchdog-test.c | 25 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/tools/testing/selftests/watchdog/watchdog

Re: [PATCH 2/2] selftests: watchdog: Add command line option to show watchdog_info

2019-09-16 Thread George G. Davis
Hello, On Mon, Sep 16, 2019 at 07:26:41AM -0600, shuah wrote: > On 9/7/19 2:58 AM, Eugeniu Rosca wrote: > >diff --git a/tools/testing/selftests/watchdog/watchdog-test.c > >b/tools/testing/selftests/watchdog/watchdog-test.c > >index 6ed822dc..f45e510500c0 100644 > >---

Re: [PATCH 2/2] selftests: watchdog: Add command line option to show watchdog_info

2019-09-16 Thread George G. Davis
Hello Shuah, On Mon, Sep 16, 2019 at 10:05:17AM -0600, shuah wrote: > On 9/16/19 7:57 AM, Eugeniu Rosca wrote: > >Hi Shuah, > >CC George > > > >On Mon, Sep 16, 2019 at 07:26:41AM -0600, shuah wrote: > >[..] > >>> case 'f': > >>> /* Handled above */ >

[PATCH] selftests: watchdog: Add optional file argument

2019-08-28 Thread George G. Davis
Some systems have multiple watchdog devices where the first device registered is assigned to the /dev/watchdog device file. In order to test other watchdog devices, add an optional file argument for selecting non-default watchdog devices for testing. Signed-off-by: George G. Davis --- tools

[PATCH 3/3] staging: most: trivial: fix a couple of typos

2019-06-05 Thread George G. Davis
Fix the following typos: "comoponent" -> "component" "communiction" -> "communication" Cc: Jiri Kosina Signed-off-by: George G. Davis --- drivers/staging/most/Documentation/ABI/configfs-most.txt | 16 drive

[PATCH 2/3] treewide: trivial: fix s/poped/popped/ typo

2019-06-05 Thread George G. Davis
Fix a couple of s/poped/popped/ typos. Cc: Jiri Kosina Signed-off-by: George G. Davis --- Documentation/arm/mem_alignment | 2 +- arch/x86/kernel/kprobes/core.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/arm/mem_alignment b/Documentation/arm

[PATCH 1/3] ARM64: trivial: s/TIF_SECOMP/TIF_SECCOMP/ comment typo fix

2019-06-05 Thread George G. Davis
Fix a s/TIF_SECOMP/TIF_SECCOMP/ comment typo Cc: Jiri Kosina Signed-off-by: George G. Davis --- arch/arm64/include/asm/thread_info.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/include/asm/thread_info.h b/arch/arm64/include/asm/thread_info.h index

[PATCH v2] scripts/checkstack.pl: Fix arm64 wrong or unknown architecture

2019-06-03 Thread George G. Davis
xpressions in the same way it was fixed for the `ARCH=x86` case via commit fda9f9903be6 ("scripts/checkstack.pl: automatically handle 32-bit and 64-bit mode for ARCH=x86"). Suggested-by: Masahiro Yamada Signed-off-by: George G. Davis --- v1: - https://patchwork.kernel.org/patch/10970393/ v2: -

Re: [RFC][PATCH] Makefile: Fix checkstack.pl arm64 wrong or unknown architecture

2019-06-03 Thread George G. Davis
Hello Masahiro, On Sat, Jun 01, 2019 at 11:09:15AM +0900, Masahiro Yamada wrote: > On Sat, Jun 1, 2019 at 2:45 AM George G. Davis > wrote: > > > Following this pattern, does this work for you? > > > > > > diff --git a/scripts/checkstack.pl b/scripts/check

Re: [PATCH v3 1/3] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Paweł, I would suggest dropping sentence punctuation in the patch subject line: Input: atmel_mxt_ts: Add support for optional regulators. More below... On Tue, Jul 31, 2018 at 05:18:09PM +0200, Paweł Chmiel wrote: > This patch adds optional regulators, which can be used to power > up

Re: [PATCH v3 2/3] Input: atmel_mxt_ts: Wait for device be ready for communication

2018-08-22 Thread George G. Davis
Hello Paweł, On Tue, Jul 31, 2018 at 05:18:10PM +0200, Paweł Chmiel wrote: > According to documentation, device isn't ready for communication, > until firmware asserts the CHG line. Add missing wait for this. > > Signed-off-by: Paweł Chmiel > --- > drivers/input/touchscreen/atmel_mxt_ts.c | 11

Re: [PATCH v2 1/2] Input: atmel_mxt_ts: Add support for optional regulators.

2018-08-22 Thread George G. Davis
Hello Nick, On Tue, Jul 17, 2018 at 10:00:05PM +0100, Nick Dyer wrote: > On Tue, Jul 17, 2018 at 08:16:25PM +0200, Paweł Chmiel wrote: > > This patch adds optional regulators, which can be used to power > > up touchscreen. After enabling regulators, we need to wait 150msec. > > This value is

[PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
since 'ret' will in fact be initialized after exiting the 'while()' loop. Squelch the smatch error since it is a false positive. Fixes: 7ca5f6be7900 ("i2c: recovery: add get_bus_free callback") Signed-off-by: George G. Davis --- drivers/i2c/i2c-core-base.c | 2 +- 1 file changed, 1 inse

Re: [PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
On Fri, Oct 05, 2018 at 06:28:41PM +0200, Wolfram Sang wrote: > On Fri, Oct 05, 2018 at 12:18:25PM -0400, George G. Davis wrote: > > The following smatch error was introduced by commit 7ca5f6be7900 ("i2c: > > recovery: add get_bus_free callback"): > > >

Re: [PATCH] serial: sh-sci: disable DMA for uart_console

2019-05-13 Thread George G. Davis
Hello Wolfram, On Mon, May 13, 2019 at 03:51:14PM +0200, Wolfram Sang wrote: > On Thu, May 09, 2019 at 10:43:30AM -0400, George G. Davis wrote: > > As noted in commit 84b40e3b57ee ("serial: 8250: omap: Disable DMA for > > console UART"), UART console lines use low-level

[PATCH v2] serial: sh-sci: disable DMA for uart_console

2019-05-13 Thread George G. Davis
MA support for UART console lines. Fixes: https://patchwork.kernel.org/patch/10929511/ Reported-by: Michael Rodin Tested-by: Eugeniu Rosca Reviewed-by: Simon Horman Reviewed-by: Wolfram Sang Cc: sta...@vger.kernel.org Signed-off-by: George G. Davis --- v2: Clarify comment regarding DMA support

Re: [PATCH v2] serial: sh-sci: disable DMA for uart_console

2019-05-14 Thread George G. Davis
Hello Geert, On Tue, May 14, 2019 at 10:28:34AM +0200, Geert Uytterhoeven wrote: > Hi George, > > On Mon, May 13, 2019 at 5:48 PM George G. Davis > wrote: > > As noted in commit 84b40e3b57ee ("serial: 8250: omap: Disable DMA for > > console UART"), UART co

Re: [PATCH v2] serial: sh-sci: disable DMA for uart_console

2019-05-14 Thread George G. Davis
Hello Wolfram, On Tue, May 14, 2019 at 05:54:51PM +0200, Wolfram Sang wrote: > > > > > Fixes: https://patchwork.kernel.org/patch/10929511/ > > > > > > I don't think this is an appropriate reference, as it points to a patch > > > that > > > was never applied. > > > > I included it as a link to

[PATCH v3] serial: sh-sci: disable DMA for uart_console

2019-05-14 Thread George G. Davis
MA support for UART console lines. Reported-by: Michael Rodin Link: https://patchwork.kernel.org/patch/10929511/ Tested-by: Eugeniu Rosca Reviewed-by: Simon Horman Reviewed-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven Cc: sta...@vger.kernel.org Signed-off-by: George G. Davis --- v2: Clari

Re: [PATCH] i2c: core: squelch error: uninitialized symbol 'ret'

2018-10-05 Thread George G. Davis
On Fri, Oct 05, 2018 at 07:43:01PM +0200, Wolfram Sang wrote: > > > Feel free to ignore if this is a topic already discussed in other > > threads. To be honest, I did not search the mailing list to determine if > > this particular issue was already discussed. > > Not this particular issue, but

lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Greetings, As recommended in the thread "SPDX-License-Identifier” [1], I would like to followup with the maintainers of the file drivers/staging/lustre/lustre/include/lustre_dlm_flags.h regarding the GPLv3 license statement contained in that file [2]. Is there an GPLv3 exception clause which

Re: lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Hello Greg, On Tue, May 12, 2015 at 03:31:17PM -0700, Greg Kroah-Hartman wrote: > On Tue, May 12, 2015 at 06:05:24PM -0400, George G. Davis wrote: > > Greetings, > > > > As recommended in the thread "SPDX-License-Identifier” [1], I would like to > > followup with

lustre_dlm_flags.h GPLv3 license clarification

2015-05-12 Thread George G. Davis
Greetings, Resending this using the correct version of get_maintainers.pl. As recommended in the thread "SPDX-License-Identifier” [1], I would like to followup with the maintainers of the file drivers/staging/lustre/lustre/include/lustre_dlm_flags.h regarding the GPLv3 license statement

Re: lustre_dlm_flags.h GPLv3 license clarification

2015-05-13 Thread George G. Davis
Hello Oleg, On Wed, May 13, 2015 at 01:53:35AM +, Drokin, Oleg wrote: > Hello! > > On May 12, 2015, at 6:43 PM, George G. Davis wrote: > > > Greetings, > > > > Resending this using the correct version of get_maintainers.pl. > > > > As recommended

[PATCH] mm: cma: Fix totalcma_pages to include DT defined CMA regions

2015-01-09 Thread George G. Davis
by all CMA regions. Signed-off-by: George G. Davis --- mm/cma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/cma.c b/mm/cma.c index a85ae28..75016fd 100644 --- a/mm/cma.c +++ b/mm/cma.c @@ -199,6 +199,7 @@ int __init cma_init_reserved_mem(phys_addr_t base, phys_addr_t

[PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
Export of_reserved_mem_device_{init,release} so that modules can initialize and release their assigned per-device cma_area. Signed-off-by: George G. Davis --- drivers/of/of_reserved_mem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/of/of_reserved_mem.c b/drivers

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: > On Fri, Jan 9, 2015 at 8:29 AM, George G. Davis wrote: > > Export of_reserved_mem_device_{init,release} so that modules > > can initialize and release their assigned per-device cma_area. > > I believ

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 01:33:10PM -0500, George G. Davis wrote: > On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: > > On Fri, Jan 9, 2015 at 8:29 AM, George G. Davis wrote: > > > Export of_reserved_mem_device_{init,release} so that modules > > >

Re: [PATCH] drivers: of: Export of_reserved_mem_device_{init,release}

2015-01-09 Thread George G. Davis
On Fri, Jan 09, 2015 at 04:29:43PM -0600, Rob Herring wrote: > On Fri, Jan 9, 2015 at 4:05 PM, George G. Davis wrote: > > On Fri, Jan 09, 2015 at 01:33:10PM -0500, George G. Davis wrote: > >> On Fri, Jan 09, 2015 at 11:53:01AM -0600, Rob Herring wrote: > >> >

[RFC][PATCH] mmc: core: Disable HPI for certain Hynix eMMC cards

2016-06-09 Thread George G. Davis
is disabled. Signed-off-by: Pratibhasagar V Signed-off-by: Subhash Jadavani [gdavis: Forward port and cleanup https://gitlab.com/k2wl/g2_kernel/commit/84af3731019921a28d595dbf6cbf00539706a42c] Signed-off-by: George G. Davis --- drivers/mmc/card/block.c | 6 -- drivers/mmc/core/mmc.c

[PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Convert the i.MX FEC driver and DT files to use the GPIOD API which automatically handles GPIO polarity. Signed-off-by: George G. Davis --- Documentation/devicetree/bindings/net/fsl-fec.txt | 7 ++- arch/arm/boot/dts/imx25-karo-tx25.dts | 3 ++- arch/arm/boot/dts/imx25-pdk.dts

Re: [PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Hello Fabio, On Tue, May 31, 2016 at 02:26:54PM -0300, Fabio Estevam wrote: > On Tue, May 31, 2016 at 2:17 PM, George G. Davis > wrote: > > Convert the i.MX FEC driver and DT files to use the GPIOD API > > which automatically handles GPIO polarity. > > > >

Re: [PATCH] net: fec: dt: convert phy-reset-gpios to use GPIOD API

2016-05-31 Thread George G. Davis
Hello Russell, On Tue, May 31, 2016 at 07:26:42PM +0100, Russell King - ARM Linux wrote: > On Tue, May 31, 2016 at 02:24:50PM -0400, George G. Davis wrote: > > Hello Fabio, > > > > On Tue, May 31, 2016 at 02:26:54PM -0300, Fabio Estevam wrote: > > > On Tue, May