Subject: Re: [PATCH] staging: pi433: Removed some obsolete or duplicated
 defines; moved two defines to better locations
To: Greg KH <gregkh@linuxfoundation.org>,
        Marcus Wolf <linux@wolf-entwicklungen.de>
Cc: dan.carpenter@oracle.com, devel@driverdev.osuosl.org,
        linux-kernel@vger.kernel.org
References: <1512227648-16008-1-git-send-email-linux@wolf-entwicklungen.de>
 <20171202155549.GA5960@kroah.com>
From: Marcus Wolf <marcus.wolf@smarthome-wolf.de>
Message-ID: <30f27e3f-bd83-f44b-f0b1-f3c97fa2373c@smarthome-wolf.de>
Date: Sat, 2 Dec 2017 18:00:28 +0200
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <20171202155549.GA5960@kroah.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: de-DE
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 997
Lines: 36

Hi Greg,

for me the action was "clean up the defines in rf69.c". So for me it was 
fine, that two defines were moved and several other deleted at the same.

Is it ok for you, or should I split the patch belated?

Cheers,

Marcus

Am 02.12.2017 um 17:55 schrieb Greg KH:
> On Sat, Dec 02, 2017 at 05:14:08PM +0200, Marcus Wolf wrote:
>> The define FIFO_SIZE was moved to rf69_registers.h. Although it is not a register,
>> it is a value, that is given by hardware (like the registers).
>>
>> The define FIFO_THRESHOLD was moved to pi433_if.c, since it is a value, that is
>> freely choosen by the interface implementation. The better the response time of
>> the driver, the lower threshold can be set.
> 
> Shouldn't this be two separate patches?
> 
> Remember, each patch just does one thing.
> 
> thanks,
> 
> greg k-h
> 

-- 
Smarthome-Wolf UG (haftungsbeschränkt)
Helene-Lange-Weg 23
80637 München
Amtsgericht München, HRB 223529
Umastzsteuer-ID: DE304719911
Geschäftsführer: Marcus Wolf