drivers/clocksource/timer-of.h:35:28: error: field 'clkevt' has incomplete type

Hi Daniel, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d0d6ab53c9abd7dd1070f43a0455328874127ba8 commit: dc11bae78529526605c5c45c369c9512fd012093 clocksource/drivers: Add timer-of common init routine date: 3 mo

Re: [PATCH][next] IB/core: fix spelling mistake: "aceess" -> "access"

On Fri, Sep 01, 2017 at 09:22:14AM +0100, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake in WARN message > > Signed-off-by: Colin Ian King > --- > drivers/infiniband/core/uverbs_ioctl_merge.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Thanks, Reviewe

Re: [PATCH v7 00/19] simplify crypto wait for async op

On Thu, Aug 31, 2017 at 3:31 PM, Harsh Jain wrote: > HI Gilad, > > I think we need an update in ESP also. Now EBUSY return means driver > has accepted, Packet should not be dropped in > > esp_output_tail() function. Good catch. You are right and the same holds true for ah_output() in ah4.c. But

Re: [PATCH] virt/kvm avoids oops by adding parameter checking

Hi nixiaoming, [auto build test ERROR on v4.13-rc6] [also build test ERROR on next-20170901] [cannot apply to kvm/linux-next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/nixiaoming/virt-kvm-a

Re: [PATCH] virt/kvm avoids oops by adding parameter checking

Hi nixiaoming, [auto build test ERROR on v4.13-rc6] [also build test ERROR on next-20170901] [cannot apply to kvm/linux-next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/nixiaoming/virt-kvm-a

Re: fs/binfmt_flat.c:828:9: error: void value not ignored as it ought to be

On 09/02/17 18:48, kbuild test robot wrote: > Hi Al, > > FYI, the error/warning still remains. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: d0d6ab53c9abd7dd1070f43a0455328874127ba8 > commit: 468138d78510688fb5476f98d23f11ac6a63229a binfmt_flat

Re: v4.11.12-rt10 - hotplug lockdep splat

On Thu, 2017-08-31 at 18:18 +0200, Sebastian Andrzej Siewior wrote: > On 2017-08-23 11:53:44 [+0200], Mike Galbraith wrote: > > virt box reminded me this morning to report this gripe. > > if you can reproduce it, then this should make it go away: Missed a spot.  With this on top, lockdep went sil

[PATCH 06/12] media: rc-sysfs-nodes.rst: better use literals

A literal box provides a better visual when pdf and html output is generated for things like the output of a sysfs devnode. It alsod matches other conventions used within the media book. So, use it. While here, use literals for protocol names. Signed-off-by: Mauro Carvalho Chehab --- Documenta

[PATCH 04/12] media: v4l uAPI docs: adjust some tables for PDF output

On tests with Spinx 1.4, some tables are still writing text outside cells. Adjust those tables. PS.: As this was revisited several times, I suspect that this will only be fully fixed if we add tabularcolumns to all tables at the V4L2 part of the book. Signed-off-by: Mauro Carvalho Chehab --- Do

[PATCH 10/12] media: v4l2-event.rst: adjust table to fit on PDF output

The tables there don't quite fit on PDF output. Adjust it by adding a tabularcolumns macro. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/kapi/v4l2-event.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/media/kapi/v4l2-event.rst b/Documentation/media/kapi/

[PATCH 09/12] media: docs: don't show ToC for each part on PDF output

The "Table of Contents" of a PDF file is generated only once, at the beginning fo the output. It doesn't produce it on each part. So, don't output this text on each part of the document. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/cec-drivers/index.rst | 4 +++- Do

[PATCH 02/12] media: vidioc-querycap: use a more realistic value for KERNEL_VERSION

In the past, V4L2 versions were 0.x.y, but that changed years ago. Since Kernel 3.1, however, the numbering schema was changed to match the Kernel version. However, the presented example still uses the old numerating schema, with is a misleading information. So, update it to the new schema. Sign

[PATCH 03/12] media: vidioc-g-tuner.rst: Fix table number of cols

The Tuner Audio Matrix table is broken: the first row has 7 columns instead of 6, causing it to be parsed wrong and displayed very badly on PDF output. Fix it and adjust the table to look nice at PDF output Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/vidioc-g-tuner.rst

[PATCH 00/12] media documentation ReST fixes

This series contain media documentation ReST fixes, mostly related to table parsing and the corresponding PDF output (tested with Sphinx 1.4). It also contains a few documentation trivial improvements, from a few minor issues I noticed when reading the PDF output text on some places. Probably the

[PATCH 01/12] media: v4l uAPI: add descriptions for arguments to all ioctls

Several ioctls are missing descriptions for the third argument of the ioctl() command. They should have a description, as otherwise the output won't be ok, and will sound like something is missing. So, add them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/vidioc-create

[PATCH 11/12] media: em28xx-cardlist.rst: update to reflect last changes

There were some new board additions. Update the cardlist accordingly. Signed-off-by: Mauro Carvalho Chehab --- .../media/v4l-drivers/em28xx-cardlist.rst | 210 +++-- 1 file changed, 106 insertions(+), 104 deletions(-) diff --git a/Documentation/media/v4l-drivers/em28xx-

[PATCH 05/12] media: docs: fix PDF build with Sphinx 1.4

changeset 70b074df4ed1 ("media: fix pdf build with Spinx 1.6") caused a regression at Sphinx 1.4 PDF build: although it produces a full document in batch mode, it returns errors on interactive mode: [63] Runaway argument? {\relax ! Paragraph ended before \multicolum

[PATCH 07/12] media: mc uapi: adjust some table sizes for PDF output

Some cells are too small to fit the text written to it. Increase it. No text changes. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/mediactl/media-ioc-enum-entities.rst | 2 +- Documentation/media/uapi/mediactl/media-ioc-g-topology.rst| 2 +- Documentation/media/uapi/med

[PATCH 12/12] media: docs-rst: cardlists: change their format to flat-tables

Most tables there don't fit on 80 columns. Some are very big. While those tables are actually generated via scripts, every time a new board is added, the entire table could be reformatted. >From the diffstat PoV, that's bad, as it is hard to see what happened. One such example is at changeset 486

[PATCH 08/12] media: cec uapi: Adjust table sizes for PDF output

Several tables at this media book chapter have issues when PDF is produced. Adjust them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/cec/cec-ioc-adap-g-log-addrs.rst | 7 +-- Documentation/media/uapi/cec/cec-ioc-dqevent.rst | 9 + Documentation/media/ua

Re: [RFC Part1 PATCH v3 16/17] X86/KVM: Provide support to create Guest and HV shared per-CPU variables

On 9/1/17 10:21 PM, Andy Lutomirski wrote: > On Fri, Sep 1, 2017 at 3:52 PM, Brijesh Singh wrote: >> Hi Boris, >> >> On 08/30/2017 12:46 PM, Borislav Petkov wrote: >>> On Wed, Aug 30, 2017 at 11:18:42AM -0500, Brijesh Singh wrote: I was trying to avoid mixing early and no-early set_memory_d

[PATCH] X.509: Fix the buffer overflow in the utility function for OID string

From: Takashi Iwai The sprint_oid() utility function doesn't properly check the buffer size that it causes that the warning in vsnprintf() be triggered. For example on v4.1 kernel: [ 49.612536] [ cut here ] [ 49.612543] WARNING: CPU: 0 PID: 2357 at lib/vsprintf.c:1867

fs/binfmt_flat.c:828:9: error: void value not ignored as it ought to be

Hi Al, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: d0d6ab53c9abd7dd1070f43a0455328874127ba8 commit: 468138d78510688fb5476f98d23f11ac6a63229a binfmt_flat: flat_{get,put}_addr_from_rp() should be able to fail date

[RFC v2 0/6] PMC/PUNIT IPC driver cleanup

From: Kuppuswamy Sathyanarayanan Hi All, Currently intel_pmc_ipc.c, intel_punit_ipc.c, intel_scu_ipc.c drivers implements the same IPC features. This code duplication could be avoided if we implement the IPC driver as a generic library and let custom device drivers use API provided by generic

[RFC v2 3/6] platform/x86: intel_pmc_ipc: Use regmap calls for GCR updates

From: Kuppuswamy Sathyanarayanan Currently, update_no_reboot_bit() function implemented in this driver uses mutex_lock to protect its register updates. But this function is called with in atomic context in iTCO_wdt_start() and iTCO_wdt_stop() functions in iTCO_wdt.c driver, which in turn causes "

[RFC v2 4/6] platform: x86: Add generic Intel IPC driver

From: Kuppuswamy Sathyanarayanan Currently intel_scu_ipc.c, intel_pmc_ipc.c and intel_punit_ipc.c redundantly implements the same IPC features and has lot of code duplication between them. This driver addresses this issue by grouping the common IPC functionalities under the same driver. Signed-o

[RFC v2 6/6] platform/x86: intel_pmc_ipc: Use generic Intel IPC device calls

From: Kuppuswamy Sathyanarayanan Removed redundant IPC helper functions and refactored the driver to use generic IPC device driver APIs. This patch also cleans-up PMC IPC user drivers to use APIs provided by generic IPC driver. Signed-off-by: Kuppuswamy Sathyanarayanan --- arch/x86/include/a

[RFC v2 5/6] platform/x86: intel_punit_ipc: Use generic intel ipc device calls

From: Kuppuswamy Sathyanarayanan Removed redundant IPC helper functions and refactored the driver to use APIs provided by generic IPC driver. This patch also cleans-up PUNIT IPC user drivers to use APIs provided by generic IPC driver. Signed-off-by: Kuppuswamy Sathyanarayanan --- arch/x86/inc

[RFC v2 2/6] platform/x86: intel_pmc_ipc: Use MFD framework to create dependent devices

From: Kuppuswamy Sathyanarayanan Currently, we have lot of repetitive code in dependent device resource allocation and device creation handling code. This logic can be improved if we use MFD framework for dependent device creation. This patch adds this support. Signed-off-by: Kuppuswamy Sathyana

[RFC v2 1/6] platform/x86: intel_pmc_ipc: Use devm_* calls in driver probe function

From: Kuppuswamy Sathyanarayanan This patch cleans up unnecessary free/alloc calls in ipc_plat_probe(), ipc_pci_probe() and ipc_plat_get_res() functions by using devm_* calls. This patch also adds proper error handling for failure cases in ipc_pci_probe() function. Signed-off-by: Kuppuswamy Sat

Re: [PATCH] ACPICA: Check whether ACPI is disabled before getting a table

Hi Borislav, [auto build test ERROR on pm/linux-next] [also build test ERROR on v4.13-rc7 next-20170901] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Borislav-Petkov/ACPICA-Check-whether-ACPI-

[PATCH] kernel: Emphasize the return value of READ_ONCE() is honored

READ_ONCE() is used around in kernel to provide a control dependency, and to make the control dependency valid, we must 1) make the load of READ_ONCE() actually happen and 2) make sure compilers take the return value of READ_ONCE() serious. 1) is already done and commented, and in current implement

