{"push_id":1551180032,"size":37,"distinct_size":37,"ref":"refs/heads/cm-linux-3.18.y","head":"ad2958b89f5c1b475a7fdbfd259c5e3db634eb78","before":"a900600b721f87c211b3f396c3321bbf31dbd6f8","commits":[{"sha":"af93ebdd005bea26ae4b2a1d83151410641e9ec6","author":{"name":"Andrey Ryabinin","email":"fa6f000972208fd46fdef68b746f6e1f56abbfc2@samsung.com"},"message":"module: fix types of device tables aliases\n\ncommit 6301939d97d079f0d3dbe71e750f4daf5d39fc33 upstream.\n\nMODULE_DEVICE_TABLE() macro used to create aliases to device tables.\nNormally alias should have the same type as aliased symbol.\n\nDevice tables are arrays, so they have 'struct type##_device_id[x]'\ntypes. Alias created by MODULE_DEVICE_TABLE() will have non-array type -\n\t'struct type##_device_id'.\n\nThis inconsistency confuses compiler, it could make a wrong assumption\nabout variable's size which leads KASan to produce a false positive report\nabout out of bounds access.\n\nFor every global variable compiler calls __asan_register_globals() passing\ninformation about global variable (address, size, size with redzone, name\n...) __asan_register_globals() poison symbols redzone to detect possible\nout of bounds accesses.\n\nWhen symbol has an alias __asan_register_globals() will be called as for\nsymbol so for alias.  Compiler determines size of variable by size of\nvariable's type.  Alias and symbol have the same address, so if alias have\nthe wrong size part of memory that actually belongs to the symbol could be\npoisoned as redzone of alias symbol.\n\nBy fixing type of alias symbol we will fix size of it, so\n__asan_register_globals() will not poison valid memory.\n\nSigned-off-by: Andrey Ryabinin <a.ryabinin@samsung.com>\nCc: Dmitry Vyukov <dvyukov@google.com>\nCc: Konstantin Serebryany <kcc@google.com>\nCc: Dmitry Chernenkov <dmitryc@google.com>\nSigned-off-by: Andrey Konovalov <adech.fo@gmail.com>\nCc: Yuri Gribov <tetra2005@gmail.com>\nCc: Konstantin Khlebnikov <koct9i@gmail.com>\nCc: Sasha Levin <sasha.levin@oracle.com>\nCc: Christoph Lameter <cl@linux.com>\nCc: Joonsoo Kim <iamjoonsoo.kim@lge.com>\nCc: Dave Hansen <dave.hansen@intel.com>\nCc: Andi Kleen <andi@firstfloor.org>\nCc: Ingo Molnar <mingo@elte.hu>\nCc: Thomas Gleixner <tglx@linutronix.de>\nCc: \"H. Peter Anvin\" <hpa@zytor.com>\nCc: Christoph Lameter <cl@linux.com>\nCc: Pekka Enberg <penberg@kernel.org>\nCc: David Rientjes <rientjes@google.com>\nSigned-off-by: Andrew Morton <akpm@linux-foundation.org>\nSigned-off-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/af93ebdd005bea26ae4b2a1d83151410641e9ec6"},{"sha":"e2dee7fb89d91333f0ae01efa17d8a943910d4d0","author":{"name":"Andrew Morton","email":"1c34db6615bc25a29a6a7c99217d037c3d74d16a@linux-foundation.org"},"message":"MODULE_DEVICE_TABLE: fix some callsites\n\ncommit 0f989f749b51ec1fd94bb5a42f8ad10c8b9f73cb upstream.\n\nThe patch \"module: fix types of device tables aliases\" newly requires that\ninvocations of\n\nMODULE_DEVICE_TABLE(type, name);\n\ncome *after* the definition of `name'.  That is reasonable, but some\ndrivers weren't doing this.  Fix them.\n\nCc: James Bottomley <James.Bottomley@HansenPartnership.com>\nCc: Andrey Ryabinin <a.ryabinin@samsung.com>\nCc: David Miller <davem@davemloft.net>\nCc: Hans Verkuil <hverkuil@xs4all.nl>\nAcked-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>\nSigned-off-by: Andrew Morton <akpm@linux-foundation.org>\nSigned-off-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/e2dee7fb89d91333f0ae01efa17d8a943910d4d0"},{"sha":"ce08f0e982eea39a2f6ea67c859812f01ad92143","author":{"name":"Tillmann Heidsieck","email":"53353d82af20d238ba8bc5d0847571be840c0b44@leenox.de"},"message":"atm: iphase: fix misleading indention\n\ncommit cbb41b91e68a302087762823136c9067138cff7c upstream.\n\nFix a smatch warning:\ndrivers/atm/iphase.c:1178 rx_pkt() warn: curly braces intended?\n\nThe code is correct, the indention is misleading. In case the allocation\nof skb fails, we want to skip to the end.\n\nSigned-off-by: Tillmann Heidsieck <theidsieck@leenox.de>\nSigned-off-by: David S. Miller <davem@davemloft.net>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/ce08f0e982eea39a2f6ea67c859812f01ad92143"},{"sha":"65cdae9cbe797c33a574e4beaabba4049f4f5eec","author":{"name":"Dan Carpenter","email":"60b2bf2bc0bec4ca4a4b046ca7ef9255248abff8@oracle.com"},"message":"paride: fix the \"verbose\" module param\n\ncommit 946e87981942552e526aca9cb6204f02a6c847cb upstream.\n\nThe verbose module parameter can be set to 2 for extremely verbose\nmessages so the type should be int instead of bool.\n\nSigned-off-by: Dan Carpenter <dan.carpenter@oracle.com>\nCc: Tim Waugh <tim@cyberelk.net>\nSigned-off-by: Andrew Morton <akpm@linux-foundation.org>\nSigned-off-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/65cdae9cbe797c33a574e4beaabba4049f4f5eec"},{"sha":"3f6aec579184989184437fdbfa9b81875d5f7735","author":{"name":"Arnd Bergmann","email":"72a0f6d3b182607e6d5c44b93e1abb7b23ee18f3@arndb.de"},"message":"net: caif: fix misleading indentation\n\ncommit 8e0cc8c326d99e41468c96fea9785ab78883a281 upstream.\n\ngcc points out code that is not indented the way it is\ninterpreted:\n\nnet/caif/cfpkt_skbuff.c: In function 'cfpkt_setlen':\nnet/caif/cfpkt_skbuff.c:289:4: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation]\n    return cfpkt_getlen(pkt);\n    ^~~~~~\nnet/caif/cfpkt_skbuff.c:286:3: note: ...this 'else' clause, but it is not\n   else\n   ^~~~\n\nIt is clear from the context that not returning here would be\na bug, as we'd end up passing a negative length into a function\nthat takes a u16 length, so it is not missing curly braces\nhere, and I'm assuming that the indentation is the only part\nthat's wrong about it.\n\nSigned-off-by: Arnd Bergmann <arnd@arndb.de>\nSigned-off-by: David S. Miller <davem@davemloft.net>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/3f6aec579184989184437fdbfa9b81875d5f7735"},{"sha":"e6ec9d2751d086e1dafc25812d8f786a1dfc2b46","author":{"name":"Linus Torvalds","email":"ead7104581934751970e9decb2db5deb20a9e407@linux-foundation.org"},"message":"Disable \"frame-address\" warning\n\ncommit 124a3d88fa20e1869fc229d7d8c740cc81944264 upstream.\n\nNewer versions of gcc warn about the use of __builtin_return_address()\nwith a non-zero argument when \"-Wall\" is specified:\n\n  kernel/trace/trace_irqsoff.c: In function ‘stop_critical_timings’:\n  kernel/trace/trace_irqsoff.c:433:86: warning: calling ‘__builtin_return_address’ with a nonzero argument is unsafe [-Wframe-address]\n     stop_critical_timing(CALLER_ADDR0, CALLER_ADDR1);\n  [ .. repeats a few times for other similar cases .. ]\n\nIt is true that a non-zero argument is somewhat dangerous, and we do not\nactually have very many uses of that in the kernel - but the ftrace code\ndoes use it, and as Stephen Rostedt says:\n\n \"We are well aware of the danger of using __builtin_return_address() of\n  > 0.  In fact that's part of the reason for having the \"thunk\" code in\n  x86 (See arch/x86/entry/thunk_{64,32}.S).  [..] it adds extra frames\n  when tracking irqs off sections, to prevent __builtin_return_address()\n  from accessing bad areas.  In fact the thunk_32.S states: 'Trampoline to\n  trace irqs off.  (otherwise CALLER_ADDR1 might crash)'.\"\n\nFor now, __builtin_return_address() with a non-zero argument is the best\nwe can do, and the warning is not helpful and can end up making people\nmiss other warnings for real problems.\n\nSo disable the frame-address warning on compilers that need it.\n\nAcked-by: Steven Rostedt <rostedt@goodmis.org>\nSigned-off-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/e6ec9d2751d086e1dafc25812d8f786a1dfc2b46"},{"sha":"81b6187123b1e765f2a74b2cfca603de02753321","author":{"name":"David Miller","email":"79caf591e37e1438a2d55ae59670d5cd0d717764@davemloft.net"},"message":"netfilter: Fix switch statement warnings with recent gcc.\n\ncommit c1f866767777d1c6abae0ec57effffcb72017c00 upstream.\n\nMore recent GCC warns about two kinds of switch statement uses:\n\n1) Switching on an enumeration, but not having an explicit case\n   statement for all members of the enumeration.  To show the\n   compiler this is intentional, we simply add a default case\n   with nothing more than a break statement.\n\n2) Switching on a boolean value.  I think this warning is dumb\n   but nevertheless you get it wholesale with -Wswitch.\n\nThis patch cures all such warnings in netfilter.\n\nSigned-off-by: David S. Miller <davem@davemloft.net>\nAcked-by: Pablo Neira Ayuso <pablo@netfilter.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/81b6187123b1e765f2a74b2cfca603de02753321"},{"sha":"371fdd96c182e166b4e65a3724da94745e69f3b0","author":{"name":"Jiri Slaby","email":"5983af6a8e8283aa60de579a10bd4a0e541489a0@suse.cz"},"message":"Bluetooth: make hci_test_bit's addr const\n\ncommit 9391976a4da0d2a30abdb8d2704cfc7bf4bf9aab upstream.\n\ngcc5 warns about passing a const array to hci_test_bit which takes a\nnon-const pointer:\nnet/bluetooth/hci_sock.c: In function ‘hci_sock_sendmsg’:\nnet/bluetooth/hci_sock.c:955:8: warning: passing argument 2 of ‘hci_test_bit’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-array-qualifiers]\n        &hci_sec_filter.ocf_mask[ogf])) &&\n        ^\nnet/bluetooth/hci_sock.c:49:19: note: expected ‘void *’ but argument is of type ‘const __u32 (*)[4] {aka const unsigned int (*)[4]}’\n static inline int hci_test_bit(int nr, void *addr)\n                   ^\n\nSo make 'addr' 'const void *'.\n\nSigned-off-by: Jiri Slaby <jslaby@suse.cz>\nSigned-off-by: Marcel Holtmann <marcel@holtmann.org>\nCc: Gustavo Padovan <gustavo@padovan.org>\nCc: Johan Hedberg <johan.hedberg@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/371fdd96c182e166b4e65a3724da94745e69f3b0"},{"sha":"8b367e00da604df489c8574bb1e21fab5bda29f3","author":{"name":"Arnd Bergmann","email":"72a0f6d3b182607e6d5c44b93e1abb7b23ee18f3@arndb.de"},"message":"ata: hpt366: fix constant cast warning\n\ncommit 6ec0a86c645be3fce7ade42f165a6a417c3503b1 upstream.\n\ngcc-5.x warns about a preexisting problem in the hpt36x pata driver:\n\ndrivers/ata/pata_hpt366.c: In function 'hpt36x_init_one':\ndrivers/ata/pata_hpt366.c:376:9: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-array-qualifiers]\n\nOther ata drivers have the same problem, as ata_pci_bmdma_init_one\ntakes a non-const pointer, and they solve it by using a cast to\nturn that pointer into a normal non-const pointer.\n\nI also tried to change the ata core code to make host->private_data\na const pointer, but that quickly got out of hand, as some other\ndrivers expect it to be writable, so I ended up using the same\nhack as the others here.\n\nSigned-off-by: Arnd Bergmann <arnd@arndb.de>\nAcked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>\nSigned-off-by: Tejun Heo <tj@kernel.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/8b367e00da604df489c8574bb1e21fab5bda29f3"},{"sha":"8db58d097880ca79e32245dbc437712bdb82412e","author":{"name":"James C Boyd","email":"8783fd213de1d3e42d071ba0d346930d1dd67515@gmail.com"},"message":"HID: hid-input: Add parentheses to quell gcc warning\n\ncommit 09a5c34e8d6b05663ec4c3d22b1fbd9fec89aaf9 upstream.\n\nGCC reports a -Wlogical-not-parentheses warning here; therefore\nadd parentheses to shut it up and to express our intent more.\n\nSigned-off-by: James C Boyd <jcboyd.dev@gmail.com>\nSigned-off-by: Jiri Kosina <jkosina@suse.cz>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/8db58d097880ca79e32245dbc437712bdb82412e"},{"sha":"a46b14e0896358f70588f7086256118c71731ce7","author":{"name":"Tomer Barletz","email":"e10f6faa10e4bf8cf96098da170594e6bf863d8c@gmail.com"},"message":"ALSA: oxygen: Fix logical-not-parentheses warning\n\ncommit 8ec7cfce3762299ae289c384e281b2f4010ae231 upstream.\n\nThis fixes the following warning, that is seen with gcc 5.1:\nwarning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses].\n\nSigned-off-by: Tomer Barletz <barletz@gmail.com>\nSigned-off-by: Takashi Iwai <tiwai@suse.de>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/a46b14e0896358f70588f7086256118c71731ce7"},{"sha":"696ebb3a3a1f252301299c3168139dd20b6363da","author":{"name":"Juergen Gross","email":"d3dfe5c5fac772d904d4932d138136d040bb63c7@suse.com"},"message":"x86/xen: fix upper bound of pmd loop in xen_cleanhighmap()\n\ncommit 1cf38741308c64d08553602b3374fb39224eeb5a upstream.\n\nxen_cleanhighmap() is operating on level2_kernel_pgt only. The upper\nbound of the loop setting non-kernel-image entries to zero should not\nexceed the size of level2_kernel_pgt.\n\nReported-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Juergen Gross <jgross@suse.com>\nSigned-off-by: David Vrabel <david.vrabel@citrix.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/696ebb3a3a1f252301299c3168139dd20b6363da"},{"sha":"fb095146dd0e9e8c3a11295effeefdb0eda3a4bb","author":{"name":"Hans Verkuil","email":"aeb1fa9b7f4f5be44f239b597abfb2c97d8cd0b0@cisco.com"},"message":"s3c-camif: fix compiler warnings\n\ncommit 7e0d4e92269e574e50a58041fac4cc75a149828c upstream.\n\nFix these compiler warnings that appeared after switching to gcc-5.1.0:\n\ndrivers/media/platform/s3c-camif/camif-capture.c: In function 'sensor_set_power':\ndrivers/media/platform/s3c-camif/camif-capture.c:118:10: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]\n  if (!on == camif->sensor.power_count)\n          ^\ndrivers/media/platform/s3c-camif/camif-capture.c: In function 'sensor_set_streaming':\ndrivers/media/platform/s3c-camif/camif-capture.c:134:10: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]\n  if (!on == camif->sensor.stream_count)\n          ^\n\nSigned-off-by: Hans Verkuil <hans.verkuil@cisco.com>\nCc: Kamil Debski <k.debski@samsung.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/fb095146dd0e9e8c3a11295effeefdb0eda3a4bb"},{"sha":"8ce8274bbc3c8715410b047258b7108de4025efc","author":{"name":"David S. Miller","email":"79caf591e37e1438a2d55ae59670d5cd0d717764@davemloft.net"},"message":"netfilter; Add some missing default cases to switch statements in nft_reject.\n\ncommit 129d23a56623eea0947a05288158d76dc7f2f0ac upstream.\n\nThis fixes:\n\n====================\nnet/netfilter/nft_reject.c: In function ‘nft_reject_dump’:\nnet/netfilter/nft_reject.c:61:2: warning: enumeration value ‘NFT_REJECT_TCP_RST’ not handled in switch [-Wswitch]\n  switch (priv->type) {\n  ^\nnet/netfilter/nft_reject.c:61:2: warning: enumeration value ‘NFT_REJECT_ICMPX_UNREACH’ not handled in switch [-Wswi\\\ntch]\nnet/netfilter/nft_reject_inet.c: In function ‘nft_reject_inet_dump’:\nnet/netfilter/nft_reject_inet.c:105:2: warning: enumeration value ‘NFT_REJECT_TCP_RST’ not handled in switch [-Wswi\\\ntch]\n  switch (priv->type) {\n  ^\n====================\n\nSigned-off-by: David S. Miller <davem@davemloft.net>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/8ce8274bbc3c8715410b047258b7108de4025efc"},{"sha":"d3eb0660cac8063329ca53df6d1134d5133bc3ca","author":{"name":"Tomer Barletz","email":"e10f6faa10e4bf8cf96098da170594e6bf863d8c@gmail.com"},"message":"mtd: blkdevs: fix switch-bool compilation warning\n\ncommit cc7fce80229067890365c1ee196be5d304d36dea upstream.\n\nWith gcc 5.1 I get:\nwarning: switch condition has boolean value [-Wswitch-bool]\n\nSigned-off-by: Tomer Barletz <barletz@gmail.com>\nSigned-off-by: Brian Norris <computersforpeace@gmail.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/d3eb0660cac8063329ca53df6d1134d5133bc3ca"},{"sha":"25f6501da53fa43eca5a29508bd73a7cdf86de69","author":{"name":"Linus Torvalds","email":"ead7104581934751970e9decb2db5deb20a9e407@linux-foundation.org"},"message":"media: remove unused variable that causes a warning\n\ncommit 1d11437f4fd02f9b5d3749675a1232216787dcc6 upstream.\n\nMy 'allmodconfig' build is _almost_ free of warnings, and most of the\nremaining ones are for legacy drivers that just do bad things that I\ncan't find it in my black heart to care too much about.  But this one\nwas just annoying me:\n\n   drivers/media/v4l2-core/videobuf2-core.c:3256:26: warning: unused variable ‘fileio’ [-Wunused-variable]\n\nbecause commit 0e661006370b (\"[media] vb2: fix 'UNBALANCED' warnings\nwhen calling vb2_thread_stop()\") removed all users of 'fileio' and\ninstead calls \"__vb2_cleanup_fileio(q)\" to clean up q->fileio.  But the\nnow unused 'fileio' variable was left around.\n\nSigned-off-by: Linus Torvalds <torvalds@linux-foundation.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/25f6501da53fa43eca5a29508bd73a7cdf86de69"},{"sha":"ed3bd8d2c166ed94bd7208b0fcf3b7bb7a6cdeb8","author":{"name":"David Malcolm","email":"31454f2ecb110b774e5f4b6650cc7a0d89765954@redhat.com"},"message":"drivers/net/ethernet/dec/tulip/uli526x.c: fix misleading indentation in uli526x_timer\n\ncommit e1395a321eab1a7833d82e952eb8255e0a1f03cb upstream.\n\nThis code in drivers/net/ethernet/dec/tulip/uli526x.c\nfunction \"uli526x_timer\":\n\n  1086          } else\n  1087                  if ((tmp_cr12 & 0x3) && db->link_failed) {\n  [...snip...]\n  1109                  }\n  1110                  else if(!(tmp_cr12 & 0x3) && db->link_failed)\n  1111                  {\n  [...snip...]\n  1117                  }\n  1118                  db->init=0;\n\nis misleadingly indented: the\n  db->init=0\nis indented as if part of the else clause at line 1086, but it is\nindependent of it (no braces before the \"if\" at line 1087).\n\nThis patch fixes the indentation to reflect the actual meaning of the code,\nthough is it actually meant to be part of the \"else\" clause?  (I'm a\ncompiler developer, not a kernel person).  It also adds spaces around\nthe assignment, to placate checkpatch.pl.\n\nSeen via an experimental new gcc warning I'm working on for gcc 6,\n-Wmisleading-indentation, using gcc r223098 adding\n-Werror=misleading-indentation to KBUILD_CFLAGS in Makefile.\nThe experimental GCC emits this warning (as an error), rightly IMHO:\n\ndrivers/net/ethernet/dec/tulip/uli526x.c: In function ‘uli526x_timer’:\ndrivers/net/ethernet/dec/tulip/uli526x.c:1118:3: error: statement is\nindented as if it were guarded by... [-Werror=misleading-indentation]\n   db->init=0;\n    ^\ndrivers/net/ethernet/dec/tulip/uli526x.c:1086:4: note: ...this ‘else’\nclause, but it is not\n  } else\n     ^\n\nHope this is helpful\nDave\n\nSigned-off-by: David Malcolm <dmalcolm@redhat.com>\nSigned-off-by: David S. Miller <davem@davemloft.net>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/ed3bd8d2c166ed94bd7208b0fcf3b7bb7a6cdeb8"},{"sha":"f513703e2366c0e2f9a0e29926eabdfb55aecfc6","author":{"name":"Arnd Bergmann","email":"72a0f6d3b182607e6d5c44b93e1abb7b23ee18f3@arndb.de"},"message":"iwlegacy: avoid warning about missing braces\n\ncommit 2cce76c3fab410520610a7d2f52faebc3cfcf843 upstream.\n\ngcc-6 warns about code in il3945_hw_txq_ctx_free() being\nsomewhat ambiguous:\n\ndrivers/net/wireless/intel/iwlegacy/3945.c:1022:5: warning: suggest explicit braces to avoid ambiguous 'else' [-Wparentheses]\n\nThis adds a set of curly braces to avoid the warning.\n\nSigned-off-by: Arnd Bergmann <arnd@arndb.de>\nAcked-by: Stanislaw Gruszka <sgruszka@redhat.com>\nSigned-off-by: Kalle Valo <kvalo@codeaurora.org>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/f513703e2366c0e2f9a0e29926eabdfb55aecfc6"},{"sha":"b927840c1a76639555621c8636a68c85159d6a69","author":{"name":"Tim Gardner","email":"aca0c50b96e1994e1ffe56274804b9dca8a7dcf7@canonical.com"},"message":"be2iscsi: Fix bogus WARN_ON length check\n\ncommit dd29dae00d39186890a5eaa2fe4ad8768bfd41a9 upstream.\n\ndrivers/scsi/be2iscsi/be_main.c: In function 'be_sgl_create_contiguous':\ndrivers/scsi/be2iscsi/be_main.c:3187:18: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]\n  WARN_ON(!length > 0);\n\ngcc version 5.2.1\n\nSigned-off-by: Tim Gardner <tim.gardner@canonical.com>\nCc: Jayamohan Kallickal <jayamohan.kallickal@avagotech.com>\nCc: Minh Tran <minh.tran@avagotech.com>\nCc: John Soni Jose <sony.john-n@avagotech.com>\nCc: \"James E.J. Bottomley\" <JBottomley@odin.com>\nReported-by: Joel Stanley <joel@jms.id.au>\nReviewed-by: Manoj Kumar <manoj@linux.vnet.ibm.com>\nSigned-off-by: Martin K. Petersen <martin.petersen@oracle.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/b927840c1a76639555621c8636a68c85159d6a69"},{"sha":"696e37395a2a4b72e48a995f3a45312d34326c11","author":{"name":"Joe Perches","email":"d2ff8d83efb8d5aeb4f4e85388fa7e6f5a2df730@perches.com"},"message":"i40e: Reduce stack in i40e_dbg_dump_desc\n\ncommit e6c97234d1b18d4751671df15d52e29daa8a7ba8 upstream.\n\nReduce stack use by using kmemdup and not using a very\nlarge struct on stack.\n\nIn function ‘i40e_dbg_dump_desc’:\nwarning: the frame size of 8192 bytes is larger than 2048 bytes [-Wframe-larger-than=]\n\nSigned-off-by: Joe Perches <joe@perches.com>\nTested-by: Jim Young <jamesx.m.young@intel.com>\nSigned-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>\nSigned-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>","distinct":true,"url":"https://api.github.com/repos/akca/android_kernel_xiaomi_msm8996/commits/696e37395a2a4b72e48a995f3a45312d34326c11"}]}
{"actor":{"display_login":"akca"}}