Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Daniel Stone
Hi, On 17 February 2017 at 14:56, Ville Syrjälä wrote: > On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: >> If we're talking fixed point reprsentation, ChromeOS is using this : >> >>

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Daniel Stone
Hi, On 17 February 2017 at 14:56, Ville Syrjälä wrote: > On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: >> If we're talking fixed point reprsentation, ChromeOS is using this : >> >> https://cs.chromium.org/chromium/src/ui/ozone/platform/drm/gpu/drm_device.cc?q=DrmDevice=209 >

[PATCH v2] gpio: xgene: mark PM functions as __maybe_unused

2017-02-17 Thread Arnd Bergmann
When CONFIG_PM_SLEEP is disabled, we get a warning about unused functions: drivers/gpio/gpio-xgene.c:155:12: warning: 'xgene_gpio_resume' defined but not used [-Wunused-function] static int xgene_gpio_resume(struct device *dev) ^ drivers/gpio/gpio-xgene.c:142:12:

[PATCH v2] gpio: xgene: mark PM functions as __maybe_unused

2017-02-17 Thread Arnd Bergmann
When CONFIG_PM_SLEEP is disabled, we get a warning about unused functions: drivers/gpio/gpio-xgene.c:155:12: warning: 'xgene_gpio_resume' defined but not used [-Wunused-function] static int xgene_gpio_resume(struct device *dev) ^ drivers/gpio/gpio-xgene.c:142:12:

[PATCH] Revert "ARM: at91/dt: sama5d2: Use new compatible for ohci node"

2017-02-17 Thread Romain Izard
This reverts commit cab43282682e ("ARM: at91/dt: sama5d2: Use new compatible for ohci node") It depends from commit 7150bc9b4d43 ("usb: ohci-at91: Forcibly suspend ports while USB suspend") which was reverted and implemented differently. With the new implementation, the compatible string must

[PATCH] Revert "ARM: at91/dt: sama5d2: Use new compatible for ohci node"

2017-02-17 Thread Romain Izard
This reverts commit cab43282682e ("ARM: at91/dt: sama5d2: Use new compatible for ohci node") It depends from commit 7150bc9b4d43 ("usb: ohci-at91: Forcibly suspend ports while USB suspend") which was reverted and implemented differently. With the new implementation, the compatible string must

[PATCH] vmxnet3: prevent building with 64K pages

2017-02-17 Thread Arnd Bergmann
I got a warning about broken code on ARM64 with 64K pages: drivers/net/vmxnet3/vmxnet3_drv.c: In function 'vmxnet3_rq_init': drivers/net/vmxnet3/vmxnet3_drv.c:1679:29: error: large integer implicitly truncated to unsigned type [-Werror=overflow] rq->buf_info[0][i].len = PAGE_SIZE; 'len'

[PATCH] vmxnet3: prevent building with 64K pages

2017-02-17 Thread Arnd Bergmann
I got a warning about broken code on ARM64 with 64K pages: drivers/net/vmxnet3/vmxnet3_drv.c: In function 'vmxnet3_rq_init': drivers/net/vmxnet3/vmxnet3_drv.c:1679:29: error: large integer implicitly truncated to unsigned type [-Werror=overflow] rq->buf_info[0][i].len = PAGE_SIZE; 'len'

[PATCH net-next v2 4/6] net: dsa: mv88e6xxx: move VTU code to its own file

2017-02-17 Thread Vivien Didelot
Move the Global (1) VTU related code to its own file. Use this opportunity to provide a cleaner API for the VTU, by renaming a few underscore prefixed functions, split the data member of the mv88e6xxx_vtu_entry structure into membership (for 802.1q) and port state (for 802.1s) members, and add a

[PATCH net-next v2 4/6] net: dsa: mv88e6xxx: move VTU code to its own file

2017-02-17 Thread Vivien Didelot
Move the Global (1) VTU related code to its own file. Use this opportunity to provide a cleaner API for the VTU, by renaming a few underscore prefixed functions, split the data member of the mv88e6xxx_vtu_entry structure into membership (for 802.1q) and port state (for 802.1s) members, and add a

[PATCH net-next v2 5/6] net: dsa: mv88e6xxx: add VTU ops

2017-02-17 Thread Vivien Didelot
Because there are several variant of the VTU operations and because checking for the presence of an STU is not enough, add new ops to the info structure to describe the VTU operations that a chip supports. Signed-off-by: Vivien Didelot ---

[PATCH net-next v2 5/6] net: dsa: mv88e6xxx: add VTU ops

2017-02-17 Thread Vivien Didelot
Because there are several variant of the VTU operations and because checking for the presence of an STU is not enough, add new ops to the info structure to describe the VTU operations that a chip supports. Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6xxx/chip.c | 58

[PATCH net-next v2 3/6] net: dsa: mv88e6xxx: add support for 6390 ATU Move

