On Mon, Oct 01, 2018 at 05:37:58PM +0800, Icenowy Zheng wrote:
> >> dtb-$(CONFIG_MACH_SUN9I) += \
> >> sun9i-a80-optimus.dtb \
> >> sun9i-a80-cubieboard4.dtb \
> >> diff --git a/arch/arm/dts/sun50i-a64-pinebook.dts
> >b/arch/arm/dts/sun50i-a64-pinebook.dts
> >> new file mode 100644
> >> index 0000000000..48bee4cc7f
> >> --- /dev/null
> >> +++ b/arch/arm/dts/sun50i-a64-pinebook.dts
> >> @@ -0,0 +1,99 @@
> >> +/*
> >> + * Copyright (c) 2016 ARM 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.
> >> + */
> >> +
> >> +/dts-v1/;
> >> +
> >> +#include <dt-bindings/pwm/pwm.h>
> >> +#include "sun50i-a64-pine64.dts"
> >> +
> >> +/ {
> >> + model = "Pinebook";
> >> + compatible = "pine64,pinebook", "allwinner,sun50i-a64";
> >> +
> >> + aliases {
> >> + serial0 = &uart0;
> >> + i2c0 = "/i2c@1f02400";
> >> + };
> >> +
> >> + vdd_bl: regulator@0 {
> >> + compatible = "regulator-fixed";
> >> + regulator-name = "bl-3v3";
> >> + regulator-min-microvolt = <3300000>;
> >> + regulator-max-microvolt = <3300000>;
> >> + gpio = <&pio 7 6 GPIO_ACTIVE_HIGH>; /* PH6 */
> >> + enable-active-high;
> >> + };
> >> +
> >> +
> >> + backlight: backlight {
> >> + compatible = "pwm-backlight";
> >> + pwms = <&pwm 0 50000 0>;
> >> + brightness-levels = <0 10 20 30 40 50 60 70 100>;
> >> + default-brightness-level = <3>;
> >> + enable-gpios = <&pio 3 23 GPIO_ACTIVE_HIGH>; /* PD23 */
> >> + power-supply = <&vdd_bl>;
> >> + };
> >> +
> >> + chosen {
> >> + stdout-path = "serial0:115200n8";
> >> + };
> >> +
> >> + memory {
> >> + reg = <0x40000000 0x40000000>;
> >> + };
> >> +};
> >> +
> >> +&pwm {
> >> + status = "okay";
> >> +};
> >> +
> >> +&r_i2c {
> >> + pinctrl-names = "default";
> >> + pinctrl-0 = <&r_i2c_pins_a>;
> >> + status = "okay";
> >> + anx6345: edp-bridge@38 {
> >> + compatible = "analogix,anx6345";
> >> + reg = <0x38>;
> >> + reset-gpios = <&pio 3 24 GPIO_ACTIVE_LOW>; /* PD24 */
> >> + status = "okay";
> >> + };
> >> +};
> >
> >I'm not sure why that DT is that different from the one we have in
> >Linux. Ideally, they should be identical.
>
> Linux one has no ANX6345 yet, so they're not identical.Right, but it's far from being the only difference. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
