Received-SPF: pass (google.com: domain of linux-kernel+bounces-4260-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <d03e3dfc-ba3c-4474-8094-94c5557021f2@phytec.com>
Date: Mon, 18 Dec 2023 10:47:38 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/2] *** Add fixed-paritions to phyCORE-AM62x and
 phyCORE-AM64x ***
To: Andrew Davis <afd@ti.com>, nm@ti.com, vigneshr@ti.com, kristo@kernel.org,
 robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, upstream@lists.phytec.de, w.egorov@phytec.de
References: <20231215191838.1925082-1-nmorrisson@phytec.com>
 <9f86870c-88cc-43ea-800c-3a05482953aa@ti.com>
Content-Language: en-US
From: Nathan Morrisson <nmorrisson@phytec.com>
In-Reply-To: <9f86870c-88cc-43ea-800c-3a05482953aa@ti.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?NDFjVHk2cCtKMXMrNCtvV2xaMEpnbmMwYnFZbTNUdjdYd3V3aWlOeW5hNG12?=
 =?utf-8?B?eG5FbDQxcjRENEVTL3RPby83TVk1Y3BYVzFtd25CWXFYWXRXV0UwTE5ldlJQ?=
 =?utf-8?B?Nko5MlFVMGpQUWpWVXMxRDBSZE5NSGdkSkZHRWY4eWdwRUExbFRqZHZUNEY4?=
 =?utf-8?B?c2FLc2pURFV5VXBOdVV3ZTRJUmYxbFF0cHY1MmQ0aEhFNVBGMkl1OHVRMGlN?=
 =?utf-8?B?akJqcFo0OTNLck03SzR1YTd3S2NxLzFlNnQ0b2NOMGZuMy9ud2ZLUmc4aVUx?=
 =?utf-8?B?TDMxays1aTcvTHFmb01hYVU2SVg1TFBCNUVPZDdGWURROFRYSmZ0endWNnF2?=
 =?utf-8?B?NkZWSmZVaXl6YUxLaWdxcVlXeU1BbGNVY05KL2lUbEtOUG9EWGZTdnlDTjYy?=
 =?utf-8?B?djdvaCtvK2JRWGRFN3F6NzNnNEFCMExJYW5nSEEvVGpJc0w3RTU2R1FUbEhG?=
 =?utf-8?B?QWpTeXAxTG9tZk9QcHNkUmhzanNMVlZZOWVLTDF2dEFicGUzb1IycVRYVFQ4?=
 =?utf-8?B?dU5wMGRIVFVDbE1GUUZybEFHbm40RE55RTFxVzFXejJiR2szUFBYcVRjU3JT?=
 =?utf-8?B?blp4UUpEQ25tR2VrUU1SRnNtTTJHUzhtckpoMmlIelkzM2Y3NjlJbkVOQU1W?=
 =?utf-8?B?ZGF5eW1tTktXUTFsN1QxcUdCS0dMTFlxRHRjUnY2MW5VS25pYUtMNERONkhX?=
 =?utf-8?B?UkJBNlpnanl3SkdRSjAvUHNOM05sUlQyRE5qVnVQOEtMbE5waU9YUnM0MEgr?=
 =?utf-8?B?amRzYnJ2d0VmSU8wODI3YU1KQ205Z2IreVRXTEpidEpRb1hCMFBiaHp3dTEr?=
 =?utf-8?B?bTlMY0pIc3dHMkVFc3RCdlZtM0lsdENOcE9vRGdxK3Y2MjJHWGhraFA1NDNT?=
 =?utf-8?B?WUFzWGtGekc4eHRPUXlEdlkraGFsNVlPaGo2NEpLb09rNVhodWU0dUNpaEM4?=
 =?utf-8?B?ekZCc1pkVnlncUw5N25hSHdNaUJqYnNlTlVMcmZOZ3hYZWl5VU45NTU2ajhY?=
 =?utf-8?B?OGtFOVJncE02dmJKaFdlaG8rbVI4blh3aW1iUmRqNTl3YlNLTWsvT0lOUDVs?=
 =?utf-8?B?akFOMnR4eCs4Q29FZWlveVpsSXJjRGJOeEkzMHpSanF0ZVR6TjY1cU0rbUVQ?=
 =?utf-8?B?YXJjdDVHd2Y0c1lPM2Z1OWNmWXR5L1h4M0V5QUZ0VmpaL2ZSMWdHOG5UR3NV?=
 =?utf-8?B?RTVEZkJjSHNUN3Q1elZvQzE5MnFOVTJBQythQWZBVHl6dTVObTQrbVh0NGds?=
 =?utf-8?B?U3I0bWNWVDdwaFozWU9zRVlaVTVxRzhDVFR3K3Fwc2RBSHFVWU43aExDcTRa?=
 =?utf-8?B?MW1CRGlGd0RObC81cjI5SHhzbk5XaHZZTTRkNkZvazF1dHh6NDMrZDdkQXpw?=
 =?utf-8?B?VEtLc2hYK3ZRbmFHSVREQ3pLbzhvajcwYVd5emFsZmMrd1hscXE4eE9oYU91?=
 =?utf-8?B?S2ZadUVOeFhRQ0poSmtkMnR1dmpnS0QvWC9PM1dBZWh1bGkvdkxwelBzRGZr?=
 =?utf-8?B?SFhDbUpPck1KMVliL1BQZ3dwS0FES2ptTnlmbkl5RFl3Yi9aT2I3N1E1STBh?=
 =?utf-8?B?dm9yVHFVTjF2QkV3K21QSnN6eGdvZlcvQm1sUHZLanAvMkYrbzY3aEE5cndE?=
 =?utf-8?B?bThiZURvbjYyRmYxL3BxMU9Ma1l2MlRJUXhPU1I1M2ZaRklYMW4vQ2EweXFJ?=
 =?utf-8?B?R256MXN6cmgyU3hvWTliVHY3YUE5MEdSSEk4SVo3cldhUHR4RFZ1SHQ2c0NQ?=
 =?utf-8?B?c1hsZ2Y4QlV2alZyc242Z21wVG9WVTZ5bkVzZkM1WSt6NER6UjdNUVJYWSts?=
 =?utf-8?B?WjRCNmV0RDR2cUpUQ0hJbnVIa2xwK05wQkk5eFI5czQzQ2dEWkZnTTJRWlYr?=
 =?utf-8?B?L3hXTXE5MmFJQ2lTYlFDTllUdUUvTFM0Z09UTEFQRVcxTDJFNnJWR1lMT3hr?=
 =?utf-8?B?MjJNalhtVFc5SkpPelhxYWF1WEJ2Nmg2UTFlYXhqQUhUUGhSU3BRSzQrdVEx?=
 =?utf-8?B?YVZyaFBRRGtiTlZkN3dPNnI0Y28zUnd0MGljOG9IMzJBanF5SGRPSXowUGw5?=
 =?utf-8?B?U0tGTDdZRCtockdyQmV2b09zM1U5b3dNZTJ4ckNvdlRNbGQvMHFiMXp2VExT?=
 =?utf-8?Q?O4Om8guL5MY1tYTStGOoOM8CA?=
