Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <907b6b75-55bc-b38c-442b-4ccb036a3690@gmail.com>
Date:   Tue, 10 Jan 2023 00:27:16 -0600
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH 1/2] of: create of_root if no dtb provided
Content-Language: en-US
To:     =?UTF-8?B?Q2zDqW1lbnQgTMOpZ2Vy?= <clement.leger@bootlin.com>
Cc:     Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, Lizhi Hou <lizhi.hou@xilinx.com>,
        Allan Nielsen <allan.nielsen@microchip.com>,
        Horatiu Vultur <horatiu.vultur@microchip.com>,
        Steen Hegelund <steen.hegelund@microchip.com>,
        Thomas Petazzoni <thomas.petazzoni@bootlin.com>
References: <20220624034327.2542112-1-frowand.list@gmail.com>
 <20220624034327.2542112-2-frowand.list@gmail.com>
 <20220624141320.3c473605@fixe.home>
 <6d40876c-2751-01bb-94ab-7c9ab90e636f@gmail.com>
 <20221011092654.6c7d7ec3@fixe.home> <20230109094009.3878c30e@fixe.home>
From:   Frank Rowand <frowand.list@gmail.com>
In-Reply-To: <20230109094009.3878c30e@fixe.home>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 1/9/23 02:40, Clément Léger wrote:
> Le Tue, 11 Oct 2022 09:26:54 +0200,
> Clément Léger <clement.leger@bootlin.com> a écrit :
> 
>> Le Fri, 24 Jun 2022 11:44:07 -0500,
>> Frank Rowand <frowand.list@gmail.com> a écrit :
>>
>>> On 6/24/22 08:13, Clément Léger wrote:  
>>>> Le Thu, 23 Jun 2022 22:43:26 -0500,
>>>> frowand.list@gmail.com a écrit :
>>>>     
>>>>>  
>>>>> +/*
>>>>> + * __dtb_empty_root_begin[] magically created by cmd_dt_S_dtb in
>>>>> + * scripts/Makefile.lib
>>>>> + */
>>>>> +extern void *__dtb_empty_root_begin;
>>>>> +
>>>>>  /*
>>>>>   * of_fdt_limit_memory - limit the number of regions in the /memory node
>>>>>   * @limit: maximum entries
>>>>> @@ -1332,8 +1338,13 @@ bool __init early_init_dt_scan(void *params)
>>>>>   */
>>>>>  void __init unflatten_device_tree(void)
>>>>>  {    
>>>>   
>>
>>
>> Any news on this series ?
>>
> 
> Hi Frank,
> 
> Do you plan on resubmitting this series ? If not, could I resubmit it
> after fixing problems that were raised in the review ?
>> Thanks,
> 

Thanks for the prod.  I'll re-spin it.

If I properly captured all the comments, I'll have to implement
Rob's suggestion:

  "either CONFIG_OF_FLATTREE or CONFIG_OF_EARLY_FLATTREE will need 
  to become user selectable."

-Frank