From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: [PATCH 1/2] GenWQE: Fix endian issues detected by sparse Date: Tue, 7 Jan 2014 09:41:31 +0300 Message-ID: <20140107064130.GA5494@mwanda> References: <1387553171-31469-1-git-send-email-haver@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from aserp1040.oracle.com ([141.146.126.69]:21730 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752631AbaAGGl2 (ORCPT ); Tue, 7 Jan 2014 01:41:28 -0500 Content-Disposition: inline In-Reply-To: <1387553171-31469-1-git-send-email-haver@linux.vnet.ibm.com> Sender: linux-next-owner@vger.kernel.org List-ID: To: Frank Haverkamp Cc: fengguang.wu@intel.com, jim.epost@gmail.com, sfr@canb.auug.org.au, gregkh@linuxfoundation.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, kbuild-all@01.org Could you also fix this Smatch warning? drivers/misc/genwqe/card_dev.c:658 do_flash_update() warn: maybe return -EFAULT instead of the bytes remaining? Also we shouldn't be doing dev_err() on copy_to/from_user() problems. The user can trigger those and flood dmesg. It is a DoS (annoying). regards, dan carpenter