2023-07-05 14:21:12

by Avram Lubkin

[permalink] [raw]
Subject: Re: Bug introduced in 724ba67 (ARM: dts: Move .dts files to vendor sub-directories)

Resending in plain text mode

SImple bug introduced in 724ba6751532055db75992fc6ae21c3e322e94a7.

The file
arch/arm/boot/dts/vexpress-v2m-rs1.dtsi
was moved to
arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi

However,
arch/arm64/boot/dts/arm/vexpress-v2m-rs1.dtsi
is a symlink and still points to the old location
../../../../arm/boot/dts/vexpress-v2m-rs1.dtsi

My backup software is complaining, though I expect it will cause more
significant issues for others.

Apologies if someone already reported. I didn't see it when I searched.


2023-07-06 08:55:04

by Sudeep Holla

[permalink] [raw]
Subject: Re: Bug introduced in 724ba67 (ARM: dts: Move .dts files to vendor sub-directories)

On Wed, Jul 05, 2023 at 10:16:19AM -0400, Avram Lubkin wrote:
> Resending in plain text mode
>
> SImple bug introduced in 724ba6751532055db75992fc6ae21c3e322e94a7.
>
> The file
> arch/arm/boot/dts/vexpress-v2m-rs1.dtsi
> was moved to
> arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi
>
> However,
> arch/arm64/boot/dts/arm/vexpress-v2m-rs1.dtsi
> is a symlink and still points to the old location
> ../../../../arm/boot/dts/vexpress-v2m-rs1.dtsi
>

It doesn't break any compilation as it is taken care. I will submit a patch
to remove the symlink soon. Thanks for the report.


--
Regards,
Sudeep

2023-07-13 18:47:43

by Avram Lubkin

[permalink] [raw]
Subject: Re: Bug introduced in 724ba67 (ARM: dts: Move .dts files to vendor sub-directories)

Thanks for submitting a patch. I saw it was acked. Will that get
merged soon? It shouldn't need to for the next merge window since it's
fixing an issue, right?
Just wondering if I need to work around this or not.

Thanks,
Avram

On Thu, Jul 6, 2023 at 4:45 AM Sudeep Holla <[email protected]> wrote:
>
> On Wed, Jul 05, 2023 at 10:16:19AM -0400, Avram Lubkin wrote:
> > Resending in plain text mode
> >
> > SImple bug introduced in 724ba6751532055db75992fc6ae21c3e322e94a7.
> >
> > The file
> > arch/arm/boot/dts/vexpress-v2m-rs1.dtsi
> > was moved to
> > arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi
> >
> > However,
> > arch/arm64/boot/dts/arm/vexpress-v2m-rs1.dtsi
> > is a symlink and still points to the old location
> > ../../../../arm/boot/dts/vexpress-v2m-rs1.dtsi
> >
>
> It doesn't break any compilation as it is taken care. I will submit a patch
> to remove the symlink soon. Thanks for the report.
>
>
> --
> Regards,
> Sudeep

2023-07-14 11:44:05

by Sudeep Holla

[permalink] [raw]
Subject: Re: Bug introduced in 724ba67 (ARM: dts: Move .dts files to vendor sub-directories)

On Thu, Jul 13, 2023 at 01:38:48PM -0400, Avram Lubkin wrote:
> Thanks for submitting a patch. I saw it was acked. Will that get
> merged soon? It shouldn't need to for the next merge window since it's
> fixing an issue, right?

Yes I will send it as fix soon, just queued for -next testing last night.

> Just wondering if I need to work around this or not.

Not for long, I hope it will be merged soonish after I send pull request.

--
Regards,
Sudeep