[PATCH 14/16] staging: gasket: interrupt: refactor PCI MSIX-specific handler code

From: Todd Poynor Split interrupt handler into PCI MSIX-specific and generic functions, for adding non-MSIX handlers in the future. Move MSIX init code together,, out of generic init path. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_interrupt.c | 48

[PATCH 13/16] staging: gasket: core: rename lookup_internal_desc to be PCI-specific

From: Todd Poynor Rename lookup_internal_desc() to lookup_pci_internal_desc() to reflect use for PCI devices only, in prep for non-PCI devices in the future. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 11/16] staging: gasket: core: factor out generic device add code from PCI code

From: Todd Poynor Split out generic gasket device add code from the code for adding a PCI gasket device, in prep for other gasket device types in the future. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 76 ++-- 1 file changed, 48 insertions

[PATCH 12/16] staging: gasket: core: factor out generic device remove code from PCI

From: Todd Poynor Separate code for generic parts of gasket device removal sequence from the PCI device removal code, in prep for non-PCI devices later. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 20 ++-- 1 file changed, 10 insertions(+), 10

[PATCH 06/16] staging: gasket: remove gasket_exit()

From: Todd Poynor Remove now-empty gasket_exit() function. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/gasket/gasket_core.c b/drivers/staging/gasket/gasket_core.c index 0fe5b86b294c8..aee819f379e9a

[PATCH 00/16] staging: gasket: return of the son of cleanups

From: Todd Poynor Remove extraneous memory barriers, refactor PCI-specific code in prep for platform devices in the near future, general cleanups, and make de facto maintainership official. Todd Poynor (16): MAINTAINERS: Switch a maintainer for drivers/staging/gasket staging: gasket: core

[PATCH 07/16] staging: gasket: page table: remove unnecessary NULL check

From: Todd Poynor gasket_alloc_coherent_memory remove unnecessary NULL check for coherent_pages. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/gasket/gasket_page_table.c b

[PATCH 03/16] staging: gasket: core: fix line continuation indent in gasket_alloc_dev

From: Todd Poynor Previous cleanups missed a case of multi-line function call with line continuation parameters not aligned per kernel style. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 05/16] staging: gasket: core: remove ftrace-style debug logs

From: Todd Poynor Remove debug logs that only indicate the name of the entered function, in favor of using ftrace for function tracing style logs. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/gasket

[PATCH 02/16] staging: gasket: core: remove debug log that could crash

From: Todd Poynor A debug log in gasket_alloc_dev() is issued regardless of whether the device pointer used returned success or error. The log isn't that useful anyway, remove it. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 -- 1 file changed, 2 deletions(-)

Re: [PATCH 01/16] MAINTAINERS: Switch a maintainer for drivers/staging/gasket

or these so I suggested he send an Acked-by as well, can drop that. > > > On Thu, Aug 9, 2018 at 8:20 PM, Todd Poynor wrote: > > > From: Todd Poynor > > > > > > Todd Poynor takes over for John Joseph. > > > > > > Signed-off-by: John Joseph

[PATCH] Staging: rtlwifi: efuse: fixed a line length coding style issue

Fixed a code style issue. Line length over 80 characters. Signed-off-by: Tom Todd --- drivers/staging/rtlwifi/efuse.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtlwifi/efuse.c b/drivers/staging/rtlwifi/efuse.c index 1dc71455f270..49ec9728fb04 100644

[PATCH 02/10] staging: gasket: core: convert to standard logging

From: Todd Poynor Use standard logging functions, drop use of gasket log functions. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 295 --- 1 file changed, 134 insertions(+), 161 deletions(-) diff --git a/drivers/staging/gasket/gasket_core.c b

[PATCH 01/10] staging: gasket: save struct device for a gasket device

From: Todd Poynor Save the struct device pointer to a gasket device in gasket's metadata, to facilitate use of standard logging calls and in anticipation of non-PCI gasket devices in the future. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 5 +++-- drivers/st

[PATCH 09/10] staging: gasket: TODO: remove entry for convert to standard logging

From: Todd Poynor Gasket/apex drivers now use standard logging, remove TODO entry for this. Signed-off-by: Todd Poynor --- drivers/staging/gasket/TODO | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/gasket/TODO b/drivers/staging/gasket/TODO index d3c44ca4fda25

