Re: [PATCH 4.4 000/124] 4.4.146-stable review

2018-08-04 Thread Nathan Chancellor
On Sat, Aug 04, 2018 at 10:59:49AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.146 release. > There are 124 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/32] 4.9.118-stable review

2018-08-04 Thread Nathan Chancellor
On Sat, Aug 04, 2018 at 11:00:50AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.118 release. > There are 32 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 00/17] 4.9.119-stable review

2018-08-07 Thread Nathan Chancellor
On Tue, Aug 07, 2018 at 08:51:36PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.119 release. > There are 17 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/12] 4.4.147-stable review

2018-08-07 Thread Nathan Chancellor
On Tue, Aug 07, 2018 at 08:52:04PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.147 release. > There are 12 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 3.18 00/85] 3.18.118-stable review

2018-08-07 Thread Nathan Chancellor
On Tue, Aug 07, 2018 at 08:51:02PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.118 release. > There are 85 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/39] 4.9.110-stable review

2018-06-24 Thread Nathan Chancellor
On Sun, Jun 24, 2018 at 11:23:47PM +0800, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.110 release. > There are 39 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH] Trivial numbering change in comments.

2018-08-28 Thread Nathan Chancellor
for adding comments like what changed between versions of the patch or maybe something like "I'm not sure this change is correct, it could also be done via , I'd like some review". Small nits in the grand scheme of things but they'll come in handy as you develop more and more complex patches and series. Reviewed-by: Nathan Chancellor Cheers! Nathan

Re: [PATCH 4.9 00/25] 4.9.123-stable review

2018-08-21 Thread Nathan Chancellor
On Tue, Aug 21, 2018 at 08:21:14AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.123 release. > There are 25 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 000/130] 4.9.124-stable review

2018-08-23 Thread Nathan Chancellor
On Thu, Aug 23, 2018 at 09:51:56AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.124 release. > There are 130 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 3.18 00/56] 3.18.120-stable review

2018-08-26 Thread Nathan Chancellor
On Sun, Aug 26, 2018 at 08:44:21AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.120 release. > There are 56 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 3.18 00/27] 3.18.117-stable review

2018-07-27 Thread Nathan Chancellor
On Fri, Jul 27, 2018 at 12:26:37PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.117 release. > There are 27 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 00/23] 4.4.145-stable review

2018-07-27 Thread Nathan Chancellor
On Fri, Jul 27, 2018 at 12:09:01PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.145 release. > There are 23 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 00/33] 4.9.116-stable review

2018-07-27 Thread Nathan Chancellor
On Fri, Jul 27, 2018 at 12:08:41PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.116 release. > There are 33 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH] kernel.h: Disable -Wreturn-stack-address for _THIS_IP_

2018-07-30 Thread Nathan Chancellor
On Mon, Jul 30, 2018 at 10:06:20AM -0700, Nick Desaulniers wrote: > Starting with Clang-7.0, _THIS_IP_ generates -Wreturn-stack-address > warnings for almost every translation unit. In general, I'd prefer to > leave this on (returning the address of a stack allocated variable is in > general a bad

Re: [PATCH] kernel.h: Disable -Wreturn-stack-address for _THIS_IP_

2018-07-30 Thread Nathan Chancellor
On Mon, Jul 30, 2018 at 12:48:06PM -0700, Nick Desaulniers wrote: > On Mon, Jul 30, 2018 at 11:39 AM Nathan Chancellor > wrote: > > > > On Mon, Jul 30, 2018 at 10:06:20AM -0700, Nick Desaulniers wrote: > > > Starting with Clang-7.0, _THIS_IP_ generates -Wreturn-stack

Re: [PATCH 3.18 00/29] 3.18.116-stable review

2018-07-20 Thread Nathan Chancellor
On Fri, Jul 20, 2018 at 02:10:55PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.116 release. > There are 29 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 00/31] 4.4.143-stable review

