
On Mon, Jul 29, 2019 at 6:17 PM Joe Hershberger joe.hershberger@ni.com wrote:
On Thu, Jul 25, 2019 at 4:42 PM Anatolij Gustschin agust@denx.de wrote:
On Tue, 23 Jul 2019 04:26:17 +0000 Joe Hershberger joe.hershberger@ni.com wrote: ...
+config MV88E61XX_88E6020_FAMILY
bool "Marvell MV88E6020 family support."
help
The driver supports 6172/6176/6240/6352 devices in the
default configuration. Select this option to enable support
for 6250/6220/6020/6070/6071 switches.
Is there a rhyme or reason to the model numbers here? This option seems oddly named, especially since the help doesn't have the model numbers in numerical order. Can you make it a choice instead?
We want to be able to use single U-Boot images which support different switches, a choice will make it not possible.
I don't see how choice is any different than what you have here in that respect. You have a hard-coded selection among families. If you have a need to run the same binary on multiple switches, then this option should be removed and the pre-init should be detecting the target.
Any response here?
-Joe
-- Anatolij _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot