Subject: Re: [PATCH v2 1/1] ARM: dts: dra7: add entry for bb2d module
To:     Yongqin Liu <yongqin.liu@linaro.org>,
        Tony Lindgren <tony@atomide.com>
Cc:     linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, Gowtham Tammana <g-tammana@ti.com>,
        Jyri Sarha <jsarha@ti.com>
References: <20210921071807.30978-1-narmstrong@baylibre.com>
 <20210921071807.30978-2-narmstrong@baylibre.com>
 <YV1UdSVOrZ3B9pq/@atomide.com>
 <CAMSo37UN78k=WE0CwRyNNV3P9kau+JzVZ7mHOMMvh5Bn=+=jAQ@mail.gmail.com>
From:   Neil Armstrong <narmstrong@baylibre.com>
Organization: Baylibre
Message-ID: <78b51650-0e32-e81f-0191-2222580e7343@baylibre.com>
Date:   Tue, 23 Nov 2021 13:47:47 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.13.0
MIME-Version: 1.0
In-Reply-To: <CAMSo37UN78k=WE0CwRyNNV3P9kau+JzVZ7mHOMMvh5Bn=+=jAQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi,

On 23/11/2021 13:17, Yongqin Liu wrote:
> Hi, Neil, Tony
> 
> # sorry for the confusion if you have received the mail which I sent
> before, which is not in plain text mode.
> 
> We have one out of tree change to enable the SGX544 gpu for the
> beagleboard-X15 Android build,
>     https://android-review.linaro.org/c/kernel/common/+/20521/11/arch/arm/boot/dts/dra7.dtsi
> 
> and that seems to conflict with this BB2D enabling change,
> Could you please help give some suggestions on how we should update our patch
> to make it work with BB2D, without the revert of this change?

This BB2D patch alters the target-module@59000000 while your SGX
change alters the target-module@56000000.

Please rebase your patches.

Neil

> 
> On Wed, 6 Oct 2021 at 15:47, Tony Lindgren <tony@atomide.com> wrote:
>>
>> * Neil Armstrong <narmstrong@baylibre.com> [210921 10:18]:
>>> From: Gowtham Tammana <g-tammana@ti.com>
>>>
>>> BB2D is a Vivante GC 2D Accelerator.
>>> This adds the node to the dts file within a target module node.
>>> Crossbar index number is used for interrupt mapping.
>>
>> Thanks applying into omap-for-v5.16/dt.
>>
>> Tony
>