2018-07-20 Thread Nathan Chancellor
On Fri, Jul 20, 2018 at 02:13:30PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.143 release. > There are 31 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 00/66] 4.9.114-stable review

2018-07-20 Thread Nathan Chancellor
On Fri, Jul 20, 2018 at 02:13:17PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.114 release. > There are 66 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH v2 2/2] kernel.h: Disable -Wreturn-stack-address for _THIS_IP_

2018-07-30 Thread Nathan Chancellor
xpression or macro, hence we use _Pragma, which is > its raison d'ĂȘtre: https://gcc.gnu.org/onlinedocs/cpp/Pragmas.html > > We also can't use compiler specific pragma's without triggering > -Werror=unknown-pragmas in other compilers, so use __diag. > > Cc: sta...@

Re: [PATCH v2 1/2] compiler-clang.h: Add CLANG_VERSION and __diag macros

2018-07-30 Thread Nathan Chancellor
l.org # 4.17, 4.14, 4.9, 4.4 > Suggested-by: Nathan Chancellor > Signed-off-by: Nick Desaulniers Tested-and-reviewed-by: Nathan Chancellor > --- > include/linux/compiler-clang.h | 19 +++ > include/linux/compiler_types.h | 4 > 2 files changed, 23 insertion

Re: [PATCH v2 2/2] kernel.h: Disable -Wreturn-stack-address for _THIS_IP_

2018-07-31 Thread Nathan Chancellor
On Tue, Jul 31, 2018 at 09:48:57AM -0700, Nick Desaulniers wrote: > Does anyone understand this error? The code looks just fine to me, > and the source file doesn't conflict with any of the macros I've added > (certainly not in any way that could cause an indentation error as > reported here). Un

Re: [PATCH 4.9 000/144] 4.9.117-stable review

2018-08-01 Thread Nathan Chancellor
On Wed, Aug 01, 2018 at 06:50:27PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.117 release. > There are 144 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 000/107] 4.4.144-stable review

2018-07-23 Thread Nathan Chancellor
On Mon, Jul 23, 2018 at 02:40:54PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.144 release. > There are 107 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/28] 4.9.115-stable review

2018-07-23 Thread Nathan Chancellor
On Mon, Jul 23, 2018 at 02:25:00PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.115 release. > There are 28 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/53] 4.4.113-stable review

2018-01-22 Thread Nathan Chancellor
On Mon, Jan 22, 2018 at 09:39:52AM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.113 release. > There are 53 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/22] 4.4.111-stable review

2018-01-08 Thread Nathan Chancellor
On Mon, Jan 08, 2018 at 01:59:27PM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.111 release. > There are 22 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/74] 4.4.114-stable review

2018-01-29 Thread Nathan Chancellor
On Mon, Jan 29, 2018 at 01:56:05PM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.114 release. > There are 74 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/87] 4.4.112-stable review

2018-01-15 Thread Nathan Chancellor
On Mon, Jan 15, 2018 at 01:33:59PM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.112 release. > There are 87 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/36] 4.4.121-stable review

2018-03-09 Thread Nathan Chancellor
On Fri, Mar 09, 2018 at 04:18:16PM -0800, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.121 release. > There are 36 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/36] 4.4.121-stable review

2018-03-09 Thread Nathan Chancellor
On Fri, Mar 09, 2018 at 05:03:12PM -0800, Greg Kroah-Hartman wrote: > On Fri, Mar 09, 2018 at 05:50:35PM -0700, Nathan Chancellor wrote: > > On Fri, Mar 09, 2018 at 04:18:16PM -0800, Greg Kroah-Hartman wrote: > > > This is the start of the stable review cycle for the 4.4.121 re

Re: [PATCH] staging: android: ashmem: Remove deadlock

2018-03-07 Thread Nathan Chancellor
out_unlock; > - } > - > /* per custom, you can pass zero for len to mean "everything onward" */ > if (!pin.len) > pin.len = PAGE_ALIGN(asma->size) - pin.offset; > -- > 2.16.2.395.g2e18187dfd-goog > Hey Paul, Looks like this same patch is already in Greg's tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?id=740a5759bf222332fbb5eda42f89aa25ba38f9b2 Cheers! Nathan Chancellor