[PATCH 08/10] staging: gasket: remove gasket logging header

From: Todd Poynor Gasket logging functions no longer used. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_logging.h | 64 - 1 file changed, 64 deletions(-) delete mode 100644 drivers/staging/gasket/gasket_logging.h diff --git a/drivers/staging/gasket

[PATCH 06/10] staging: gasket: sysfs: convert to standard logging

From: Todd Poynor Drop gasket logging calls in favor of standard logging. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_sysfs.c | 73 +-- 1 file changed, 35 insertions(+), 38 deletions(-) diff --git a/drivers/staging/gasket/gasket_sysfs.c b/drivers

[PATCH 05/10] staging: gasket: page table: convert to standard logging

From: Todd Poynor Replace gasket logging calls with standard logging calls. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 131 + 1 file changed, 54 insertions(+), 77 deletions(-) diff --git a/drivers/staging/gasket/gasket_page_table.c b

[PATCH 10/10] staging: gasket: don't print device addresses as kernel pointers

From: Todd Poynor Print device addresses as unsigned long, not as kernel pointers. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/gasket/gasket_page_table.c b/drivers

[PATCH 03/10] staging: gasket: interrupt: convert to standard logging

From: Todd Poynor Convert gasket logging calls to standard functions. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_interrupt.c | 67 +++ 1 file changed, 32 insertions(+), 35 deletions(-) diff --git a/drivers/staging/gasket/gasket_interrupt.c b/drivers

[PATCH 04/10] staging: gasket: ioctl: convert to standard logging

From: Todd Poynor Replace gasket logging calls with standard logging calls. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/drivers/staging/gasket/gasket_ioctl.c b/drivers

[PATCH 00/10] staging: gasket: logging cleanups

From: Todd Poynor Kill off gasket logging functions, convert to standard. Fixup a few formatting/style problems in the process. Todd Poynor (10): staging: gasket: save struct device for a gasket device staging: gasket: core: convert to standard logging staging: gasket: interrupt: convert

[PATCH 07/10] staging: gasket: apex: convert to standard logging

From: Todd Poynor Drop gasket logging calls in favor of standard logging. Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 61 1 file changed, 26 insertions(+), 35 deletions(-) diff --git a/drivers/staging/gasket/apex_driver.c b/drivers

Re: [PATCH 06/10] staging: gasket: sysfs: convert to standard logging

On Fri, Jul 27, 2018 at 8:07 AM Greg Kroah-Hartman wrote: > > On Thu, Jul 26, 2018 at 08:07:33PM -0700, Todd Poynor wrote: > > From: Todd Poynor > > > > Drop gasket logging calls in favor of standard logging. > > > > Signed-off-by: Todd Poynor > > ---

Re: [PATCH 01/10] staging: gasket: save struct device for a gasket device

On Fri, Jul 27, 2018 at 8:09 AM Greg Kroah-Hartman wrote: > > On Thu, Jul 26, 2018 at 08:07:28PM -0700, Todd Poynor wrote: > > From: Todd Poynor > > > > Save the struct device pointer to a gasket device in gasket's metadata, > > to facilitate use of standard l

[PATCH 4/5] staging: gasket: page table: remove code for "no dma_ops"

From: Todd Poynor Remove code with TODOs on it for working around apparent problems previously seen in a qemu environment where dma_ops was not set correctly. There is no user of this in the current code. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2

[PATCH 5/5] staging: gasket: core: hold reference on device kobj while in use

From: Todd Poynor Hold a reference on the struct device kobject while a pointer to that device is in use by gasket. Reported-by: Greg Kroah-Hartman Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a

[PATCH 3/5] staging: gasket: sysfs: remove unnecessary NULL check on device ptr

From: Todd Poynor The device pointer passed into get_mapping() will never be NULL; the check is unnecessary. Reported-by: Greg Kroah-Hartman Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_sysfs.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/gasket

[PATCH 1/5] staging: gasket: sysfs: remove check for refcount already zero

From: Todd Poynor Remove the check for refcount already zero, which shouldn't be necessary. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_sysfs.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/gasket/gasket_sysfs.c b/drivers/staging/gasket/gasket_sy

[PATCH 0/5] staging: gasket: fixes and cleanups

