Re: [PATCH v4 1/3] arm: dts: mt8135: Move pinfunc to include/dt-bindings/pinctrl

From: Linus Walleij
Date: Wed Aug 11 2021 - 04:49:08 EST


On Thu, Aug 5, 2021 at 11:02 AM Matthias Brugger <matthias.bgg@xxxxxxxxx> wrote:

> Hi Linus,
>
> On 04/08/2021 06:40, Hsin-Yi Wang wrote:
> > Move mt8135-pinfunc.h into include/dt-bindings/pinctrl so that we can
> > include it in yaml examples.
> >
> > Signed-off-by: Hsin-Yi Wang <hsinyi@xxxxxxxxxxxx>
> > ---
> > arch/arm/boot/dts/mt8135.dtsi | 2 +-
> > .../boot/dts => include/dt-bindings/pinctrl}/mt8135-pinfunc.h | 0
> > 2 files changed, 1 insertion(+), 1 deletion(-)
> > rename {arch/arm/boot/dts => include/dt-bindings/pinctrl}/mt8135-pinfunc.h (100%)
> >
>
> If that's fine with you, I'll take patch 1+2 through my tree. IMHO the best for
> patch 3 would be to go through your tree.
>
> Sounds good?

It can't be done that way. Patch 3 depends on patch 1+2 to get the
include file into the right place for patch 3/3 to compile. (YAML
check.)

I will apply all three and then provide an immutable branch that you
can pull in to your tree as well. This is usually what we do with
cross-tree dependencies.

Yours,
Linus Walleij