Re: [PATCH 4.4 00/72] 4.4.127-stable review

2018-04-06 Thread Nathan Chancellor
On Fri, Apr 06, 2018 at 03:23:01PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.127 release. > There are 72 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 000/190] 4.4.128-stable review

2018-04-11 Thread Nathan Chancellor
On Wed, Apr 11, 2018 at 08:34:06PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.128 release. > There are 190 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 000/134] 4.4.123-stable review

2018-03-19 Thread Nathan Chancellor
On Mon, Mar 19, 2018 at 07:04:43PM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.123 release. > There are 134 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 000/134] 4.4.123-stable review

2018-03-20 Thread Nathan Chancellor
On Tue, Mar 20, 2018 at 06:32:00AM -0700, Guenter Roeck wrote: > On 03/19/2018 11:04 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 4.4.123 release. > > There are 134 patches in this series, all will be posted as a response > > to this one. If anyone has any

Re: [PATCH 4.4 00/97] 4.4.124-stable review

2018-03-23 Thread Nathan Chancellor
On Fri, Mar 23, 2018 at 10:53:47AM +0100, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.124 release. > There are 97 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

[PATCH] staging: rtl8723bs: Mark ACPI table declaration as used

2018-09-26 Thread Nathan Chancellor
LE. $ nm -S drivers/staging/rtl8723bs/os_dep/sdio_intf.o | grep acpi 0040 R __mod_acpi__acpi_ids_device_table Link: https://github.com/ClangBuiltLinux/linux/issues/169 Suggested-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- drivers/staging/rtl8723bs/os

[PATCH] staging: rtl8188eu: Use proper enum in rtl8188eu_config_rf_reg

2018-09-26 Thread Nathan Chancellor
tl_rfreg_delay(adapt, RF90_PATH_A, addr | maskforphyset, ~~~^~~ 1 warning generated. Avoid this by using the equivalent value from the expected type, rf_radio_path: RF90_PATH_A = RF_PATH_A = 0 Link: https://github.com/ClangBuiltLinux/linux/issues/164 Signed-off-b

Re: [PATCH] IB/mlx4: Avoid implicit enumerated type conversion

2018-09-26 Thread Nathan Chancellor
On Mon, Sep 24, 2018 at 08:37:22PM -0600, Jason Gunthorpe wrote: > On Mon, Sep 24, 2018 at 03:29:38PM -0700, Nick Desaulniers wrote: > > On Mon, Sep 24, 2018 at 3:27 PM Nathan Chancellor > > wrote: > > > > > > On Mon, Sep 24, 2018 at 03:24:36PM -0700, Nick Desau

[PATCH] RDMA/qedr: Explicitly cast pkt->tx_dest to qed_ll2_tx_dest

2018-09-26 Thread Nathan Chancellor
; [-Wenum-conversion] ll2_tx_pkt.tx_dest = pkt->tx_dest; ~ ~^~~ 1 warning generated. Avoid this warning by explicitly casting pkt->tx_dest to qed_112_tx_dest, which has the expected values from the type qed_roce_ll2_tx_dest. Reported-by: Nick Desaulniers Signed

[PATCH] mfd: cros_ec: Avoid unneeded internal declaration warning

2018-09-26 Thread Nathan Chancellor
by adding it to the cros_ec_dev_driver definition under the id_table member like all other platform drivers. Signed-off-by: Nathan Chancellor --- I looked at several drivers with platform_device_id defintions and I didn't really find any where the definition wasn't then added to the pl

[PATCH v2] IB/rxe: Avoid implicit enum conversions in rxe_init functions

2018-09-26 Thread Nathan Chancellor
on type 'enum rxe_port_param' to different enumeration type 'enum ib_mtu' [-Wenum-conversion] ib_mtu_enum_to_int(RXE_PORT_ACTIVE_MTU); ~~ ^~~ 5 warnings generated. Use the appropriate values from the expected en

[PATCH v3] IB/rxe: Remove unnecessary enum values

2018-09-26 Thread Nathan Chancellor
rated type so no conversion needs to happen then remove the unneeded definitions. Reported-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Don't cast, just use the expecting enumerated value directly, per Jason's request v2 -> v3: * Conv

[PATCH v2] drm/amd/display: Use proper enums in process_channel_reply

2018-09-27 Thread Nathan Chancellor
e current enum is incorrect, it should be from aux_transaction_reply, so use AUX_TRANSACTION_REPLY_HPD_DISCON. Reported-by: Nick Desaulniers Suggested-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Rather than change status to an integer, use the proper enumerated type fr

Re: [PATCH v2] drm/amd/display: Use proper enums in process_channel_reply

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 11:06:33AM -0700, Nathan Chancellor wrote: > Clang warns when one enumerated type is implicitly converted to another. > > drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c:315:19: warning: > implicit conversion from enumeratio

Re: [PATCH 4.4 00/28] 4.4.159-stable review

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 11:06:24AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.159 release. > There are 28 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 00/44] 4.9.130-stable review

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 11:03:49AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.130 release. > There are 44 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.14 00/64] 4.14.73-stable review

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 11:03:17AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.73 release. > There are 64 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH] IB/mlx4: Avoid implicit enumerated type conversion

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 01:13:31PM -0700, Nick Desaulniers wrote: > On Wed, Sep 26, 2018 at 9:48 PM Jason Gunthorpe wrote: > > > > On Wed, Sep 26, 2018 at 06:08:03PM -0700, Nathan Chancellor wrote: > > > On Mon, Sep 24, 2018 at 08:37:22PM -0600, Jason Gunthorpe wrote: >