2017-02-17 Thread Vivien Didelot
The 6390 family of Marvell chips uses 5 bits to describe the ToPort and FromPort values of PortVec in the ATU Move operation, while older switches use 0xf. Add an atu_move_port_mask member in the info structure to describe that. Signed-off-by: Vivien Didelot

[PATCH net-next v2 3/6] net: dsa: mv88e6xxx: add support for 6390 ATU Move

2017-02-17 Thread Vivien Didelot
The 6390 family of Marvell chips uses 5 bits to describe the ToPort and FromPort values of PortVec in the ATU Move operation, while older switches use 0xf. Add an atu_move_port_mask member in the info structure to describe that. Signed-off-by: Vivien Didelot ---

[PATCH net-next v2 0/6] net: dsa: mv88e6xxx: 88E6390 ATU/VTU

2017-02-17 Thread Vivien Didelot
This series adds support to 88E6390 for the ATU (MAC address databases) and VTU (VLAN databases), as well as providing a cleaner API for both. The 6390 family of Marvell switch chip handles the ATU Move operation differently and has a different usage of the VTU Data registers. v2: remove unsused

[PATCH net-next v2 6/6] net: dsa: mv88e6xxx: add support for 6390 VTU

2017-02-17 Thread Vivien Didelot
The 6390 family of chips use only 2 of the 3 VTU Data registers to pack the MemberTag and PortState VLAN data. This means that they must be written or read before or after each VTU/STU operations. Implement this variant to add support for VTU with such chips. Signed-off-by: Vivien Didelot

[PATCH net-next v2 0/6] net: dsa: mv88e6xxx: 88E6390 ATU/VTU

2017-02-17 Thread Vivien Didelot
This series adds support to 88E6390 for the ATU (MAC address databases) and VTU (VLAN databases), as well as providing a cleaner API for both. The 6390 family of Marvell switch chip handles the ATU Move operation differently and has a different usage of the VTU Data registers. v2: remove unsused

[PATCH net-next v2 6/6] net: dsa: mv88e6xxx: add support for 6390 VTU

2017-02-17 Thread Vivien Didelot
The 6390 family of chips use only 2 of the 3 VTU Data registers to pack the MemberTag and PortState VLAN data. This means that they must be written or read before or after each VTU/STU operations. Implement this variant to add support for VTU with such chips. Signed-off-by: Vivien Didelot ---

[PATCH net-next v2 1/6] net: dsa: mv88e6xxx: add port mask helper

2017-02-17 Thread Vivien Didelot
Add a mv88e6xxx_port_mask() helper to get the bitmask of ports in a switch chip, that will be used in several features. Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6xxx/mv88e6xxx.h | 5 + drivers/net/dsa/mv88e6xxx/port.c | 2 +- 2 files

[PATCH net-next v2 2/6] net: dsa: mv88e6xxx: move ATU code in its own file

2017-02-17 Thread Vivien Didelot
Move the Global (1) ATU related code in its own file, and export the necessary primitives. Use that opportunity to provide a cleaner API for the ATU, by renaming a few underscore prefixed functions, and members of the mv88e6xxx_atu_entry structures. Signed-off-by: Vivien Didelot

[PATCH net-next v2 2/6] net: dsa: mv88e6xxx: move ATU code in its own file

2017-02-17 Thread Vivien Didelot
Move the Global (1) ATU related code in its own file, and export the necessary primitives. Use that opportunity to provide a cleaner API for the ATU, by renaming a few underscore prefixed functions, and members of the mv88e6xxx_atu_entry structures. Signed-off-by: Vivien Didelot ---

[PATCH net-next v2 1/6] net: dsa: mv88e6xxx: add port mask helper

2017-02-17 Thread Vivien Didelot
Add a mv88e6xxx_port_mask() helper to get the bitmask of ports in a switch chip, that will be used in several features. Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6xxx/mv88e6xxx.h | 5 + drivers/net/dsa/mv88e6xxx/port.c | 2 +- 2 files changed, 6 insertions(+), 1

Re: [PATCH 07/10] perf: Directly pass PERF_AUX_* flags to perf_aux_output_end

2017-02-17 Thread Alexander Shishkin
Will Deacon writes: > On Fri, Feb 17, 2017 at 04:06:59PM +0200, Alexander Shishkin wrote: >> Will Deacon writes: >> >> > On Fri, Feb 17, 2017 at 03:40:23PM +0200, Alexander Shishkin wrote: >> >> Will Deacon writes: >> >> >> >> >

Re: [PATCH v3 3/3] perf: replace _SC_NPROCESSORS_CONF with max_present_cpu in cpu_topology_map

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 12:10:26PM +0100, Jan Stancek wrote: SNIP > - int cpu_nr = ph->env.nr_cpus_online; > + int cpu_nr = ph->env.nr_cpus_avail; > > nr = ph->env.nr_sibling_cores; > str = ph->env.sibling_cores; > @@ -1792,7 +1786,7 @@ static int

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Lionel Landwerlin
On 17/02/17 14:56, Ville Syrjälä wrote: On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: On 17/02/17 13:54, Brian Starkey wrote: What's the verdict? We've got [1] which is about to become another (driver) implementation - better to change before that merges than after I

