Re: [PATCH] staging: pi433: Removed some obsolete or duplicated defines; moved two defines to better locations

[Marcus Wolf]

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