Hi all,
For sched_switch tracepoint, I guess maybe it would be more reasonable
to show guest samples together with host samples by default, instead of
setting an option to display guest samples when we meet with this
confused situation.
To take care of this issue, the code should be changed as
On Fri, 2017-10-20 at 12:21 -0500, Gustavo A. R. Silva wrote:
> In preparation to enabling -Wimplicit-fallthrough, mark switch cases
> where we are expecting to fall through.
>
> Signed-off-by: Gustavo A. R. Silva
> ---
> This code was tested by compilation only (GCC 7.2.0 was used).
> Please, ve
On 23.10.2017 09:39, Nikolay Borisov wrote:
>
>
> On 19.10.2017 22:11, Christian Brauner wrote:
>> - Add a struct containing two pointer to extents and wrap both the static
>> extent
>> array and the struct into a union. This is done in preparation for bumping
>> the
>> {g,u}idmap limits
> On 10/20/2017 12:37 AM, Elena Reshetova wrote:
> > atomic_t variables are currently used to implement reference
> > counters with the following properties:
> > - counter is initialized to 1 using atomic_set()
> > - a resource is freed upon counter reaching zero
> > - once counter reaches zero,
On Tue, Oct 10, 2017 at 01:23:01PM +0800, Baolin Wang wrote:
> +++ b/drivers/dma/sprd-dma.c
> @@ -0,0 +1,988 @@
> +/*
> + * Copyright (C) 2017 Spreadtrum Communications Inc.
> + *
> + * SPDX-License-Identifier: (GPL-2.0+ OR MIT)
MIT? See comment for MODULE_LICENSE
> +/* SPRD_DMA_CHN_CFG register
On 19.10.2017 22:11, Christian Brauner wrote:
> - Add a struct containing two pointer to extents and wrap both the static
> extent
> array and the struct into a union. This is done in preparation for bumping
> the
> {g,u}idmap limits for user namespaces.
> - Add brackets around anonymous un
On Mon, Oct 23, 2017 at 08:53:35AM +0900, Byungchul Park wrote:
> On Fri, Oct 20, 2017 at 07:44:51AM -0700, Christoph Hellwig wrote:
> > The Subject prefix for this should be "block:".
> >
> > > @@ -945,7 +945,7 @@ int submit_bio_wait(struct bio *bio)
> > > {
> > > struct submit_bio_ret ret;
>
> On Fri, 2017-10-20 at 10:37 +, Reshetova, Elena wrote:
> > > On Fri, 2017-10-20 at 10:23 +0300, Elena Reshetova wrote:
> > > > atomic_t variables are currently used to implement reference
> > > > counters with the following properties:
> > > > - counter is initialized to 1 using atomic_set()
> From: Elena Reshetova
> Date: Fri, 20 Oct 2017 10:23:34 +0300
>
> > Note: these are the last patches related to networking that perform
> > conversion of refcounters from atomic_t to refcount_t.
> > In contrast to the core network refcounter conversions that
> > were merged earlier, these are
On Mon, Oct 23, 2017 at 11:13:23AM +0530, Kishon Vijay Abraham I wrote:
> EPF devices use EPC devices which in turn use the actual platform device for
> configuring the hardware. IMO the devices in one layer shouldn't have to
> explicitly use devices in another layer other than using clearly define
Hi Bin,
Yes, that's it. thanks.
--
tanxiaofei
On 2017/10/23 9:34, Li Bin wrote:
>
>
> on 2017/10/21 23:35, Tejun Heo wrote:
>> On Fri, Oct 20, 2017 at 02:57:18PM +0800, tanxiaofei wrote:
>>> Hi Tejun,
>>>
>>> Any comments about this?
>>
>> I think you're confused, or at least can't understand
On Tue, Oct 17, 2017 at 01:06:34AM +0200, Stefan Brüns wrote:
> To avoid introduction of a new compatible for each small SoC/DMA controller
> variation, move the definition of the channel count to the devicetree.
>
> The number of vchans is no longer explicit, but limited by the highest
> port/DMA
The latest maintenance release Git v2.14.3 is now available at
the usual places.
The tarballs are found at:
https://www.kernel.org/pub/software/scm/git/
The following public repositories all have a copy of the 'v2.14.3'
tag and the 'maint' branch that the tag points at:
url = https://kern
>>> I think that there is any benefit of this patch.
>>> Also, as I commented, it make the code more complicated.
>>
>> We have got different software development opinions about
>
> I agree absolutely. So, anyone can suggest the opinion and send patches.
> But, all patches have to get the review f
On 2017년 10월 23일 14:57, SF Markus Elfring wrote:
>> I think that there is any benefit of this patch.
>> Also, as I commented, it make the code more complicated.
>
> We have got different software development opinions about
I agree absolutely. So, anyone can suggest the opinion and send patches.
B
On Tue, Oct 03, 2017 at 10:52:56AM +0530, Anup Patel wrote:
> This patchset does few more improvements to Broadcom SBA RAID
> driver.
>
> The patches are based on Linux-4.14-rc3 and can also be found
> at sba-raid-imp2-v1 branch of:
> https://github.com/Broadcom/arm64-linux.git
Applied, thanks
-
> I think that there is any benefit of this patch.
> Also, as I commented, it make the code more complicated.
We have got different software development opinions about
the shown change possibilities then.
Regards,
Markus
On 2017년 10월 23일 14:45, SF Markus Elfring wrote:
>> As you commented, this patch might remove the redundant error message.
>> But, it makes the code more complicated in side of readability.
>
> Do you try to avoid duplicated code any more in other circumstances?
I usually used the goto statement
> As I already commented on your other patch related to extcon-max14577,
> this patch might remove the redundant error message.
I hope that more software improvements can be achieved in this design direction.
> But, it makes the code more complicated in side of readability. I prefer
> existing
> As you commented, this patch might remove the redundant error message.
> But, it makes the code more complicated in side of readability.
Do you try to avoid duplicated code any more in other circumstances?
Regards,
Markus
Hi,
On Wednesday 11 October 2017 10:15 PM, Robin Murphy wrote:
> On 11/10/17 09:00, Kishon Vijay Abraham I wrote:
>> pci-epc-core.c invokes of_dma_configure in order to configure
>> the coherent_dma_mask/dma_mask of endpoint function device. This is
>> required for dma_alloc_coherent to succeed in
Dear all,
I got the reviews from all maintainers related to this patch.
So, I make the immutable branch on extcon.git and then send
the pull request in order to prevent the possible merge conflict.
Best Regards,
Chanwoo Choi
Samsung Electronics
The following changes since commit 2bd6bf03f4c1c59
Hi Thierry,
Just have a ping on this issue.
Regards,
Zhi
On Wed, 2017-09-20 at 16:48 +0800, Zhi Mao wrote:
> Hi Thierry,
>
> Just a gentle ping on this issue.
> Would you please have a review to this patch?
>
> Regards,
> Zhi
>
>
> On Tue, 2017-08-22 at 10:09 +0800, Zhi Mao wrote:
> > Add su
On Sun, Oct 22, 2017 at 8:10 PM, Minchan Kim wrote:
> On Fri, Oct 20, 2017 at 10:59:31PM +0300, Kirill A. Shutemov wrote:
>> With boot-time switching between paging mode we will have variable
>> MAX_PHYSMEM_BITS.
>>
>> Let's use the maximum variable possible for CONFIG_X86_5LEVEL=y
>> configuratio
On Fri, Oct 20, 2017 at 10:17:00AM +0200, Michal Hocko wrote:
> On Fri 20-10-17 15:22:14, Xishi Qiu wrote:
> > On 2017/10/20 10:13, Joonsoo Kim wrote:
> >
> > > On Thu, Oct 19, 2017 at 02:21:18PM +0200, Michal Hocko wrote:
> [...]
> > >> >From 8cbd811d741f5dd93d1b21bb3ef94482a4d0bd32 Mon Sep 17 00
On Fri, Oct 20, 2017 at 09:02:20AM +0200, Michal Hocko wrote:
> On Fri 20-10-17 15:50:14, Joonsoo Kim wrote:
> > On Fri, Oct 20, 2017 at 07:59:22AM +0200, Michal Hocko wrote:
> > > On Fri 20-10-17 11:13:29, Joonsoo Kim wrote:
> > > > On Thu, Oct 19, 2017 at 02:21:18PM +0200, Michal Hocko wrote:
> >
Hi Arnaldo and Milian,
On Fri, Oct 20, 2017 at 10:39:35AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Fri, Oct 20, 2017 at 01:38:23PM +0200, Milian Wolff escreveu:
> > On Freitag, 20. Oktober 2017 12:21:35 CEST Milian Wolff wrote:
> > > On Donnerstag, 19. Oktober 2017 17:01:08 CEST Namhyung Kim wr
On Fri, 20 Oct 2017 18:29:33 +0200
Christoph Hellwig wrote:
> On Fri, Oct 20, 2017 at 08:23:02AM -0700, Dan Williams wrote:
> > Yes, however it seems these drivers / platforms have been living with
> > the lack of struct page for a long time. So they either don't use DAX,
> > or they have a const
On 10/22/17 20:19, Lixin Wang wrote:
> If a node with no properties is dynamically added, then a property is
> dynamically added to the node, then the property is dynamically removed,
> the result will be node->properties == NULL and node->deadprops != NULL.
>
> Add a separate function to release
On 10/20/2017 07:27 PM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki
>
> The special value of 0 for device resume latency PM QoS means
> "no restriction", but there are two problems with that.
>
> First, device resume latency PM QoS requests with 0 as the
> value are always put in front
> The commit a76caf55e5b35 ("thermal: Add devfreq cooling") is able
> to disable OPP as a cooling device. In result, both update_devfreq()
> and {min|max}_freq_show() have to consider the 'opp->available'
> status of each OPP.
>
> So, this patch adds the 'scaling_{min|max}_freq' to struct devfreq
On 10/22/2017 07:55 PM, Jarkko Sakkinen wrote:
> On Fri, Oct 20, 2017 at 07:32:42AM -0700, Dave Hansen wrote:
>> I've always been curious, and the changelog and thread are curiously
>> oblique on this topic: what the heck does this driver use pipes *for*?
> For communication with the process hostin
On Wednesday 18 October 2017 07:44 AM, Chanwoo Choi wrote:
> Gently Ping.
>
> Dear Kishon,
>
> Could you please review this patch for 'drivers/phy/*'?
sorry for the delay.. here it is
Acked-by: Kishon Vijay Abraham I
>
> Regards,
> Chanwoo Choi
>
> On 2017년 10월 12일 12:40, Chanwoo Choi wrot
Move the search for instruction dead_end into helper function
__add_dead_ends(), instead of duplicating the search for unreachable
and reachable sections in add_dead_ends().
Cc: Ingo Molnar
Signed-off-by: Kamalesh Babulal
---
tools/objtool/check.c | 108 +++--
On Thu, Oct 19, 2017 at 8:54 PM, Timur Tabi wrote:
> On 10/18/17 3:57 PM, Amit Kucheria wrote:
>>
>> Enable panic and disk activity triggers to tie to LED activity
>
>
> Could you provide some explanation as to why we want this enabled for ARM64?
> I don't have a problem with the change itself, bu
Hi,
On Wed, Oct 18, 2017 at 7:54 PM, Andy Shevchenko
wrote:
> While Rafael is looking for a solution, can you in meantime gather the
> following on the affected hardware and share it via some resource on
> Internet?
>
> 1. % acpidump -o tables.dat # tables.dat is point of interest
https://gist.g
On Fri, Oct 20, 2017 at 10:31:47AM -0700, Tony Lindgren wrote:
> * Joonsoo Kim [171019 18:53]:
> > Oops... I made a mistak. Could you test with reverting commit
> > c977ee2803787363187d6aca9cebdabc793c6531 ("omap: forcibly call
> > save_secure_ram_context() for test") in that branch?
> > Without r
On Thu, Oct 19, 2017 at 5:30 PM, Guodong Xu wrote:
>
>
> On Thu, Oct 19, 2017 at 4:57 AM, Amit Kucheria
> wrote:
>>
>> Blink the LED on a kernel panic.
>>
>> Signed-off-by: Amit Kucheria
>> ---
>> arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> d
From: "Gustavo A. R. Silva"
Date: Fri, 20 Oct 2017 19:43:11 -0500
> Use BUG_ON instead of if condition followed by BUG in do_setlink.
>
> This issue was detected with the help of Coccinelle.
>
> Signed-off-by: Gustavo A. R. Silva
Applied.
Add clock controller nodes for MT2712, include topckgen, infracfg,
pericfg, mcucfg and apmixedsys. This patch also add six oscillators that
provide clocks for MT2712.
Signed-off-by: Weiyi Lu
---
arch/arm64/boot/dts/mediatek/mt2712e.dtsi | 115 ++
1 file changed, 115 i
There are dependent clock jpgdec/audio in scpsys on MT2712,
and will exist three dependent clocks on MT2712 VDEC.
Signed-off-by: Weiyi Lu
---
drivers/soc/mediatek/mtk-scpsys.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/soc/mediatek/mtk-scpsys.c
b/drivers/so
MT2712 add "set/clear" bus control register to each control register set
instead of providing only one "enable" control register, we could avoid
the read-modify-write racing by declaring "bus_prot_reg_update" as "false"
in scp_soc_data or declaring as "true" to use the legacy update method.
By impr
add scpsys driver for MT2712
Signed-off-by: Weiyi Lu
---
drivers/soc/mediatek/mtk-scpsys.c | 106 +++---
1 file changed, 100 insertions(+), 6 deletions(-)
diff --git a/drivers/soc/mediatek/mtk-scpsys.c
b/drivers/soc/mediatek/mtk-scpsys.c
index 7e7e4ec..2a0c3a7 1
Add MT2712 clock support, include topckgen, apmixedsys,
infracfg, pericfg, mcucfg and subsystem clocks.
Signed-off-by: Weiyi Lu
---
drivers/clk/mediatek/Kconfig | 50 ++
drivers/clk/mediatek/Makefile|8 +
drivers/clk/mediatek/clk-mt2712-bdp.c| 102 +++
drivers/
Add MT2712 clock dt-bindings, include topckgen, apmixedsys,
infracfg, pericfg, mcucfg and subsystem clocks.
Signed-off-by: Weiyi Lu
---
include/dt-bindings/clock/mt2712-clk.h | 427 +
1 file changed, 427 insertions(+)
create mode 100644 include/dt-bindings/clock/
Add power dt-bindings for MT2712.
Acked-by: Rob Herring
Signed-off-by: Weiyi Lu
---
.../devicetree/bindings/soc/mediatek/scpsys.txt| 3 +++
include/dt-bindings/power/mt2712-power.h | 26 ++
2 files changed, 29 insertions(+)
create mode 100644 include/dt-bindi
add power controller node for MT2712
Signed-off-by: Weiyi Lu
---
arch/arm64/boot/dts/mediatek/mt2712e.dtsi | 16
1 file changed, 16 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
b/arch/arm64/boot/dts/mediatek/mt2712e.dtsi
index d2ee3cd..fa87f92 100644
--
This patch adds the binding documentation for apmixedsys, bdpsys,
imgsys, imgsys, infracfg, mcucfg, mfgcfg, mmsys, pericfg, topckgen,
vdecsys and vencsys for Mediatek MT2712.
Acked-by: Rob Herring
Signed-off-by: Weiyi Lu
---
.../bindings/arm/mediatek/mediatek,apmixedsys.txt | 1 +
.../binding
This series is based on v4.14-rc1 and composed of
clock control (PATCH 1-4) and scpsys control (PATCH 5-9)
changes since v5:
- Refine bus protection with proper variable name
and better implementation for the if statement.
changes since v4:
- Refine scpsys and infracfg for bus protection by pas
On (10/22/17 22:30), Baoquan He wrote:
> Signed-off-by: Baoquan He
Reviewed-by: Sergey Senozhatsky
-ss
> ---
> kernel/printk/printk_safe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/printk/printk_safe.c b/kernel/printk/printk_safe.c
> index 3cdaea
Hi lkml,
Linux 4.13.8 hangs on boot very early, so no output or log entries and
just a blank screen after bootloader, on Surface Book.
https://bugzilla.kernel.org/show_bug.cgi?id=197359
A parallel bug report on Ubuntu suggests the problem starts at 4.13.0
and persists to 4.14.0.
https://bugs.la
Currently we are using the .pad section to ensure the zImage file size
aligned to 8 bytes.
But that is not needed when CONFIG_EFI_STUB enabled, as it would
followed by a 512 bytes aligned .data section.
Signed-off-by: Jeffy Chen
---
Changes in v2:
Remove .pad section when CONFIG_EFI_STUB enable
Hi Ard,
On 10/22/2017 09:01 PM, Ard Biesheuvel wrote:
On 22 October 2017 at 13:47, Russell King - ARM Linux
wrote:
On Sun, Oct 22, 2017 at 12:01:13PM +0100, Ard Biesheuvel wrote:
On 18 October 2017 at 06:01, Jeffy Chen wrote:
The zImage file size should be aligned.
Fixes: e4bae4d0b5f3 ("ar
On 10/10/17 12:19 AM, Mark Brown wrote:
Hi Santosh,
After merging the keystone tree, today's linux-next build
(multi_v7_defconfig) failed like this:
DTC arch/arm/boot/dts/keystone-k2g-evm.dtb
Error: /home/broonie/tmpfs/next/arch/arm/boot/dts/keystone-k2g-evm.dts:150.1-15
syntax error
FA
On 10/19/2017 09:03 PM, Michal Hocko wrote:
> On Thu 19-10-17 20:26:57, Anshuman Khandual wrote:
>> Its already assumed that the PageActive flag is clear on the input
>> page, hence page_lru(page) will pick the base LRU for the page. In
>> the same way page_lru(page) will pick active base LRU, once
If a node with no properties is dynamically added, then a property is
dynamically added to the node, then the property is dynamically removed,
the result will be node->properties == NULL and node->deadprops != NULL.
Add a separate function to release the properties in both lists.
Signed-off-by: L
On Fri, Oct 20, 2017 at 10:59:31PM +0300, Kirill A. Shutemov wrote:
> With boot-time switching between paging mode we will have variable
> MAX_PHYSMEM_BITS.
>
> Let's use the maximum variable possible for CONFIG_X86_5LEVEL=y
> configuration to define zsmalloc data structures.
>
> The patch introd
On 10/19/2017 08:59 PM, Michal Hocko wrote:
> On Thu 19-10-17 14:03:14, Anshuman Khandual wrote:
>> The function pagevec_lru_move_fn() actually moves pages from various
>> per cpu pagevecs into per node lruvecs with a custom function which
>> knows how to handle individual pages present in any give
On 10/20/2017 04:54 AM, Mike Kravetz wrote:
> On 10/17/2017 10:55 PM, Anshuman Khandual wrote:
>> This adds two tests to validate mirror functionality with mremap()
>> system call on shared and private anon mappings. After the commit
>> dba58d3b8c5 ("mm/mremap: fail map duplication attempts for pri
Hi all,
On Fri, Oct 20, 2017 at 3:34 PM, Bjorn Andersson
wrote:
> On Fri 20 Oct 07:43 PDT 2017, Georgi Djakov wrote:
>
>> Hi,
>>
>> On 09/08/2017 08:18 PM, Georgi Djakov wrote:
>> > This patch introduce a new API to get requirements and configure the
>> > interconnect buses across the entire chip
On Fri, Oct 20, 2017 at 02:42:54PM +, alexander.stef...@infineon.com wrote:
> > The TPM burstcount status indicates the number of bytes that can
> > be sent to the TPM without causing bus wait states. Effectively,
> > it is the number of empty bytes in the command FIFO.
> >
> > This patch opt
On Fri, Oct 20, 2017 at 07:32:42AM -0700, Dave Hansen wrote:
> I've always been curious, and the changelog and thread are curiously
> oblique on this topic: what the heck does this driver use pipes *for*?
For communication with the process hosting the launch enclave.
/Jarkko
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.timers
head: 0f0f9ec319ea27dba44d16c643805821005f4955
commit: 7f6f735b72fb372668f6526167a7f6ae4b39ab87 [28/37] hrtimer: Implement
support for softirq based hrtimers
reproduce: make htmldocs
All warnings (new ones prefixed b
My sincere apology if this email will cause any inconvenience. I have a
proposal which I think might be of interest to you, I have a client who
is searching for Good Business Owners that have got lucrative
projects/business ideas he can invest his funds on. My client has great
interest to invest
Hi Rob & Ulf,
Gentle ping...
Any other comments about this series of patch ?
On Mon, 2017-10-16 at 09:46 +0800, Chaotian Jing wrote:
> Change the comptiable for support of multi-platform
> Make compatible explicit, as MMC host of mt8173 has difference with
> mt8135(mt8173 supports hs400 and hs400_
Hi Maxime,
On 17 October 2017 at 20:06, Maxime Ripard
wrote:
> The TCON supports the LVDS interface to output to a panel or a bridge.
> Let's add support for it.
>
> Signed-off-by: Maxime Ripard
> ---
> drivers/gpu/drm/sun4i/Makefile | 1 +-
> drivers/gpu/drm/sun4i/sun4i_lvds.c | 183
Hi Fengguang,
Sorry for that bug.
I had find the cause reason and I will send out new patch for this today.
Thanks.
Lixin
-Original Message-
From: Fengguang Wu [mailto:fengguang...@intel.com]
Sent: Monday, October 23, 2017 2:29 AM
To: Wang, Alan 1. (NSB - CN/Hangzhou)
Cc: w...@linux.i
From: Kees Cook
Date: Fri, 20 Oct 2017 13:47:08 -0700
> While the work callback uses the urb to find cardstate from bas_cardstate,
> this may not be valid for timer callbacks. Instead, introduce a direct
> pointer back to the cardstate from bas_cardstate for use in timer
> callbacks.
>
> Reporte
On Sun, Oct 22, 2017 at 02:34:56PM +, Bart Van Assche wrote:
> On Sat, 2017-10-21 at 11:23 +0900, Byungchul Park wrote:
> > On Sat, Oct 21, 2017 at 4:58 AM, Bart Van Assche
> > wrote:
> > > As explained in another e-mail thread, unlike the lock inversion checking
> > > performed by the <= v4.
On 10/21/17 at 12:20am, Yinghai Lu wrote:
> On Thu, Oct 19, 2017 at 10:52 PM, Dave Young wrote:
> > Now crashkernel=X will fail if there's not enough memory at low region
> > (below 896M) when trying to reserve large memory size. One can use
> > crashkernel=xM,high to reserve it at high region (>
Hi Vinod,
On 10 October 2017 at 13:23, Baolin Wang wrote:
> This patch adds the DMA controller driver for Spreadtrum SC9860 platform.
>
> Signed-off-by: Baolin Wang
> ---
> Changes since v3:
> - Remove redundant local 'mask' and 'val' variables.
> - Simplify sprd_dma_get_req_type() function.
>
Hi,
On Thu, 2017-10-19 at 16:54 +0200, Philipp Zabel wrote:
> Hi Laurent,
>
> On Thu, 2017-10-19 at 16:39 +0300, Laurent Pinchart wrote:
> > Hi Philipp,
> >
> > On Thursday, 19 October 2017 16:01:54 EEST Philipp Zabel wrote:
> > > On Thu, 2017-10-19 at 13:26 +0200, Matthias Brugger wrote:
> > >
A bit late, but this has bitten us again recently. below are two
typos/nitpicks comments
On Mon, Jun 19, 2017 at 11:55:13AM +0200, Benjamin Tissoires wrote:
> This is something that bothered us from a long time. When hid-input
'for a long time'
> doesn't know how to map a usage, it uses *_MISC.
Quoting David Miller :
From: "Gustavo A. R. Silva"
Date: Sun, 22 Oct 2017 20:08:40 -0500
Code refactoring in order to make it easier to read and maintain.
Signed-off-by: Gustavo A. R. Silva
Gustavo, always when reposting a new version of a patch that is part of
a series you must _always_
These patches makes the devfreq to use the OPP interface and clean-up codes.
[Detaild Descripion]
The commit a76caf55e5b3 ("thermal: Add devfreq cooling") provides
the devfreq cooling device by using the OPP interface such as
dev_pm_opp_disable() and dev_pm_opp_enable(). It means that
the OPP inte
The commit a76caf55e5b35 ("thermal: Add devfreq cooling") is able
to disable OPP as a cooling device. In result, both update_devfreq()
and {min|max}_freq_show() have to consider the 'opp->available'
status of each OPP.
So, this patch adds the 'scaling_{min|max}_freq' to struct devfreq
in order to
Prior to that, the min/max_freq of the devfreq device are always zero
before the user changes the min/max_freq through sysfs entries.
It might make the confusion for the min/max_freq.
This patch initializes the available min/max_freq by using the OPP
during adding the devfreq device.
Signed-off-b
This reverts commit 3104fa3081126c9bda35793af5f335d0ee0d5818.
The {min|max}_freq_show() show the stored value of the struct devfreq.
But, if the drivers/thermal/devfreq_cooling.c disables the specific
frequency value, {min|max}_freq_show() have to check this situation
before showing the stored val
on 2017/10/21 23:35, Tejun Heo wrote:
> On Fri, Oct 20, 2017 at 02:57:18PM +0800, tanxiaofei wrote:
>> Hi Tejun,
>>
>> Any comments about this?
>
> I think you're confused, or at least can't understand what you're
> trying to say. Can you create a rero?
>
Hi Tejun,
The case is as following:
The commit a76caf55e5b35 ("thermal: Add devfreq cooling") allows
the devfreq device to use the cooling device. When the cooling down
are required, the devfreq_cooling.c disables the OPP entry with
the dev_pm_opp_disable(). In result, 'available_frequencies'[1]
sysfs node never came to show the all
Prior to that, the devfreq device uses the governor name when adding
the itself. In order to prevent the mistake used the wrong governor name,
this patch defines the governor name as a constant and then uses them
instead of using the string directly.
Signed-off-by: Chanwoo Choi
Acked-by: MyungJoo
This patch changes the return type of devfreq_set_freq_table()
from 'void' to 'int' in order to check whether it fails or not.
And This patch just removes the 'devfreq' prefix and the description
of function. Because the helper functions are only used by the devfreq.
Signed-off-by: Chanwoo Choi
The freq_table array of each devfreq device is always not NULL.
In result, it is unneeded to check whether profile->freq_table
is NULL or not.
Signed-off-by: Chanwoo Choi
Acked-by: MyungJoo Ham
---
drivers/devfreq/devfreq.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --gi
From: "Gustavo A. R. Silva"
Date: Sun, 22 Oct 2017 20:08:40 -0500
> Code refactoring in order to make it easier to read and maintain.
>
> Signed-off-by: Gustavo A. R. Silva
Gustavo, always when reposting a new version of a patch that is part of
a series you must _always_ repost the entire patc
Hi Markus,
As I already commented on your other patch related to extcon-max14577,
this patch might remove the redundant error message. But, it makes
the code more complicated in side of readability. I prefer existing code.
On 2017년 10월 22일 22:40, SF Markus Elfring wrote:
> From: Markus Elfring
>
Code refactoring in order to make it easier to read and maintain.
Signed-off-by: Gustavo A. R. Silva
---
This code was tested by compilation only.
Changes in v2:
Make use of the swap macro and remove inline keyword.
net/netrom/nr_route.c | 59 ++
On Sun, Oct 22, 2017 at 12:05:55AM +0200, Lukasz Majewski wrote:
> This commit adds device tree description of Liebherr's Display5 board.
>
> Signed-off-by: Lukasz Majewski
> Reviewed-by: Fabio Estevam
Applied, thanks.
This is only ever set to LDLM_FL_WAIT_NOREPROC, so we can remove the arg
and discard any code that is only run when it doesn't have that value.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 110 ---
1 file changed, 21 insertions(+), 89 deleti
With this field gone, we don't need local variables 'imp' or 'obd'
any more.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 21 +++--
1 file changed, 3 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c
Use list_for_each_entry variants to
avoid the explicit list_entry() calls.
This allows us to use list_for_each_entry_safe_from()
instread of adding a local list-walking macro.
Also improve some comments so that it is more obvious
that the locks are sorted per-owner and that we need
to find the ins
The only value ever passed in LDLM_FL_WAIT_NOREPROC, so assume that
instead of passing it.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 36 ++-
1 file changed, 16 insertions(+), 20 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ld
Now that the code has been simplified, 'ownlocks' is not
necessary.
The loop which sets it exits with 'lock' having the same value as
'ownlocks', or pointing to the head of the list if ownlocks is NULL.
The current code then tests ownlocks and sets 'lock' to exactly the
value that it currently ha
Hi Linus,
2017-10-16 17:24 GMT+09:00 Linus Walleij :
> On Thu, Oct 12, 2017 at 1:36 PM, Masahiro Yamada
> wrote:
>
>> Changes in v8:
>> - Move dt-binding header to binding commit
>> - Add MAINTAINERS entry for the binding doc
>> - Add a consumer example to the binding doc
>
> This v8 looks
This file contains quite a bit of dead code and unused variables.
This patch series cleans it up in various ways.
It should change behaviour at all, just code
readability/maintainabilty.
I sent the back in July but got not response, possibly because there
were included with other patches which cau
'overlaps' is never used, only incremented.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c
b/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c
index 2d1fa2b33129.
'work_list' is only set to NULL, and is never used.
So discard it.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c
b/drivers/staging/lustre/lu
This arg is used to return an error code, but the returned code is never
looked at. So there is no point returning it.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/lu
it is only ever set to '1', so we can just assume that and remove the code.
Signed-off-by: NeilBrown
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_flock.c
b/drive
On Thu, 19 Oct 2017, Johannes Weiner wrote:
> David would have really liked for this patchset to include knobs to
> influence how the algorithm picks cgroup victims. The rest of us
> agreed that this is beyond the scope of these patches, that the
> patches don't need it to be useful, and that ther
On Wed, Oct 18, 2017 at 08:40:59AM +0200, Lothar Waßmann wrote:
> > So should I drop all interrupts-extended patches from you?
> >
> Yes, please. Since this patch, and the corresponding patch for tx53
> also fixed the interrupt flags, I will send new patches to only fix
> the interrupt flags.
Okay
1 - 100 of 280 matches
Mail list logo
