Re: [PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

On Tue, Oct 17, 2017 at 12:15 AM, Sean Paul wrote: > On Sat, Oct 14, 2017 at 2:36 PM, Harsha Sharma > wrote: >> Replace instances of drm_framebuffer_reference/unreference() with >> *_get/put() suffixes and drm_dev_unref with *_put() suffix >> because get/put is shorter

[PATCH] drm/tinydrm: Replace list_for_each with list_for_each_entry

@ identifier s.i,s.f; expression s.e; statement S; @@ list_for_each_entry(i,e,f) - { S - } Signed-off-by: Harsha Sharma --- drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c b

[PATCH] drm/omap: Replace list_for_each with list_for_each_entry

@ identifier s.i,s.f; expression s.e; statement S; @@ list_for_each_entry(i,e,f) - { S - } Signed-off-by: Harsha Sharma --- drivers/gpu/drm/omapdrm/dss/display.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/dss/display.c b/dri

[PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma --- Changes in v5: -rebase drm_dev_put change on drm-tip Changes in v4: -change one instance of *_put to *_get Changes in

[PATCH v4] drm/amd/powerplay: Remove unnecessary cast on void pointer

Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma --- Changes in v4: -Removed git diff warning "No newline at end of file" Changes in v3: -Remove

[PATCH v3] drm/amd/powerplay: Remove unnecessary cast on void pointer

Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma --- Changes in v3: -Removed unnecessary lines -Remove more useless casts Changes in v2: -Remove unnecess

[PATCH v2] drm/amd/powerplay: Remove unnecessary cast on void pointer

Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma --- Changes in v2: -Remove unnecessary parentheses -Remove one more useless cast drivers/gpu/drm/

Re: [PATCH v4] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

On Mon, Oct 9, 2017 at 5:36 PM, Harsha Sharma wrote: > Replace instances of drm_framebuffer_reference/unreference() with > *_get/put() suffixes and drm_dev_unref with *_put() suffix > because get/put is shorter and consistent with the > kernel use of *_get/put suffixes. > Done

[PATCH] drm/amd/powerplay: Remove unnecessary cast on void pointer

Done with following coccinelle patch @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T*)x)->f | - (T*) e ) Signed-off-by: Harsha Sharma --- drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c| 6 +++--- drivers/gpu/drm/amd/powerplay/hw

[PATCH v2] drm: Replace kzalloc with kcalloc

Prefer kcalloc over kzalloc to allocate an array. This patch fixes checkcpatch issue. Signed-off-by: Harsha Sharma --- Changes in v2: -kcalloc will take 3 arguments drivers/gpu/drm/drm_crtc_helper.c | 4 ++-- drivers/gpu/drm/drm_fb_helper.c| 2 +- drivers/gpu/drm/drm_plane_helper.c | 2

[PATCH] netfilter: nf_conntrack_h323: Remove typedef struct

ccinelle.T2 = T[:-2]; else: coccinelle.T2 = T; print T, coccinelle.T2 @r2@ type r1.T; identifier c1.T2; @@ -typedef struct + T2 { ... } -T ; @r3@ type r1.T; identifier c1.T2; @@ -T +struct T2 Signed-off-by: Harsha Sharma --- net/netfilter/nf_conntrack_h323_asn1.c | 80 +---

[PATCH] drm: Replace kzalloc with kcalloc

Prefer kcalloc over kzalloc to allocate an array. This patch fixes checkcpatch issue. Signed-off-by: Harsha Sharma --- drivers/gpu/drm/drm_crtc_helper.c | 4 ++-- drivers/gpu/drm/drm_fb_helper.c| 2 +- drivers/gpu/drm/drm_plane_helper.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions

[PATCH v4] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma --- Changes in v4: -change one instance of *_put to *_get Changes in v3: -Removed changes in selftests Changes in v2

[PATCH v3] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma --- Changes in v3: -Removed changes in selftests Changes in v2: -Added cocinelle patch in log message -cc to all driver

[PATCH v3 06/10] staging: rtl8723bs: Add space after ','

Space is required after ',' according to linux-kernel coding style. Signed-off-by: Harsha Sharma --- Other patches from this patchset have already been merged and only this one is remaining. Changes in v3: -Change log message and rebase against staging-testing Changes in v2: -Reba

Re: [PATCH v2] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

On Fri, Sep 29, 2017 at 10:00 AM, Harsha Sharma wrote: > Replace instances of drm_framebuffer_reference/unreference() with > *_get/put() suffixes and drm_dev_unref with *_put() suffix > because get/put is shorter and consistent with the > kernel use of *_get/put suffixes. > Done

Re: [PATCH v2 06/10] staging: rtl8723bs: Add space after ','