Re: [PATCH 07/10] perf: Directly pass PERF_AUX_* flags to perf_aux_output_end

2017-02-17 Thread Alexander Shishkin
Will Deacon writes: > On Fri, Feb 17, 2017 at 04:06:59PM +0200, Alexander Shishkin wrote: >> Will Deacon writes: >> >> > On Fri, Feb 17, 2017 at 03:40:23PM +0200, Alexander Shishkin wrote: >> >> Will Deacon writes: >> >> >> >> > @@ -485,7 +485,8 @@ int intel_bts_interrupt(void) >> >> >

Re: [PATCH v3 3/3] perf: replace _SC_NPROCESSORS_CONF with max_present_cpu in cpu_topology_map

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 12:10:26PM +0100, Jan Stancek wrote: SNIP > - int cpu_nr = ph->env.nr_cpus_online; > + int cpu_nr = ph->env.nr_cpus_avail; > > nr = ph->env.nr_sibling_cores; > str = ph->env.sibling_cores; > @@ -1792,7 +1786,7 @@ static int

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Lionel Landwerlin
On 17/02/17 14:56, Ville Syrjälä wrote: On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: On 17/02/17 13:54, Brian Starkey wrote: What's the verdict? We've got [1] which is about to become another (driver) implementation - better to change before that merges than after I

Re: [PATCH v4 00/36] i.MX Media Driver

2017-02-17 Thread Philipp Zabel
On Fri, 2017-02-17 at 14:22 +0200, Sakari Ailus wrote: > Hi Philipp, Steve and Russell, > > On Fri, Feb 17, 2017 at 12:43:38PM +0100, Philipp Zabel wrote: > > On Thu, 2017-02-16 at 14:27 -0800, Steve Longerbeam wrote: > > > > > > On 02/16/2017 02:20 PM, Russell King - ARM Linux wrote: > > > > On

Re: [PATCH v4 00/36] i.MX Media Driver

2017-02-17 Thread Philipp Zabel
On Fri, 2017-02-17 at 14:22 +0200, Sakari Ailus wrote: > Hi Philipp, Steve and Russell, > > On Fri, Feb 17, 2017 at 12:43:38PM +0100, Philipp Zabel wrote: > > On Thu, 2017-02-16 at 14:27 -0800, Steve Longerbeam wrote: > > > > > > On 02/16/2017 02:20 PM, Russell King - ARM Linux wrote: > > > > On

[PATCH] smartpqi: fix time handling

2017-02-17 Thread Arnd Bergmann
When we have turned off RTC support, the smartpqi driver fails to build: ERROR: "rtc_time64_to_tm" [drivers/scsi/smartpqi/smartpqi.ko] undefined! This is easily avoided by using the generic 'struct tm' based helper rather than the RTC specific one. While fixing this, I noticed that even though

[PATCH] smartpqi: fix time handling

2017-02-17 Thread Arnd Bergmann
When we have turned off RTC support, the smartpqi driver fails to build: ERROR: "rtc_time64_to_tm" [drivers/scsi/smartpqi/smartpqi.ko] undefined! This is easily avoided by using the generic 'struct tm' based helper rather than the RTC specific one. While fixing this, I noticed that even though

Re: [PATCH 1/5] KVM: change API for requests to match bit operations

2017-02-17 Thread Radim Krčmář
2017-02-17 10:30+0100, Cornelia Huck: > On Thu, 16 Feb 2017 17:04:45 +0100 > Radim Krčmář wrote: > >> kvm_make_request was a wrapper that added barriers to bit_set and >> kvm_check_request did the same for bit_test and bit_check, but the name >> was not very obvious and we

Re: [PATCH 1/5] KVM: change API for requests to match bit operations

2017-02-17 Thread Radim Krčmář
2017-02-17 10:30+0100, Cornelia Huck: > On Thu, 16 Feb 2017 17:04:45 +0100 > Radim Krčmář wrote: > >> kvm_make_request was a wrapper that added barriers to bit_set and >> kvm_check_request did the same for bit_test and bit_check, but the name >> was not very obvious and we were also lacking

Re: [PATCH 09/35] x86: Convert remaining uses of pr_warning to pr_warn

2017-02-17 Thread Steven Rostedt
On Thu, 16 Feb 2017 23:11:22 -0800 Joe Perches wrote: > diff --git a/arch/x86/mm/testmmiotrace.c b/arch/x86/mm/testmmiotrace.c > index 38868adf07ea..4a55e453296d 100644 > --- a/arch/x86/mm/testmmiotrace.c > +++ b/arch/x86/mm/testmmiotrace.c > @@ -121,9 +121,8 @@ static int

Re: [PATCH 09/35] x86: Convert remaining uses of pr_warning to pr_warn

