lkml.org 
[lkml]   [2017]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 2/5] arm64: dts: exynos: make tm2 and tm2e independent from each other
    Hi, 

    On Fri, Jan 06, 2017 at 09:09:11AM +0200, Krzysztof Kozlowski wrote:
    > On Fri, Jan 06, 2017 at 12:59:06PM +0900, Jaechul Lee wrote:
    > > From: Andi Shyti <andi.shyti@samsung.com>
    > >
    > > Currently tm2e dts includes tm2 but there are some differences
    > > between the two boards and tm2 has some properties that tm2e
    > > doesn't have.
    > >
    > > That's why it's important to keep the two dts files independent
    > > and put all the commonalities in a tm2-common.dtsi file.
    > >
    > > At the current status the only two differences between the two
    > > dts files (besides the board name) are ldo31 and ldo38.
    > >
    > > Signed-off-by: Andi Shyti <andi.shyti@samsung.com>
    > > Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
    > > ---
    > > .../boot/dts/exynos/exynos5433-tm2-common.dtsi | 1118 +++++++++++++++++++
    > > arch/arm64/boot/dts/exynos/exynos5433-tm2.dts | 1153 +-------------------
    >
    > Like talking to a wall. Without any feedback. If my instructions were
    > wrong (and it is not possible to detect rename) then please say it (you
    > can add personal comments after separator ---).

    no Krzysztof, I'm sorry, but this patch has been formatted with
    the diff algorithm *you* asked, -B50% both version 2 (where you
    didn't comment) and version 3. If you still don't like it, please
    don't blame me, blame the algorithm.

    Now we can stay here at trying random diff algorithms (as they
    give more or less the same result) or you tell me which exact
    algorithm you want me to use. Besides, for me it's clear,
    tm2-common is all new, while in tm2 you have on one side the '-'
    (if it applies nothing has changed) on the bottom the '+'.

    Andi

    \
     
     \ /
      Last update: 2017-01-06 08:59    [W:3.081 / U:0.152 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site