On Wed, Sep 13, 2017 at 2:11 AM, Greg KH wrote: > On Tue, Sep 12, 2017 at 07:05:23PM +0530, Harsha Sharma wrote: >> Space required after ',' >> >> Signed-off-by: Harsha Sharma >> --- >> Changes in v2: >> -Rebase against staging-testing and solve

[PATCH v3] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma --- Changes in v3: -Solve merge conflicts Changes in v2: -Fix alignment issues drivers/gpu/drm/tinydrm/mi0283qt.c | 8 drivers/gpu/drm/tinydrm

[PATCH] test: shell: execute shell/run-tests.sh from any directory

Update shell/run-tests.sh to refer /src/nft with a relative path Signed-off-by: Harsha Sharma --- tests/shell/run-tests.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/shell/run-tests.sh b/tests/shell/run-tests.sh index 4eba0a8..dbddd8d 100755 --- a/tests/shell/run

[PATCH] INSTALL: Update dependency list and configure with libxtables support

Add configure with lixtables in INSTALL and required dependencies for the same Signed-off-by: Harsha Sharma --- INSTALL | 11 +++ 1 file changed, 11 insertions(+) diff --git a/INSTALL b/INSTALL index 3e9a6ad..04981f1 100644 --- a/INSTALL +++ b/INSTALL @@ -18,6 +18,12 @@ Installation

[PATCH 2/3] evaluate: Place constant on right side in comparison

Comparisons should place the constant on the right side of the test as per linux-kernel coding style Signed-off-by: Harsha Sharma --- src/evaluate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/evaluate.c b/src/evaluate.c index 5624ca2..b783054 100644 --- a/src

[PATCH 3/3] evaluate: make pointers in string arrays constant

static const char * array should probably be static const char * const array as per linux-kernel coding style Signed-off-by: Harsha Sharma --- src/evaluate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/evaluate.c b/src/evaluate.c index b783054..f48801a 100644

[PATCH 1/3] evaluate: Remove unnecessary spaces

Code indent should use tabs wherever possible Signed-off-by: Harsha Sharma --- src/evaluate.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/evaluate.c b/src/evaluate.c index e767542..5624ca2 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -122,7 +122,7

[PATCH 0/3] evaluate: Follow linux-kernel coding style

Issues found using checkpatch.pl As per linux-kernel coding style, code indent should use tabs wherever possible and avoid unnecessary spaces. Comparisons shoukd place the constant on the right side of the test. static const char * array should be static const * char const array Harsha Sharma

[PATCH] exthdr: Add support for reserved header and address

Add support for IPV6 type 0 routing header reserved field and address unable to test it with nft-test.py Signed-off-by: Harsha Sharma --- include/exthdr.h | 2 ++ src/exthdr.c | 7 +-- tests/py/ip6/rt.t | 2 ++ 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/include

[PATCH v2] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma --- Changes in v2: -Fix alignment issues drivers/gpu/drm/tinydrm/mi0283qt.c | 8 drivers/gpu/drm/tinydrm/repaper.c | 26

[PATCH v2] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

; @@ ( -drm_framebuffer_unreference(ex); +drm_framebuffer_put(ex); | -drm_dev_unref(ex); +drm_dev_put(ex); | -drm_framebuffer_reference(ex); +drm_framebuffer_get(ex); ) Signed-off-by: Harsha Sharma --- Changes in v2: -Added coccinelle patch in log message -cc to all driver-specific mailing lists

[PATCH] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

Replace instances of drm_framebuffer_reference/unreference() with *_get/put() suffixes and drm_dev_unref with *_put() suffix because get/put is shorter and consistent with the kernel use of *_get/put suffixes . Signed-off-by: Harsha Sharma --- drivers/gpu/drm/i915/i915_pci.c

[PATCH] iptables: Constify option struct

; expression e; position p; @@ e = i@p @bad@ position p != {r1.p,ok1.p}; identifier r1.i; @@ e@i@p @depends on !bad disable optional_qualifier@ identifier r1.i; @@ static +const struct option i[] = { ... }; Signed-off-by: Harsha Sharma --- iptables/ip6tables.c | 2 +- iptables/iptables.c | 2

[PATCH] staging: iio: trigger: Move header file content to source file

The contents of the header file are used only by this single source file. Moved content into iio-trig-bfin-timer.c and removed iio-trig-bfin-timer.h Signed-off-by: Harsha Sharma --- drivers/staging/iio/trigger/iio-trig-bfin-timer.c | 20 ++- drivers/staging/iio/trigger/iio-trig

[PATCH v2] drm/tegra: Replace dev_* with DRM_DEV_*

Replace all occurences of dev_info/err/dbg with DRM_DEV_INFO/ ERROR/DEBUG as we have DRM_DEV_* variants of drm print macros Done using following coccinelle semantic patch @r@ @@ ( -dev_info +DRM_DEV_INFO | -dev_err +DRM_DEV_ERROR | -dev_dbg +DRM_DEV_DEBUG ) Signed-off-by: Harsha Sharma