[PATCH 1/2] ARM: dts: omap3-evm-37xx: Add common processor module support

This commit moves common OMAP3-EVM processor module device tree data to a separate include file. This will allow for 'omap3-evm.dts' to use device tree data that is unique to the OMAP3530 version of the processor module, while making use of the work already done for the 'omap3-evm-37xx.dts'. --- a

[PATCH 0/2] Update TMDSEVM3530 support for omap3-evm

This patch set allows TMDSEVM3530(omap3-evm.dts) to boot using common processor module data that is shared with 'omap3-evm-37xx.dts'. A new common file for processor module data is introduced to help facilitate the updated OMAP3530 support. Derald D. Woods (2): ARM: dts: omap3-evm-37xx: Add comm

[PATCH 2/2] ARM: dts: omap3-evm: Add OMAP3530 specific device tree processor data

This commit allows OMAP3530 variants to use common data that is available in 'omap3-evm-processor-common.dtsi'. It adds proper pinmux macros for 'omap3_pmx_core2' on OMAP3430. The Micron NAND chip is also added for the TMDSEVM3530 processor module. --- arch/arm/boot/dts/omap3-evm.dts | 76

[PATCH] workqueue: Fix flag collision

Commit 0a94efb5acbb ("workqueue: implicit ordered attribute should be overridable") introduced a __WQ_ORDERED_EXPLICIT flag but gave it the same value as __WQ_LEGACY. I don't believe these were intended to mean the same thing, so renumber __WQ_ORDERED_EXPLICIT. Fixes: 0a94efb5acbb ("workqueue: im