Re: [PATCH] RDMA/qedr: Explicitly cast pkt->tx_dest to qed_ll2_tx_dest

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 01:28:12PM -0700, Nick Desaulniers wrote: > On Wed, Sep 26, 2018 at 6:18 PM Nathan Chancellor > wrote: > > > > Clang warns when one enumerated type is explicitly converted to another. > > > > drivers/infiniband/hw/qedr/qedr_roce_

Re: [PATCH] IB/mlx4: Avoid implicit enumerated type conversion

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 01:34:16PM -0700, Nick Desaulniers wrote: > On Thu, Sep 27, 2018 at 1:28 PM Nathan Chancellor > wrote: > > > > On Thu, Sep 27, 2018 at 01:13:31PM -0700, Nick Desaulniers wrote: > > > On Wed, Sep 26, 2018 at 9:48 PM Jason Gunthorpe wrote: >

[PATCH v2] IB/mlx4: Avoid implicit enumerated type conversion

2018-09-27 Thread Nathan Chancellor
gs. Make that clear to Clang by changing the create_flags type to int. Reported-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Use int instead of u32 since enums are restricted to this range, as suggested by Nick. include/rdma/ib_verbs.h | 2 +- 1 file changed,

[PATCH v2] RDMA/qedr: Remove enumerated type qed_roce_ll2_tx_dest

2018-09-27 Thread Nathan Chancellor
from qed_ll2_tx_dest in their place. Reported-by: Nick Desaulniers Signed-off-by: Nathan Chancellor --- v1 -> v2: * Rather than using an explicit cast, just convert the uses to the appropriate values and delete the duplicated enum. drivers/infiniband/hw/qedr/qedr_roce_cm.c |

Re: [PATCH] scsi: bfa: Avoid implicit enum conversion in bfad_im_post_vendor_event

2018-09-27 Thread Nathan Chancellor
On Thu, Sep 27, 2018 at 04:35:37PM -0700, Nick Desaulniers wrote: > On Tue, Sep 25, 2018 at 9:58 PM Nathan Chancellor > wrote: > > > > Clang warns when one enumerated type is implicitly converted to another. > > > > drivers/scsi/bfa/bfa_fcs_lport.c:379:26: warning

