On 10/1/18 12:49 PM, Lidza Louina wrote:
On Mon, Oct 1, 2018 at 8:09 AM Mark Hounschell <mailto:ma...@compro.net>> wrote:
On 9/28/18 3:59 PM, Lidza Louina wrote:
> I haven't done work on this driver in a long time. I looked up the
> devices, and they seem
On 9/28/18 3:59 PM, Lidza Louina wrote:
I haven't done work on this driver in a long time. I looked up the
devices, and they seem to already have an existing Linux driver:
https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
. They will not respond
to an inquiry concerning it or its firmware.
Signed-off-by: Mark Hounschell <ma...@compro.net>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
Cc: Ben Hutchings <b...@decadent.org.uk>
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAI
On 09/15/2015 09:05 AM, Greg Kroah-Hartman wrote:
On Tue, Sep 15, 2015 at 08:56:10AM -0400, Mark Hounschell wrote:
Removes myself from the MAINTAINERS file for the dgap driver.
There appears to be no way to get the firmware files required
by the dgap driver into the linux-firmware tree
On 03/15/2015 08:07 AM, Mark Hounschell wrote:
On 03/14/2015 04:44 AM, Greg KH wrote:
On Fri, Mar 13, 2015 at 04:55:55PM -0400, Mark Hounschell wrote:
On 03/12/2015 12:14 PM, Giedrius Statkevičius wrote:
On 2015.03.12 12:08, Greg KH wrote:
On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius
On 03/14/2015 04:44 AM, Greg KH wrote:
On Fri, Mar 13, 2015 at 04:55:55PM -0400, Mark Hounschell wrote:
On 03/12/2015 12:14 PM, Giedrius Statkevičius wrote:
On 2015.03.12 12:08, Greg KH wrote:
On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius Statkevičius wrote:
Remove BOARD_FAILED
On 03/12/2015 12:14 PM, Giedrius Statkevičius wrote:
On 2015.03.12 12:08, Greg KH wrote:
On Mon, Mar 09, 2015 at 06:29:38PM +0200, Giedrius Statkevičius wrote:
Remove BOARD_FAILED and don't save dgnc_boards which failed to
initialize.
Assign the result of kzalloc() to brd in
On 12/03/2014 06:37 PM, Joe Perches wrote:
On Wed, 2014-12-03 at 21:30 +, Sean Cleator wrote:
A patch to fix the rest of the long line warnings in the dgnc_cls.h file
found by the checkpatch.pl tool
checkpatch is a brainless little tool.
You should prefer to develop a readable style
On 10/29/2014 05:22 AM, Greg KH wrote:
On Sun, Oct 26, 2014 at 11:08:54AM +0900, Daeseok Youn wrote:
Re-arrange the functions for removing forward declarations.
Tested-by: Mark Hounschell ma...@compro.net
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
RESEND: This patch is tested all
On 10/21/2014 07:33 AM, David Binderman wrote:
Hello there,
1.
[linux-3.18-rc1/drivers/staging/dgap/dgap.c:6692]: (warning) Logical
disjunction always evaluates to true: conc_type != 65 || conc_type != 66.
Source code is
if (conc_type == 0 || conc_type != CX ||
On 10/14/2014 08:01 AM, Mark Hounschell wrote:
On 10/13/2014 10:04 PM, Greg KH wrote:
On Mon, Oct 13, 2014 at 07:56:38AM -0700, Joe Perches wrote:
On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
Hi,
2014-10-13 12:25 GMT+09:00 Greg KH gre...@linuxfoundation.org:
On Mon, Oct 13, 2014
On 10/13/2014 10:04 PM, Greg KH wrote:
On Mon, Oct 13, 2014 at 07:56:38AM -0700, Joe Perches wrote:
On Mon, 2014-10-13 at 17:01 +0900, DaeSeok Youn wrote:
Hi,
2014-10-13 12:25 GMT+09:00 Greg KH gre...@linuxfoundation.org:
On Mon, Oct 13, 2014 at 11:34:25AM +0900, Daeseok Youn wrote:
On 07/31/2014 07:14 PM, DaeSeok Youn wrote:
Hi, Mark
2014-07-31 21:44 GMT+09:00 Mark Hounschell ma...@compro.net:
On 07/31/2014 12:02 AM, Daeseok Youn wrote:
When a configration file is parsed with dgap_parsefile(),
makes nodes for saving configrations for board.
Making a node will allocate
On 07/31/2014 12:02 AM, Daeseok Youn wrote:
When a configration file is parsed with dgap_parsefile(),
makes nodes for saving configrations for board.
Making a node will allocate node memory and strings for saving
configrations with kstrdup().
So these are freed when dgap is unloaded or failed
On 07/16/2014 08:42 PM, DaeSeok Youn wrote:
2014-07-16 23:17 GMT+09:00 Mark Hounschell ma...@compro.net:
On 07/16/2014 05:26 AM, DaeSeok Youn wrote:
2014-07-16 8:50 GMT+09:00 Greg KH gre...@linuxfoundation.org:
On Wed, Jul 16, 2014 at 08:21:30AM +0900, DaeSeok Youn wrote:
Hi,
2014-07-16
On 07/16/2014 09:35 PM, Daeseok Youn wrote:
When a configration file is parsed with dgap_parsefile(),
makes nodes for saving configrations for board.
Making a node will allocate node memory and strings for saving
configrations with kstrdup().
So these are freed when dgap is unloaded or
On 07/15/2014 11:30 AM, Greg KH wrote:
On Tue, Jul 15, 2014 at 06:14:25PM +0900, Daeseok Youn wrote:
When a configration file is parsed with dgap_parsefile(),
makes nodes for saving configrations for board.
configuration files should not be parsed in the kernel at all. That
logic should be
On 07/16/2014 05:26 AM, DaeSeok Youn wrote:
2014-07-16 8:50 GMT+09:00 Greg KH gre...@linuxfoundation.org:
On Wed, Jul 16, 2014 at 08:21:30AM +0900, DaeSeok Youn wrote:
Hi,
2014-07-16 0:29 GMT+09:00 Greg KH gre...@linuxfoundation.org:
On Tue, Jul 15, 2014 at 06:11:44PM +0900, Daeseok Youn
Hi Dan,
I've been away for a while but should be able to find some time to work
on the dgap driver some more. Do you have a TODO list?
Thanks
Mark
___
devel mailing list
de...@linuxdriverproject.org
On 05/28/2014 06:11 AM, Dan Carpenter wrote:
On Wed, May 28, 2014 at 06:29:38PM +0900, DaeSeok Youn wrote:
In your patch it has:
+ dgap_tty_uninit(brd, false);
But it should only be false if dgap_tty_init() failed. If
dgap_tty_register_ports() fails then it should be true. Another
refactors NULL conditionals
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 64 ++---
1 file changed, 32 insertions(+), 32
Simplify ugly dgap_find_config function
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 84 +++--
1 file changed, 43 insertions
Replaces ENXIO with EIO errno
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 44 ++--
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/drivers/staging/dgap
Removes more extra white space and tabs
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 178 +---
1 file changed, 86 insertions(+), 92 deletions(-)
diff --git a/drivers
This patch replaces double negativeconditionals
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff
Simplify dgap_config_get_useintr and dgap_config_get_altpin
changing the case statement to an if statement.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 10
We don't need to use GFP_ATOMIC at driver load time
so use GFP_KERNEL instead.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 8
1 file changed, 4 insertions
On 05/22/2014 04:38 PM, Dan Carpenter wrote:
On Thu, May 22, 2014 at 01:49:22PM -0400, Mark Hounschell wrote:
I understand that unnecessarily initializing them is wrong. But if they
do need initialized, is it preferred to do it in the declaration or in
the code before it is used?
Which ever
This patch fixes an smatch warning about a missing break
in a case statement.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 6 ++
1 file changed, 2 insertions
This patch fixes a smatch warning about an unsigned
integer being tested for less than zero.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.h | 2 +-
1 file changed, 1
The sniff buffer was a /proc filesystem file. This
module no longer uses /proc. This patch removes
the deaed code associated with the sniff buffer.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
This patch removes a false comment
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c
index b3ad3ac..6816cf7
This patch removes an unnecessary test. We've already
tested it by the time we get here with no possiblity
of it being changed.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap
On 05/21/2014 03:59 PM, Dan Carpenter wrote:
On Wed, May 21, 2014 at 01:03:09PM -0400, Mark Hounschell wrote:
This patch fixes an smatch warning about a missing break
in a case statement.
I would prefer if we leave the code as-is and fix Smatch. The fix will
be to read the previous line
Removes dgap_savestring function to use kstrdup instead.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 44 +++-
1 file changed
On 05/15/2014 06:49 PM, Dan Carpenter wrote:
On Fri, May 16, 2014 at 12:27:01AM +0200, Christian Engelmayer wrote:
Hi,
As I got a bit queued up for dgap, I just noticed that commit 542f3d5a states
to add Mark to the maintainers list for dgap, while the patch addresses the
dgnc entry. Was
On 05/16/2014 07:52 AM, Mark Hounschell wrote:
On 05/15/2014 06:49 PM, Dan Carpenter wrote:
On Fri, May 16, 2014 at 12:27:01AM +0200, Christian Engelmayer wrote:
Hi,
As I got a bit queued up for dgap, I just noticed that commit 542f3d5a
states
to add Mark to the maintainers list for dgap
Add Mark Hounschell to the MAINTAINERs list for the dgap driver
Signed-off-by: Mark Hounschell ma...@compro.net
Suggested-by: Dan Carpenter dan.carpen...@oracle.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b
I have some patches submitted almost 3 weeks ago that I have not
received any email responses to. When should I think about resubmitting
them? Or should I just be more patient?
Thanks
Mark
___
devel mailing list
de...@linuxdriverproject.org
This patch just backs out a change in a previous patch
that generated a new sparse warning.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 04/25/2014 09:44 AM, Dan Carpenter wrote:
On Fri, Apr 25, 2014 at 09:38:48AM -0400, Mark Hounschell wrote:
This patch just backs out a change in a previous patch
that generated a new sparse warning.
Could you redo the previous change instead? Or has it already been
applied?
I have
This patch fixes a few more sparse warnings related to __iomem
version 2 excludes a change that actually caused a new sparse
warning.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 37
On 04/25/2014 08:59 AM, Dan Carpenter wrote:
On Fri, Apr 25, 2014 at 08:29:41AM -0400, Mark Hounschell wrote:
On 04/25/2014 07:02 AM, DaeSeok Youn wrote:
Hi, Dan.
2014-04-25 18:26 GMT+09:00 Dan Carpenter dan.carpen...@oracle.com:
Mark, maybe you should add yourself to the MAINTAINERS entry
Remove 2 unnessessary conditionals. They are always
false in this code path.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 32 ++--
1 file changed, 14 insertions(+), 18 deletions
Check the return values of dgap_tty_register and
dgap_finalize_board_init. If they fail for a
particular board we should not be able to try
to use that board. IE. no device entries shall
be created for that board.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre
patch 1: check return values of dgap_tty_register and dgap_finalize_board_init
patch 2: removes 2 unnecessary conditionals
patch 3: only allow driver to configure MAXBOARDS (32)
--
1.8.1.4
___
devel mailing list
de...@linuxdriverproject.org
This patch just cleans up the ugly dgap_init_one function
Signed-off-by: Mark Hounschell ma...@compro.net
Suggested-by: Dan Carpenter dan.carpen...@oracle.com
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.c | 19 ---
1 file changed, 8 insertions(+), 11
Add Mark Hounschell to the MAINTAINERs list for the dgap driver
Signed-off-by: Mark Hounschell ma...@compro.net
Suggested-by: Dan Carpenter dan.carpen...@oracle.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b
I get the sparse warning:
drivers/staging/dgap/dgap.c:367:25: warning: too long initializer-string
for array of char
It's referring to INIT_C_CC in the following code segment.
static struct ktermios DgapDefaultTermios = {
.c_iflag = (DEFAULT_IFLAGS), /* iflags */
This patch gets rid of the obvious CamelCases
from both dgap.c and dgap.h
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Suggested-by: Dan Carpenter dan.carpen...@oracle.com
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap
This patch fixes a few more sparse warnings related to __iomem
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 37 +++--
1 file changed
This patch fixes a sparse warning:
warning: symbol 'dgap_poll_lock' was not declared. Should it be static?
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Oops, ignore this. I got the subject wrong. resending...
Mark
On 04/24/2014 10:31 AM, Mark Hounschell wrote:
This patch fixes a sparse warning:
warning: symbol 'dgap_poll_lock' was not declared. Should it be static?
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman
This patch fixes a sparse warning:
warning: symbol 'dgap_poll_lock' was not declared. Should it be static?
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
This patch fixes sparse warnings:
warning: Using plain integer as NULL pointer
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 32
1 file changed, 16 insertions(+), 16 deletions
This patch changes the way we insure the config data
is a string. Clearly this was just wrong. After a
certain number of loads/unloads various OOPs were
generated indicating something other than this driver
had a problem. It was this driver.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested
This patch fixes an OOPS caused by a pointer being
changed between the malloc and free.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Reported-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging
This patch series fixes 2 different BUGS that didn't
reveal themselves until the 3.15 series started.
patch 1 fixes how we insure the config data
is a string. The original code is clearly
just wrong. After many loads/unloads of the
driver, various OOPs would occure. None of
which indicated this
This patch fixes sparse warnings for the
re_map_membase and re_map_port variables.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 24
drivers/staging/dgap/dgap.h | 4 ++--
2 files
This patch fixes sparse warnings for the entire bs_t structure
This entire structure defines a hardware segment
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 22
This patch fixes sparse warnings for the entire cm_t structure
This entire structure defines a hardware segment
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 8
The latest staging tree based on 3.15-rc1 has revealed a bug that I
introduced into the dgap driver way back. Various oops occur when
loading and unloading the driver multiple times in succession. The oops
never actually indicate anything dgap driver related. When I send a
patch to fix the
On 04/15/2014 06:01 AM, Dan Carpenter wrote:
On Mon, Apr 14, 2014 at 03:17:19PM -0400, Mark Hounschell wrote:
On 04/14/2014 11:49 AM, Greg Kroah-Hartman wrote:
On Tue, Mar 25, 2014 at 04:38:14PM -0400, Mark Hounschell wrote:
The config file is currently read for each board found.
It only
On 04/14/2014 12:00 PM, Greg Kroah-Hartman wrote:
On Thu, Apr 03, 2014 at 08:57:56AM -0400, Mark Hounschell wrote:
This patch removes the uchar define in dgap.h and
changes all uses of uchar to u8.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
On 04/14/2014 11:49 AM, Greg Kroah-Hartman wrote:
On Tue, Mar 25, 2014 at 04:38:14PM -0400, Mark Hounschell wrote:
The config file is currently read for each board found.
It only needs to be read one time. The buffer it is read
into can now be freed immediately after it is parsed
instead
This patch removes the uchar typedef in dgap.h
and changes all uses of it to u8.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 100 ++--
drivers/staging/dgap/dgap.h | 65
On 03/28/2014 07:34 AM, Dan Carpenter wrote:
These patches are fine and they were applied already.
On Wed, Mar 12, 2014 at 12:50:55PM -0400, Mark Hounschell wrote:
@@ -4368,15 +4364,16 @@ static void dgap_do_bios_load(struct board_t *brd,
uchar __user *ubios, int len)
/*
* Checks
On 03/28/2014 09:08 AM, Mark Hounschell wrote:
On 03/28/2014 07:34 AM, Dan Carpenter wrote:
These patches are fine and they were applied already.
On Wed, Mar 12, 2014 at 12:50:55PM -0400, Mark Hounschell wrote:
@@ -4368,15 +4364,16 @@ static void dgap_do_bios_load(struct board_t *brd,
uchar
This patch fixes checkpatch warnings about using volatiles in dgap.h
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Tested-by: Mark Hounschell ma...@compro.net
---
drivers/staging/dgap/dgap.h | 152 ++--
1
This patch fixes a checkpatch warning about adding typedefs
in dgap.h
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap.h
patch 1: remove some extra empty lines not previously caught.
patch 2: the firmware loading provess loads the config file
for each board found. It only needs to load it once.
patch 3: fixes a checkpatch warning about a macro needing parenthesis
patch 4: removes volatiles from dgap.h as
This patch fixes a checkpatch warning
Macros with complex values should be enclosed in parenthesis
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
This patch removes excessive empty lines from dgap.h
not previously caught.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.h | 26 --
1 file changed, 26 deletions(-)
diff --git a/drivers
This patch just fixes a pr_info. brd-state is not proper
at this point in time.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers
The config file is currently read for each board found.
It only needs to be read one time. The buffer it is read
into can now be freed immediately after it is parsed
instead of at driver unload time.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
patch 1 removes some unused code in dgap.c
patch 2 removes some unneeded comments in dgap.c
patch 3 fixes some whitespace warnings by checkpatch in dgap.h
patch 4 fixes several 80+ char lines in dgap.h.
Note that many of these 80+ char lines
are lines using volatile which also
Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 107 +++-
1 file changed, 35 insertions(+), 72 deletions(-)
diff --git a/drivers/staging/dgap/dgap.c b
patch 1 re-factors some code to fix warnings by
checkpatch about 80+ char lines. This patch was
suggested by Dan Carpenter. A test case for this
patch has been written and used and used to show
the patch to be correct.
patch 2 gets rid of the DGAP_LOCK and DGAP_UNLOCK macros
patch 3 re-factors
This patch replaces the DGAP_LOCK and DGAP_UNLOCK macros with
spin_lock_irqsave spin_unlock_irqrestore.
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 442 ++--
drivers
This patch fixes more 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 40 +---
1 file
This patch fixes all the simple_strtol and simple_strtoul warnings
as reported by checkpatch. After this patch, checkpatch will report
zero errors or warnings on dgap.c.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre
On 03/07/2014 05:46 PM, Dan Carpenter wrote:
I still prefer my patch. Unless I'm missing something? Could you
review it and I will submit it as a patch tomorrow.
regards,
dan carpenter
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c
index d00283a226a2..783cd75a2d96
patches 1-3 remove most of the unneeded state machine
tracking code and variables previously used in the old
firmware loading process.
patch 4 adds back in a pr_info that provided information
on each board found and configured. This got removed in
a previous patch that removed the debug and
The only driver states we need now is READY or !READY.
No need for a sysfs method of seeing those 2 states so
the sysfs entries for dgap_driver_state is also removed.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre
This patch removes more unneeded brd-state states
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 35 +--
drivers/staging/dgap/dgap.h
This patch removes unused brd-state states previously
used in the userland firmware download process.
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 5 -
drivers
This patch adds a pr_info per board that indicates
board number, type, etc..
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/staging/dgap/dgap.c b
Patch 1 - removes some unused defines in dgap.h
Patch 2 - removes some unneeded code supporting
the old firmware loading process.
--
1.8.1.4
___
devel mailing list
de...@linuxdriverproject.org
This patch removes more unneeded code that was
supporting the old firmware loading process
Signed-off-by: Mark Hounschell ma...@compro.net
Tested-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 149
This patch removes some unused defines in dgap.h
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.h | 53 ++---
1 file changed, 2 insertions(+), 51 deletions(-)
diff --git
This patch removes the unused DGAP_TRYLOCK macro
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/dgap/dgap.h b/drivers/staging/dgap/dgap.h
index
This patch gets rid of the DGAP_SPINLOCK_INIT macro
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 4 ++--
drivers/staging/dgap/dgap.h | 1 -
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git
On 03/08/2014 04:59 AM, Mark Hounschell wrote:
On 03/07/2014 05:46 PM, Dan Carpenter wrote:
I still prefer my patch. Unless I'm missing something? Could you
review it and I will submit it as a patch tomorrow.
I've reviewed it and see no obvious problems. If you like it better, I'm
OK
This patch fixes a few 80+ char lines
as reported by checkpatch
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 55 -
1 file changed, 30 insertions(+), 25 deletions
This patch fixes a few more 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 26 +-
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git
This patch fixes a few more 80+ char lines as reported by checkpatch
Signed-off-by: Mark Hounschell ma...@compro.net
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
---
drivers/staging/dgap/dgap.c | 162 ++--
1 file changed, 113 insertions(+), 49
On 03/07/2014 05:46 PM, Dan Carpenter wrote:
I still prefer my patch. Unless I'm missing something? Could you
review it and I will submit it as a patch tomorrow.
regards,
dan carpenter
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c
index d00283a226a2..783cd75a2d96
This is just for comments. The indentation was alread so far
over 80 chars I (as recommended earlier) make a couple of
new functions. Somehow I don't think this is proper and there
is probably a better way to do this. This is untested.
Signed-off-by: Mark Hounschell ma...@compro.net
---
drivers
On 03/06/2014 01:17 AM, Daeseok Youn wrote:
coccinelle warning:
drivers/staging/dgap/dgap.c:782:3-7: WARNING:
casting value returned by k[cmz]alloc to (char *) is useless.
drivers/staging/dgap/dgap.c:776:2-16: WARNING:
casting value returned by k[cmz]alloc to (struct board_t *) is useless.
On 03/05/2014 04:39 PM, Dan Carpenter wrote:
Btw, if you don't get any messages from me that means I have given your
patch the stamp of approval. So good job on your previous patchset. :)
On Wed, Mar 05, 2014 at 03:54:49PM -0500, Mark Hounschell wrote:
@@ -1613,7 +1616,8 @@ static void
1 - 100 of 209 matches
Mail list logo
