lkml.org 
[lkml]   [2014]   [Oct]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/2] ARM: dts: vf500/vf610: support VF500 SoC
    Am 2014-10-02 17:28, schrieb Arnd Bergmann:
    > On Thursday 02 October 2014 16:55:23 Stefan Agner wrote:
    >> The VF500 is essentially the same SoC, but with only one core and
    >> without L1 cache. The VF610 is therefore a superset of the VF500.
    >> Move allmost all periperals to vf500.dtsi which is then included
    >> and enhanced by vf610.dtsi.
    >>
    >> Signed-off-by: Stefan Agner <stefan@agner.ch>
    >> ---
    >> Somehow using -M -B switches create a unapplyable patch, hence I
    >> generated one without those options. Sorry about that.
    >>
    >> arch/arm/boot/dts/vf500.dtsi | 498 ++++++++++++++++++++++++++++++++++++++++++
    >> arch/arm/boot/dts/vf610.dtsi | 507 +------------------------------------------
    >> 2 files changed, 510 insertions(+), 495 deletions(-)
    >> create mode 100644 arch/arm/boot/dts/vf500.dtsi
    >
    > I think it would be better to create one extra file that contains the common
    > parts and is included by both vf500 and vf610.
    >

    I also thought about that. But when looking at the product variants, its
    clear that VF500 really contains only features, which are part of all
    other Vybrids (VF3xx series excluded, but these series is not really
    suitable for Linux due to lack of external memory). So vf500.dtsi would
    always be a oneliner (#include <vf.dtsi>).

    Also I disliked this idea because of the somewhat unclear naming
    (vf.dtsi? vfxxx.dtsi?)

    --
    Stefan


    \
     
     \ /
      Last update: 2014-10-02 18:21    [W:6.830 / U:0.056 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site