Re: [PATCH v4 2/5] drivers/of: do not add memory for unavailable nodes

From: Reza Arbab
Date: Sat Oct 22 2016 - 21:51:25 EST


Hi Alistair,

On Fri, Oct 21, 2016 at 05:22:54PM +1100, Alistair Popple wrote:
From what I can tell it seems that kernels without this patch will try and use this memory even if it is marked in the device-tree as status="disabled" which could lead to problems for older kernels when we start exporting this property from firmware.

Arguably this might not be such a problem in practice as we probably don't have many (if any) existing kernels that will boot on hardware exporting these properties.

Yes, I think you've got it right.

However given this patch seems fairly independent perhaps it is worth sending as a separate fix if it is not going to make it into this release?

Michael,

If this set as a whole is going to miss the release, would it be helpful for me to resend 1/5 and 2/5 as a separate set? They are the minimum needed to prevent the possible forward compatibility issue Alistair describes.

--
Reza Arbab