[PATCH v2] scsi: bfa: Avoid implicit enum conversion in bfad_im_post_vendor_event

2018-09-27 Thread Nathan Chancellor
n_type's type in bfa_aen_entry_s as members that hold enumerated types should be int. Link: https://github.com/ClangBuiltLinux/linux/issues/147 Signed-off-by: Nathan Chancellor --- v1 -> v2: * Update aen_type's type in bfa_aen_entry_s to match evt drivers/scsi/bfa/bfa_defs_svc.h |

[PATCH] staging: bcm2835-camera: Avoid unneeded internal declaration warning

2018-09-27 Thread Nathan Chancellor
op because V4L2_CID_MPEG_VIDEO_BITRATE_MODE is defined as a MMAL_CONTROL_TYPE_STD_MENU, which does not pass the imenu definition along to v4l2_ctrl_new in bm2835_mmal_init_controls. Link: https://github.com/ClangBuiltLinux/linux/issues/122 Signed-off-by: Nathan Chancellor --- drivers/staging/vc04_servic

Re: [PATCH] mfd: cros_ec: Avoid unneeded internal declaration warning

2018-09-27 Thread Nathan Chancellor
On Wed, Sep 26, 2018 at 08:33:17PM -0700, Nathan Chancellor wrote: > Clang warns: > > drivers/mfd/cros_ec_dev.c:509:40: warning: variable 'cros_ec_id' is not > needed and will not be emitted [-Wunneeded-internal-declaration] > static const struct plat

Re: [PATCH] staging: bcm2835-camera: Avoid unneeded internal declaration warning

2018-09-28 Thread Nathan Chancellor
On Fri, Sep 28, 2018 at 10:04:29AM +0100, Dave Stevenson wrote: > Hi Nate > > Thanks for the patch. > > On Fri, 28 Sep 2018 at 01:53, Nathan Chancellor > wrote: > > > > Clang warns: > > > > drivers/staging/vc04_services/bcm2835-camera/controls.c:59:1

[PATCH] soc/tegra: Fix terminating condition

2018-11-21 Thread Nathan Chancellor
infinite loop and QEMU no longer boots... https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/160242918 Check that the first character of the string isn't null so that the loop properly terminates. Fixes: c57eff9503a5 ("soc/tegra: refactor soc_is_tegra()") Signed-off-by:

[PATCH 4.14] kbuild: allow to use GCC toolchain not in Clang search path

2018-11-28 Thread Nathan Chancellor
in /usr/bin. Link: https://github.com/ClangBuiltLinux/linux/issues/78 Signed-off-by: Stefan Agner Reviewed-and-tested-by: Nick Desaulniers Signed-off-by: Masahiro Yamada [nc: Adjust context] Signed-off-by: Nathan Chancellor --- Makefile | 8 +--- 1 file changed, 5 insertions(+), 3

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Nathan Chancellor
On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > > > > > On 29/11/18 04:19, Nick Desaulniers wrote: > > > Fixes the warning produced from Clang: > > > ./include/asm-generic/io.h:711:9: warning: value size does not mat

Re: [PATCH] arm64: io: specify asm operand width for __iormb()

2018-11-29 Thread Nathan Chancellor
On Thu, Nov 29, 2018 at 04:13:37PM +, Will Deacon wrote: > On Thu, Nov 29, 2018 at 09:10:39AM -0700, Nathan Chancellor wrote: > > On Thu, Nov 29, 2018 at 10:49:03AM +, Will Deacon wrote: > > > On Thu, Nov 29, 2018 at 09:03:54AM +, Julien Thierry wrote: > > >

Re: [PATCH] pinctrl: zynq: Use define directive for PIN_CONFIG_IO_STANDARD

