Thread (35 messages) 35 messages, 3 authors, 2016-01-21

[PATCH 1/2] ARM: dts: kirkwood: split lswxl into linkstation lswsxl and lswxl

flat view
STALE3924d

From: andrew@lunn.ch (Andrew Lunn)
Date: 2016-01-19 14:48:46

On Tue, Jan 19, 2016 at 11:15:35PM +0900, Roger Shimizu wrote:
On Tue, Jan 19, 2016 at 11:05 PM, Andrew Lunn [off-list ref] wrote:
quoted
On Tue, Jan 19, 2016 at 10:49:01PM +0900, Roger Shimizu wrote:
quoted
With a few gpio/led-pin changes, including fix for the issue mentioned
in Debian BTS #810894.
Hi Roger

This fix should be a separate patch. We can then get it applied to
stable, so it makes its way back to older kernels.
Dear Andrew,

So this patchset would be reviewed or not?
I will review it and give comments, but won't accept it.
Or I have to create another patchset including both fix for
"kirkwood-lswxl.dts" and the dts split?
Yes please. And the fix patch should come first, and have a fixes:
tag.

As a general rule of thumb, it is better to have lots of small, easy
to review patches, than an couple of big patch. So i would probably
have:

1) Fix
2) License change.
3) Split patch LS-WXL/WSXL
4) Split patch LS-WVL/VL

The license change needs a different sort of review to the other
patches, so having it as a separate patch helps with the review
process.

	Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help