<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/drivers/net/mvpp2.c, branch main</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/drivers/net/mvpp2.c?h=main</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/drivers/net/mvpp2.c?h=main'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2026-06-10T09:23:26Z</updated>
<entry>
<title>net: mvpp2: fix NULL pointer dereference in mvpp2_phy_connect</title>
<updated>2026-06-10T09:23:26Z</updated>
<author>
<name>Vincent Jardin</name>
<email>vjardin@free.fr</email>
</author>
<published>2025-12-16T00:53:52Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=50b3c87e0e2b6037dfa4dd9ee1e0ddb42165e835'/>
<id>urn:sha1:50b3c87e0e2b6037dfa4dd9ee1e0ddb42165e835</id>
<content type='text'>
Fix two NULL pointer dereferences in mvpp2_phy_connect():

1. port-&gt;phy_dev-&gt;dev is used in dev_warn() but
   port-&gt;phy_dev is not assigned yet (assigned later at line below).

2. port-&gt;phy_dev-&gt;dev is used in dev_err() inside
   the "if (!phy_dev)" block, which means phy_dev is NULL.

Both cases would cause a crash if the PHY detection fails or returns
a generic PHY. Use the already available 'dev' parameter instead.

Fixes: 9db60ee470c2 ("net: mvpp2: Convert netdev_xxx to dev_xxx")
Signed-off-by: Vincent Jardin &lt;vjardin@free.fr&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: convert FDT access to ofnode API</title>
<updated>2026-06-03T14:55:55Z</updated>
<author>
<name>Peng Fan</name>
<email>peng.fan@nxp.com</email>
</author>
<published>2026-05-28T08:00:24Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=dafa6a36037b516bed3c4f578c69e0c5c8017acb'/>
<id>urn:sha1:dafa6a36037b516bed3c4f578c69e0c5c8017acb</id>
<content type='text'>
Convert mvpp2 driver from legacy fdtdec/fdt_* APIs to the ofnode-based
interfaces.

Replace usage of dev_of_offset(), fdtdec_lookup_phandle(),
fdtdec_get_int(), fdt_parent_offset(), and related helpers with their
ofnode equivalents, including dev_ofnode(), ofnode_parse_phandle(),
ofnode_read_s32_default(), ofnode_get_parent(), and
ofnode_for_each_subnode().

Remove direct dependencies on gd-&gt;fdt_blob.

Main changes:
- Use ofnode_valid() instead of integer checks for node presence
- Switch fixed-link detection to ofnode_find_subnode()
- Replace uclass_get_device_by_of_offset() with
  uclass_get_device_by_ofnode()
- Update subnode iteration and device binding to use ofnode

No functional changes.

Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Signed-off-by: Peng Fan &lt;peng.fan@nxp.com&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: Use dev_read_addr_index_ptr()</title>
<updated>2026-06-03T14:55:55Z</updated>
<author>
<name>Peng Fan</name>
<email>peng.fan@nxp.com</email>
</author>
<published>2026-05-28T08:00:23Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f603d10d72bf6a341b2af238693f17e671e4bc07'/>
<id>urn:sha1:f603d10d72bf6a341b2af238693f17e671e4bc07</id>
<content type='text'>
Use dev_read_addr_index_ptr() which supports both live device tree and
flat DT backends, avoiding direct dependency on devfdt_* helpers.