2018-11-16 Thread Nathan Chancellor
On Fri, Nov 16, 2018 at 09:40:45AM +0100, Michal Simek wrote: > On 09. 11. 18 16:36, Nathan Chancellor wrote: > > On Fri, Nov 09, 2018 at 10:33:00AM +0100, Michal Simek wrote: > >> On 08. 11. 18 16:01, Nathan Chancellor wrote: > >>> On Thu, Nov 08, 2018 at 07:45:

Re: [PATCH] geneve: Add missing braces in addr6 initializer

2018-11-12 Thread Nathan Chancellor
On Mon, Nov 12, 2018 at 11:19:17PM +0100, Stefano Brivio wrote: > On Mon, 12 Nov 2018 15:12:48 -0700 > Nathan Chancellor wrote: > > > Clang warns: > > > > drivers/net/geneve.c:428:29: error: suggest braces around initialization > > of su

[PATCH 2/2] ARM: Wrap '--pic-veneer' with ld-option

2018-12-04 Thread Nathan Chancellor
This flag is not supported by lld: ld.lld: error: unknown argument: --pic-veneer Signed-off-by: Nathan Chancellor --- arch/arm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/Makefile b/arch/arm/Makefile index e2a0baf36766..4fab2aa29570 100644 --- a

[PATCH 1/2] ARM: Remove '-p' from LDFLAGS

2018-12-04 Thread Nathan Chancellor
around so just remove it. Commit 1a381d4a0a9a ("arm64: remove no-op -p linker flag") did the same for arm64. Signed-off-by: Nathan Chancellor --- arch/arm/Makefile | 2 +- arch/arm/boot/compressed/Makefile | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git

Re: [PATCH 2/2] ARM: Wrap '--pic-veneer' with ld-option

2018-12-05 Thread Nathan Chancellor
On Wed, Dec 05, 2018 at 08:37:05AM +0100, Ard Biesheuvel wrote: > On Wed, 5 Dec 2018 at 02:42, Nathan Chancellor > wrote: > > > > This flag is not supported by lld: > > > > ld.lld: error: unknown argument: --pic-veneer > > > > Signed-off-by: Natha

Re: [PATCH 2/2] ARM: Wrap '--pic-veneer' with ld-option

2018-12-05 Thread Nathan Chancellor
On Wed, Dec 05, 2018 at 09:09:56AM +0100, Ard Biesheuvel wrote: > (+ Arnd) > > On Wed, 5 Dec 2018 at 09:06, Nathan Chancellor > wrote: > > > > On Wed, Dec 05, 2018 at 08:37:05AM +0100, Ard Biesheuvel wrote: > > > On Wed, 5 Dec 2018 at 02:42, Nathan Chancellor

Re: [PATCH] pinctrl: generic: Avoid several implicit enum conversions

2018-11-09 Thread Nathan Chancellor
On Fri, Nov 09, 2018 at 10:29:02AM +0100, Linus Walleij wrote: > On Thu, Nov 1, 2018 at 1:03 AM Nathan Chancellor > wrote: > > [Me] > > > A slightly lesser evil variant is to add a few PIN_CONFIG_CUSTOM_1 > > > PIN_CONFIG_CUSTOM_2 etc at the end of the enum an

Re: [PATCH] pinctrl: zynq: Use define directive for PIN_CONFIG_IO_STANDARD

2018-11-09 Thread Nathan Chancellor
On Fri, Nov 09, 2018 at 10:33:00AM +0100, Michal Simek wrote: > On 08. 11. 18 16:01, Nathan Chancellor wrote: > > On Thu, Nov 08, 2018 at 07:45:42AM +0100, Michal Simek wrote: > >> On 07. 11. 18 18:48, Nick Desaulniers wrote: > >>> On Wed, Nov 7, 2018 at 1:01

Re: [PATCH 3.18 00/85] 3.18.114-stable review

2018-07-01 Thread Nathan Chancellor
On Sun, Jul 01, 2018 at 06:01:18PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.114 release. > There are 85 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 000/105] 4.4.139-stable review

