Jerome Brunet <jbru...@baylibre.com> writes:

> Like order boards, order nodes by address then node names then aliases.
>
> Signed-off-by: Jerome Brunet <jbru...@baylibre.com>
> ---
>  .../boot/dts/amlogic/meson-g12a-sei510.dts    | 92 +++++++++----------
>  1 file changed, 46 insertions(+), 46 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts 
> b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
> index 34b40587e5ef..61fb30047d7f 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
> @@ -14,10 +14,6 @@
>       compatible = "seirobotics,sei510", "amlogic,g12a";
>       model = "SEI Robotics SEI510";
>  
> -     aliases {
> -             serial0 = &uart_AO;
> -     };
> -
>       adc_keys {
>               compatible = "adc-keys";
>               io-channels = <&saradc 0>;
> @@ -31,13 +27,8 @@
>               };
>       };
>  
> -     ao_5v: regulator-ao_5v {
> -             compatible = "regulator-fixed";
> -             regulator-name = "AO_5V";
> -             regulator-min-microvolt = <5000000>;
> -             regulator-max-microvolt = <5000000>;
> -             vin-supply = <&dc_in>;
> -             regulator-always-on;
> +     aliases {
> +             serial0 = &uart_AO;
>       };

minor nit: I kind of like "aliases" and "chosen" at the top since they
are kind of special nodes, but honestly, I can't think of a really good
reason other than personal preference, so keeping things sorted as
you've done here is probably better.

Kevin

Reply via email to