2017-02-17 Thread Steven Rostedt
On Thu, 16 Feb 2017 23:11:22 -0800 Joe Perches wrote: > diff --git a/arch/x86/mm/testmmiotrace.c b/arch/x86/mm/testmmiotrace.c > index 38868adf07ea..4a55e453296d 100644 > --- a/arch/x86/mm/testmmiotrace.c > +++ b/arch/x86/mm/testmmiotrace.c > @@ -121,9 +121,8 @@ static int __init init(void) >

Re: [PATCH 5/5] KVM: add kvm_request_pending

2017-02-17 Thread Radim Krčmář
2017-02-17 10:51+0100, Andrew Jones: > On Thu, Feb 16, 2017 at 05:04:49PM +0100, Radim Krčmář wrote: > ... >> diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h >> index 2cc438685af8..563cf964dc5c 100644 >> --- a/include/linux/kvm_host.h >> +++ b/include/linux/kvm_host.h >> @@

Re: [PATCH 5/5] KVM: add kvm_request_pending

2017-02-17 Thread Radim Krčmář
2017-02-17 10:51+0100, Andrew Jones: > On Thu, Feb 16, 2017 at 05:04:49PM +0100, Radim Krčmář wrote: > ... >> diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h >> index 2cc438685af8..563cf964dc5c 100644 >> --- a/include/linux/kvm_host.h >> +++ b/include/linux/kvm_host.h >> @@

Re: [PATCH 33/35] kernel/trace: Convert remaining uses of pr_warning to pr_warn

2017-02-17 Thread Steven Rostedt
On Thu, 16 Feb 2017 23:11:46 -0800 Joe Perches wrote: > To enable eventual removal of pr_warning > > This makes pr_warn use consistent for kernel/trace > > Prior to this patch, there were 2 uses of pr_warning and > 79 uses of pr_warn in kernel/trace > > Signed-off-by: Joe

Re: [PATCH 33/35] kernel/trace: Convert remaining uses of pr_warning to pr_warn

2017-02-17 Thread Steven Rostedt
On Thu, 16 Feb 2017 23:11:46 -0800 Joe Perches wrote: > To enable eventual removal of pr_warning > > This makes pr_warn use consistent for kernel/trace > > Prior to this patch, there were 2 uses of pr_warning and > 79 uses of pr_warn in kernel/trace > > Signed-off-by: Joe Perches Acked-by:

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Ville Syrjälä
On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: > On 17/02/17 13:54, Brian Starkey wrote: > > What's the verdict? We've got [1] which is about to become another > > (driver) implementation - better to change before that merges than > > after I guess. > > > > -Brian > > > > [1]

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Ville Syrjälä
On Fri, Feb 17, 2017 at 02:42:26PM +, Lionel Landwerlin wrote: > On 17/02/17 13:54, Brian Starkey wrote: > > What's the verdict? We've got [1] which is about to become another > > (driver) implementation - better to change before that merges than > > after I guess. > > > > -Brian > > > > [1]

Re: [PATCH] platform/x86: ideapad-laptop: Add sysfs interface for touchpad state

2017-02-17 Thread Ritesh Raj Sarraf
On Fri, 2017-02-17 at 14:35 +0530, Ritesh Raj Sarraf wrote: > > Please use: > > > > static DEVICE_ATTR_RW(touchpad_mode); > > > > You'll need to move the show_ and store_ prefixes to be postfixes, see > > toshiba-acpi.c for several examples. > > > > Thank you. I am building the kernel the

Re: [PATCH] platform/x86: ideapad-laptop: Add sysfs interface for touchpad state

2017-02-17 Thread Ritesh Raj Sarraf
On Fri, 2017-02-17 at 14:35 +0530, Ritesh Raj Sarraf wrote: > > Please use: > > > > static DEVICE_ATTR_RW(touchpad_mode); > > > > You'll need to move the show_ and store_ prefixes to be postfixes, see > > toshiba-acpi.c for several examples. > > > > Thank you. I am building the kernel the

Re: [PATCH v2 1/3] watchdog: sama5d4: Cleanup init

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: .config is used to cache a part of WDT_MR at probe time and is not used afterwards. Also functions are used while they always return 0. Simplify the flow by having everything in .probe(). Does that really improve anything ? It makes the code

Re: [PATCH v2 1/3] watchdog: sama5d4: Cleanup init

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: .config is used to cache a part of WDT_MR at probe time and is not used afterwards. Also functions are used while they always return 0. Simplify the flow by having everything in .probe(). Does that really improve anything ? It makes the code

[PATCH v3] Add sysfs interface for touchpad state

2017-02-17 Thread Ritesh Raj Sarraf
Lenovo Yoga (many variants: Yoga, Yoga2 Pro, Yoga2 13, Yoga3 Pro, Yoga 3 14 etc) has multiple modles that are a hybrid laptop, working in laptop mode as well as tablet mode. Currently, there is no easy interface to determine the touchpad status, which in case of the Yoga family of machines, can

[PATCH v3] Add sysfs interface for touchpad state