2018-07-01 Thread Nathan Chancellor
On Sun, Jul 01, 2018 at 06:01:10PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.139 release. > There are 105 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 000/101] 4.9.111-stable review

2018-07-01 Thread Nathan Chancellor
On Sun, Jul 01, 2018 at 06:20:46PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.111 release. > There are 101 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 3.18 00/23] 3.18.115-stable review

2018-07-10 Thread Nathan Chancellor
On Tue, Jul 10, 2018 at 08:24:33PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.115 release. > There are 23 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/52] 4.9.112-stable review

2018-07-10 Thread Nathan Chancellor
On Tue, Jul 10, 2018 at 08:24:28PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.112 release. > There are 52 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/47] 4.4.140-stable review

2018-07-10 Thread Nathan Chancellor
On Tue, Jul 10, 2018 at 08:24:24PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.140 release. > There are 47 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 3.18 00/45] 3.18.110-stable review

2018-05-24 Thread Nathan Chancellor
On Thu, May 24, 2018 at 11:38:08AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.110 release. > There are 45 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 00/92] 4.4.133-stable review

2018-05-24 Thread Nathan Chancellor
On Thu, May 24, 2018 at 11:37:37AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.133 release. > There are 92 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.14 000/165] 4.14.44-stable review

2018-05-24 Thread Nathan Chancellor
On Thu, May 24, 2018 at 11:36:46AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.44 release. > There are 165 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 3.18 00/23] 3.18.109-stable review

2018-05-14 Thread Nathan Chancellor
On Mon, May 14, 2018 at 08:48:29AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.109 release. > There are 23 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.4 00/56] 4.4.132-stable review

2018-05-14 Thread Nathan Chancellor
On Mon, May 14, 2018 at 08:48:05AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.132 release. > There are 56 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.14 00/62] 4.14.41-stable review

2018-05-14 Thread Nathan Chancellor
On Mon, May 14, 2018 at 08:48:16AM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.41 release. > There are 62 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 3/7] staging: ks7010: Remove unnecessary limit checks

2018-05-14 Thread Nathan Chancellor
On Mon, May 14, 2018 at 04:17:36PM +0300, Dan Carpenter wrote: > On Sun, May 06, 2018 at 03:03:00PM -0700, Nathan Chancellor wrote: > > uwrq is an unsigned 32-bit integer, it cannot be less than zero. > > > > Signed-off-by: Nathan Chancellor > > --- > > drivers

[PATCH] staging: android: ion: Check return value of ion_buffer_kmap_get

2018-05-14 Thread Nathan Chancellor
GCC warns that vaddr is set but unused. Check the return value of ion_buffer_kmap_get to make vaddr useful and make sure everything is properly configured before beginning a DMA. Suggested-by: Laura Abbott Signed-off-by: Nathan Chancellor --- drivers/staging/android/ion/ion.c | 10

Kconfig warnings with GCC 8.1.0

2018-05-30 Thread Nathan Chancellor
Hi everyone, My apologies if this has already been reported in some capacity, I searched the mailing list and patchwork but I didn't see anything. With GCC 8.1.0, I am starting to see the following warnings from Kconfig: CCscripts/kconfig/zconf.tab.c LEX scripts/kconfig/zconf.lex.c

[PATCH] kconfig: Avoid format overflow warning from GCC 8.1

2018-06-02 Thread Nathan Chancellor
"%s.tmpconfig.%d", dirname, (int)getpid()); ^~~ Increase the size of tmpname and newname to make GCC happy. Cc: sta...@vger.kernel.org Signed-off-by: Nathan Chancellor --- scripts/kconfig/confdata.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/kconfig/confdata

Re: [PATCH 4.9 00/31] 4.9.108-stable review

2018-06-12 Thread Nathan Chancellor
On Tue, Jun 12, 2018 at 06:46:03PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.108 release. > There are 31 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/24] 4.4.137-stable review

2018-06-12 Thread Nathan Chancellor
On Tue, Jun 12, 2018 at 06:51:44PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.137 release. > There are 24 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 3.18 00/21] 3.18.113-stable review

