[PATCH] staging: rtl8723bs: fix pointer declaration style

2021-02-04 Thread Ayush
Fix some pointer declarations where '*' is not adjacent to data name. This fixes checkpatch.pl error: "POINTER_LOCATION: "foo * bar" should be "foo *bar"" Signed-off-by: Ayush --- compile tested only. drivers/staging/rtl8723bs/include/hal_intf.h | 8 1 file changed, 4 insertions(+), 4

Re: [PATCH] staging: qlge/qlge_main: Use min_t instead of min

2021-02-04 Thread Nathan Chancellor
On Fri, Feb 05, 2021 at 03:24:51AM +0530, ameynarkhed...@gmail.com wrote: > From: Amey Narkhede > > Use min_t instead of min function in qlge/qlge_main.c > Fixes following checkpatch.pl warning: > WARNING: min() should probably be min_t(int, MAX_CPUS, num_online_cpus()) > > Signed-off-by: Amey N

Re: [PATCH] drm/bridge: anx7625: enable DSI EOTP

2021-02-04 Thread Nicolas Boichat
On Thu, Feb 4, 2021 at 8:59 PM Andrzej Hajda wrote: > > > W dniu 04.02.2021 o 13:34, Nicolas Boichat pisze: > > On Thu, Feb 4, 2021 at 8:07 PM Robert Foss wrote: > >> Hi Xin, > >> > >> Thanks for the patch. > >> > >> On Thu, 28 Jan 2021 at 12:17, Xin Ji wrote: > >>> Enable DSI EOTP feature for f

[PATCH] staging: qlge/qlge_main: Use min_t instead of min

2021-02-04 Thread ameynarkhede02
From: Amey Narkhede Use min_t instead of min function in qlge/qlge_main.c Fixes following checkpatch.pl warning: WARNING: min() should probably be min_t(int, MAX_CPUS, num_online_cpus()) Signed-off-by: Amey Narkhede --- drivers/staging/qlge/qlge_main.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH] staging: rtl8723bs: remove unused structures

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann Building this with 'make W=1' produces a couple of warnings: rtl8723bs/include/ieee80211.h:730:1: warning: alignment 1 of 'struct ieee80211_assoc_request_frame' is less than 2 [-Wpacked-not-aligned] rtl8723bs/include/ieee80211.h:737:1: warning: alignment 1 of 'struct ieee80

[PATCH] staging: vt665x: fix alignment constraints