2017-02-17 Thread Ritesh Raj Sarraf
Lenovo Yoga (many variants: Yoga, Yoga2 Pro, Yoga2 13, Yoga3 Pro, Yoga 3 14 etc) has multiple modles that are a hybrid laptop, working in laptop mode as well as tablet mode. Currently, there is no easy interface to determine the touchpad status, which in case of the Yoga family of machines, can

Re: [PATCH v2 3/3] watchdog: sama5d4: Implement resume hook

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: When resuming for the deepest state on sama5d2, it is necessary to restore MR as the registers are lost. Signed-off-by: Alexandre Belloni --- Changes in v2: - cache mr beofre suspending

Re: [PATCH v2 3/3] watchdog: sama5d4: Implement resume hook

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: When resuming for the deepest state on sama5d2, it is necessary to restore MR as the registers are lost. Signed-off-by: Alexandre Belloni --- Changes in v2: - cache mr beofre suspending drivers/watchdog/sama5d4_wdt.c | 32

[PATCH] sched/core: include asm/paravirt.h

2017-02-17 Thread Arnd Bergmann
One of the headers that got removed was needed after all, depending on the configuration: kernel/sched/core.c: In function 'update_rq_clock_task': kernel/sched/core.c:198:50: error: 'paravirt_steal_rq_enabled' undeclared (first use in this function); did you mean 'arch_local_irq_enable'? This

[PATCH] sched/core: include asm/paravirt.h

2017-02-17 Thread Arnd Bergmann
One of the headers that got removed was needed after all, depending on the configuration: kernel/sched/core.c: In function 'update_rq_clock_task': kernel/sched/core.c:198:50: error: 'paravirt_steal_rq_enabled' undeclared (first use in this function); did you mean 'arch_local_irq_enable'? This

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 11:41:28AM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Feb 17, 2017 at 03:33:27PM +0100, Jiri Olsa escreveu: > > On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > > > Boris

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 11:41:28AM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Feb 17, 2017 at 03:33:27PM +0100, Jiri Olsa escreveu: > > On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > > > Boris

Re: [PATCH 2/2] x86/xen: use capabilities instead of fake cpuid values

2017-02-17 Thread Juergen Gross
On 17/02/17 15:30, Boris Ostrovsky wrote: > > > On 02/17/2017 09:19 AM, Juergen Gross wrote: >> On 17/02/17 15:05, Boris Ostrovsky wrote: >>> >>> >>> On 02/17/2017 02:36 AM, Juergen Gross wrote: When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In

Re: [PATCH 2/2] x86/xen: use capabilities instead of fake cpuid values

2017-02-17 Thread Juergen Gross
On 17/02/17 15:30, Boris Ostrovsky wrote: > > > On 02/17/2017 09:19 AM, Juergen Gross wrote: >> On 17/02/17 15:05, Boris Ostrovsky wrote: >>> >>> >>> On 02/17/2017 02:36 AM, Juergen Gross wrote: When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In

Re: [PATCH 07/10] perf: Directly pass PERF_AUX_* flags to perf_aux_output_end

2017-02-17 Thread Will Deacon
On Fri, Feb 17, 2017 at 04:06:59PM +0200, Alexander Shishkin wrote: > Will Deacon writes: > > > On Fri, Feb 17, 2017 at 03:40:23PM +0200, Alexander Shishkin wrote: > >> Will Deacon writes: > >> > >> > @@ -485,7 +485,8 @@ int intel_bts_interrupt(void) >

Re: [PATCH 07/10] perf: Directly pass PERF_AUX_* flags to perf_aux_output_end

2017-02-17 Thread Will Deacon
On Fri, Feb 17, 2017 at 04:06:59PM +0200, Alexander Shishkin wrote: > Will Deacon writes: > > > On Fri, Feb 17, 2017 at 03:40:23PM +0200, Alexander Shishkin wrote: > >> Will Deacon writes: > >> > >> > @@ -485,7 +485,8 @@ int intel_bts_interrupt(void) > >> > return handled; >

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Lionel Landwerlin
On 17/02/17 13:54, Brian Starkey wrote: What's the verdict? We've got [1] which is about to become another (driver) implementation - better to change before that merges than after I guess. -Brian [1] https://lkml.org/lkml/2017/2/13/304 On Wed, Feb 15, 2017 at 11:56:55AM +, Daniel Stone

Re: [PATCH v2] drm/color: Document CTM eqations

2017-02-17 Thread Lionel Landwerlin
On 17/02/17 13:54, Brian Starkey wrote: What's the verdict? We've got [1] which is about to become another (driver) implementation - better to change before that merges than after I guess. -Brian [1] https://lkml.org/lkml/2017/2/13/304 On Wed, Feb 15, 2017 at 11:56:55AM +, Daniel Stone

[PATCH] fbdev: ssd1307fb: include linux/gpio/consumer.h