From: Todd Poynor The fun continues with gasket+apex: remove dead code and unnecessary stuff, fixup apex PCI class for devices that advertise class 0 (undefined), and make sure the struct device doesn't go away on us. Most of these from review comments of previous patch series. Todd Poyn

[PATCH 2/5] staging: gasket: apex: fixup undefined PCI class

From: Todd Poynor Apex chips with class 0 (PCI_CLASS_NOT_DEFINED) fixed up to PCI_CLASS_SYSTEM_OTHER to enable PCI resource assignments. Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/staging/gasket

[PATCH 0/1 v2] staging: gasket: fixes and cleanups

From: Todd Poynor The fun continues with gasket+apex: remove dead code and unnecessary stuff, fixup apex PCI class for devices that advertise class 0 (undefined), and make sure the struct device doesn't go away on us. Most of these from review comments of previous patch series. Changed pa

[PATCH 1/1] staging: gasket: core: hold reference on device while in use

From: Todd Poynor Hold a reference on the struct device while a pointer to that device is in use by gasket. Reported-by: Greg Kroah-Hartman Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 04/13] staging: gasket: core: allow root access based on user namespace

From: Todd Poynor Use user namespace to determine whether gasket device file opener is root, allowing root access to containers, if necessary. Reported-by: Dmitry Torokhov Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 10 -- 1 file changed, 8 insertions(+), 2

[PATCH 05/13] staging: gasket: apex: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 64 +--- 1 file changed, 10 inser

[PATCH 06/13] staging: gasket: core: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 151 ++- 1 file changed, 31 inser

