doc: release-notes-3.6: Update release notes for NXP additions #69100
Conversation
mmahadevan108
requested review from
dleach02,
manuargue,
bperseghetti,
yvanderv,
danieldegrasse,
decsny and
dbaluta
doc/releases/release-notes-3.6.rst
Outdated
| * Added a driver for NXP ENET which is a rework of the old ``eth_mcux.c`` driver which | ||
| had become unmaintainable due to fundamental problems with the lack of PHY abstraction. | ||
| ``eth_mcux.c`` and the corresponding compatible ``nxp,kinetis-ethernet`` will be deprecated | ||
| and this new driver will be supported instead. |
There was a problem hiding this comment.
Probably deserves a mention in the migration guide
There was a problem hiding this comment.
@decsny will work to add it to the migration guide.
There was a problem hiding this comment.
@mmahadevan108, this line item needs "EXPERIMENTAL" as this driver is still in experimental state.
@MaureenHelm, if the driver is experimental, would we put something in the migration guide to highlight how to use it?
There was a problem hiding this comment.
The migration guide is only for instructions needed for migrating. I'd put experimental drivers in the release notes.
There was a problem hiding this comment.
I'd out experimental drivers in the release notes.
Not sure what you are saying...
There was a problem hiding this comment.
Sorry, meant to write "put", not "out".
There was a problem hiding this comment.
I have added a mention that this driver is experimental.
doc/releases/release-notes-3.6.rst
Outdated
| * NXP: Enabled :ref:`linkserver<linkserver-debug-host-tools>` to be the default runner, | ||
| if none was set in the board's ``board.cmake`` file. |
There was a problem hiding this comment.
Didn't linkserver become the default runner on all supported boards?
There was a problem hiding this comment.
It was made the default for west command in this release.
There was a problem hiding this comment.
That needs to go in the migration guide then.
There was a problem hiding this comment.
Moved this portion to the migration guide.
mmahadevan108
dismissed stale reviews from dleach02 and henrikbrixandersen
via
d692400
mmahadevan108
requested review from
henrikbrixandersen,
MaureenHelm and
dleach02
doc/releases/migration-guide-3.6.rst
Outdated
| @@ -19,6 +19,8 @@ Boards | |||
|
|
|||
| * The deprecated Nordic SoC Kconfig option ``NRF_STORE_REBOOT_TYPE_GPREGRET`` has been removed, | |||
| applications that use this should switch to using the :ref:`boot_mode_api` instead. | |||
| * NXP: Enabled :ref:`linkserver<linkserver-debug-host-tools>` to be the default runner, | |||
| if none was set in the board's ``board.cmake`` file. | |||
There was a problem hiding this comment.
I don't understand the last part. The linkserver was made the default no matter what was set before?
There was a problem hiding this comment.
Rewording but this was set as the default only on NXP boards and not all of them yet. Still a work in progress. Will be clarifying the language here.
There was a problem hiding this comment.
I think brix is pointing out that currently the line here says "if the board did not have a default runner, we set it to linkserver", and that does not reflect the reality which is actually "we set many nxp boards to default runner linkserver regardless of whether they already had a default"
There was a problem hiding this comment.
This was a copy paste from the commit. I have updated the description with more information.
NXP additions for 3.6 release Signed-off-by: Mahesh Mahadevan <[email protected]>
NXP additions for 3.6 release