2017-02-17 Thread Arnd Bergmann
Changing this driver to the gpiod API requires the use of the new-style header, depending on the configuration: drivers/video/fbdev/ssd1307fb.c: In function 'ssd1307fb_probe': drivers/video/fbdev/ssd1307fb.c:569:15: error: implicit declaration of function 'devm_gpiod_get_optional';did you mean

[PATCH] fbdev: ssd1307fb: include linux/gpio/consumer.h

2017-02-17 Thread Arnd Bergmann
Changing this driver to the gpiod API requires the use of the new-style header, depending on the configuration: drivers/video/fbdev/ssd1307fb.c: In function 'ssd1307fb_probe': drivers/video/fbdev/ssd1307fb.c:569:15: error: implicit declaration of function 'devm_gpiod_get_optional';did you mean

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:33:27PM +0100, Jiri Olsa escreveu: > On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > > Boris asked for default -a option in case we monitor > > > only uncore events. While

[PATCH] RDMA/bnxt_re: add DCB dependency

2017-02-17 Thread Arnd Bergmann
When CONFIG_DCB is disabled, we get a link error: drivers/infiniband/built-in.o: In function `bnxt_re_setup_qos': trace.c:(.text+0x155774): undefined reference to `dcb_ieee_getapp_mask' trace.c:(.text+0x155774): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:33:27PM +0100, Jiri Olsa escreveu: > On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > > Boris asked for default -a option in case we monitor > > > only uncore events. While

[PATCH] RDMA/bnxt_re: add DCB dependency

2017-02-17 Thread Arnd Bergmann
When CONFIG_DCB is disabled, we get a link error: drivers/infiniband/built-in.o: In function `bnxt_re_setup_qos': trace.c:(.text+0x155774): undefined reference to `dcb_ieee_getapp_mask' trace.c:(.text+0x155774): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol

Re: [PATCH v2 2/3] watchdog: sama5d4: Fix setting timeout when watchdog is disabled

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: The datasheet states: "When setting the WDDIS bit, and while it is set, the fields WDV and WDD must not be modified." Ensure WDDIS is not set when changing the timeout and set it afterwards if the watchdog was disabled. Signed-off-by: Alexandre

Re: [PATCH v2 2/3] watchdog: sama5d4: Fix setting timeout when watchdog is disabled

2017-02-17 Thread Guenter Roeck
On 02/16/2017 11:30 AM, Alexandre Belloni wrote: The datasheet states: "When setting the WDDIS bit, and while it is set, the fields WDV and WDD must not be modified." Ensure WDDIS is not set when changing the timeout and set it afterwards if the watchdog was disabled. Signed-off-by: Alexandre

Re: v4.10-rc8 (-rc6) boot regression on Intel desktop, does not boot after cold boots, boots after reboot

2017-02-17 Thread Frederic Weisbecker
On Thu, Feb 16, 2017 at 08:06:04PM +0100, Pavel Machek wrote: > On Thu 2017-02-16 18:25:35, Pavel Machek wrote: > > Hi! > > > > > > > 4.10-rc4 broken > > > > > 4.10-rc3 ok > > > > > > > > Hmm. If those actually end up being reliable, then there's not a whole > > > > lot in between them wrt PCI

Re: v4.10-rc8 (-rc6) boot regression on Intel desktop, does not boot after cold boots, boots after reboot

2017-02-17 Thread Frederic Weisbecker
On Thu, Feb 16, 2017 at 08:06:04PM +0100, Pavel Machek wrote: > On Thu 2017-02-16 18:25:35, Pavel Machek wrote: > > Hi! > > > > > > > 4.10-rc4 broken > > > > > 4.10-rc3 ok > > > > > > > > Hmm. If those actually end up being reliable, then there's not a whole > > > > lot in between them wrt PCI

[PATCH] IB/hns: include linux/module.h

2017-02-17 Thread Arnd Bergmann
I ran into a build error on arm64 randconfig testing: infiniband/hw/hns/hns_roce_main.c:539:1: error: data definition has no type or storage class [-Werror] infiniband/hw/hns/hns_roce_main.c:539:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int]

[PATCH] IB/hns: include linux/module.h

2017-02-17 Thread Arnd Bergmann
I ran into a build error on arm64 randconfig testing: infiniband/hw/hns/hns_roce_main.c:539:1: error: data definition has no type or storage class [-Werror] infiniband/hw/hns/hns_roce_main.c:539:1: error: type defaults to 'int' in declaration of 'MODULE_DEVICE_TABLE' [-Werror=implicit-int]

[PATCH v1 2/2] drm: sti: make driver use devm_of_platform_populate()

2017-02-17 Thread Benjamin Gaignard
This make sure that of_platform_depopulate() is called if an error occur in probe after populating the date from the device tree. Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/sti/sti_drv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v1 2/2] drm: sti: make driver use devm_of_platform_populate()

2017-02-17 Thread Benjamin Gaignard
This make sure that of_platform_depopulate() is called if an error occur in probe after populating the date from the device tree. Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/sti/sti_drv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH v1 1/2] of: add devm_ functions for populate and depopulate