[PATCH] drm/tegra: Replace dev_* with DRM_DEV_*

Replace all occurences of dev_info/err/dbg with DRM_DEV_INFO/ ERROR/DEBUG as we have DRM_DEV_* variants of drm print macros Done using following coccinelle semantic patch @r@ @@ ( -dev_info +DRM_DEV_INFO | -dev_err +DRM_DEV_ERROR | -dev_dbg +DRM_DEV_DEBUG ) Signed-off-by: Harsha Sharma

[PATCH] drm/tinydrm: Replace dev_error with DRM_DEV_ERROR

Convert instances of dev_error to DRM_DEV_ERROR as we have DRM_DEV_ERROR variants of drm print macros. Signed-off-by: Harsha Sharma --- drivers/gpu/drm/tinydrm/mi0283qt.c | 8 drivers/gpu/drm/tinydrm/repaper.c | 26 +- drivers/gpu/drm/tinydrm/st7586.c | 6

[PATCH v2] staging: vc04_services: Remove typedef struct

ccinelle.T2 = T[:-2]; else: coccinelle.T2 = T; print T, coccinelle.T2 @r2@ type r1.T; identifier c1.T2; @@ -typedef struct + T2 { ... } -T ; @r3@ type r1.T; identifier c1.T2; @@ -T +struct T2 Signed-off-by: Harsha Sharma --- Changes in v2: -Convert structure name to lowercase .../vc04_se

[PATCH] staging: vc04_services: Remove typedef struct

ccinelle.T2 = T[:-2]; else: coccinelle.T2 = T; print T, coccinelle.T2 @r2@ type r1.T; identifier c1.T2; @@ -typedef struct + T2 { ... } -T ; @r3@ type r1.T; identifier c1.T2; @@ -T +struct T2 Signed-off-by: Harsha Sharma --- .../vc04_services/interface/vchiq_arm/vchiq_shim.c | 44 +++-

[PATCH] staging: rtlwifi: delete double assignment

e2; | (<+...++e1...+>)=e2; | (<+...--e1...+>)=e2; | e1=e2; e1 = <+...e1...+>; | *e1=e2; *e1=e3; ) Signed-off-by: Harsha Sharma --- drivers/staging/rtlwifi/base.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/rtlwifi/base.c b/drivers/staging/rtlwifi/base.c

[PATCH] staging: rtl8723bs: Remove unused variable ret

Remove unused variable ret as it is not used anywhere. Remove multiple blank lines. Done using following coccinelle semantic patch @@ type T; identifier i; constant C; @@ ( extern T i; | - T i; <+... when != i - i = C; ...+> ) Signed-off-by: Harsha Sharma --- drivers/staging/rtl

[PATCH] staging: rtl8723bs: Merge assignment with return

Merge assignment with return statement to directly return the value. Done using following coccinelle semantic patch @@ local idexpression ret; expression e; @@ -ret = +return e; -return ret; Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 20

[PATCH v2 06/10] staging: rtl8723bs: Add space after ','

Space required after ',' Signed-off-by: Harsha Sharma --- Changes in v2: -Rebase against staging-testing and solve merge conflicts drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl872

[PATCH] staging: rtl8723bs: Remove unnecessary spaces

Space between function name and open parentheses '(' is prohibited. Space is required around most binary operators '=', '==', '+=', '<', ':', '+', '-' Space required before '&', '*' Space i

[PATCH v3 07/10] staging: rtl8723bs: Remove unnecessary space

Remove space between function name and open parenthesis '(' Remove space before ';', '++', ',' Signed-off-by: Harsha Sharma --- Change in v3: -Fix small spelling mistake Change in v2: -Merge patches 07/10 and 08/10 drivers/staging/rtl8723bs/os_dep/os_i

[PATCH v2 07/10] staging: rtl8723bs: Remove unneccesary space

Remove space between function name and open parenthesis '(' Remove space before ';', '++', ',' Signed-off-by: Harsha Sharma --- Change in v2: -Merge patches 07/10 and 08/10 drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file change

[PATCH v3] staging: rtl8723bs: Change coding style to (foo *bar)

This coding style (foo *bar) is more common for the kernel code. Change foo* bar to foo *bar. Change foo * bar to foo *bar. Change (foo*) to (foo *). Signed-off-by: Harsha Sharma --- Change in v3: -Rebase against staging-testing branch Change in v2: -Updated log message and subject drivers

[PATCH 10/10] staging: rtl8723bs: Remove unnecessary blank lines

Blank lines aren't necessary after an open brace and before a close brace Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/o

[PATCH 09/10] staging: rtl8723bs: Remove unneccesary braces and change position of open brace