Re: [PATCH 0/3] Add missing types to checkpatch.pl --list-types

On Sat, 2017-09-02 at 23:03 +0200, Jean Delvare wrote: > Oh, btw, is there any purpose in listing a number in front of each > type? Not really. I did that because I wanted a header and because I had no idea how many of those types existed and I was too lazy to count. > It makes it look like one

Re: [I2C] About warning 'DMA-API: device driver maps memory from stack'

Hi, nice to see someone else interested in the I2C & DMA topic. Please check this series which I sent out recently: "[RFC PATCH v4 0/6] i2c: document DMA handling and add helpers for it" In that series, I proposed... > 3. kmalloc data buffer instead of local variables buf in function > i2c_smb

Re: [PATCH 0/3] Add missing types to checkpatch.pl --list-types

Oh, btw, is there any purpose in listing a number in front of each type? It makes it look like one can pass that number instead of the type name, however it doesn't work, and I don't think it should as the numbering isn't stable and could change with any update of the script. Can't we simplify the

[PATCH 7/7] [media] Mantis: Delete an unnecessary variable initialisation in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 22:05:22 +0200 The variable "err" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/mantis_cards.c | 2 +- 1 file changed, 1 i

[PATCH 6/7] [media] Mantis: Improve a size determination in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:56:07 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 5/7] [media] Mantis: Delete an error message for a failed memory allocation in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:55:05 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/mantis_cards.c | 4 +--- 1 file changed, 1 ins

