From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752619AbaIATLV (ORCPT ); Mon, 1 Sep 2014 15:11:21 -0400 Received: from mxout5.netvision.net.il ([194.90.6.65]:54105 "EHLO mxout5.netvision.net.il" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752518AbaIATLU (ORCPT ); Mon, 1 Sep 2014 15:11:20 -0400 MIME-version: 1.0 Content-transfer-encoding: 7BIT Content-type: text/plain; CHARSET=US-ASCII; format=flowed Message-id: <5404C116.9020300@gmail.com> Date: Mon, 01 Sep 2014 21:55:18 +0300 From: Eli Billauer User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.12) Gecko/20100907 Fedora/3.0.7-1.fc12 Thunderbird/3.0.7 To: Dan Carpenter Cc: gregkh@linuxfoundation.org, arnd@arndb.de, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: xillybus: Move out of staging References: <1409471821-11809-1-git-send-email-eli.billauer@gmail.com> <20140901121318.GG5122@mwanda> In-reply-to: <20140901121318.GG5122@mwanda> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, Dan. Thanks a lot for reviewing the driver. There are indeed a few things that need fixing. I'll be back with patches and responses in a matter of a few days. As for the --strict option, I wasn't aware of it. I learn something new every time. :) Thanks again, Eli On 01/09/14 15:13, Dan Carpenter wrote: > Pretty nice. This is very special purpose hardware and the UAPI for > this is fine. The documentation seems good. I had some minor style > comments but nothing major stands out. > > These days I quite like the --strict option for checkpatch.pl. > > for i in $(find drivers/staging/xillybus/ -name \*\.c) > do ./scripts/checkpatch.pl --strict -f $i > done > >