* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
@ 2019-03-30 6:15 Troy Benjegerdes
2019-03-31 22:02 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Troy Benjegerdes @ 2019-03-30 6:15 UTC (permalink / raw)
To: u-boot
I attempted to merge in the latest master branch into
https://github.com/sifive/u-boot/tree/sandbox and I got
the following error, which seems somewhat broken..
Device Tree Source is not correctly specified.
Please define 'CONFIG_DEFAULT_DEVICE_TREE'
or build with 'DEVICE_TREE=<device_tree>' argument
dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
Does anyone have any idea what would have caused that?
Also any feedback or pointers on why the serial_sifive.c
driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
without CONFIG_DM_SERIAL does seem to work might be appreciated.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
2019-03-30 6:15 [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures Troy Benjegerdes
@ 2019-03-31 22:02 ` Tom Rini
2019-04-02 20:16 ` Troy Benjegerdes
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2019-03-31 22:02 UTC (permalink / raw)
To: u-boot
On Fri, Mar 29, 2019 at 11:15:48PM -0700, Troy Benjegerdes wrote:
> I attempted to merge in the latest master branch into
> https://github.com/sifive/u-boot/tree/sandbox and I got
> the following error, which seems somewhat broken..
>
>
> Device Tree Source is not correctly specified.
> Please define 'CONFIG_DEFAULT_DEVICE_TREE'
> or build with 'DEVICE_TREE=<device_tree>' argument
>
> dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
>
>
> Does anyone have any idea what would have caused that?
>
> Also any feedback or pointers on why the serial_sifive.c
> driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
> without CONFIG_DM_SERIAL does seem to work might be appreciated.
You'll need to do something like
commit 89c2b5c02049aea746b1edee0b4e1d8519dec2f4
Author: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Thu Mar 14 14:58:33 2019 +0900
ARM: fix arch/arm/dts/Makefile
Since commit 27cb7300ffda ("Ensure device tree DTS is compiled"),
build succeeds irrespective of the correctness of Makefile.
In fact, you can compile any defconfig without adding any entry in
arch/*/dts/Makefile.
As a result, a lot of wrong code have been merged unnoticed.
I am going to revert that commit, and lots of hidden issues have
come to light:
[1] Typos
armada-3720-uDPU.dts, sun8i-a83t-tbs-a711.dts
use the extension ".dts" instead of ".dtb"
[2] DTB is associated to undefined CONFIG option
For example, mx6sllevk_defconfig defines CONFIG_MX6SLL, but
associates its device tree to CONFIG_MX6SL, which is undefined.
[3] Lots of entries are missing
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Chris Packham <judge.packham@gmail.com>
[trini: add imx6ul pico dtbs]
Signed-off-by: Tom Rini <trini@konsulko.com>
But to arch/riscv/dts/Makefile
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190331/245a6dd7/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
2019-03-31 22:02 ` Tom Rini
@ 2019-04-02 20:16 ` Troy Benjegerdes
2019-04-03 11:18 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Troy Benjegerdes @ 2019-04-02 20:16 UTC (permalink / raw)
To: u-boot
> On Mar 31, 2019, at 5:02 PM, Tom Rini <trini@konsulko.com> wrote:
>
> On Fri, Mar 29, 2019 at 11:15:48PM -0700, Troy Benjegerdes wrote:
>
>> I attempted to merge in the latest master branch into
>> https://github.com/sifive/u-boot/tree/sandbox and I got
>> the following error, which seems somewhat broken..
>>
>>
>> Device Tree Source is not correctly specified.
>> Please define 'CONFIG_DEFAULT_DEVICE_TREE'
>> or build with 'DEVICE_TREE=<device_tree>' argument
>>
>> dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
>>
>>
>> Does anyone have any idea what would have caused that?
>>
>> Also any feedback or pointers on why the serial_sifive.c
>> driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
>> without CONFIG_DM_SERIAL does seem to work might be appreciated.
>
> You'll need to do something like
>
> commit 89c2b5c02049aea746b1edee0b4e1d8519dec2f4
> Author: Masahiro Yamada <yamada.masahiro@socionext.com>
> Date: Thu Mar 14 14:58:33 2019 +0900
>
> ARM: fix arch/arm/dts/Makefile
>
> Since commit 27cb7300ffda ("Ensure device tree DTS is compiled"),
> build succeeds irrespective of the correctness of Makefile.
>
> In fact, you can compile any defconfig without adding any entry in
> arch/*/dts/Makefile.
>
> As a result, a lot of wrong code have been merged unnoticed.
>
> I am going to revert that commit, and lots of hidden issues have
> come to light:
>
> [1] Typos
>
> armada-3720-uDPU.dts, sun8i-a83t-tbs-a711.dts
>
> use the extension ".dts" instead of ".dtb"
>
> [2] DTB is associated to undefined CONFIG option
>
> For example, mx6sllevk_defconfig defines CONFIG_MX6SLL, but
> associates its device tree to CONFIG_MX6SL, which is undefined.
>
> [3] Lots of entries are missing
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Reviewed-by: Chris Packham <judge.packham@gmail.com>
> [trini: add imx6ul pico dtbs]
> Signed-off-by: Tom Rini <trini@konsulko.com>
>
> But to arch/riscv/dts/Makefile
>
> --
> Tom
I ended up reverting that commit, and we now have https://github.com/sifive/u-boot/tree/sandbox booting to a prompt based on upstream master.
It still needs the macb driver ported over, and probably re-worked to do what Rockchip boards do to put the Denali memory init code in the device tree.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
2019-04-02 20:16 ` Troy Benjegerdes
@ 2019-04-03 11:18 ` Tom Rini
2019-04-03 11:21 ` Masahiro Yamada
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2019-04-03 11:18 UTC (permalink / raw)
To: u-boot
On Tue, Apr 02, 2019 at 03:16:10PM -0500, Troy Benjegerdes wrote:
>
> > On Mar 31, 2019, at 5:02 PM, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Fri, Mar 29, 2019 at 11:15:48PM -0700, Troy Benjegerdes wrote:
> >
> >> I attempted to merge in the latest master branch into
> >> https://github.com/sifive/u-boot/tree/sandbox and I got
> >> the following error, which seems somewhat broken..
> >>
> >>
> >> Device Tree Source is not correctly specified.
> >> Please define 'CONFIG_DEFAULT_DEVICE_TREE'
> >> or build with 'DEVICE_TREE=<device_tree>' argument
> >>
> >> dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
> >>
> >>
> >> Does anyone have any idea what would have caused that?
> >>
> >> Also any feedback or pointers on why the serial_sifive.c
> >> driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
> >> without CONFIG_DM_SERIAL does seem to work might be appreciated.
> >
> > You'll need to do something like
> >
> > commit 89c2b5c02049aea746b1edee0b4e1d8519dec2f4
> > Author: Masahiro Yamada <yamada.masahiro@socionext.com>
> > Date: Thu Mar 14 14:58:33 2019 +0900
> >
> > ARM: fix arch/arm/dts/Makefile
> >
> > Since commit 27cb7300ffda ("Ensure device tree DTS is compiled"),
> > build succeeds irrespective of the correctness of Makefile.
> >
> > In fact, you can compile any defconfig without adding any entry in
> > arch/*/dts/Makefile.
> >
> > As a result, a lot of wrong code have been merged unnoticed.
> >
> > I am going to revert that commit, and lots of hidden issues have
> > come to light:
> >
> > [1] Typos
> >
> > armada-3720-uDPU.dts, sun8i-a83t-tbs-a711.dts
> >
> > use the extension ".dts" instead of ".dtb"
> >
> > [2] DTB is associated to undefined CONFIG option
> >
> > For example, mx6sllevk_defconfig defines CONFIG_MX6SLL, but
> > associates its device tree to CONFIG_MX6SL, which is undefined.
> >
> > [3] Lots of entries are missing
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> > Reviewed-by: Chris Packham <judge.packham@gmail.com>
> > [trini: add imx6ul pico dtbs]
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> >
> > But to arch/riscv/dts/Makefile
>
> I ended up reverting that commit, and we now have
> https://github.com/sifive/u-boot/tree/sandbox booting to a prompt
> based on upstream master.
Note that you just need something like:
obj-y += first.dtb \
second.dtb
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190403/a37cf0fa/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
2019-04-03 11:18 ` Tom Rini
@ 2019-04-03 11:21 ` Masahiro Yamada
2019-04-03 11:29 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Masahiro Yamada @ 2019-04-03 11:21 UTC (permalink / raw)
To: u-boot
On Wed, Apr 3, 2019 at 8:19 PM Tom Rini <trini@konsulko.com> wrote:
>
> On Tue, Apr 02, 2019 at 03:16:10PM -0500, Troy Benjegerdes wrote:
> >
> > > On Mar 31, 2019, at 5:02 PM, Tom Rini <trini@konsulko.com> wrote:
> > >
> > > On Fri, Mar 29, 2019 at 11:15:48PM -0700, Troy Benjegerdes wrote:
> > >
> > >> I attempted to merge in the latest master branch into
> > >> https://github.com/sifive/u-boot/tree/sandbox and I got
> > >> the following error, which seems somewhat broken..
> > >>
> > >>
> > >> Device Tree Source is not correctly specified.
> > >> Please define 'CONFIG_DEFAULT_DEVICE_TREE'
> > >> or build with 'DEVICE_TREE=<device_tree>' argument
> > >>
> > >> dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
> > >>
> > >>
> > >> Does anyone have any idea what would have caused that?
> > >>
> > >> Also any feedback or pointers on why the serial_sifive.c
> > >> driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
> > >> without CONFIG_DM_SERIAL does seem to work might be appreciated.
> > >
> > > You'll need to do something like
> > >
> > > commit 89c2b5c02049aea746b1edee0b4e1d8519dec2f4
> > > Author: Masahiro Yamada <yamada.masahiro@socionext.com>
> > > Date: Thu Mar 14 14:58:33 2019 +0900
> > >
> > > ARM: fix arch/arm/dts/Makefile
> > >
> > > Since commit 27cb7300ffda ("Ensure device tree DTS is compiled"),
> > > build succeeds irrespective of the correctness of Makefile.
> > >
> > > In fact, you can compile any defconfig without adding any entry in
> > > arch/*/dts/Makefile.
> > >
> > > As a result, a lot of wrong code have been merged unnoticed.
> > >
> > > I am going to revert that commit, and lots of hidden issues have
> > > come to light:
> > >
> > > [1] Typos
> > >
> > > armada-3720-uDPU.dts, sun8i-a83t-tbs-a711.dts
> > >
> > > use the extension ".dts" instead of ".dtb"
> > >
> > > [2] DTB is associated to undefined CONFIG option
> > >
> > > For example, mx6sllevk_defconfig defines CONFIG_MX6SLL, but
> > > associates its device tree to CONFIG_MX6SL, which is undefined.
> > >
> > > [3] Lots of entries are missing
> > >
> > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> > > Reviewed-by: Chris Packham <judge.packham@gmail.com>
> > > [trini: add imx6ul pico dtbs]
> > > Signed-off-by: Tom Rini <trini@konsulko.com>
> > >
> > > But to arch/riscv/dts/Makefile
> >
> > I ended up reverting that commit, and we now have
> > https://github.com/sifive/u-boot/tree/sandbox booting to a prompt
> > based on upstream master.
>
> Note that you just need something like:
> obj-y += first.dtb \
> second.dtb
It should be:
dtb-y += first.dtb \
second.dtb
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures
2019-04-03 11:21 ` Masahiro Yamada
@ 2019-04-03 11:29 ` Tom Rini
0 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2019-04-03 11:29 UTC (permalink / raw)
To: u-boot
On Wed, Apr 03, 2019 at 08:21:34PM +0900, Masahiro Yamada wrote:
> On Wed, Apr 3, 2019 at 8:19 PM Tom Rini <trini@konsulko.com> wrote:
> >
> > On Tue, Apr 02, 2019 at 03:16:10PM -0500, Troy Benjegerdes wrote:
> > >
> > > > On Mar 31, 2019, at 5:02 PM, Tom Rini <trini@konsulko.com> wrote:
> > > >
> > > > On Fri, Mar 29, 2019 at 11:15:48PM -0700, Troy Benjegerdes wrote:
> > > >
> > > >> I attempted to merge in the latest master branch into
> > > >> https://github.com/sifive/u-boot/tree/sandbox and I got
> > > >> the following error, which seems somewhat broken..
> > > >>
> > > >>
> > > >> Device Tree Source is not correctly specified.
> > > >> Please define 'CONFIG_DEFAULT_DEVICE_TREE'
> > > >> or build with 'DEVICE_TREE=<device_tree>' argument
> > > >>
> > > >> dts/Makefile:28: recipe for target 'arch/riscv/dts/fu540.dtb' failed
> > > >>
> > > >>
> > > >> Does anyone have any idea what would have caused that?
> > > >>
> > > >> Also any feedback or pointers on why the serial_sifive.c
> > > >> driver with CONFIG_DM_SERIAL would not work, but serial_hifive.c
> > > >> without CONFIG_DM_SERIAL does seem to work might be appreciated.
> > > >
> > > > You'll need to do something like
> > > >
> > > > commit 89c2b5c02049aea746b1edee0b4e1d8519dec2f4
> > > > Author: Masahiro Yamada <yamada.masahiro@socionext.com>
> > > > Date: Thu Mar 14 14:58:33 2019 +0900
> > > >
> > > > ARM: fix arch/arm/dts/Makefile
> > > >
> > > > Since commit 27cb7300ffda ("Ensure device tree DTS is compiled"),
> > > > build succeeds irrespective of the correctness of Makefile.
> > > >
> > > > In fact, you can compile any defconfig without adding any entry in
> > > > arch/*/dts/Makefile.
> > > >
> > > > As a result, a lot of wrong code have been merged unnoticed.
> > > >
> > > > I am going to revert that commit, and lots of hidden issues have
> > > > come to light:
> > > >
> > > > [1] Typos
> > > >
> > > > armada-3720-uDPU.dts, sun8i-a83t-tbs-a711.dts
> > > >
> > > > use the extension ".dts" instead of ".dtb"
> > > >
> > > > [2] DTB is associated to undefined CONFIG option
> > > >
> > > > For example, mx6sllevk_defconfig defines CONFIG_MX6SLL, but
> > > > associates its device tree to CONFIG_MX6SL, which is undefined.
> > > >
> > > > [3] Lots of entries are missing
> > > >
> > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> > > > Reviewed-by: Chris Packham <judge.packham@gmail.com>
> > > > [trini: add imx6ul pico dtbs]
> > > > Signed-off-by: Tom Rini <trini@konsulko.com>
> > > >
> > > > But to arch/riscv/dts/Makefile
> > >
> > > I ended up reverting that commit, and we now have
> > > https://github.com/sifive/u-boot/tree/sandbox booting to a prompt
> > > based on upstream master.
> >
> > Note that you just need something like:
> > obj-y += first.dtb \
> > second.dtb
>
> It should be:
>
> dtb-y += first.dtb \
> second.dtb
Ah right, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190403/c3ff7c48/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-04-03 11:29 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-30 6:15 [U-Boot] Riscv: CONFIG_DEFAULT_DEVICE_TREE failures Troy Benjegerdes
2019-03-31 22:02 ` Tom Rini
2019-04-02 20:16 ` Troy Benjegerdes
2019-04-03 11:18 ` Tom Rini
2019-04-03 11:21 ` Masahiro Yamada
2019-04-03 11:29 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.