[PATCH 4/7] [media] Hopper: Delete an unnecessary variable initialisation in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:50:55 +0200 The variable "err" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/hopper_cards.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 3/7] [media] Hopper: Adjust a null pointer check in two functions

From: Markus Elfring Date: Sat, 2 Sep 2017 21:25:50 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code pla

[PATCH 2/7] [media] Hopper: Improve a size determination in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:21:22 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/7] [media] Hopper: Delete an error message for a failed memory allocation in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:19:05 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/hopper_cards.c | 1 - 1 file changed, 1 deleti

[PATCH 0/7] [media] Mantis: Adjustments for three function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 22:18:22 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (7): Delete an error message for a failed memory allocation in hopper_pci_probe() Improve a size determination in hopper_pci_probe() A

[PATCH] serial: sccnxp: Fix error handling in sccnxp_probe()

sccnxp_probe() returns result of regulator_disable() that may lead to returning zero, while device is not properly initialized. Also the driver enables clocks, but it does not disable it. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- driver

[PATCH v2 2/2] locking/refcounts, x86/asm: Enable CONFIG_ARCH_HAS_REFCOUNT

With the section inlining bug fixed for the x86 refcount protection, we can turn the config back on. Cc: Mike Galbraith Signed-off-by: Kees Cook --- arch/x86/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index eaa8ff41f424..c6acd

[PATCH v2 0/2] locking/refcounts, x86/asm: Use unique .text section for refcount exceptions

This splits the earlier patch to have the Kconfig enablement separate, as requested by Ingo. Thanks! -Kees

[PATCH v2 1/2] locking/refcounts, x86/asm: Use unique .text section for refcount exceptions

Using .text.unlikely for refcount exceptions isn't safe because gcc may move entire functions into .text.unlikely (e.g. in6_dev_dev()), which would cause any uses of a protected refcount_t function to stay inline with the function, triggering the protection unconditionally: .section

[PATCH v6 03/12] ASoC: add new ac97 bus support

Add the new ac97 bus support, with ac97 bus automatic probing. Signed-off-by: Robert Jarzmik Acked-by: Charles Keepax --- sound/Kconfig | 2 ++ sound/Makefile | 1 + 2 files changed, 3 insertions(+) diff --git a/sound/Kconfig b/sound/Kconfig index ee2e69a9ecd1..141b145c4195 100644 --- a/sound

[PATCH v6 02/12] ALSA: ac97: add an ac97 bus

AC97 is a bus for sound usage. It enables for a AC97 AC-Link to link one controller to 0 to 4 AC97 codecs. The goal of this new implementation is to implement a device/driver model for AC97, with an automatic scan of the bus and automatic discovery of AC97 codec devices. Signed-off-by: Robert Jar