2021-02-04 Thread Arnd Bergmann
From: Arnd Bergmann multiple structures contains a ieee80211_rts structure, which is required to have at least two byte alignment, but are annotated with a __packed attribute to force single-byte alignment: staging/vt6656/rxtx.h:98:1: warning: alignment 1 of 'struct vnt_rts_g' is less than 2 [-

Re: [PATCH] staging: media: atomisp: fix coding style issues in timer.c

2021-02-04 Thread Greg KH
On Sun, Jan 31, 2021 at 11:55:29PM +0530, Ayush wrote: > - Fix unneeded brace in if condition(also, brace was on next line). > - Fix leading space warning before struct ia_css_clock_tick *curr_ts. > > compile tested only (on next-20210129) > > Signed-off-by: Ayush > --- > .../staging/media/atom

[PATCH RFC] staging: comedi: fix coding style issues

2021-02-04 Thread Ayush
fix IF_0 and IF_1 warnings by checkpatch.pl Signed-off-by: Ayush --- compile tested only. drivers/staging/comedi/drivers/dt2801.c | 29 - 1 file changed, 29 deletions(-) diff --git a/drivers/staging/comedi/drivers/dt2801.c b/drivers/staging/comedi/drivers/dt2801.c inde

Re: [PATCH] staging: rtl8192e, rtl8192u: use correct notation to define pointer

2021-02-04 Thread Muhammad Usama Anjum
> Sparse's warning is not about changing the definition of this member > as if it was the argument of a function. It's about how can you use > an array of structure when this structure has a flexible member. We have the following structures in drivers/staging/rtl8192e. (I've simplified them for sh

Re: [PATCH v3 3/7] drivers/acpi: convert seqno to use seqnum_ops

2021-02-04 Thread Rafael J. Wysocki
Hi Shuah, First off, please indicate the component in the subject, for example: "ACPI: extlog: convert seqno to use seqnum_ops" On Wed, Feb 3, 2021 at 7:12 PM Shuah Khan wrote: > > Sequence Number api provides interfaces for unsigned atomic up counters > leveraging atomic_t and atomic64_t ops u

Re: [PATCH] drm/bridge: anx7625: enable DSI EOTP

2021-02-04 Thread Andrzej Hajda
W dniu 04.02.2021 o 13:34, Nicolas Boichat pisze: > On Thu, Feb 4, 2021 at 8:07 PM Robert Foss wrote: >> Hi Xin, >> >> Thanks for the patch. >> >> On Thu, 28 Jan 2021 at 12:17, Xin Ji wrote: >>> Enable DSI EOTP feature for fixing some panel screen constance >>> shift issue. >>> Removing MIPI fla

Re: [PATCH] drm/bridge: anx7625: enable DSI EOTP

2021-02-04 Thread Nicolas Boichat
On Thu, Feb 4, 2021 at 8:07 PM Robert Foss wrote: > > Hi Xin, > > Thanks for the patch. > > On Thu, 28 Jan 2021 at 12:17, Xin Ji wrote: > > > > Enable DSI EOTP feature for fixing some panel screen constance > > shift issue. > > Removing MIPI flag MIPI_DSI_MODE_EOT_PACKET to enable DSI EOTP. > > I

Re: [PATCH RFC] staging: comedi: fix coding style issues

2021-02-04 Thread Greg KH
On Thu, Feb 04, 2021 at 05:43:37PM +0530, Ayush wrote: > fix IF_0 and IF_1 warnings by checkpatch.pl What is that? > Signed-off-by: Ayush > --- > compile tested only. When you send something with "RFC" that means you do not think it is good enough to be merged, and so I will just delete it. Al

Re: [PATCH v4 2/3] drm/bridge: anx7625: fix not correct return value

2021-02-04 Thread Robert Foss
Hi Xin, On Thu, 28 Jan 2021 at 04:12, Xin Ji wrote: > > At some time, the original code may return non zero value, force return 0 > if operation finished Missing "." at end of line. Other than that, this patch looks fine. Feel free to add my r-b. Reviewed-by: Robert Foss > > Signed-off-by: Xi

[PATCH v14] staging: fbtft: add tearing signal detect

2021-02-04 Thread Carlis
From: Carlis For st7789v IC, when we need continuous full screen refresh, it is best to wait for the tearing effect line signal to arrive to avoid screen tearing. Signed-off-by: Carlis --- v14: change to define TE completion and TE irq only in st7789v. v13: change TE completion to par data stru

Re: [PATCH] drm/bridge: anx7625: enable DSI EOTP

2021-02-04 Thread Robert Foss
Hi Xin, Thanks for the patch. On Thu, 28 Jan 2021 at 12:17, Xin Ji wrote: > > Enable DSI EOTP feature for fixing some panel screen constance > shift issue. > Removing MIPI flag MIPI_DSI_MODE_EOT_PACKET to enable DSI EOTP. I don't think I quite understand how removing the MIPI_DSI_MODE_EOT_PACKE

Re: [PATCH v3 1/7] seqnum_ops: Introduce Sequence Number Ops

2021-02-04 Thread Peter Zijlstra
On Wed, Feb 03, 2021 at 11:11:57AM -0700, Shuah Khan wrote: > +static inline u32 seqnum32_inc(struct seqnum32 *seq) > +{ > + atomic_t val = ATOMIC_INIT(seq->seqnum); > + > + seq->seqnum = (u32) atomic_inc_return(&val); > + if (seq->seqnum >= UINT_MAX) > + pr_info("Sequence N

[PATCH] staging: rtl8188eu: Add Edimax EW-7811UN V2 to device table

2021-02-04 Thread Martin Kaiser
The Edimax EW-7811UN V2 uses an RTL8188EU chipset and works with this driver. Signed-off-by: Martin Kaiser --- drivers/staging/rtl8188eu/os_dep/usb_intf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/rtl8188eu/os_dep/usb_intf.c b/drivers/staging/rtl8188eu/os_dep/usb_intf.

Re: [PATCH v7 10/14] phy: phy-hi3670-usb3: move driver from staging into phy

2021-02-04 Thread Vinod Koul
On 29-01-21, 16:03, Mauro Carvalho Chehab wrote: > The phy USB3 driver for Hisilicon 970 (hi3670) is ready > for mainstream. Mode it from staging into the main driver's > phy/ directory. Acked-By: Vinod Koul -- ~Vinod ___ devel mailing list de...@linu