Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230326001535.149539-1-bigbeeshane@gmail.com>
 <20230326001535.149539-2-bigbeeshane@gmail.com> <10930783-e1dd-5e75-a2cc-a09af862d949@linaro.org>
 <CABnpCuCzdbWTTp1Gc6wWPsB80J5GYw1QzKtZzzyEYU456E=1JA@mail.gmail.com>
 <1b9c8e19-10f2-824d-9b50-51e7a9287bb1@linaro.org> <CABnpCuAWm7jh19JKukOquPnZCwHoJispgDPGJzjYy6T_BZSnbg@mail.gmail.com>
 <717bfba9-1d70-ef09-9193-853a57117926@linaro.org>
In-Reply-To: <717bfba9-1d70-ef09-9193-853a57117926@linaro.org>
From:   Shane Francis <bigbeeshane@gmail.com>
Date:   Sun, 26 Mar 2023 18:51:29 +0100
Message-ID: <CABnpCuA7V9ti6BOoz+3Mq-f=PwgxL2K7rOhjo6tJ4QquSZbaSA@mail.gmail.com>
Subject: Re: [PATCH 2/2] dt-bindings: clock: update rk3588 clock definitions
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        heiko@sntech.de, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

> Bootloader also does not use mainline DTS, so where is exactly the proble=
m?

In this case uboot does load the target DTB, and it seems a common
function in rockchips
downstream boot process, see :

https://github.com/rockchip-linux/u-boot/blob/ef1dd650042f61915c4859ecc9462=
3a09a3529fa/arch/arm/mach-rockchip/kernel_dtb.c#L70

It's a pretty nasty process, but unfortunately some devices will be
stuck with it I guess, and in
those cases they will have issues booting a mainline kernel.


Regards


On Sun, Mar 26, 2023 at 6:23=E2=80=AFPM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 26/03/2023 18:59, Shane Francis wrote:
> > Hi Krzysztof
> >
> >
> >> So mention this in the commit msg.
> >
> >> Then commit msg should also mention it.
> >
> > Sorry for not expanding on this more in the initial commit message, I w=
ill
> > expand on this in the next patch set.
> >
> > However I think in general for most modern platforms it can be assumed
> > that replacing the bootloader is not always something that is achievabl=
e
> > for one reason or another
>
> Bootloader also does not use mainline DTS, so where is exactly the proble=
m?
>
> Best regards,
> Krzysztof
>