[PATCH v6 00/12] AC97 device/driver model revamp

Hi Lars, Mark, Charles, Lee, This is a revision for Lee, only targetted at wm97xx-core changes suggested by Lee. I hope I have every concern covered Lee, just le me know if it suits you. For easier spotting, I included in [1] the diff from the last serie. I kept Charles ack for patch 6/12 follow

[PATCH v6 01/12] ALSA: ac97: split out the generic ac97 registers

Split out from the ac97_codec.h the ac97 generic registers, which can be used by a codec, typically a generic ac97 codec, and by the ac97 bus, to scan an ac97 AC-Link. This split encompasses all the AC97 standard registers, but not the codec specific ones. In order to have a clean split between f

[PATCH v6 05/12] Input: wm97xx: split out touchscreen registering

wm97xx-core does several things in it initialization : - touchscreen input device setup - battery device creation As the wm97xx is actually a multi-function device handling an audio codec, a touchscreen, a gpio block and an ADC, reshape the probing to isolate what is truly input/touchscreen spec

[PATCH v6 09/12] ASoC: wm9712: add ac97 new bus support

Add support for the new ac97 bus model, where devices are automatically discovered on AC-Links. Signed-off-by: Robert Jarzmik --- Since v4: - added #ifdef spotted by Charles --- sound/soc/codecs/Kconfig | 3 ++- sound/soc/codecs/wm9712.c | 39 ++- 2 files c

[PATCH v6 07/12] Input: wm97xx: add new AC97 bus support

This adds support for the new AC97 bus code, which discovers the devices rather than uses platform data. As part of this discovery, it enables a multi-function device wm97xx, which supports touchscreen, battery, ADC and an audio codec. This patch adds the code to bind the touchscreen "cell" as the

[PATCH v6 08/12] ASoC: wm9713: add ac97 new bus support