[PATCH 09/13] staging: gasket: interrupt: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_interrupt.c | 18 +- 1 file changed, 1 insertion(+

[PATCH 00/13] staging: gasket: fixes and cleanups

From: Todd Poynor Fixes for device reference counting and root access based on user namespace for containers, plus cleanups of comments, forward declarations for static functions (more forthcoming) and multi-line continuation style (more to come). Todd Poynor (13): staging: gasket: core: hold

[PATCH 07/13] staging: gasket: ioctl: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 51 +-- 1 file changed, 9 inser

[PATCH 12/13] staging: gasket: apex: remove static function forward declarations

From: Todd Poynor Remove forward declarations of static functions, move code to avoid forward references, for kernel style. Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 416 --- 1 file changed, 190 insertions(+), 226 deletions(-) diff --git a

[PATCH 11/13] staging: gasket: TODO: remove entry for static function kernel docs

From: Todd Poynor Remove the TODO entry for simplifying kernel doc style comments for static functions, now that this has been addressed. Signed-off-by: Todd Poynor --- drivers/staging/gasket/TODO | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/gasket/TODO b/drivers/staging

[PATCH 08/13] staging: gasket: page table: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 323 +++-- 1 file changed, 48 inser

[PATCH 13/13] staging: gasket: apex: fix function param line continuation style

From: Todd Poynor Fix multi-line alignment formatting to look like: int ret = long_function_name(device, VARIABLE1, VARIABLE2, VARIABLE3, VARIABLE4); Many of these TODO items were previously cleaned up during the conversion to standard logging functions

[PATCH 10/13] staging: gasket: sysfs: simplify comments for static functions

From: Todd Poynor Static functions don't need kernel doc formatting, can be simplified. Reformat comments that can be single-line. Remove extraneous text. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_sysfs.c | 28 --- 1 file changed, 4 inser

[PATCH 03/13] staging: gasket: page table: hold references to device and pci_dev

From: Todd Poynor Hold references to the struct device and the pci_dev for the page table while the data structures contian pointers to these. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a

[PATCH 01/13] staging: gasket: core: hold reference to pci_dev while used

From: Todd Poynor Hold a reference on the struct pci_dev while a pointer to it is held in the gasket data structures. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gasket

[PATCH 02/13] staging: gasket: sysfs: hold reference to device while in use

From: Todd Poynor Hold a reference to the struct device while a gasket sysfs mapping exists for the device and a pointer to the struct is kept in the mapping data structures. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_sysfs.c | 11 +-- 1 file changed, 5 insertions

Re: [PATCH 04/13] staging: gasket: core: allow root access based on user namespace

Hi Dmitry, On Mon, Jul 30, 2018 at 10:57 AM Dmitry Torokhov wrote: > > Hi Todd, > > On Sun, Jul 29, 2018 at 12:37 PM Todd Poynor wrote: > > @@ -1064,7 +1067,8 @@ static int gasket_open(struct inode *inode, struct > > file *filp) > > char task_name[TASK

Re: [PATCH resend] staging: gasket: remove X86 Kconfig restriction

On Wed, Jul 18, 2018 at 6:18 PM, kbuild test robot wrote: > Hi Todd, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on staging/staging-testing] > [also build test ERROR on next-20180718] > [cannot apply to v4.18-rc5] > [if your patch is

Re: [PATCH 16/32] staging: gasket: always allow root open for write

On Thu, Jul 19, 2018 at 2:29 AM, Greg Kroah-Hartman wrote: > On Tue, Jul 17, 2018 at 01:56:56PM -0700, Todd Poynor wrote: >> From: Todd Poynor >> >> Always allow root to open device for writing. >> >> Drop special-casing of ioctl permissions for root vs. owner. &

Re: [PATCH 17/32] staging: gasket: annotate ioctl arg with __user

On Thu, Jul 19, 2018 at 2:37 AM, Greg Kroah-Hartman wrote: > On Tue, Jul 17, 2018 at 01:56:57PM -0700, Todd Poynor wrote: >> From: Todd Poynor >> >> For sparse checking. > > Close, but you can do better :) > >> >> Reported-by: Dmitry Torokhov >> S

[PATCH 11/20] staging: gasket: fix comment syntax in apex.h

From: Todd Poynor Use kernel-style multi-line comment syntax. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/gasket/apex.h b/drivers

[PATCH 10/20] staging: gasket: remove else clause after return in if clause

From: Todd Poynor Else after return is unnecessary and may cause static code checkers to complain. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff

[PATCH 04/20] staging: gasket: fix class create bug handling

From: Todd Poynor class_create() never returns NULL, and this driver should never return PTR_ERR(NULL) anyway. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor Reviewed-by: Dmitry Torokhov --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1

[PATCH 00/20 v4] staging: gasket: sundry fixes and fixups

From: Todd Poynor Various fixes mainly from the chromium review of the gasket and apex drivers. More to come. Todd Poynor (20): staging: gasket: allow compile for ARM64 in Kconfig staging: gasket: gasket_enable_dev remove unnecessary variable staging: gasket: remove code for no physical

[PATCH 19/20] staging: gasket: common ioctl dispatcher add __user annotations

From: Todd Poynor Add __user annotation to gasket core common ioctl pointer arguments for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 8 +--- drivers/staging/gasket/gasket_ioctl.h | 4

[PATCH 20/20] staging: gasket: common ioctls add __user annotations

From: Todd Poynor Add __user annotation to gasket common ioctl pointer arguments for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 102 ++ 1 file changed, 55 insertions

[PATCH 02/20] staging: gasket: gasket_enable_dev remove unnecessary variable

From: Todd Poynor Remove unnecessary variable, pass constant param instead. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging

[PATCH 07/20] staging: gasket: gasket_mmap return error instead of valid BAR index

From: Todd Poynor When offset to be mapped matches both a BAR region and a coherent mapped region return an error as intended, not the BAR index. Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 18/20] staging: gasket: apex ioctl add __user annotations

From: Todd Poynor Add __user annotation to ioctl pointer argument, for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff

[PATCH 12/20] staging: gasket: remove unnecessary parens in page table code

From: Todd Poynor gasket_alloc_coherent_memory() extra parentheses in statement. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 13/20] staging: gasket: gasket_mmap use PAGE_MASK

From: Todd Poynor gasket_mmap use PAGE_MASK, instead of performing math on PAGE_SIZE, for simplicity and clarity. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 16/20] staging: gasket: always allow root open for write

From: Todd Poynor Always allow root to open device for writing. Drop special-casing of ioctl permissions for root vs. owner. Convert to bool types as appropriate. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c

[PATCH 17/20] staging: gasket: top ioctl handler add __user annotations

From: Todd Poynor Add __user annotation to gasket_core top-level ioctl handling pointer arguments, for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 6 -- drivers/staging/gasket/gasket_core.h

[PATCH 15/20] staging: gasket: fix multi-line comment syntax in gasket_core.h

From: Todd Poynor Use consistent kernel-style multi-line comment syntax. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/gasket