Follow linux-kernel code style for conditional statements Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 63 + 1 file changed, 19 insertions(+), 44 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers

[PATCH 08/10] staging: rtl8723bs: Remove unnecessary space

Remove space before ';', '++', ',' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/o

[PATCH 07/10] staging: rtl8723bs: Removed unneccesary space

Remove space between function name and open parenthesis '(' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl872

[PATCH 06/10] staging: rtl8723bs: Add space after ','

Space required after ',' Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c ind

[PATCH 05/10] staging: rtl8723bs: Add space between concatenated strings

Use spaces between concatenated strings Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 50 ++--- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs

[PATCH 04/10] staging: rtl8723bs: Remove unnecessary spaces at the start of line

No spaces at the start of a line Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index 5ceccd7..97ff3ab

[PATCH 03/10] staging: rtl8723bs: Add spaces around '?', '==', '||', '!='

Use spaces around most binary operators Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c index

[PATCH 02/10] staging: rtl8723bs: Add spaces around '+', '|', '*'

Use one space around most binary operators Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs/os_dep/os_intfs.c

[PATCH 01/10] staging: rtl8723bs: Add spaces around '='

Use one space around (on each side of) '=' operator Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging

[PATCH 00/10] staging: rtl8723bs: Follow linux-kernel coding style

e a single statement will do and put the opening brace last on the line, and put the closing brace first. Unnecessary spaces and blank lines should be avoided. Harsha Sharma (10): staging: rtl8723bs: Add spaces around '=' staging: rtl8723bs: Add spaces around '+', '|

[PATCH] staging: rtl8723bs: Add spaces around '=', '|', '<<'

Use one space around most binary operators Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs

[PATCH v2] staging: rtl8723bs: Change coding style to (foo *bar)

This coding style (foo *bar) is more common for the kernel code. Change foo* bar to foo *bar. Change foo * bar to foo *bar. Change (foo*) to (foo *). Signed-off-by: Harsha Sharma --- Change in v2: -Updated log message and subject drivers/staging/rtl8723bs/os_dep/os_intfs.c | 10 +- 1

[PATCH] staging: rtl8723bs: Change foo* bar to foo *bar

This coding style (foo *bar) is more common for the kernel code. Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs

[PATCH] staging: rtl8723bs: Do not initialise statics to 0

Static variables are initialised to 0 by gcc Signed-off-by: Harsha Sharma --- drivers/staging/rtl8723bs/os_dep/os_intfs.c | 48 ++--- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/staging/rtl8723bs/os_dep/os_intfs.c b/drivers/staging/rtl8723bs

[PATCH] staging: sm750fb: Remove extra blank lines

This was reported by checkpatch.pl Signed-off-by: Harsha Sharma --- drivers/staging/sm750fb/sm750_cursor.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/sm750fb/sm750_cursor.c b/drivers/staging/sm750fb/sm750_cursor.c index aa47a16..6b27b06 100644 --- a/drivers/staging

[PATCH] staging: unisys: visorbus: Declared char * array as static const

State explicitly that individual entries in array will not change. Signed-off-by: Harsha Sharma --- drivers/staging/unisys/visorbus/visorchipset.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys

[PATCH v3] staging: typec: tcpm: Rewrite comparison to NULL pointer

Make code more concise and readable Signed-off-by: Harsha Sharma --- Change in v3: -Change in subject and log message Change in v2: -Change in subject -Change in log message drivers/staging/typec/tcpm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH v2] staging: typec: tcpm: Comparison to NULL pointer

Makes code more concise and readable Signed-off-by: Harsha Sharma --- Change in v2: -Change in subject -Change in log message drivers/staging/typec/tcpm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/typec/tcpm.c b/drivers/staging/typec/tcpm.c index

[PATCH] staging: typec: tcpm: Comparison to NULL changed as "!port->partner_altmode[pmdata->altmodes]"

Fixes coding style issue Signed-off-by: Harsha Sharma --- drivers/staging/typec/tcpm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/typec/tcpm.c b/drivers/staging/typec/tcpm.c index 630fe75..9f4a5a5 100644 --- a/drivers/staging/typec/tcpm.c +++ b/drivers

[PATCH v6] staging: pi433: Fixes minor typo errors

Fixes checkpatch warning -- "occured" and "succesfully" are misspelled Signed-off-by: Harsha Sharma --- Changes in v6: -Added changes from other versions Changes in v5: -Signed-off properly with correct author name in .gitconfig Changes in v4: -Correct the format of the s

[PATCH v5] staging: pi433: Fixes minor typo errors

Fixes checkpatch warning -- "occured" and "succesfully" are misspelled Signed-off-by: Harsha Sharma --- Changes in v5: -Correct the format of the subject. -Signed-off properly with correct author name in .gitconfig drivers/staging/pi433/pi433_if.c | 4 ++-- 1 file changed