Add support for the new ac97 bus model, where devices are automatically discovered on AC-Links. Signed-off-by: Robert Jarzmik Acked-by: Charles Keepax --- sound/soc/codecs/Kconfig | 3 ++- sound/soc/codecs/wm9713.c | 39 +++ 2 files changed, 29 insertions(+

[PATCH v6 12/12] ASoC: pxa: switch to new ac97 bus support

Switch to the new ac97 bus support in sound/ac97 instead of the legacy snd_ac97 one. Signed-off-by: Robert Jarzmik --- Since v1: split into 2 patches, the former being XXX ac97 codec agnostic Since v2: fix driver unregistration Since v3: added platform driver data for controller removal --- soun

[PATCH v6 10/12] ASoC: wm9705: add private structure

Add a private data structure. This is a preparation for a codec which would need an another data on top of snd_ac97, which will be the case when an MFD wm97xx device will probe wm9705. Signed-off-by: Robert Jarzmik Acked-by: Charles Keepax --- sound/soc/codecs/wm9705.c | 36

[PATCH v6 11/12] ASoC: wm9705: add ac97 new bus support

Add support for the new ac97 bus model, where devices are automatically discovered on AC-Links. Signed-off-by: Robert Jarzmik Acked-by: Charles Keepax --- Since v4: - added #ifdef spotted by Charles - tentatively kept Charles's ack --- sound/soc/codecs/Kconfig | 3 ++- sound/soc/codecs/wm97

[PATCH v6 06/12] mfd: wm97xx-core: core support for wm97xx Codec

The WM9705, WM9712 and WM9713 are highly integrated codecs, with an audio codec, DAC and ADC, GPIO unit and a touchscreen interface. Historically the support was spread across drivers/input/touchscreen and sound/soc/codecs. The sharing was done through ac97 bus sharing. This model will not withsta

[PATCH v6 04/12] ASoC: arm: make pxa2xx-ac97-lib ac97 codec agnostic

All pxa library functions don't use the input parameters for nothing but slot number. This simplifies their prototypes, and makes them usable by both the legacy ac97 bus and the new ac97 bus. Signed-off-by: Robert Jarzmik --- include/sound/pxa2xx-lib.h | 15 +-- sound/arm/pxa2xx-ac9

Re: [PATCH] locking/refcounts, x86/asm: Use unique .text section for refcount exceptions

On Sat, Sep 2, 2017 at 3:29 AM, Ingo Molnar wrote: > > * Kees Cook wrote: > >> Using .text.unlikely for refcount exceptions isn't safe because gcc may >> move entire functions into .text.unlikely (e.g. in6_dev_get()), which >> would cause any uses of a protected refcount_t function to stay inline

Re: [PATCH][DEBUG] x86/refcount: split up refcount saturation handling

On Sat, Sep 2, 2017 at 3:35 AM, Ingo Molnar wrote: > > * Kees Cook wrote: > >> In support of debugging the problems Mike Galbraith has seen with >> x86-refcount vs gcc vs network refcounts... >> >> This minimizes the differences between unchecked-refcount and x86-refcount >> by changing the refco

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

Hi, On 02-09-17 21:06, Guenter Roeck wrote: On Sat, Sep 02, 2017 at 05:59:14PM +0200, Hans de Goede wrote: Hi, On 02-09-17 16:59, Guenter Roeck wrote: On 09/01/2017 02:48 PM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C polarity/r

Re: [PATCH 0/3] Add missing types to checkpatch.pl --list-types

Hi Joe, On Sat, 02 Sep 2017 09:25:29 -0700, Joe Perches wrote: > On Sat, 2017-09-02 at 17:51 +0200, Jean Delvare wrote: > > [PATCH 1/3] checkpatch: fix typo in comment > > [PATCH 2/3] checkpatch: rename variables to avoid confusion > > [PATCH 3/3] checkpatch: add 6 missing types to --list-types

Re: [PATCH 02/11] mux: core: Add support for getting a mux controller on a non DT platform

Hi, On 09/01/2017 02:48 PM, Hans de Goede wrote: On non DT platforms we cannot get the mux_chip by pnode. Other subsystems (regulator, clock, pwm) have the same problem and solve this by allowing platform / board-setup code to add entries to a lookup table and then use this table to look things

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

On Sat, Sep 02, 2017 at 05:59:14PM +0200, Hans de Goede wrote: > Hi, > > On 02-09-17 16:59, Guenter Roeck wrote: > >On 09/01/2017 02:48 PM, Hans de Goede wrote: > >>Add MUX_USB_* state constant defines, which can be used by USB > >>device/host and Type-C polarity/role/altmode mux drivers and consu

[PATCH 3/3] [media] cx18: Adjust ten checks for null pointers

From: Markus Elfring Date: Sat, 2 Sep 2017 19:49:23 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus fix the affected source code plac

[PATCH 2/3] [media] cx18: Improve a size determination in cx18_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 19:42:12 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/3] [media] cx18: Delete an error message for a failed memory allocation in cx18_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 19:39:56 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/cx18/cx18-driver.c | 6 ++ 1 file changed, 2 inse

[PATCH 0/3] [media] cx18: Adjustments for seven function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 20:03:45 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation Improve a size determination Adjust ten checks for null pointers drivers

Re: [PATCH] usb: gadget: pch_udc: add checks for dma mapping errors

