Hi,
On Thu, Sep 23, 2021 at 09:57:29PM +0300, dsankou...@gmail.com wrote:
> From: Dzmitry Sankouski <dsankou...@gmail.com>
>
> Signed-off-by: Dzmitry Sankouski <dsankou...@gmail.com>
> Cc: Ramon Fried <rfried....@gmail.com>
> Cc: Tom Rini <tr...@konsulko.com>
> ---
> arch/arm/mach-snapdragon/pinctrl-sdm845.c | 44 +++++++++++++++++++
> arch/arm/mach-snapdragon/pinctrl-snapdragon.c | 1 +
> arch/arm/mach-snapdragon/pinctrl-snapdragon.h | 1 +
> drivers/gpio/msm_gpio.c | 1 +
> drivers/gpio/pm8916_gpio.c | 8 ++--
> 5 files changed, 52 insertions(+), 3 deletions(-)
> create mode 100644 arch/arm/mach-snapdragon/pinctrl-sdm845.c
>
> diff --git a/arch/arm/mach-snapdragon/pinctrl-sdm845.c
> b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
> new file mode 100644
> index 0000000000..6d66582aa6
> --- /dev/null
> +++ b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
> @@ -0,0 +1,44 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Qualcomm SDM845 pinctrl
> + *
> + * (C) Copyright 2021 Dzmitry Sankouski <dsankou...@gmail.com>
> + *
> + */
> +
> +#include "pinctrl-snapdragon.h"
> +#include <common.h>
> +
> +#define MAX_PIN_NAME_LEN 32
> +static char pin_name[MAX_PIN_NAME_LEN];
Since this is used before relocation (when setting up pinctrl for
serial) you should really add __section(".data") here. I would expect
that you are currently corrupting some random part of your appended DTB
and are just lucky enough this does not cause any crashes. :)
See:
https://source.denx.de/u-boot/u-boot/-/commit/548b89f8ad396d238d594f6f6fa1df902b74a303
Thanks,
Stephan