[PATCH 14/20] staging: gasket: remove extra parens in gasket_write_mappable_regions

From: Todd Poynor Remove unneeded parentheses around subexpressions. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gasket

[PATCH 08/20] staging: gasket: apex_clock_gating simplify logic, reduce indentation

From: Todd Poynor Collapse together two checks and return immediately, avoid conditional indentation for most of function code. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 43 +--- 1 file

[PATCH 06/20] staging: gasket: don't treat no device reset callback as an error

From: Todd Poynor It is not an error for a device to not have a reset callback registered. Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/gasket

[PATCH 05/20] staging: gasket: remove unnecessary code in coherent allocator

From: Todd Poynor Remove extraneous statement in gasket_config_coherent_allocator() Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers

[PATCH 03/20] staging: gasket: remove code for no physical device

From: Todd Poynor gasket_enable_dev code for enabling a gasket device with no physical PCI device registered shouldn't be necessary. Reported-by: Greg Kroah-Hartman Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 12 +--- 1 file changed, 1 insertion(+

[PATCH 09/20] staging: gasket: gasket page table functions use bool return type

From: Todd Poynor Convert from int to bool return type for gasket page table functions that return values used as booleans. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 58 +++--- drivers

[PATCH 01/20] staging: gasket: allow compile for ARM64 in Kconfig

From: Todd Poynor The gasket and apex drivers are also to be used on ARM64 architectures. Signed-off-by: Todd Poynor --- drivers/staging/gasket/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gasket/Kconfig b/drivers/staging/gasket/Kconfig index

[PATCH 00/14] staging: gasket: assorted cleanups

From: Todd Poynor Various fixups arising from the chromium review of the gasket and apex drivers. Todd Poynor (14): staging: gasket: fix check_and_invoke_callback log param staging: gasket: remove duplicate call to retrieve device callback staging: gasket: gasket_handle_ioctl fix ioctl

[PATCH 13/20] staging: gasket: gasket_mmap use PAGE_MASK

From: Todd Poynor gasket_mmap use PAGE_MASK, instead of performing math on PAGE_SIZE, for simplicity and clarity. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 04/14] staging: gasket: avoid copy to user on error in coherent alloc config

From: Todd Poynor gasket_config_coherent_allocator() on error return the error to caller without copying a possibly-update DMA address back to userspace. Reported-by: Dmitry Torokhov Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 4 +++- 1 file changed, 3 insertions

[PATCH 11/14] staging: gasket: ioctl common: convert various logs to debug level

From: Todd Poynor Debugging information is improperly logged at non-debug log level in a number of places, and some logs regarding error conditions may be generated too frequently, such that these could cause performance problems and/or obscure other logs. Convert these to debug log level

[PATCH 01/14] staging: gasket: fix check_and_invoke_callback log param

From: Todd Poynor The message should be passed the callback function pointer, not the pointer to the gasket device. Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH 09/20] staging: gasket: gasket page table functions use bool return type

From: Todd Poynor Convert from int to bool return type for gasket page table functions that return values used as booleans. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 58 +++--- drivers

[PATCH 01/20] staging: gasket: allow compile for ARM64 in Kconfig

From: Todd Poynor The gasket and apex drivers are also to be used on ARM64 architectures. Signed-off-by: Todd Poynor --- drivers/staging/gasket/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gasket/Kconfig b/drivers/staging/gasket/Kconfig index

[PATCH 18/20] staging: gasket: apex ioctl add __user annotations

From: Todd Poynor Add __user annotation to ioctl pointer argument, for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff

[PATCH 06/20] staging: gasket: don't treat no device reset callback as an error

From: Todd Poynor It is not an error for a device to not have a reset callback registered. Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/staging/gasket

[PATCH 17/20] staging: gasket: top ioctl handler add __user annotations

From: Todd Poynor Add __user annotation to gasket_core top-level ioctl handling pointer arguments, for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 6 -- drivers/staging/gasket/gasket_core.h

[PATCH 20/20] staging: gasket: common ioctls add __user annotations

From: Todd Poynor Add __user annotation to gasket common ioctl pointer arguments for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 102 ++ 1 file changed, 55 insertions

[PATCH 19/20] staging: gasket: common ioctl dispatcher add __user annotations

