Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1625673

Re: [PATCH] ARM: dts: rockchip: reuse firefly dtsi

From Heiko Stuebner <heiko@sntech.de>
Newsgroups linux.kernel
Subject Re: [PATCH] ARM: dts: rockchip: reuse firefly dtsi
Date 2017-04-19 00:30 +0200
Message-ID <txJNg-3f2-13@gated-at.bofh.it> (permalink)
References <txAgW-64O-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Eddie,

Am Dienstag, 18. April 2017, 20:15:27 CEST schrieb Eddie Cai:
> firefly reload is very similar with firefly board, so reuse firefly dtsi
> 
> Signed-off-by: Eddie Cai <eddie.cai.linux@gmail.com>
> ---
>  arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi | 310 ------------------
>  arch/arm/boot/dts/rk3288-firefly-reload.dts       | 368 ++--------------------

I would disagree and remember having a similar discussion when the reload-
support was initially submitted. Please keep in mind that the firefly-
reload is a som+baseboard combination, so somebody could (or maybe
already has) create a completely different baseboard for the som that
does not have any similarities with the original firefly.
The previous firefly being a real single board.

We also don't combine rock2 and firefly and other boards following the
general rk3288 design guidelines :-) and the original firefly and reload are
very different boards if you look at them.


Heiko

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] ARM: dts: rockchip: reuse firefly dtsi Eddie Cai <eddie.cai.linux@gmail.com> - 2017-04-18 14:20 +0200
  Re: [PATCH] ARM: dts: rockchip: reuse firefly dtsi Heiko Stuebner <heiko@sntech.de> - 2017-04-19 00:30 +0200
    Re: [PATCH] ARM: dts: rockchip: reuse firefly dtsi Eddie Cai <eddie.cai.linux@gmail.com> - 2017-04-19 02:50 +0200
      Re: [PATCH] ARM: dts: rockchip: reuse firefly dtsi Heiko Stuebner <heiko@sntech.de> - 2017-04-19 12:00 +0200

csiph-web