On 24.08.2017 04:02, Jack Pham wrote: > On Thu, Aug 24, 2017 at 01:47:08AM +0300, Alexey Khoroshilov wrote: >> There are no checks for dma mapping errors in pch_udc. >> Tha patch adds the checks and error handling code. >> Compile tested only. >> >> Found by Linux Driver Verification project (linux

Re: [PATCH 19/25] can/bcm: Replace hrtimer_tasklet with softirq based hrtimer

On 08/31/2017 02:23 PM, Anna-Maria Gleixner wrote: From: Thomas Gleixner Switch the timer to CLOCK_MONOTONIC_SOFT, which executed the timer callback in softirq context and remove the hrtimer_tasklet. Signed-off-by: Thomas Gleixner Signed-off-by: Anna-Maria Gleixner Cc: Oliver Hartkopp Ack

Re: [PATCH v8 02/28] x86/boot: Relocate definition of the initial state of CR0

On Thu, 2017-08-31 at 11:51 +0200, Borislav Petkov wrote: > On Wed, Aug 30, 2017 at 09:04:18PM -0700, Ricardo Neri wrote: > > Thank you! Is it necessary for me to submit a v9 with these updates? > > Perhaps I can make these updates in branch for the maintainers to pull > > when/if this series is ac

Re: [PATCH v2] PCIe AER: report uncorrectable errors only to the functions that logged the errors

On Fri, Sep 01, 2017 at 11:39:35AM +, Gabriele Paoloni wrote: > Hi Bjorn > > Many thanks for looking at this > > > -Original Message- > > From: Bjorn Helgaas [mailto:helg...@kernel.org] > > Sent: 01 September 2017 05:43 > > To: Gabriele Paoloni > > Cc: Linuxarm; liudongdong (C); linux

[PATCH] pinctrl: uniphier: include instead of

The #includes is here to use BUILD_BUG_ON_ZERO(). Thanks to commit bc6245e5efd7 ("bug: split BUILD_BUG stuff out into "), it is now possible to reduce the number of headers pulled in. Signed-off-by: Masahiro Yamada --- drivers/pinctrl/uniphier/pinctrl-uniphier.h | 2 +- 1 file changed, 1 inse

Re: [PATCH v8 06/28] x86/mpx: Do not use SIB.index if its value is 100b and ModRM.mod is not 11b

On Thu, 2017-08-31 at 21:38 +0200, Borislav Petkov wrote: > On Fri, Aug 18, 2017 at 05:27:47PM -0700, Ricardo Neri wrote: > > Section 2.2.1.2 of the Intel 64 and IA-32 Architectures Software > > Developer's Manual volume 2A states that when ModRM.mod !=11b and > > ModRM.rm = 100b indexed register-i

Re: RFC: Revert move default dialect from CIFS to to SMB3

On Fri, Sep 1, 2017 at 10:22 PM, Andrew Bartlett wrote: > > My quick research shows: > > SMB 2.1 but not SMB3 is on: > Windows 7 > Windows 8 > Windows 2008 > Windows 2012 > Samba 3.6 and earlier (SMB1 only by default) > > SMB3 is on: > Windows 8.1 > Windows 2012 R2 > Windows 10 > Windows

Re: printk: what is going on with additional newlines?

On Fri, Sep 1, 2017 at 11:12 PM, Tetsuo Handa wrote: > > I just tried to distinguish context using one "unsigned long" value > by embedding IRQ status into lower bits of "struct task_struct *". > I can change to distinguish context using multiple "unsigned long" values. I really really don't thin

Re: linux-next: Signed-off-by missing for commit in the rdma tree

On Fri, Sep 1, 2017 at 5:06 AM, Doug Ledford wrote: > On 8/31/2017 5:50 PM, Stephen Rothwell wrote: >> Hi Doug, >> >> Commit >> >> 4b9796b0a6fb ("IB/hfi1: Use accessor to determine ring size") >> >> is missing a Signed-off-by from its author. >> > > What's the best way to fix this? I can rebase

Re: [PATCH net-next 0/4] net: dsa: add master interface

Hi Vivien, On 08/31/2017 11:37 AM, Vivien Didelot wrote: > Currently the SoC network interface (called master) to which a switch > fabric hangs, has its dsa_ptr pointing to a dsa_switch_tree instance. > > This is not quite correct, because this interface is physically wired to > one of the switch

Re: [PATCH net-next 4/4] net: dsa: assign a master to slave ports

On 09/02/2017 08:34 AM, Andrew Lunn wrote: > On Thu, Aug 31, 2017 at 02:37:46PM -0400, Vivien Didelot wrote: >> Because each DSA slave port may use a different DSA master port, add a >> pointer to a master in the slave structure. This is a preparatory patch >> for multiple CPU ports. >> >> Signed

Re: [PATCH 1/2] ARM: dts: Add the CLCD controller to Cygnus.

On 09/02/2017 08:48 AM, Scott Branden wrote: > Looks good > > On Sep 1, 2017 7:10 PM, "Florian Fainelli" > wrote: > > > > On 08/31/2017 01:16 PM, Scott Branden wrote: > > Hi Eric, > > > > mode is correct, location in file needs to be moved. >

Re: [PATCH 0/3] Add missing types to checkpatch.pl --list-types

On Sat, 2017-09-02 at 17:51 +0200, Jean Delvare wrote: > [PATCH 1/3] checkpatch: fix typo in comment > [PATCH 2/3] checkpatch: rename variables to avoid confusion > [PATCH 3/3] checkpatch: add 6 missing types to --list-types Andrew, can you please pick these up? Thanks. Jean, that's a pretty obs

[PATCH v2] platform/x86: intel_cht_int33fe: Work around BIOS bug on some devices

At least one BIOS enumerates the max17047 both through the INT33FE ACPI device (it is right there in the resources table) as well as through a separate MAX17047 device. This commit checks for the max17047 already being enumerated through a separate MAX17047 ACPI device and if so it uses the i2c-cl

Re: [PATCH net] net: dsa: loop: Do not unregister invalid fixed PHY

On Sat, Sep 02, 2017 at 08:56:45AM -0700, Florian Fainelli wrote: > During error injection it was possible to crash in dsa_loop_exit() because of > an attempt to unregister an invalid PHY. We actually want to the driver > probing > in dsa_loop_init() even though fixed_phy_register() may return an

Re: [PATCH 03/11] mux: consumer.h: Add MUX_USB_* state constant defines

Hi, On 02-09-17 16:59, Guenter Roeck wrote: On 09/01/2017 02:48 PM, Hans de Goede wrote: Add MUX_USB_* state constant defines, which can be used by USB device/host and Type-C polarity/role/altmode mux drivers and consumers to ensure that they agree on the meaning of the mux_control_select() sta

[PATCH net] net: dsa: loop: Do not unregister invalid fixed PHY

During error injection it was possible to crash in dsa_loop_exit() because of an attempt to unregister an invalid PHY. We actually want to the driver probing in dsa_loop_init() even though fixed_phy_register() may return an error to exercise how DSA deals with such cases, but we should not be crash

[PATCH 3/3] checkpatch: add 6 missing types to --list-types

Unlike all other types, LONG_LINE, LONG_LINE_COMMENT and LONG_LINE_STRING are passed to WARN() through a variable. This causes the parser in list_types() to miss them and consequently they are not present in the output of --list-types. Additionally, types TYPO_SPELLING, FSF_MAILING_ADDRESS and AVO

[PATCH 7/7] [media] ov9740: Improve a size determination in ov9740_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:46:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 2/3] checkpatch: rename variables to avoid confusion

