On Thu, Jun 01, 2023 at 12:47:03PM +0530, Vinod Koul wrote: > On 31-05-23, 10:26, Krzysztof Kozlowski wrote: > > On 30/05/2023 18:24, Vinod Koul wrote: > > > This add interconnect nodes and add LMH to sc8180x SoC dtsi > > > > > > Co-developed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > > > Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > > > Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx> > > > --- > > > > I don't understand why this was split. We talked on IRC many times on > > this - artificial splits are not "release early, release often". Your > > previous patchset was correct in that approach, but why this is separate > > patch? > > Coz the patch was big to review. This is usual Linux approach to break a > change into smaller chunks for review! > We break patches into small, logical units so that it's easy to follow the thought through each step in the process of introducing a change. This is not the same thing as splitting one logical change into multiple smaller patches to keep the line count of each patch down. This just forces the reviewer to jump between emails to get the full picture of the logical change. And it's not until patch 14 that any of this content is introduced to the build system, so it's not split to ensure bisectability etc. I'd be perfectly happy to have received the dts changes as 3-4 patches. Regards, Bjorn
