Thread (22 messages) flat view 22 messages, 3 authors, 2010-12-31
DORMANTno replies

[PATCH v3 1/8] ARM: mx5: use config to define boot related addresses

From: Uwe Kleine-König <hidden>
Date: 2010-12-31 16:23:36

Hello Shawn,

On Fri, Dec 31, 2010 at 10:01:57AM +0800, Shawn Guo wrote:
On Thu, Dec 30, 2010 at 12:05:06PM +0100, Uwe Kleine-K?nig wrote:
quoted
On Thu, Dec 30, 2010 at 07:00:26PM +0800, Richard Zhao wrote:
quoted
On Thu, Dec 30, 2010 at 10:31:27AM +0100, Uwe Kleine-K?nig wrote:
quoted
looks good

Acked-by: Uwe Kleine-K?nig <redacted>
Thanks.
For such patch acked, do I need to re-send it out when I send out v4 version
of the patch series?
As I'm not the imx-maintainer, I'd say yes, send it out (with my ack
added) in v4.
Richard was trying to understand the correct approach for sending
patch series.

I used to only send the updated patches in the new patch series
since the last series version, while Richard prefer to send all
the patches in the very single version, even most of them are not
changed since the last version.

I think the sending delta way could save bandwidth and ease reviewing
a little bit, while sending all may be easy for people to pick up
the patches. So what is the preference for you guys?
It's more usual to always send all patches of the series and that's what
I prefer.  IIRC Russell pointed that out for one of your series, too.
Optimally document for unchanged patches that they are unchanged.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help