Yes that all looks good. I don't see anything that stands out as wrong.
But, OkiePC and JeremyM are bringing up potential issues with the choices for IP addresses. I'm not sure how fluent you are with IP addressing. I apologize if this gets too technical. It would help if we knew two additional settings for your system. What is your subnet mask and what is your Gateway Address? These are configuration settings for the PLCs Ethernet modules.
In most networks we have a common subnet mask that is used. That is typically 255.255.255.0. There are devices that use this mask by default. That mask allows for a network containing 256 IP (254 useable) addresses.
For example, this might result in 192.168.1.0 through 192.168.1.255 as our available range. This is often called our "subnet". 192.168.2.0 through 192.168.2.255 is an entirely different subnet. Devices on those two subnets would not be able to communicate to each other without the use of a expensive router (routers are also identified as a Gateway). Unless...
Unless a less common subnet mask were used. Something like 255.255.0.0. That mask allows for 65,536 (65,534 useable) IP addresses. With that mask our subnet would start at 192.168.0.0 and end at 192.168.255.255. The first and last addresses on any subnet are reserved, so the usable addresses would be 192.168.0.1 through 192.168.255.254. This would be a potentially huge network that could possibly have issues with broadcast traffic. Not at all recommended for a control system.
We could also use 255.255.252.0. That mask allows for 1,024 (1,022 useable) IP addresses. With that mask our subnet would start at 192.168.0.0 and end at 192.168.3.255. The first and last addresses on any subnet are reserved, so the usable addresses would be 192.168.0.1 through 192.168.3.254. Also not great for a control system.
If there is a router in the system, then that helps. A router can connect multiple subnets together and permit them to exchange data. But ideally, I want my PLCs on the same subnet so they don't have to go through another device to reach their destinations. Routers add a bit of a delay that is usually not desirable in a control system.
The other issue is that as I mentioned there are "usable" IP addresses and reserved IP addresses. The first and the last address on a subnet are reserved. So if we are using 255.255.255.0 as our subnet mask for the 192.168.1.0 subnet, 192.168.1.1 through 192.168.1.254 are usable. But, the .1 address is commonly used for routers so we usually don't use that address. This is not a hard and fast rule. But it is a common practice to reserve the .1 address for a router to use.
What I have described here is just using some addresses as examples. It can get much more complex that what I have detailed. Ideally, try to make sure your PLCs are on the same subnet whenever possible.
OG