The variable name "$msg_type" is sometimes used to set the message type, and sometimes used to set the message level. This works but is kind of confusing. Use "$msg_level" in the latter case instead, to make the code clearer. Signed-off-by: Jean Delvare Cc: Andy Whitcroft Cc: Joe Perches --- s

[PATCH 1/3] checkpatch: fix typo in comment

Signed-off-by: Jean Delvare Cc: Andy Whitcroft Cc: Joe Perches --- scripts/checkpatch.pl |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- linux-4.13-rc7.orig/scripts/checkpatch.pl 2017-07-30 21:40:36.0 +0200 +++ linux-4.13-rc7/scripts/checkpatch.pl2017-09-01 15:4

[PATCH 6/7] [media] ov9740: Delete an error message for a failed memory allocation in ov9740_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:44:36 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/soc_camera/ov9740.c | 4 +--- 1 file changed, 1 inser

[PATCH 5/7] [media] ov9640: Improve a size determination in ov9640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:37:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 0/3] Add missing types to checkpatch.pl --list-types

[PATCH 1/3] checkpatch: fix typo in comment [PATCH 2/3] checkpatch: rename variables to avoid confusion [PATCH 3/3] checkpatch: add 6 missing types to --list-types --- scripts/checkpatch.pl | 37 +++-- 1 file changed, 19 insertions(+), 18 deletions(-) -- Jean Del

Re: [PATCH 4.9 01/12] p54: memset(0) whole array

On Thu, 2017-08-31 at 09:40 -0700, Joe Perches wrote: > On Thu, 2017-08-31 at 17:44 +0200, Greg Kroah-Hartman wrote: > > 4.9-stable review patch. If anyone has any objections, please let me know. > > > > -- > > > > From: Jiri Slaby > > > > commit 6f17581788206444cbbcdbc107498f8

[PATCH 4/7] [media] ov9640: Delete an error message for a failed memory allocation in ov9640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:34:27 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/soc_camera/ov9640.c | 5 + 1 file changed, 1 inse

  1   2   >