On 2019/5/31 17:54, Salil Mehta wrote:
>> From: netdev-ow...@vger.kernel.org On Behalf Of Yunsheng Lin
>> Sent: Friday, May 31, 2019 10:01 AM
>> To: da...@davemloft.net
>> Cc: hkallwe...@gmail.com; f.faine...@gmail.com;
>> step...@networkplumber.org; net...@vger.kernel.org; linux-
>>
On Mon, Jun 03, 2019 at 12:39:18AM +0300, Vladimir Oltean wrote:
> This is helpful for e.g. draining per-driver (not per-port) tagger
> queues.
>
> Signed-off-by: Vladimir Oltean
> ---
> Changes in v2:
>
> Patch is new.
>
> include/net/dsa.h | 1 +
> net/dsa/dsa2.c| 3 +++
> 2 files
From: Jonathan Marek
Add support for the Synaptics RMI4 touchscreen that is found on the
Nexus 5.
Signed-off-by: Jonathan Marek
Signed-off-by: Brian Masney
---
This is to be applied on top of the display patch series:
https://lore.kernel.org/lkml/20190531094619.31704-1-masn...@onstation.org/
Add support for USB networking as a module to qcom_defconfig since its
a useful feature to have for development purposes.
Signed-off-by: Brian Masney
---
This is to be applied on top of the display patch series:
https://lore.kernel.org/lkml/20190531094619.31704-1-masn...@onstation.org/
From: Stefano Garzarella
Date: Fri, 31 May 2019 15:39:51 +0200
> @@ -434,7 +434,9 @@ void virtio_transport_set_buffer_size(struct vsock_sock
> *vsk, u64 val)
> if (val > vvs->buf_size_max)
> vvs->buf_size_max = val;
> vvs->buf_size = val;
> +
Hi, Andrey
> -Original Message-
> From: Andrey Smirnov
> Sent: Saturday, June 1, 2019 3:04 AM
> To: Anson Huang
> Cc: Rob Herring ; Mark Rutland
> ; Shawn Guo ; Sascha
> Hauer ; Sascha Hauer ;
> Fabio Estevam ; manivannan.sadhasi...@linaro.org;
> bruno.thom...@gmail.com; Aisheng Dong ;
Hi Herbert,
Today's linux-next merge of the crypto tree got a conflict in:
crypto/crypto_wq.c
between commit:
2874c5fd2842 ("treewide: Replace GPLv2 boilerplate/reference with SPDX - rule
152")
from Linus' tree and commit:
3e56e168638b ("crypto: cryptd - move kcrypto_wq into cryptd")
From: Colin King
Date: Fri, 31 May 2019 14:27:38 +0100
> From: Colin Ian King
>
> The variable rc is assigned with a value that is never read and
> it is re-assigned a new value later on. The assignment is redundant
> and can be removed.
>
> Addresses-Coverity: ("Unused value")
>
From: Arnd Bergmann
Date: Fri, 31 May 2019 14:32:52 +0200
> [resending, rebased on top of today's net-next]
>
> The following changes since commit 7b3ed2a137b077bc0967352088b0adb6049eed20:
>
> Merge branch '100GbE' of
> git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/next-queue
>
From: Anson Huang
This patch fixes below build warning with "W=1":
arch/arm64/boot/dts/freescale/imx8mm.dtsi:203.6-754.4:
Warning (unit_address_vs_reg): /soc: node has a reg or
ranges property, but no unit name
arch/arm64/boot/dts/freescale/imx8mm.dtsi:209.23-388.5:
Warning
On Mon, Jun 3, 2019 at 2:15 AM Greg KH wrote:
>
> On Sun, Jun 02, 2019 at 09:03:46PM +0900, Masahiro Yamada wrote:
> > On Sun, Jun 2, 2019 at 4:17 PM Greg KH wrote:
> > >
> > > The following changes since commit
> > > 2f4c53349961c8ca480193e47da4d44fdb8335a8:
> > >
> > > Merge tag
On Sun, 2 Jun 2019, Geert Uytterhoeven wrote:
> Hi Finn,
>
> On Sun, Jun 2, 2019 at 3:29 AM Finn Thain
> wrote:
> > A system bus error during a PDMA transfer can mess up the calculation
> > of the transfer residual (the PDMA handshaking hardware lacks a byte
> > counter). This results in
On 5/29/19 11:46 AM, Krzysztof Kozlowski wrote:
On Mon, 27 May 2019 at 04:46, Matheus Castello wrote:
For configuration of fuel gauge alert for a low level state of charge
interrupt we add a function to config level threshold and a device tree
binding property to set it in flatned device
Hi all,
Tyring to fetch the mvebu tree
(git://git.infradead.org/linux-mvebu.git#for-next) for the past several
days produces this error message:
fatal: Couldn't find remote ref refs/heads/for-next
--
Cheers,
Stephen Rothwell
pgpMhaRmqo7UE.pgp
Description: OpenPGP digital signature
On Mon, 27 May 2019 at 04:45, Matheus Castello wrote:
For configure low level state of charge threshold alert signaled from
max17040 we add "maxim,alert-low-soc-level" property.
Signed-off-by: Matheus Castello
---
.../power/supply/max17040_battery.txt | 28 +++
1
On Mon, 3 Jun 2019 at 00:40, Vladimir Oltean wrote:
>
> Although meta frames are configured to be sent at SJA1105_META_DMAC
> (01-80-C2-00-00-0E) which is a multicast MAC address that would also be
> trapped by the switch to the CPU, were it to receive it on a front-panel
> port, meta frames are
Quoting Matthew Wilcox (2019-03-07 15:30:51)
> diff --git a/mm/huge_memory.c b/mm/huge_memory.c
> index 404acdcd0455..aaf88f85d492 100644
> --- a/mm/huge_memory.c
> +++ b/mm/huge_memory.c
> @@ -2456,6 +2456,9 @@ static void __split_huge_page(struct page *page, struct
> list_head *list,
>
All NVIDIA Tegra SoC generations provide IOMMU support for the video
decoder engine. Document new optional device-tree property that connects
VDE with the IOMMU provider.
Signed-off-by: Dmitry Osipenko
---
Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt | 2 ++
1 file changed, 2
Frequent IOMMU remappings take about 50% of CPU usage because there is
quite a lot to remap. Defer dmabuf's unmapping by 5 seconds in order to
mitigate the mapping overhead which goes away completely and driver works
as fast as in a case of a disabled IOMMU. The case of a disabled IOMMU
should
The video decoder driver was tested by time and works absolutely fine.
The reason why it is in staging is because it doesn't provide common V4L
interface yet, this shouldn't stop driver enabling in the defconfig since
our userspace (libvdpau-tegra) provides combined acceleration of decoding
and
The __packed macro isn't available in userspace with the kernel headers.
Checkpatch asks to use the macro, which is unwanted in a case of a UAPI
header. There is no much benefit in a tight packing of the structures,
hence let's pack them manually to cleanup things a tad. Note that there
is no
Hello,
This series cleans up some of minor shortcomings that are caused by
checkpatch recommendations that are not very applicable for the driver.
Then IOMMU support is added to the driver and now it can handle sparse
memory buffers that GPU hands to VDE in a default kernel configuration
on
The BIT macro isn't available in userspace. Checkpatch complains about
shifts being used instead of the macro and people are starting to send
patches without realizing that it's a UAPI header file. Hence let's
replace the BIT macro with a hex values to make everyone happy.
Signed-off-by: Dmitry
Enable IOMMU support for the video decoder.
Signed-off-by: Dmitry Osipenko
---
arch/arm/boot/dts/tegra30.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/tegra30.dtsi b/arch/arm/boot/dts/tegra30.dtsi
index 934caa83c8db..ce162125e7bf 100644
---
All Tegra's could provide memory isolation for the video decoder
hardware using IOMMU, it is also required for Tegra30+ in order
to handle sparse dmabuf's which GPU exports in a default kernel
configuration.
Inspired-by: Thierry Reding
Signed-off-by: Dmitry Osipenko
---
This patchset adds the following:
- A timecounter/cyclecounter based PHC for the free-running
timestamping clock of this switch.
- A state machine implemented in the DSA tagger for SJA1105, which
keeps track of metadata follow-up Ethernet frames (the switch's way
of transmitting RX
On RX, timestamping is done by programming the switch to send "meta"
follow-up Ethernet frames (which contain partial RX timestamps) after
each link-local frame that was trapped to the CPU port through MAC
filtering. This includes PTP frames.
Meta frame reception relies on the hardware keeping
This removes the existing implementation from tag_sja1105, which was
partially incorrect (it was not changing the MAC header offset, thereby
leaving it to point 4 bytes earlier than it should have).
This overwrites the VLAN tag by moving the Ethernet source and
destination MACs 4 bytes to the
This function will be reused from the .port_rxtstamp callback to see if
the received SKB can be timestamped by the switch.
Signed-off-by: Vladimir Oltean
---
Changes in v2:
Patch is new.
include/linux/dsa/sja1105.h | 15 +++
net/dsa/tag_sja1105.c | 15 ---
2
This is a cosmetic patch, pre-cursor to making another change to the
General Parameters Table (incl_srcpt) which does not logically pertain
to the sja1105_change_tpid function name, but not putting it there would
otherwise create a need of resetting the switch twice.
So simply move the existing
For drivers that use deferred_xmit for PTP frames (such as sja1105),
there is no need to perform matching between PTP frames and their egress
timestamps, since the sending process can be serialized.
In that case, it makes sense to have the pointer to the skb clone that
DSA made directly in the
Although meta frames are configured to be sent at SJA1105_META_DMAC
(01-80-C2-00-00-0E) which is a multicast MAC address that would also be
trapped by the switch to the CPU, were it to receive it on a front-panel
port, meta frames are conceptually not link-local frames, they only
carry their RX
Without noticing any particular issue, this patch ensures that
management traffic is treated with the maximum priority on RX by the
switch. This is generally desirable, as the driver keeps a state
machine that waits for metadata follow-up frames as soon as a management
frame is received.
The design of this PHC driver is influenced by the switch's behavior
w.r.t. timestamping. It exposes two PTP counters, one free-running
(PTPTSCLK) and the other offset- and frequency-corrected in hardware
through PTPCLKVAL, PTPCLKADD and PTPCLKRATE. The MACs can sample either
of these for frame
The incl_srcpt setting makes the switch mangle the destination MACs of
multicast frames trapped to the CPU - a primitive tagging mechanism that
works even when we cannot use the 802.1Q software features.
The downside is that the two multicast MAC addresses that the switch
traps for L2 PTP
This is helpful for e.g. draining per-driver (not per-port) tagger
queues.
Signed-off-by: Vladimir Oltean
---
Changes in v2:
Patch is new.
include/net/dsa.h | 1 +
net/dsa/dsa2.c| 3 +++
2 files changed, 4 insertions(+)
diff --git a/include/net/dsa.h b/include/net/dsa.h
index
Commit c62b96050bee ("media: ov6650: Register with asynchronous
subdevice framework") carelessly requested creation of a video device
node by setting a V4L2_SUBDEV_FL_HAS_DEVNODE flag. The driver is not
ready for that as it doesn't implement proper locking required for
serialization of IOCTLs.
Hmm. Fairly calm week, and rc3 is almost exactly the same size as rc2
was. Which is a bit unusual - usually rc2 is calm, and then rc3 is
when people have started finding problems and we get a more active
week.
But far be it for me to complain about a calm rc week, so I won't.
Nothing
On Sat, Jun 1, 2019 at 10:56 PM Herbert Xu wrote:
>
> You can't then go and decide to remove the compiler barrier! To do
> that you'd need to audit every single use of rcu_read_lock in the
> kernel to ensure that they're not depending on the compiler barrier.
What's the possible case where it
From: Yonglong Liu
Date: Fri, 31 May 2019 16:59:50 +0800
> When doing a loopback test at copper ports, the serdes loopback
> and the phy loopback will fail, because of the adjust link had
> not finished, and phy not ready.
>
> Adds sleep between adjust link and test process to fix it.
>
>
From: Nikita Yushchenko
Date: Fri, 31 May 2019 10:35:14 +0300
> When non-bridged, non-vlan'ed mv88e6xxx port is moving down, error
> message is logged:
>
> failed to kill vid 0081/0 for device eth_cu_1000_4
>
> This is caused by call from __vlan_vid_del() with vin set to zero, over
> call
From: Horatiu Vultur
Date: Fri, 31 May 2019 09:16:55 +0200
> This patch series enables hardware offload for flower filter used in
> traffic controller on MSCC Ocelot board.
>
> v2->v3 changes:
> - remove the check for shared blocks
>
> v1->v2 changes:
> - when declaring variables use reverse
Hi Janusz,
On Sun, Jun 02, 2019 at 11:58:23AM +0200, Janusz Krzysztofik wrote:
> Hi Sakari,
>
> On Sunday, June 2, 2019 12:37:55 AM CEST Sakari Ailus wrote:
> >
> > ... I realised that the subtle effect of "media:
> > ov6650: Register with asynchronous subdevice framework" is that the driver
>
On Sun, Jun 2, 2019 at 3:51 PM Linus Walleij wrote:
> On Wed, May 29, 2019 at 4:53 PM Thierry Reding
> wrote:
>
> > From: Thierry Reding
> >
> > Hierarchical IRQ domains can be used to stack different IRQ controllers
> > on top of each other. One specific use-case where this can be useful is
>
Hello,
syzbot found the following crash on:
HEAD commit:3ab4436f Merge tag 'nfsd-5.2-1' of git://linux-nfs.org/~bf..
git tree: upstream
console output: https://syzkaller.appspot.com/x/log.txt?x=153c64a6a0
kernel config: https://syzkaller.appspot.com/x/.config?x=50393f7bfe444ff6
On Sun, 2019-06-02 at 18:35 +0200, Michael Straube wrote:
> ETH_ALEN is defined in linux/if_ether.h which is included by
> osdep_service.h, so remove the redundant definition from ieee80211.h.
[]
> diff --git a/drivers/staging/rtl8188eu/include/ieee80211.h
>
From: Biao Huang
Date: Thu, 30 May 2019 17:43:15 +0800
> @@ -364,6 +371,15 @@ static int mediatek_dwmac_probe(struct platform_device
> *pdev)
> return 0;
> }
>
> +static int mediatek_dwmac_remove(struct platform_device *pdev)
> +{
> + int ret;
> +
> + ret =
On Tue, May 28, 2019 at 04:02:32PM -0700, Ruslan Babayev wrote:
> This allows drivers to lookup i2c adapters on ACPI based systems similar to
> of_get_i2c_adapter_by_node() with DT based systems.
>
> Signed-off-by: Ruslan Babayev
> Cc: xe-linux-exter...@cisco.com
As mentioned elsewhere, applied
On Sat, Jun 01, 2019 at 01:45:24PM +0300, Grygorii Strashko wrote:
> The Keystone 2 66AK2HK/E/L 1G Ethernet Switch Subsystems contains The
> Common Platform Time Sync (CPTS) module which is in general compatible with
> CPTS module found on TI AM3/4/5 SoCs. So, the basic support for
> Keystone 2
Quoting Matthew Wilcox (2019-06-02 11:51:50)
> On Sat, Jun 01, 2019 at 12:44:28PM +0100, Chris Wilson wrote:
> > Quoting Chris Wilson (2019-06-01 10:26:21)
> > > Quoting Matthew Wilcox (2019-03-07 15:30:51)
> > > > Transparent Huge Pages are currently stored in i_pages as pointers to
> > > >
The pull request you sent on Sun, 2 Jun 2019 19:44:42 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86-urgent-for-linus
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/7bd1d5edd0160b615ab8748cf94dabcab1fb01cb
Thank you!
--
Deet-doot-dot, I am a
The pull request you sent on Sun, 2 Jun 2019 19:39:53 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
> perf-urgent-for-linus
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/6751b8d91af515a5dc5196fe305eee0a635e2ea2
Thank you!
--
Deet-doot-dot, I
The pull request you sent on Sun, 2 Jun 2019 19:32:34 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
> core-urgent-for-linus
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/4fb5741c7c5defd88046f570694fc3249479f36f
Thank you!
--
Deet-doot-dot, I
On Sun, 2019-06-02 at 13:02 -0400, Valdis Klētnieks wrote:
> scripts/checkpatch.pl contains this code near line 3070:
>
>if ($realfile =~ /\.(h|s|S)$/) {
> $comment = '/*';
> } elsif ($realfile
Linus,
Please pull the latest x86-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
x86-urgent-for-linus
# HEAD: 2ac44ab608705948564791ce1d15d43ba81a1e38 x86/CPU/AMD: Don't force
the CPB cap when running under a hypervisor
Two fixes: a quirk for
Linus,
Please pull the latest perf-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
perf-urgent-for-linus
# HEAD: 849e96f30068d4f6f8352715e02a10533a46deba Merge tag
'perf-urgent-for-mingo-5.2-20190528' of
Linus,
Please pull the latest core-urgent-for-linus git tree from:
git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git
core-urgent-for-linus
# HEAD: 7eaf51a2e094229b75cc0c315f1cbbe2f3960058 stacktrace: Unbreak
stack_trace_save_tsk_reliable()
Fix a stack_trace_save_tsk_reliable()
The pull request you sent on Sat, 1 Jun 2019 16:22:06 -0700:
> git://git.kernel.org/pub/scm/linux/kernel/git/evalenti/linux-soc-thermal fixes
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/378e853f68e9a9548c64687880715ac3cca31c22
Thank you!
--
Deet-doot-dot, I am a
The pull request you sent on Sun, 2 Jun 2019 08:39:05 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git
> tags/spdx-5.2-rc3-2
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/a68dc6188242e1cc6f72eb3361e71633b4bc02a7
Thank you!
--
The pull request you sent on Sat, 1 Jun 2019 19:59:44 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/j.anaszewski/linux-leds.git
> tags/led-fixes-for-5.2-rc3
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/f58c356ea74feef8c2bb774dd19ded91567a5cf2
Thank you!
The pull request you sent on Sun, 2 Jun 2019 08:24:02 +0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-current
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/38baf0bb79f51b4fcbf6df8fd181441d7b5c7913
Thank you!
--
Deet-doot-dot, I am a
The pull request you sent on Sun, 2 Jun 2019 11:50:39 +0200:
> https://git.kernel.org/pub/scm/virt/kvm/kvm.git tags/for-linus
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/b44a1dd3f648a433c525efcdd6ba95ad89d50e27
Thank you!
--
Deet-doot-dot, I am a bot.
The pull request you sent on Sun, 02 Jun 2019 21:05:12 +1000:
> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git
> tags/powerpc-5.2-3
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/460b48a0fefce25beb0fc0139e721c5691d65d7f
Thank you!
--
On Sun, Jun 02, 2019 at 09:03:46PM +0900, Masahiro Yamada wrote:
> On Sun, Jun 2, 2019 at 4:17 PM Greg KH wrote:
> >
> > The following changes since commit 2f4c53349961c8ca480193e47da4d44fdb8335a8:
> >
> > Merge tag 'spdx-5.2-rc3-1' of
> >
On Sun, Jun 02, 2019 at 03:55:37PM +0530, Deepak Mishra wrote:
> This patch fixes CamelCase blnEnableRxFF0Filter by renaming it
> to enable_rx_ff0_filter in drv_types.h and related files rtl871x_cmd.c
> xmit_linux.c
> It was reported by checkpatch.pl
>
> This fix also makes enable_rx_ff0_filter a
On Sun, Jun 02, 2019 at 03:55:35PM +0530, Deepak Mishra wrote:
> This patch fixes CamelCase as reported by checkpatch.pl
> xmitThread renamed to xmit_thread
> recvThread renamed to recv_thread
>
> Signed-off-by: Deepak Mishra
> ---
> drivers/staging/rtl8712/drv_types.h | 4 ++--
> 1 file
On Sun, Jun 02, 2019 at 03:55:34PM +0530, Deepak Mishra wrote:
> This patch fixes CamelCase IsrContent to isr_content as suggested by
> checkpatch.pl
>
> Signed-off-by: Deepak Mishra
> ---
> drivers/staging/rtl8712/drv_types.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff
On Sun, Jun 02, 2019 at 03:55:33PM +0530, Deepak Mishra wrote:
> This patch fixes CamelCase renames evtThread to evt_thread in struct _adapter
> as reported by
> checkpatch.pl
> CHECK: Avoid CamelCase:
>
> Signed-off-by: Deepak Mishra
> ---
> drivers/staging/rtl8712/drv_types.h | 2 +-
> 1
On Sun, Jun 02, 2019 at 03:55:30PM +0530, Deepak Mishra wrote:
> This patch renames CamelCase ImrContent to imr_content in struct _adapter and
> related
> files drv_types.h, rtl871x_mp_ioctl.c, rtl871x_pwrctrl.h
>
> CHECK: Avoid CamelCase:
>
> Signed-off-by: Deepak Mishra
> ---
>
scripts/checkpatch.pl contains this code near line 3070:
if ($realfile =~ /\.(h|s|S)$/) {
$comment = '/*';
} elsif ($realfile =~ /\.(c|dts|dtsi)$/) {
cfs_rq_has_blocked() and others_have_blocked() are only used within
update_blocked_averages(). The !CONFIG_FAIR_GROUP_SCHED version of the
latter calls them within a #define CONFIG_NO_HZ_COMMON block, whereas
the CONFIG_FAIR_GROUP_SCHED one calls them unconditionnally.
As reported by Qian, the
The 'Exit' Debug message is superfluous ftrace can be used instead.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_core.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/pci/hotplug/ibmphp_core.c
b/drivers/pci/hotplug/ibmphp_core.c
index cd73bea12bc7..a2ea1ff6cfbc
ETH_ALEN is defined in linux/if_ether.h which is included by
osdep_service.h, so remove the redundant definition from ieee80211.h.
osdep_service.h:33:#include
etherdevice.h:25:#include
Signed-off-by: Michael Straube
---
drivers/staging/rtl8188eu/include/ieee80211.h | 1 -
1 file changed, 1
MGMT_QUEUE_NUM, ETH_TYPE_LEN and PAYLOAD_TYPE_LEN are defined but
not used in the driver code, so remove them.
Signed-off-by: Michael Straube
---
drivers/staging/rtl8188eu/include/ieee80211.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/ieee80211.h
On 30.05.2019 22:02, Nick Desaulniers wrote:
> On Mon, May 27, 2019 at 3:41 PM Stefan Agner wrote:
>>
>> OMAP2 depends on ARCH_MULTI_V6, which makes sure that the kernel is
>> compiled with -march=armv6. The compiler frontend will pass the
>> architecture to the assembler. There is no explicit
The 'Exit' Debug message is superfluous ftrace can be used instead.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_core.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/pci/hotplug/ibmphp_core.c
b/drivers/pci/hotplug/ibmphp_core.c
index cd73bea12bc7..a2ea1ff6cfbc
The pull request you sent on Sat, 01 Jun 2019 14:45:12 +0300:
> git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git scsi-fixes
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/1975b337ce26b53814f1b5c55b260649a7115393
Thank you!
--
Deet-doot-dot, I am a bot.
Fix checkpatch.pl 'line over 80 characters' Warning in ibmphp_ebda.c and
ibmphp_hpc.c
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_ebda.c | 7 +-
drivers/pci/hotplug/ibmphp_hpc.c | 107 +++---
2 files changed, 75 insertions(+), 39 deletions(-)
diff
On Sun, 2019-06-02 at 18:12 +0200, Emanuel Bennici wrote:
> Replace hardcoded function names with __func__ Macro so if the function
> name changes we have not to check all Messages and to retain the code
> structure.
trivia:
__func__ isn't a macro, it's a predefined identifier.
see the c90
On Sun, 2019-06-02 at 18:00 +0200, Emanuel Bennici wrote:
> Add a Whitespace between '-' and 'Exit' to keep the log messages consistent
[]
> diff --git a/drivers/pci/hotplug/ibmphp_core.c
> b/drivers/pci/hotplug/ibmphp_core.c
[]
> @@ -890,7 +890,7 @@ static int set_bus(struct slot *slot_cur)
>
Replace hardcoded function names with __func__ Macro so if the function
name changes we have not to check all Messages and to retain the code
structure.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/cpqphp_ctrl.c | 4 ++--
drivers/pci/hotplug/ibmphp_ebda.c | 4 ++--
Add a Whitespace between '-' and 'Exit' to keep the log messages consistent
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/hotplug/ibmphp_core.c
b/drivers/pci/hotplug/ibmphp_core.c
index
The kp_spi structure contains a member 'fifo_depth'. This member is
never used. Therefore, it should be removed.
Signed-off-by: Geordan Neukum
---
drivers/staging/kpc2000/kpc2000_spi.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/kpc2000/kpc2000_spi.c
The linux style guide prescribes that switch statements and their
subordinate case labels should be column-aligned rather than
double-indenting the case label. Make kpc2000_spi.c follow the desired
style with respect to switch/case alignment.
Signed-off-by: Geordan Neukum
---
The static function kp_spi_bytes_per_word() is defined in kpc2000_spi.c,
but it is completely unused. As this function is unused, it can and
should be removed.
Signed-off-by: Geordan Neukum
---
drivers/staging/kpc2000/kpc2000_spi.c | 14 --
1 file changed, 14 deletions(-)
diff
This patch set contains a few small fixups to the kpc_spi driver. There
is certainly nothing groundbreaking in this patch set. It is limited to
style fixups, removing unused things, and using the managed resource API
for mapping I/O space.
Geordan Neukum (5):
staging: kpc2000: kpc_spi: Remove
The kpc_spi driver does not unmap its I/O space upon error cases in the
probe() function or upon remove(). Make the driver clean up after itself
more maintainably by migrating to using the managed resource API.
Signed-off-by: Geordan Neukum
---
drivers/staging/kpc2000/kpc2000_spi.c | 3 ++-
1
The kpc2000_spi.c file contains instances of unnecessary consecutive
newlines which negatively impact the readability of the file. Remove
all unnecessary consecutive newlines.
Signed-off-by: Geordan Neukum
---
drivers/staging/kpc2000/kpc2000_spi.c | 13 -
1 file changed, 13
Replace the hardcoded Function-Names in error and debug Messages with
the __func__ Macro.
If the Function-Name changes we haven't to check all the error/ debug
messages.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_core.c | 34 +++
1 file changed, 16
'return' is not correctly used here.
This Patch replaces 'return(n)' with 'return n'.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/cpqphp_nvram.c | 48 +++---
drivers/pci/hotplug/cpqphp_pci.c | 24 +++
drivers/pci/hotplug/shpchp_ctrl.c | 2 +-
3
Replace 'return(1)' with 'return 1' because return is not a function.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/shpchp_ctrl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/hotplug/shpchp_ctrl.c
b/drivers/pci/hotplug/shpchp_ctrl.c
index
On Sun, 2019-06-02 at 08:39 +0200, Greg KH wrote:
> The second patch fixes up a number of places in the tree where people
> mistyped the string "SPDX-License-Identifier". Given that people can
> not even type their own name all the time without mistakes, this was
> bound to happen, and odds are,
Remove function hpc_get_mode1_ECC_cap since it was used by the
get_mode1_ECC_cap Callback - that was removed.
So this function is unused and can be safely removed.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/shpchp_hpc.c | 17 -
1 file changed, 17 deletions(-)
diff
This callback is never invoked/ used in this driver.
Removing this callback does not affect the driver.
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/shpchp_hpc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/pci/hotplug/shpchp_hpc.c b/drivers/pci/hotplug/shpchp_hpc.c
McBSP used to work correctly as long as compat DMA probing, removed by
commit 642aafea8889 ("ASoC: ti: remove compat dma probing"), was
available. New method of DMA probing apparently requires users to
provide channel names when registering with SDMA, while McBSP passes
NULLs. Fix it.
The same
On Fri, 2019-05-31 at 22:09 +0800, Ryder Lee wrote:
> Enable NL80211_IFTYPE_MESH_POINT and update its path.
>
> Signed-off-by: Ryder Lee
> ---
> Changes since v2 - remove unused definitions
> ---
> drivers/net/wireless/mediatek/mt76/mt7615/init.c | 6 ++
>
Check the return value of pci_hp_register() in Function
ebda_rsrc_controller()
Signed-off-by: Emanuel Bennici
---
drivers/pci/hotplug/ibmphp_ebda.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/pci/hotplug/ibmphp_ebda.c
When user space application request for change in spi clock
using ioctl, current value is taken back-up and new value is
assigned to spi->max_speed_hz, then spi_setup() function is
called with new value. If spi_setup() function fails, it needs
reverting to old spi_max_speed value only in failure
On Sat, Jun 1, 2019 at 12:05 PM Uwe Kleine-König
wrote:
>
> I didn't look into the driver to try to understand that, but the
> definitely needs a comment to explain for the next person to think they
> can do a cleanup here.
Certainly.
But if we do restore the old behaviour, there may still be
I think I´ve said most things about what I am doing to LKML now.
A correctly experienced will know much about his system and the design will not
go much from original idea.
Which is what we see with Unix. I think this was the point of its original
developers aswell, and a commentary on
101 - 200 of 259 matches
Mail list logo