X-OriginatorOrg: phytec.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1a356312-15ac-436a-3059-08dbfff9d0c3
X-MS-Exchange-CrossTenant-AuthSource: DM5PR2201MB1515.namprd22.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Dec 2023 18:47:41.9562
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 67bcab1a-5db0-4ee8-86f4-1533d0b4b5c7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: f8DeoB9hykyhW788bbTBf9ejLFGl8ziZmnLJ3dKtUs2aJ4jWUzO4cGsdrzC70xb6vfCqGGXHn0D4VAJdEINOpQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR22MB3103

Hi Andrew,

We are adding the nodes directly to the device tree to match what is 
done in other device trees, for example: k3-am642-tqma64xxl.dtsi. You 
are correct that this could be done by the bootloader instead though. 
Why do you prefer having the bootloader add the nodes?

Regards,

Nathan

On 12/15/23 11:20 AM, Andrew Davis wrote:
> On 12/15/23 1:18 PM, Nathan Morrisson wrote:
>> Add a fixed-partitions node to the NOR flash of the phyCORE-AM62x
>> and phyCORE-AM64x to enable the bootloader to fixup the partitions
>> during boot.
>>
>
> Why can't your bootloader add these nodes?
>
> Andrew
>
>> Nathan Morrisson (2):
>>    arm64: dts: ti: k3-am62-phycore-som: Add fixed-partitions to NOR 
>> Flash
>>    arm64: dts: ti: k3-am64-phycore-som: Add fixed-partitions to NOR 
>> Flash
>>
>>   arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi | 6 ++++++
>>   arch/arm64/boot/dts/ti/k3-am64-phycore-som.dtsi | 6 ++++++
>>   2 files changed, 12 insertions(+)
>>