On 2015년 09월 01일 04:57, Emil Velikov wrote: > On 31 August 2015 at 20:27, Tobias Jakobi <tjakobi@xxxxxxxxxxxxxxxxxxxxx> wrote: >> Hello! >> >> Inki Dae wrote: >>> On 2015년 08월 24일 23:13, Tobias Jakobi wrote: > >>>> + if (g2d_check_space(ctx, 7, 1)) >>>> + return -ENOSPC; >>> >>> You can make 3 and 4 patches to one. These should be same patch. >> Hmm, so which 3 (respectively 4) patches should be squashed? >> > I believe he meant "squash the introduction of the function and its > uses into a single patch". > > Not sure how much value that brings, so I'll let you guys decide on > the bike shed colour :-) That - including relevant my comments - is just my opinion so no problem to upstream it as is. This is really trivial. Thanks, Inki Dae > > -Emil > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html