2018-06-12 Thread Nathan Chancellor
On Tue, Jun 12, 2018 at 06:51:57PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.113 release. > There are 21 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/30] 4.9.109-stable review

2018-06-14 Thread Nathan Chancellor
On Thu, Jun 14, 2018 at 04:04:41PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.109 release. > There are 30 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.4 00/24] 4.4.138-stable review

2018-06-14 Thread Nathan Chancellor
On Thu, Jun 14, 2018 at 04:04:55PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.4.138 release. > There are 24 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

Re: [PATCH 4.9 000/107] 4.9.120-stable review

2018-08-14 Thread Nathan Chancellor
On Tue, Aug 14, 2018 at 07:16:23PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.120 release. > There are 107 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 000/107] 4.9.120-stable review

2018-08-14 Thread Nathan Chancellor
On Wed, Aug 15, 2018 at 02:36:00AM +0200, Sebastian Gottschall wrote: > if SWAP is disabled in kernel config, the following compile error will raise > up with this release > > arch/x86/built-in.o: in function `max_swapfile_size': (.text+0x3bba1): > undefined reference to `generic_max_swapfile_size

Re: [PATCH 3.18 00/15] 3.18.119-stable review

2018-08-16 Thread Nathan Chancellor
On Thu, Aug 16, 2018 at 08:41:37PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.18.119 release. > There are 15 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me kno

Re: [PATCH 4.9 00/15] 4.9.121-stable review

2018-08-16 Thread Nathan Chancellor
On Thu, Aug 16, 2018 at 08:41:53PM +0200, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.121 release. > There are 15 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know

[PATCH] ASoC: simple-card: Dereference pointer for memcpy sizeof in asoc_simple_card_probe

2018-12-12 Thread Nathan Chancellor
nerated. Commit 4fb7f4df49d3 ("ASoC: simple-card: use cpu/codec pointer on simple_dai_props") updated {cpu,codec}_dai to be pointers in struct simple_dai_props but didn't update these locations to dereference the pointers to get the proper size of their contents. Signed-off-by: Nat

Re: [PATCH] ASoC: simple-card: Dereference pointer for memcpy sizeof in asoc_simple_card_probe

2018-12-12 Thread Nathan Chancellor
pointer on > > simple_dai_props") updated {cpu,codec}_dai to be pointers in struct > > simple_dai_props but didn't update these locations to dereference the > > pointers to get the proper size of their contents. > > > > Signed-off-by: Nathan Chancellor > &g

Re: [PATCH] ARM: Ensure that NEON code always compiles with Clang

2018-12-17 Thread Nathan Chancellor
On Mon, Dec 17, 2018 at 01:23:52PM -0500, Nicolas Pitre wrote: > On Sat, 15 Dec 2018, Nathan Chancellor wrote: > > > While building arm32 allyesconfig, I ran into the following errors: > > > > arch/arm/lib/xor-neon.c:17:2: error: You should compile this file with &

Re: [PATCH] drm/i915: Disable -Wuninitialized for intel_breadcrumbs.o

2018-12-18 Thread Nathan Chancellor
On Tue, Dec 18, 2018 at 11:53:06AM +, Chris Wilson wrote: > Quoting Nick Desaulniers (2018-10-25 23:20:58) > > On Thu, Oct 25, 2018 at 12:36 PM Nathan Chancellor > > wrote: > > > > > > This warning is disabled by default in scripts/Makefile.extrawarn when

[PATCH] PCI: Remove unused attr variable in pci_dma_configure

2018-12-14 Thread Nathan Chancellor
Clang warns: drivers/pci/pci-driver.c:1603:21: error: unused variable 'attr' [-Werror,-Wunused-variable] Commit e5361ca29f2f ("ACPI / scan: Refactor _CCA enforcement") removed attr's use and replaced it with its assigned value so it is no longer needed. Signed-

  1   2   3   4   5   6   7   8   9   10   >