[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <58123122.8020908@gmail.com>
Date: Thu, 27 Oct 2016 09:53:54 -0700
From: Frank Rowand <frowand.list@...il.com>
To: Pantelis Antoniou <panto@...oniou-consulting.com>
Cc: Rob Herring <robh+dt@...nel.org>,
devicetree <devicetree@...r.kernel.org>,
linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 13/13] of: Remove unused variable overlay_symbols
On 10/27/16 09:27, Frank Rowand wrote:
> On 10/27/16 07:41, Pantelis Antoniou wrote:
>> Hi Frank,
>>
>>
>>> On Oct 25, 2016, at 23:59 , frowand.list@...il.com wrote:
>>>
>>> From: Frank Rowand <frank.rowand@...sony.com>
>>>
>>> This unused variable is a reminder that symbols in overlays are
>>> not available to subsequent overlays. If such a feature is
>>> desired then there are several ways it could be implemented.
>>>
>>
>> Please don’t apply that. There’s a patch that actually imports
>> the symbol table from overlays that subsequent operations
>> work.
>>
>> Please see:
>>
>> https://patchwork.kernel.org/patch/9104701/
>
> Thanks for the pointer! When the import symbols patch is applied
> then the comment in my patch header becomes incorrect. I will
> change the patch comment to act is if the import symbols patch
> is in place.
>
> But the node pointer that my patch removes is still not used
> for anything, even if the import symbols patch is applied.
>
> Am I missing something?
I was missing a later patch in the symbol import patch set that
updated resolver.c to use the imported symbols. I'll go look
at that.
Powered by blists - more mailing lists