From: Todd Poynor Add __user annotation to gasket core common ioctl pointer arguments for sparse checking. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_ioctl.c | 8 +--- drivers/staging/gasket/gasket_ioctl.h | 4

[PATCH 16/20] staging: gasket: always allow root open for write

From: Todd Poynor Always allow root to open device for writing. Drop special-casing of ioctl permissions for root vs. owner. Convert to bool types as appropriate. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c

[PATCH 15/20] staging: gasket: fix multi-line comment syntax in gasket_core.h

From: Todd Poynor Use consistent kernel-style multi-line comment syntax. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/gasket

[PATCH 12/14] staging: gasket: page table: convert various logs to debug level

From: Todd Poynor Debugging information is improperly logged at non-debug log level in a number of places, and some logs regarding error conditions may be generated too frequently, such that these could cause performance problems and/or obscure other logs. Convert these to debug log level

[PATCH 13/14] staging: gasket: page table: remove unnecessary logs

From: Todd Poynor Some error logs in page table handling code could only be hit in cases of programming errors not expected in the current code base, and aren't likely to be useful on their own. Remove these. Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c

[PATCH 14/20] staging: gasket: remove extra parens in gasket_write_mappable_regions

From: Todd Poynor Remove unneeded parentheses around subexpressions. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gasket

[PATCH 14/14] staging: gasket: apex: convert various logs to debug level

From: Todd Poynor Debugging information is improperly logged at non-debug log level in a number of places, and some logs regarding error conditions may be generated too frequently, such that these could cause performance problems and/or obscure other logs. Convert these to debug log level

[PATCH 12/20] staging: gasket: remove unnecessary parens in page table code

From: Todd Poynor gasket_alloc_coherent_memory() extra parentheses in statement. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 11/20] staging: gasket: fix comment syntax in apex.h

From: Todd Poynor Use kernel-style multi-line comment syntax. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/gasket/apex.h b/drivers

[PATCH 06/14] staging: gasket: remove unnecessary NULL checks on calls from VFS

From: Todd Poynor Remove unneeded checks for NULL pointers in struct file pointers passed from the VFS layer or the private_data that must have been properly set at file open time. Reported-by: Guenter Roeck Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 15

[PATCH 08/20] staging: gasket: apex_clock_gating simplify logic, reduce indentation

From: Todd Poynor Collapse together two checks and return immediately, avoid conditional indentation for most of function code. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 43 +--- 1 file

[PATCH 04/20] staging: gasket: fix class create bug handling

From: Todd Poynor class_create() never returns NULL, and this driver should never return PTR_ERR(NULL) anyway. Reported-by: Dmitry Torokhov Signed-off-by: Zhongze Hu Signed-off-by: Todd Poynor Reviewed-by: Dmitry Torokhov --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1

[PATCH 10/20] staging: gasket: remove else clause after return in if clause

From: Todd Poynor Else after return is unnecessary and may cause static code checkers to complain. Reported-by: Guenter Roeck Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_page_table.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff

[PATCH 09/14] staging: gasket: core: convert various logs to debug level

From: Todd Poynor Debugging information is improperly logged at non-debug log level in a number of places, and some logs regarding error conditions may be generated too frequently, such that these could cause performance problems and/or obscure other logs. Convert these to debug log level

[PATCH 08/14] staging: gasket: apex return error on sysfs show of missing attribute

From: Todd Poynor Apex sysfs show function return -ENODEV if the attribute is not present, rather than silently failing from the standpoint of the userspace accessor. Reported-by: Guenter Roeck Signed-off-by: Todd Poynor --- drivers/staging/gasket/apex_driver.c | 4 ++-- 1 file changed, 2

[PATCH 07/20] staging: gasket: gasket_mmap return error instead of valid BAR index

From: Todd Poynor When offset to be mapped matches both a BAR region and a coherent mapped region return an error as intended, not the BAR index. Signed-off-by: Simon Que Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH 07/14] staging: gasket: gasket_get_device drop check for NULL pci_dev

From: Todd Poynor The pci_dev field of a struct gasket_dev can never be NULL, there's no need to check for this in gasket_get_device(). Reported-by: Guenter Roeck Signed-off-by: Todd Poynor --- drivers/staging/gasket/gasket_core.c | 4 +--- 1 file changed, 1 insertion(+), 3 dele

<    1   2   3   4   5   6   7   8   >