From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756453AbbE2RGB (ORCPT ); Fri, 29 May 2015 13:06:01 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:46708 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751995AbbE2RFw (ORCPT ); Fri, 29 May 2015 13:05:52 -0400 Date: Fri, 29 May 2015 20:05:22 +0300 From: Dan Carpenter To: Nicholas Mc Guire Cc: Lidza Louina , devel@driverdev.osuosl.org, Greg Kroah-Hartman , driverdev-devel@linuxdriverproject.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] staging: dgnc: switch timeout to signed type Message-ID: <20150529170522.GP28762@mwanda> References: <1432917688-20228-1-git-send-email-hofrat@osadl.org> <1432917688-20228-2-git-send-email-hofrat@osadl.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1432917688-20228-2-git-send-email-hofrat@osadl.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 29, 2015 at 06:41:28PM +0200, Nicholas Mc Guire wrote: > The schedule_timeout*() helpers take the timeout as signed long, as > ch_close_delay in struct channel_t was not used for other purposes its > type was switched to signed long and the declarations fixed up. Uh, we never pass it to schedule_timeout etc and even if we did how would that matter? It's either 250 or 0. What is the bug you are trying to fix and we can help you? regards, dan carpenter