From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757822AbcCaXnH (ORCPT ); Thu, 31 Mar 2016 19:43:07 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:39073 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757313AbcCaXnE (ORCPT ); Thu, 31 Mar 2016 19:43:04 -0400 Date: Thu, 31 Mar 2016 16:43:02 -0700 From: Greg Kroah-Hartman To: Tomas Winkler Cc: Alexander Usyskin , linux-kernel@vger.kernel.org, Rasmus Villemoes Subject: Re: [char-misc-next] mei: bus: use scnprintf in *_show Message-ID: <20160331234302.GA31271@kroah.com> References: <1458808684-21817-1-git-send-email-tomas.winkler@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1458808684-21817-1-git-send-email-tomas.winkler@intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 24, 2016 at 10:38:04AM +0200, Tomas Winkler wrote: > From: Rasmus Villemoes > > There's no reason to duplicate the logic provided by scnprintf(). And really, there's no need for any of this, just use sprintf() for sysfs attributes that are the size of an integer. But I'll take this for now, just to keep people happy because "everyone knows sprintf() should never be used"... bah. greg k-h