Re: [PATCH v2 1/6] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs

Path	csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Olof Johansson <olof@lixom.net>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 1/6] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs
Date	Wed, 19 Apr 2017 15:00:03 +0200
Message-ID	<txXnd-3yQ-63@gated-at.bofh.it> (permalink)
References	<t982t-5bz-1@gated-at.bofh.it> <t982u-5bz-19@gated-at.bofh.it> <tdslY-1bG-11@gated-at.bofh.it> <tfydA-Zv-3@gated-at.bofh.it>
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=lixom-net.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Z/Ibe6Ryym0cHdeCALlaTnn8jJUn+sIGnSzO8bbHxg4=; b=NTixhFzeMEyKN2PxGCbYNpgWyP0k5jzAwtiAzmzrIBa1aTjQBfl1atn0/NhjurrcnN 2izZBaUDAw6q3U9flxAtwfcPqirG9P1nznm7wmaHBcttYdkzCBqLnOeYvMcOUOrO5+8+ mnN09qh49RbImVZIGhzlgqQCznxPlLSNjrII0uLflXo1cqOQpln4VYjL2I+97Px+dStd kQGIzDAFEk2PwL5WYURfD94htky1zYLI6KUe/3xJ94KaHxo+sUPeTzSTRvGNS8WWHYdX aShkkj4yJ41I7MmcoFoLVhsfp1Zl9gkwP7QRAszVNWxvkF4PhdQGrSrWmSImyEi+oCw7 fRVw==
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=Z/Ibe6Ryym0cHdeCALlaTnn8jJUn+sIGnSzO8bbHxg4=; b=gHk1RSNEfhCdpHuDNW50f+4+cecc5OaZwqrYP7lHNaH1lX4ACPTCBIzhZzbP9rmLb2 uwQe7S3bL0pu/SA7z9wB86s3117h9l5Nba8UEMUuZefl6ni22h9BntZ3oNHfm5kiGVmX ABw3i/IMaKAFGdhtRsmNKv/6c1lUq6oLkFfw9b8O3O4u0tYkNlg+5qFSl0MdMJIuKxpo kd+QGUx4Y0dmjK/JizS/NAisIQ+vydwLM7CL1F0HnNm7G4S1j3M+UcM0IhMXni4CN/YG j5br224akcd4Kpa1T8Sp+vriQTWMppoTgQBMXDhee7Y2EqrwEcYUS8V0xyZHiT7Y3E0M Mg2g==
X-Gm-Message-State	AN3rC/5fjlMvU2CBUZ/aWFB1pbSp4+beh3F96uoj9TPNLlyhp+YK/IbQ NPhsUg3PmhEOjlpuOpKoQLmaizvRSg==
X-Received	by 10.176.4.39 with SMTP id 36mr1056150uav.154.1492606450522; Wed, 19 Apr 2017 05:54:10 -0700 (PDT)
MIME-Version	1.0
X-Originating-IP	[222.106.197.2]
Content-Type	text/plain; charset=UTF-8
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	43
Organization	linux.* mail to news gateway
X-Original-Cc	Brian Norris <briannorris@chromium.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Doug Anderson <dianders@chromium.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "open list:ARM/Rockchip SoC..." <linux-rockchip@lists.infradead.org>, Rob Herring <robh+dt@kernel.org>, Chris Zhong <zyw@rock-chips.com>, Stephen Barber <smbarber@chromium.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, Caesar Wang <wxt@rock-chips.com>
X-Original-Date	Wed, 19 Apr 2017 21:54:09 +0900
X-Original-Message-ID	<CAOesGMirj2j5P7_Ri4CSCux+O3hH_wxvrra7-p33AdXO4Ej5Hg@mail.gmail.com>
X-Original-References	<20170210010521.78872-1-briannorris@chromium.org> <20170210010521.78872-2-briannorris@chromium.org> <CAOesGMiim2WFfEAg-bJmWTwWn69=V_GuEg6AxsxPYMyzd3KFUw@mail.gmail.com> <1529949.5djtX72aeJ@phil>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1626092

Show key headers only | View raw

Hi,

On Tue, Feb 28, 2017 at 3:20 AM, Heiko Stuebner <heiko@sntech.de> wrote:
> Hi Olof,
>
> Am Dienstag, 21. Februar 2017, 15:47:31 CET schrieb Olof Johansson:
>> On Thu, Feb 9, 2017 at 5:05 PM, Brian Norris <briannorris@chromium.org> wrote:
>> > From: Douglas Anderson <dianders@chromium.org>
>> >
>> > We'd like to be able to use the cros-ec-keyboard.dtsi and
>> > cros-ec-sbs.dtsi snippets for arm64 devices.  Currently those files live
>> > in the arm/boot/dts directory.
>> >
>> > Let's follow the convention set by commit 8ee57b8182c4 ("ARM64: dts:
>> > vexpress: Use a symlink to vexpress-v2m-rs1.dtsi from arch=arm") and use
>> > a symlink.  Note that in this case we put the files in a new
>> > "include/common" directory since these snippets may need to be
>> > referenced by dts files in many different subdirectories.
>>
>> I'd rather have something like this:
>>
>> https://marc.info/?m=147547436324674&w=2
>>
>> Instead of having everybody move things over. I.e. make it easy to
>> refer to the arm version from arm64 instead of creating a "common"
>> layer inbetween.
>
> just so it gets noticed, I've done and tested [0], which hopefully should
> implement your suggestions above.
>
> If that looks ok, how do you want that picked up? Should I just include
> them in my regular rockchip branches or do you to pick them into some
> immutable branch, if other surprise-users turn up in time for 4.12?

Sigh. I completely dropped the ball on this, and I didn't see it
included in any of your pull requests for 4.12 since I never actually
acked that approach.

I've applied the patches onto a dt/include-paths stable branch, but
we're late for merging dependent code on top of it for 4.12.


-Olof

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

Thread

Re: [PATCH v2 1/6] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs Olof Johansson <olof@lixom.net> - 2017-04-19 15:00 +0200
  Re: [PATCH v2 1/6] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs Heiko Stuebner <heiko@sntech.de> - 2017-04-19 15:30 +0200
    Re: [PATCH v2 1/6] arm64: dts: Add symlinks for cros-ec-keyboard and cros-ec-sbs Olof Johansson <olof@lixom.net> - 2017-04-19 15:40 +0200

csiph-web