2017-02-17 Thread Benjamin Gaignard
Lost of calls to of_platform_populate() are not unbalanced by a call to of_platform_depopulate(). This create issues while drivers are bind/unbind. In way to solve those issues is to add devm_of_platform_populate() which will call of_platform_depopulate() when the device is unbound from the bus.

[PATCH v1 1/2] of: add devm_ functions for populate and depopulate

2017-02-17 Thread Benjamin Gaignard
Lost of calls to of_platform_populate() are not unbalanced by a call to of_platform_depopulate(). This create issues while drivers are bind/unbind. In way to solve those issues is to add devm_of_platform_populate() which will call of_platform_depopulate() when the device is unbound from the bus.

Re: [PATCH 2/2] x86/xen: use capabilities instead of fake cpuid values

2017-02-17 Thread Boris Ostrovsky
On 02/17/2017 09:19 AM, Juergen Gross wrote: On 17/02/17 15:05, Boris Ostrovsky wrote: On 02/17/2017 02:36 AM, Juergen Gross wrote: When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the aperf/mperf feature is indicated as not being present by

Re: [PATCH 2/2] x86/xen: use capabilities instead of fake cpuid values

2017-02-17 Thread Boris Ostrovsky
On 02/17/2017 09:19 AM, Juergen Gross wrote: On 17/02/17 15:05, Boris Ostrovsky wrote: On 02/17/2017 02:36 AM, Juergen Gross wrote: When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the aperf/mperf feature is indicated as not being present by

[PATCH v1 0/2] Introduce devm_of_platform_populate() helper

2017-02-17 Thread Benjamin Gaignard
Lost of calls to of_platform_populate() are not unbalanced by a call to of_platform_depopulate(). This create issues while drivers are bind/unbind. In way to solve those issues is to add devm_of_platform_populate() which will call of_platform_depopulate() when the device is unbound from the

[PATCH v1 0/2] Introduce devm_of_platform_populate() helper

2017-02-17 Thread Benjamin Gaignard
Lost of calls to of_platform_populate() are not unbalanced by a call to of_platform_depopulate(). This create issues while drivers are bind/unbind. In way to solve those issues is to add devm_of_platform_populate() which will call of_platform_depopulate() when the device is unbound from the

[PATCH] x86/mm: trivial code cleanup for memory_map_top_doown()

2017-02-17 Thread Wei Yang
In case (last_start <= step_size), start is for sure to be 0. So, it is save to do the round_down for all cases and set start to map_start when start is smaller than map_start. >From the performance point of view, this also reduces the check on each iteration. This patch unifies the code on

[PATCH] x86/mm: trivial code cleanup for memory_map_top_doown()

2017-02-17 Thread Wei Yang
In case (last_start <= step_size), start is for sure to be 0. So, it is save to do the round_down for all cases and set start to map_start when start is smaller than map_start. >From the performance point of view, this also reduces the check on each iteration. This patch unifies the code on

[PATCH] staging: octeon: remove unused variable

2017-02-17 Thread Arnd Bergmann
A cleanup patch left one local variable without a reference: drivers/staging/octeon/ethernet-rx.c:339:28: warning: unused variable 'priv' [-Wunused-variable] This removes the declaration too. Fixes: 66812da3a689 ("staging: octeon: Use net_device_stats from struct net_device") Signed-off-by:

Re: [PATCH v7 1/2] staging: wlan-ng: move else if statement to a single line

2017-02-17 Thread Maksymilian Piechota
On Thu, Feb 16, 2017 at 10:27:24AM -0800, Greg Kroah-Hartman wrote: > On Wed, Feb 15, 2017 at 10:11:00AM -0500, Maksymilian Piechota wrote: > > move else if statement to a single line > > > > Signed-off-by: Maksymilian Piechota > > --- > >

Re: [PATCH v7 1/2] staging: wlan-ng: move else if statement to a single line

2017-02-17 Thread Maksymilian Piechota
On Thu, Feb 16, 2017 at 10:27:24AM -0800, Greg Kroah-Hartman wrote: > On Wed, Feb 15, 2017 at 10:11:00AM -0500, Maksymilian Piechota wrote: > > move else if statement to a single line > > > > Signed-off-by: Maksymilian Piechota > > --- > > drivers/staging/wlan-ng/prism2mgmt.c | 3 +-- > > 1

[PATCH] staging: octeon: remove unused variable

2017-02-17 Thread Arnd Bergmann
A cleanup patch left one local variable without a reference: drivers/staging/octeon/ethernet-rx.c:339:28: warning: unused variable 'priv' [-Wunused-variable] This removes the declaration too. Fixes: 66812da3a689 ("staging: octeon: Use net_device_stats from struct net_device") Signed-off-by:

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > Boris asked for default -a option in case we monitor > > only uncore events. While implementing that I thought > > it might be actually useful to make it

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Jiri Olsa
On Fri, Feb 17, 2017 at 11:27:47AM -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > > Boris asked for default -a option in case we monitor > > only uncore events. While implementing that I thought > > it might be actually useful to make it

