[U-Boot] please pull u-boot-samsung master

Przemyslaw Marczak p.marczak at samsung.com
Wed Nov 4 18:12:39 CET 2015


Hello Minkyu,

On 11/04/2015 04:43 PM, Minkyu Kang wrote:
> Dear Przemyslaw Marczak,
>
> On 4 November 2015 at 18:37, Przemyslaw Marczak <p.marczak at samsung.com
> <mailto:p.marczak at samsung.com>> wrote:
>
>     Hello Minkyu,
>
>     On 11/04/2015 03:05 AM, Minkyu Kang wrote:
>
>         Dear Przemyslaw Marczak,
>
>         2015년 11월 4일 수요일, Tom Rini<trini at konsulko.com
>         <mailto:trini at konsulko.com>
>
>              On Tue, Nov 03, 2015 at 02:02:57PM +0900, Minkyu Kang wrote:
>
>               > Dear Tom,
>               >
>               > The following changes since commit
>              0e067a65f57189703668826d9841fea477026bf6:
>               >
>               >   x86: Select the ns16550 debug UART for minnowmax,
>              chromebook_link (2015-10-30 18:04:14 -0400)
>               >
>               > are available in the git repository at:
>               >
>               > http://git.denx.de/u-boot-samsung
>               >
>               > for you to fetch changes up to
>              de5f9733f653091d4b227914d04432099a6448dc:
>               >
>               >   samsung: clean up checkpatch issues (2015-11-02
>         10:38:22 +0900)
>               >
>
>              NAK:
>
>              +(sandbox) ../drivers/adc/sandbox.c:11:25: fatal error:
>              sandbox-adc.h: No such file or directory
>              +(sandbox)  #include <sandbox-adc.h>
>              +(sandbox)                          ^
>              +(sandbox) compilation terminated.
>              +(sandbox) make[3]: *** [drivers/adc/sandbox.o] Error 1
>              +(sandbox) make[2]: *** [drivers/adc] Error 2
>              +(sandbox) make[1]: *** [drivers] Error 2
>
>              "sandbox: add ADC unit tests" is missing adding sandbox-adc.h
>
>
>         Could you please check it?
>
>              --
>              Tom
>
>
>         Thanks,
>         Minkyu Kang.
>
>
>         --
>         Thanks.
>         Minkyu Kang.
>
>
>     Sorry for this mistake. I have this file locally, so it build
>     properly for me.
>
>     Is that good to you if I send you a patch with this missing file, to
>     squash with one commit? Or do you prefer V3 of the whole patch-set?
>
>
> If squash your patch  then I should do push forced.
> I think it's not a good way.
> I will merge your patch separately.
>
> Thanks,
> Minkyu Kang.
> --
> from. prom.
> www.promsoft.net <http://www.promsoft.net>

Squashing would be better if we want prevent build break. This single 
file (include/sandbox-adc.h) will not conflict with any other, so it's 
easy to rebase it.
This is not a master tree, so I think that push with force will not 
cause any problems.

But if you really prefer to take it as a separate patch, then I will 
send one, with a proper commit message in a moment.

Best regards,
-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com


More information about the U-Boot mailing list