[PATCH] arm64: dts: rockchip: add roc-rk3328-cc board
Rob Herring
robh at kernel.org
Mon Jan 15 10:09:53 PST 2018
On Fri, Jan 12, 2018 at 2:33 AM, Heiko Stuebner <heiko at sntech.de> wrote:
> Hi Rob,
>
> Am Donnerstag, 11. Januar 2018, 14:30:33 CET schrieb Rob Herring:
>> > diff --git a/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts b/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts
>> > new file mode 100644
>> > index 0000000..2134baf
>> > --- /dev/null
>> > +++ b/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts
>> > @@ -0,0 +1,313 @@
>> > +/*
>> > + * Copyright (c) 2017 T-Chip Intelligent Technology Co., Ltd
>> > + *
>> > + * This file is dual-licensed: you can use it either under the terms
>> > + * of the GPL or the X11 license, at your option. Note that this dual
>> > + * licensing only applies to this file, and not this project as a
>> > + * whole.
>> > + *
>> > + * a) This library is free software; you can redistribute it and/or
>> > + * modify it under the terms of the GNU General Public License as
>> > + * published by the Free Software Foundation; either version 2 of the
>> > + * License, or (at your option) any later version.
>> > + *
>> > + * This library is distributed in the hope that it will be useful,
>> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> > + * GNU General Public License for more details.
>> > + *
>> > + * Or, alternatively,
>> > + *
>> > + * b) Permission is hereby granted, free of charge, to any person
>> > + * obtaining a copy of this software and associated documentation
>> > + * files (the "Software"), to deal in the Software without
>> > + * restriction, including without limitation the rights to use,
>> > + * copy, modify, merge, publish, distribute, sublicense, and/or
>> > + * sell copies of the Software, and to permit persons to whom the
>> > + * Software is furnished to do so, subject to the following
>> > + * conditions:
>> > + *
>> > + * The above copyright notice and this permission notice shall be
>> > + * included in all copies or substantial portions of the Software.
>> > + *
>> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> > + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>> > + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>> > + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>> > + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>> > + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> > + * OTHER DEALINGS IN THE SOFTWARE.
>> > + */
>>
>> Use SPDX license tag.
>
> It looks like Thomas' SPDX documentation patches are not yet merged
> and there were smaller protests about SPDX information only being
> available on some external website, so just for my understanding, SPDX
> is the agreed standard now, right?
His series adds the license text for all the tags which Russell said
addressed his concerns.
I think it is not yet merged mainly because Thomas has been busy with
Spectre and Meltdown.
> And judging by the SPDX conversion for Rockchip dts I have in my inbox
> the correct tags should probably be
> // SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>
> due to the
> "All devicetrees claim to be either GPL or X11 while the actual license
> text is MIT. Therefore we use MIT for the SPDX tag as X11 is clearly
> wrong."
Right.
>> > +&i2c1 {
>> > + status = "okay";
>> > +
>> > + rk805: rk805 at 18 {
>>
>> pmic at 18
>>
>> > + compatible = "rockchip,rk805";
>> > + reg = <0x18>;
>> > + interrupt-parent = <&gpio1>;
>> > + interrupts = <24 IRQ_TYPE_LEVEL_LOW>;
>> > + #clock-cells = <1>;
>> > + clock-output-names = "xin32k", "rk805-clkout2";
>> > + gpio-controller;
>> > + #gpio-cells = <2>;
>> > + pinctrl-names = "default";
>> > + pinctrl-0 = <&pmic_int_l>;
>> > + rockchip,system-power-controller;
>> > + wakeup-source;
>> > +
>> > + vcc1-supply = <&vcc_sys>;
>> > + vcc2-supply = <&vcc_sys>;
>> > + vcc3-supply = <&vcc_sys>;
>> > + vcc4-supply = <&vcc_sys>;
>> > + vcc5-supply = <&vcc_io>;
>> > + vcc6-supply = <&vcc_io>;
>> > +
>> > + regulators {
>> > + vdd_logic: DCDC_REG1 {
>>
>> Don't use '_' in node names.
>
> The driver binds against the node names including the underscores.
> (with regulator-compatible being deprecated)
Okay. You've probably told me that before (and will again next time I forget).
Rob
More information about the linux-arm-kernel
mailing list