No functional changes.

Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Signed-off-by: Peng Fan &lt;peng.fan@nxp.com&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: Use field just assigned in error test</title>
<updated>2025-08-18T13:47:57Z</updated>
<author>
<name>Andrew Goodbody</name>
<email>andrew.goodbody@linaro.org</email>
</author>
<published>2025-08-05T14:18:19Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=c6561a467c9f9a77717ddc2ef296866a4d8a6843'/>
<id>urn:sha1:c6561a467c9f9a77717ddc2ef296866a4d8a6843</id>
<content type='text'>
In mvpp2_probe the code attempts to get a value for "gop-port-id" and
assigns it to port-&gt;gop_id but it then tests port-&gt;id for being equal to
-1. That is an impossible test as port-&gt;id is a field of type u8 so
cannot be negative. Change the test to port-&gt;gop_id.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody &lt;andrew.goodbody@linaro.org&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: Cannot test unsigned variable to be negative</title>
<updated>2025-08-18T13:47:57Z</updated>
<author>
<name>Andrew Goodbody</name>
<email>andrew.goodbody@linaro.org</email>
</author>
<published>2025-08-05T14:18:18Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3aa3d37282414ea5c19ea415f46d214f4c80d8b6'/>
<id>urn:sha1:3aa3d37282414ea5c19ea415f46d214f4c80d8b6</id>
<content type='text'>
In phy_info_parse all uses of the variable phyaddr are as an int so
declaring as u32 is not useful and prevents the test for an error return
from fdtdec_get_int ever detecting an error. Change phyaddr to be an
int.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody &lt;andrew.goodbody@linaro.org&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: Return -ENOMEM for failed alloc</title>
<updated>2025-08-18T13:47:57Z</updated>
<author>
<name>Andrew Goodbody</name>
<email>andrew.goodbody@linaro.org</email>
</author>
<published>2025-08-05T14:18:17Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6781b90f89b427ec9445c43f8102ebe0a4330e54'/>
<id>urn:sha1:6781b90f89b427ec9445c43f8102ebe0a4330e54</id>
<content type='text'>
Instead of returning -1 on a failed alloc, return -ENOMEM.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody &lt;andrew.goodbody@linaro.org&gt;
</content>
</entry>
<entry>
<title>net: mvpp2: Fix impossible test</title>
<updated>2025-08-18T13:47:57Z</updated>
<author>
<name>Andrew Goodbody</name>
<email>andrew.goodbody@linaro.org</email>
</author>
<published>2025-08-05T14:18:16Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=b24c89affd9063c3f9f3ca99db075bd3bb1fc4b2'/>
<id>urn:sha1:b24c89affd9063c3f9f3ca99db075bd3bb1fc4b2</id>
<content type='text'>
You cannot test an unsigned char to be &gt;= 256. Instead make the
variables start and end to be ints.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody &lt;andrew.goodbody@linaro.org&gt;
</content>
</entry>
<entry>
<title>treewide: drop no-op prefetch() calls</title>
<updated>2025-06-18T14:05:04Z</updated>
<author>
<name>Rasmus Villemoes</name>
<email>ravi@prevas.dk</email>
</author>
<published>2025-05-07T12:12:44Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f5fa73a625f671152aa65be4b039060f8baed4cc'/>
<id>urn:sha1:f5fa73a625f671152aa65be4b039060f8baed4cc</id>
<content type='text'>
These all end up using the no-op prefetch() defined in linux/list.h,
because the only possible real implementation is in
arch/mips/include/asm/processor.h, which is behing
CONFIG_CPU_HAS_PREFETCH which is nowhere defined.

In order to be able to drop that fallback definition from list.h,
first remove all uses.

Signed-off-by: Rasmus Villemoes &lt;ravi@prevas.dk&gt;
</content>
</entry>
<entry>
<title>drivers: net: Remove duplicate newlines</title>
<updated>2024-07-22T16:53:05Z</updated>
<author>
<name>Marek Vasut</name>
<email>marek.vasut+renesas@mailbox.org</email>
</author>
<published>2024-07-20T12:40:45Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f3974bec32f7627e123d9005f2c70b9a004aa857'/>
<id>urn:sha1:f3974bec32f7627e123d9005f2c70b9a004aa857</id>
<content type='text'>
Drop all duplicate newlines. No functional change.

Signed-off-by: Marek Vasut &lt;marek.vasut+renesas@mailbox.org&gt;
</content>
</entry>
<entry>
<title>Restore patch series "arm: dts: am62-beagleplay: Fix Beagleplay Ethernet"</title>
<updated>2024-05-20T19:35:03Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2024-05-20T19:35:03Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=03de305ec48b0bb28554372abb40ccd46dbe0bf9'/>
<id>urn:sha1:03de305ec48b0bb28554372abb40ccd46dbe0bf9</id>
<content type='text'>
As part of bringing the master branch back in to next, we need to allow
for all of these changes to exist here.

Reported-by: Jonas Karlman &lt;jonas@kwiboo.se&gt;
Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
</feed>