Re: [PATCH 5/5] perf record: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:00:58PM +0100, Jiri Olsa escreveu: > Adding system wide (-a) option as a default target > if non is specified. > > # perf record > Warning: No target specified, setting system-wide collection (-a). Ditto, this warning will get annoying after a while... > ... >

Re: [PATCH 5/5] perf record: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:00:58PM +0100, Jiri Olsa escreveu: > Adding system wide (-a) option as a default target > if non is specified. > > # perf record > Warning: No target specified, setting system-wide collection (-a). Ditto, this warning will get annoying after a while... > ... >

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > Boris asked for default -a option in case we monitor > only uncore events. While implementing that I thought > it might be actually useful to make it overall default. > > # perf stat > Warning: No target specified, setting

Re: [PATCH 4/5] perf stat: Add -a as a default target

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 17, 2017 at 03:00:57PM +0100, Jiri Olsa escreveu: > Boris asked for default -a option in case we monitor > only uncore events. While implementing that I thought > it might be actually useful to make it overall default. > > # perf stat > Warning: No target specified, setting

[PATCHv3 21/33] x86/asm: remove __VIRTUAL_MASK_SHIFT==47 assert

2017-02-17 Thread Kirill A. Shutemov
We don't need it anymore. 17be0aec74fb ("x86/asm/entry/64: Implement better check for canonical addresses") made canonical address check generic wrt. address width. Signed-off-by: Kirill A. Shutemov --- arch/x86/entry/entry_64.S | 7 ++- 1 file changed, 2

[PATCHv3 21/33] x86/asm: remove __VIRTUAL_MASK_SHIFT==47 assert

2017-02-17 Thread Kirill A. Shutemov
We don't need it anymore. 17be0aec74fb ("x86/asm/entry/64: Implement better check for canonical addresses") made canonical address check generic wrt. address width. Signed-off-by: Kirill A. Shutemov --- arch/x86/entry/entry_64.S | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff

[PATCHv3 19/33] x86: convert the rest of the code to support p4d_t

2017-02-17 Thread Kirill A. Shutemov
This patch converts x86 to use proper folding of new page table level with . That's a bit of kitchen sink, but I don't see how to split it further. Signed-off-by: Kirill A. Shutemov --- arch/x86/include/asm/paravirt.h | 33 +-

[PATCHv3 19/33] x86: convert the rest of the code to support p4d_t

2017-02-17 Thread Kirill A. Shutemov
This patch converts x86 to use proper folding of new page table level with . That's a bit of kitchen sink, but I don't see how to split it further. Signed-off-by: Kirill A. Shutemov --- arch/x86/include/asm/paravirt.h | 33 +- arch/x86/include/asm/paravirt_types.h | 12 ++-

[PATCHv3 18/33] x86/xen: convert __xen_pgd_walk() and xen_cleanmfnmap() to support p4d

2017-02-17 Thread Kirill A. Shutemov
Split these helpers few per-level functions and add p4d support. Signed-off-by: Xiong Zhang [kirill.shute...@linux.intel.com: split off into separate patch] Signed-off-by: Kirill A. Shutemov --- arch/x86/xen/mmu.c | 243

[PATCHv3 18/33] x86/xen: convert __xen_pgd_walk() and xen_cleanmfnmap() to support p4d

2017-02-17 Thread Kirill A. Shutemov
Split these helpers few per-level functions and add p4d support. Signed-off-by: Xiong Zhang [kirill.shute...@linux.intel.com: split off into separate patch] Signed-off-by: Kirill A. Shutemov --- arch/x86/xen/mmu.c | 243 - arch/x86/xen/mmu.h

Re: [PREVIEW 10/10] linkage: add .cfi_{start/end}proc to ENTRY/ENDPROC

2017-02-17 Thread Jiri Slaby
On 02/17/2017, 03:07 PM, Josh Poimboeuf wrote: > On Fri, Feb 17, 2017 at 02:36:15PM +0100, Jiri Slaby wrote: >> On 02/17/2017, 02:16 PM, Josh Poimboeuf wrote: >>> On Fri, Feb 17, 2017 at 11:47:57AM +0100, Jiri Slaby wrote: This is just a preview, not to merged now, only later with DWARF

Re: [PREVIEW 10/10] linkage: add .cfi_{start/end}proc to ENTRY/ENDPROC

2017-02-17 Thread Jiri Slaby
On 02/17/2017, 03:07 PM, Josh Poimboeuf wrote: > On Fri, Feb 17, 2017 at 02:36:15PM +0100, Jiri Slaby wrote: >> On 02/17/2017, 02:16 PM, Josh Poimboeuf wrote: >>> On Fri, Feb 17, 2017 at 11:47:57AM +0100, Jiri Slaby wrote: This is just a preview, not to merged now, only later with DWARF

<    3   4   5   6   7   8   9   10   11   12   >