[U-Boot] Merge conflict on Tegra SPI between u-boot/master and u-boot-arm/master

Tom Warren twarren.nvidia at gmail.com
Thu Mar 28 16:16:37 CET 2013


Albert,


On Thu, Mar 28, 2013 at 8:08 AM, Tom Warren <twarren.nvidia at gmail.com>wrote:

> Albert,
>
>
> On Thu, Mar 28, 2013 at 6:38 AM, Albert ARIBAUD <albert.u.boot at aribaud.net
> > wrote:
>
>> Hello all,
>>
>> While preparing for an ARM PR, specifically a merge between commit
>> 417c55803118eb8e350d5ab8ba6583fb39f4e2e3 (current u-boot/master ToT)
>> and commit d53e340edf65ff253d3a7b06ebe60501045892e3 (current Tot for
>> u-boot-arm/master), I hit the following merge conflicts:
>>
>> CONFLICT (content): Merge conflict in lib/fdtdec.c
>> CONFLICT (content): Merge conflict in include/fdtdec.h
>> CONFLICT (content): Merge conflict in drivers/spi/tegra20_sflash.c
>>
>> While the first two, related to FDT, are simple (colliding additions to
>> the same enum from both branches) and can be manually resolved, the
>> third one, drivers/spi/tegra20_sflash.c, is not, and can not, at least,
>> not by me, as I am not a specialist on Tegra SPI.
>>
>> Tom (Warren), can you advise on how to best solve this?
>>
> Let me take a look  - IIRC, Simon's use of spi_alloc_slave() was the only
> change to tegra20_sflash.c (replaces the normal malloc() in
> tegra20_spi_setup_slave()).
>
> Here's a diff of the only change to tegra20_sflash.c (merging Allen's
> Tegra SPI changes and Simon's SPI changes):
>
> --- drivers/spi/tegra20_sflash.c 2013-03-22 08:43:02.373219315 -0700
> +++ /home/tom/denx/u-boot-arm/drivers/spi/tegra20_sflash.c 2013-03-28
> 08:04:04.899860607 -0700
> @@ -127,7 +127,7 @@
>   return NULL;
>   }
>
> - spi = spi_alloc_slave(struct tegra_spi_slave, bus, cs);
> + spi = malloc(sizeof(struct tegra_spi_slave));
>   if (!spi) {
>   printf("SPI error: malloc of SPI structure failed\n");
>   return NULL;
>
> Let me know if that's not enough for you to resolve this,
>
> Tom
>

Just confirmed - in tegra20_sflash.c, remove the <<<<< HEAD section (#ifdef
CONFIG_OF_CONTROL + 2 lines), and keep the "tegra: spi remove non fdt
support" section (spi->slave.bus = bus + 4 lines).

Tom

>
>> Amicalement,
>> --
>> Albert.
>>
>
>


More information about the U-Boot mailing list