struct pd_message is the format of a PD message as seen on the wire.
Signed-off-by: Guenter Roeck
---
v2: No change
v3: Fix document tag
drivers/staging/typec/pd.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/staging/typec/pd.h b/drivers/staging/typec/pd.h
index 30b32ad72acd
Commented out code can be added as needed. Drop it.
Also drop TODO and an obsolete XXX comment.
Signed-off-by: Guenter Roeck
---
v2, v3: No change
drivers/staging/typec/tcpm.c | 37 +
1 file changed, 1 insertion(+), 36 deletions(-)
diff --git a/drivers/stagi
The driver is in good enough shape to be moved out of staging.
Do it.
Signed-off-by: Guenter Roeck
---
v2: Use format-patch -M
v3: No change
drivers/staging/typec/Kconfig| 2 --
drivers/staging/typec/Makefile | 1 -
drivers/staging/typec/fusb302/T
Move tcpm (USB Type-C Port Manager) out of staging.
Signed-off-by: Guenter Roeck
---
v2: Use format-patch -M
v3: No change
drivers/staging/typec/Kconfig | 8
drivers/staging/typec/Makefile| 1 -
drivers/staging/typec/TODO
Document struct tcpc_config and struct tcpc_dev.
Drop unused TCPC_USB_SWITCH_RESTORE.
Signed-off-by: Guenter Roeck
---
v2, v3: No change
drivers/staging/typec/tcpm.h | 57 ++--
1 file changed, 50 insertions(+), 7 deletions(-)
diff --git a/drivers/staging
Fix checkpath-reported unbalanced braces in the following areas
221: FILE: drivers/staging/rtl8712/hal_init.c:221:
392: FILE: drivers/staging/rtl8712/os_intfs.c:392:
363: FILE: drivers/staging/rtl8712/rtl8712_cmd.c:363:
889: FILE: drivers/staging/rtl8712/rtl8712_recv.c:889:
902: FILE: drivers/stag
Hi Harsha,
[auto build test WARNING on v4.13]
[also build test WARNING on next-20170911]
[cannot apply to staging/staging-testing]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Harsha-Sharma
On Mon, 11 Sep 2017 20:49:27 +0200
Vincent Hervieux wrote:
> Currently atomisp module supports Intel's Baytrail SoC and contains
> some compilation switches to support Intel's Cherrytrail SoC instead.
> The patchset aims to :
> - 1/2: activate ATOMISP2400 or ATOMISP2401 from the menu.
> - 2/2: fi
The linkstatus variable in the info struct received as __le16
but handled in every other place in the driver as u16
Fix that and remove the sparse warning that occurred due to it:
prism2sta.c:1450:29: warning: incorrect type in assignment (different base
types)
prism2sta.c:1450:29:expected un
Fixed IIO_DEVICE_ATTR_NAMED API to take name as a
string instead of implicit conversion to string using
preprocessors. Added double quotes around names in
existing usage of IIO_DEVICE_ATTR_NAMED.
Signed-off-by: Himanshi Jain
---
drivers/iio/adc/ad7793.c | 2 +-
drivers/staging/iio/adc/a
From: Stephen Hemminger
Date: Wed, 6 Sep 2017 13:53:05 -0700
> When a virtual device is added dynamically (via host console), then
> the vmbus sends an offer message for the primary channel. The processing
> of this message for networking causes the network device to then
> initialize the sub ch
From: Stephen Hemminger
Date: Wed, 6 Sep 2017 13:53:06 -0700
> Only need to wakeup the initiator after all sub-channels
> are opened.
>
> Signed-off-by: Stephen Hemminger
Applied.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.
No changelog. No Signed-off-by line. Without reading too carefully, or
trying to do a build, it looks like we're activating the menu items and
then fixing the build. It should be the other way around so that we
don't break git bisect. People are always doing randconfig and the
autobuilders dete
We always need a changelog. And actually this seems a bit involved so
there is stuff to explain.
On Mon, Sep 11, 2017 at 08:51:15PM +0200, Vincent Hervieux wrote:
> Signed-off-by: Vincent Hervieux
> ---
> .../media/atomisp/pci/atomisp2/atomisp_cmd.c | 5 ++---
> .../media/atomisp/pci/ato
On Mon, Sep 11, 2017 at 11:26:01AM -0700, Greg Kroah-Hartman wrote:
> On Mon, Sep 11, 2017 at 10:37:11AM -0700, Guenter Roeck wrote:
> > On Mon, Sep 11, 2017 at 09:23:34AM -0700, Greg Kroah-Hartman wrote:
> > > On Sun, Sep 10, 2017 at 01:37:01PM -0700, Guenter Roeck wrote:
> > > > Commented out cod
Signed-off-by: Vincent Hervieux
---
.../media/atomisp/pci/atomisp2/atomisp_cmd.c | 5 ++---
.../media/atomisp/pci/atomisp2/atomisp_v4l2.c | 6 +-
.../pci/atomisp2/css2400/ia_css_acc_types.h| 1 +
.../css2400/runtime/debug/src/ia_css_debug.c | 3 ---
.../media/atom
---
drivers/staging/media/atomisp/pci/Kconfig | 23 ++
.../staging/media/atomisp/pci/atomisp2/Makefile| 10 +-
2 files changed, 32 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/atomisp/pci/Kconfig
b/drivers/staging/media/atomisp/pci/Kcon
Currently atomisp module supports Intel's Baytrail SoC and contains
some compilation switches to support Intel's Cherrytrail SoC instead.
The patchset aims to :
- 1/2: activate ATOMISP2400 or ATOMISP2401 from the menu.
- 2/2: fix compilation errors for ATOMISP2401.
I'm not so confident with patch 2
From: Armin Schoenlieb
This is a patch to the irqueue.h file that fixes up pointer style errors and
trailing whitespace errors, found by the checkpatch.pl tool
Signed-off-by: Armin Schoenlieb
---
drivers/staging/irda/include/net/irda/irqueue.h | 46 -
1 file changed, 23
On Mon, Sep 11, 2017 at 10:37:11AM -0700, Guenter Roeck wrote:
> On Mon, Sep 11, 2017 at 09:23:34AM -0700, Greg Kroah-Hartman wrote:
> > On Sun, Sep 10, 2017 at 01:37:01PM -0700, Guenter Roeck wrote:
> > > Commented out code can be added as needed. Drop it.
> > > Also drop TODO and an obsolete XXX
On Mon, Sep 11, 2017 at 7:36 PM, Sean Paul wrote:
> On Mon, Sep 11, 2017 at 12:28 PM, Srishti Sharma
> wrote:
>> Remove the variable monitor_lock as it is not used anywhere.
>>
>> Signed-off-by: Srishti Sharma
>
> Reviewed-by: Sean Paul
That variable is a left-over I've missed when I deleted
On Mon, Sep 11, 2017 at 05:55:29PM +0300, Sakari Ailus wrote:
> Hi Devid,
>
> Please see my comments below.
>
> Andy: please look for "INT5648".
Hi Sakari,
I'm replying below to your comments. I'll work on a v2 patch as soon as we get
more comments.
About "INT5648", I extracted it from the DSDT
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/s
On Mon, Sep 11, 2017 at 09:23:34AM -0700, Greg Kroah-Hartman wrote:
> On Sun, Sep 10, 2017 at 01:37:01PM -0700, Guenter Roeck wrote:
> > Commented out code can be added as needed. Drop it.
> > Also drop TODO and an obsolete XXX comment.
> >
> > Signed-off-by: Guenter Roeck
> > ---
> > drivers/st
On Mon, Sep 11, 2017 at 10:25:14PM +0530, Amit Pundir wrote:
> Hi Todd,
>
> On 11 September 2017 at 21:10, Todd Kjos wrote:
> > (resend in plain-text mode -- sorry about that)
> >
> > Amit,
> >
> > Are you sure this patch is the culprit? That is pretty surprising
> > since this change can only be
On Mon, Sep 11, 2017 at 9:54 PM, Greg KH wrote:
> On Thu, Sep 07, 2017 at 07:44:52PM +0530, Srishti Sharma wrote:
>> Use BIT(x) instead of using (1<>
>> Signed-off-by: Srishti Sharma
>> ---
>> Changes in v2:
>> - Add tab spaces before BIT macro.
>>
>> drivers/staging/ccree/ssi_cipher.h | 10 +++
Hi Todd,
On 11 September 2017 at 21:10, Todd Kjos wrote:
> (resend in plain-text mode -- sorry about that)
>
> Amit,
>
> Are you sure this patch is the culprit? That is pretty surprising
> since this change can only be hit in a uncommon case (the target node
> is valid when we start creating the
On Sun, Sep 10, 2017 at 11:04:34PM +0530, Harsha Sharma wrote:
> 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(-)
This and patch 8/10, ha
On Sun, Sep 10, 2017 at 11:02:02PM +0530, Harsha Sharma wrote:
> Space required after ','
>
> Signed-off-by: Harsha Sharma
> Acked-by: Julia Lawall
This patch does not apply, can you rebase and resend?
thanks,
greg k-h
___
devel mailing list
de...@l
On Sat, Sep 09, 2017 at 10:43:36PM +0530, Harsha Sharma wrote:
> 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 messa
On Mon, Sep 11, 2017 at 12:28 PM, Srishti Sharma wrote:
> Remove the variable monitor_lock as it is not used anywhere.
>
> Signed-off-by: Srishti Sharma
Reviewed-by: Sean Paul
> ---
> Changes in v2:
> -The variable that was not to be declared as volatile can be
> eliminated as it is not bei
On Sat, Sep 09, 2017 at 01:14:53PM -0700, Joe Perches wrote:
> Use the normal kernel macro.
> Remove the now ununsed LIST_CONTAINOR macro.
>
> Signed-off-by: Joe Perches
Doesn't apply to my tree :(
___
devel mailing list
de...@linuxdriverproject.org
ht
On Mon, Sep 11, 2017 at 9:45 PM, Srishti Sharma wrote:
> On Mon, Sep 11, 2017 at 9:41 PM, Julia Lawall wrote:
>>
>>
>> On Mon, 11 Sep 2017, Srishti Sharma wrote:
>>
>>> On Mon, Sep 11, 2017 at 9:34 PM, Greg KH wrote:
>>> > On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
>>> >> Th
Remove the variable monitor_lock as it is not used anywhere.
Signed-off-by: Srishti Sharma
---
Changes in v2:
-The variable that was not to be declared as volatile can be
eliminated as it is not being used anywhere.
drivers/staging/ccree/ssi_request_mgr.c | 1 -
1 file changed, 1 deletion(-)
On Sat, Sep 09, 2017 at 11:18:02PM -0700, Joe Perches wrote:
> Logging without newlines are still prone to interleaving.
> Add newlines where necessary.
Doesn't apply to my staging-testing branch, due to other changes in this
driver. Can you rebase it onto that branch and resend?
thanks,
greg k
On Thu, Sep 07, 2017 at 07:44:52PM +0530, Srishti Sharma wrote:
> Use BIT(x) instead of using (1<
> Signed-off-by: Srishti Sharma
> ---
> Changes in v2:
> - Add tab spaces before BIT macro.
>
> drivers/staging/ccree/ssi_cipher.h | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-
On Sun, Sep 10, 2017 at 01:37:01PM -0700, Guenter Roeck wrote:
> Commented out code can be added as needed. Drop it.
> Also drop TODO and an obsolete XXX comment.
>
> Signed-off-by: Guenter Roeck
> ---
> drivers/staging/typec/tcpm.c | 37 +
> 1 file changed, 1
On Mon, Sep 11, 2017 at 9:41 PM, Julia Lawall wrote:
>
>
> On Mon, 11 Sep 2017, Srishti Sharma wrote:
>
>> On Mon, Sep 11, 2017 at 9:34 PM, Greg KH wrote:
>> > On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
>> >> The use of volatile for the variable monitor_lock is unnecessary.
>
On Mon, 11 Sep 2017, Srishti Sharma wrote:
> On Mon, Sep 11, 2017 at 9:34 PM, Greg KH wrote:
> > On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
> >> The use of volatile for the variable monitor_lock is unnecessary.
> >>
> >> Signed-off-by: Srishti Sharma
> >> ---
> >> drivers
On Mon, Sep 11, 2017 at 12:08 PM, Srishti Sharma wrote:
> On Mon, Sep 11, 2017 at 9:34 PM, Greg KH wrote:
>> On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
>>> The use of volatile for the variable monitor_lock is unnecessary.
>>>
>>> Signed-off-by: Srishti Sharma
>>> ---
>>> dr
On Mon, Sep 11, 2017 at 9:34 PM, Greg KH wrote:
> On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
>> The use of volatile for the variable monitor_lock is unnecessary.
>>
>> Signed-off-by: Srishti Sharma
>> ---
>> drivers/staging/ccree/ssi_request_mgr.c | 2 +-
>> 1 file changed,
On Mon, Sep 11, 2017 at 09:29:31PM +0530, Srishti Sharma wrote:
> The use of volatile for the variable monitor_lock is unnecessary.
>
> Signed-off-by: Srishti Sharma
> ---
> drivers/staging/ccree/ssi_request_mgr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/
On Mon, 11 Sep 2017, Srishti Sharma wrote:
> The use of volatile for the variable monitor_lock is unnecessary.
You need to give more evidence of why this is the case. How is the
variable used? I guess this comes from checkpatch, but checkpatch has
only a local view of things, and doesn't know
The use of volatile for the variable monitor_lock is unnecessary.
Signed-off-by: Srishti Sharma
---
drivers/staging/ccree/ssi_request_mgr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ccree/ssi_request_mgr.c
b/drivers/staging/ccree/ssi_request_mgr.c
index
(resend in plain-text mode -- sorry about that)
Amit,
Are you sure this patch is the culprit? That is pretty surprising
since this change can only be hit in a uncommon case (the target node
is valid when we start creating the transaction, but dead when we
check right before sending it) so it is u
Commit 7dcf90e9e032 ("PCI: hv: Use vPCI protocol version 1.2") and
commit 628f54cc6451 ("x86/hyper-v: Support extended CPU ranges for TLB
flush hypercalls") adding the same definition came through different
trees. Fix the glitch.
Signed-off-by: Vitaly Kuznetsov
---
arch/x86/include/uapi/asm/hype
Hi Branislav,
On Thu, Sep 07, 2017 at 06:26:42PM +0200, Branislav Radocaj wrote:
> Remove casting the values returned by memory allocation functions
> like kmalloc, kzalloc, kmem_cache_alloc, kmem_cache_zalloc etc.
>
> Semantic patch information:
> This makes an effort to find cases of casting of
Hi Devid,
Please see my comments below.
Andy: please look for "INT5648".
On Sun, Sep 10, 2017 at 02:23:07PM +0200, Devid Antonio Floni wrote:
> The ov5680 5-megapixel camera sensor from OmniVision supports up to 2592x1944
> resolution and MIPI CSI-2 interface. Output format is raw sRGB/Bayer wit
On Mon, 11 Sep 2017, Srishti Sharma wrote:
> Remove extra blank line.
>
> Signed-off-by: Srishti Sharma
Acked-by: Julia Lawall
> ---
> drivers/staging/rtl8712/rtl8712_event.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8712/rtl8712_event.h
> b/drivers/staging
Remove extra blank line.
Signed-off-by: Srishti Sharma
---
drivers/staging/rtl8712/rtl8712_event.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/rtl8712/rtl8712_event.h
b/drivers/staging/rtl8712/rtl8712_event.h
index b383740..cad7085 100644
--- a/drivers/staging/rtl8712/rtl
> -Original Message-
> From: Greg KH [mailto:gre...@linuxfoundation.org]
> Sent: Monday, September 11, 2017 4:03 PM
>
> On Mon, Sep 11, 2017 at 11:55:22AM +, Laurentiu Tudor wrote:
> > Hi,
> >
> > > -Original Message-
> > > From: Greg KH [mailto:gre...@linuxfoundation.org]
>
On Mon, Sep 11, 2017 at 11:55:22AM +, Laurentiu Tudor wrote:
> Hi,
>
> > -Original Message-
> > From: Greg KH [mailto:gre...@linuxfoundation.org]
> > Sent: Thursday, August 31, 2017 7:05 PM
> >
> > On Mon, Aug 28, 2017 at 01:54:05PM +0300, laurentiu.tu...@nxp.com wrote:
> > > From: St
On 5 September 2017 at 22:51, Todd Kjos wrote:
> From: Xu YiPing
>
> commit 7a4408c6bd3e ("binder: make sure accesses to proc/thread are
> safe") made a change to enqueue tcomplete to thread->todo before
> enqueuing the transaction. However, in err_dead_proc_or_thread case,
> the tcomplete is dir
Hi,
> -Original Message-
> From: Greg KH [mailto:gre...@linuxfoundation.org]
> Sent: Thursday, August 31, 2017 7:05 PM
>
> On Mon, Aug 28, 2017 at 01:54:05PM +0300, laurentiu.tu...@nxp.com wrote:
> > From: Stuart Yoder
> >
> > Move the source files out of staging into their final locatio
Hello Yueyao Zhu,
The patch c034a43e72dd: "staging: typec: Fairchild FUSB302 Type-c
chip driver" from Apr 27, 2017, leads to the following static checker
warning:
drivers/staging/typec/fusb302/fusb302.c:180 _fusb302_log()
error: testing array offset 'chip->logbuffer_head' after us
On Sun, 10 Sep 2017, Harsha Sharma wrote:
> Space required after ','
>
> Signed-off-by: Harsha Sharma
Acked-by: Julia Lawall
> ---
> drivers/staging/rtl8723bs/os_dep/os_intfs.c | 12 ++--
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/o
On Sun, 10 Sep 2017, Harsha Sharma wrote:
> Use spaces between concatenated strings
Perhaps all the entry/exit ones could be dropped, and the ohters could be
rewritten to use more standard kernel debugging functionalities.
julia
>
> Signed-off-by: Harsha Sharma
> ---
> drivers/staging/rtl87
Hello.
On 9/10/2017 11:37 PM, Guenter Roeck wrote:
struct pd_message is the format of a PD message as seen on the wire.
Signed-off-by: Guenter Roeck
---
drivers/staging/typec/pd.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/staging/typec/pd.h b/drivers/staging/typec/pd
Hi Harsha,
[auto build test ERROR on v4.13]
[also build test ERROR on next-20170911]
[cannot apply to staging/staging-testing]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Harsha-Sharma
59 matches
Mail list logo