Hello Steen Hegelund,
The patch 3cfa11bac9bb: "net: sparx5: add the basic sparx5 driver"
from Jun 24, 2021, leads to the following static checker warning:
drivers/net/ethernet/microchip/sparx5/sparx5_main.c:760 mchp_sparx5_probe()
error: uninitialized symbol 'mac_addr'.
drivers/net/ethernet/microchip/sparx5/sparx5_main.c
661 static int mchp_sparx5_probe(struct platform_device *pdev)
662 {
663 struct initial_port_config *configs, *config;
664 struct device_node *np = pdev->dev.of_node;
665 struct device_node *ports, *portnp;
666 struct reset_control *reset;
667 struct sparx5 *sparx5;
668 int idx = 0, err = 0;
669 u8 *mac_addr;
^^^^^^^^^^^^
670
671 if (!np && !pdev->dev.platform_data)
672 return -ENODEV;
673
674 sparx5 = devm_kzalloc(&pdev->dev, sizeof(*sparx5), GFP_KERNEL);
675 if (!sparx5)
676 return -ENOMEM;
677
678 platform_set_drvdata(pdev, sparx5);
679 sparx5->pdev = pdev;
680 sparx5->dev = &pdev->dev;
681
682 /* Do switch core reset if available */
683 reset = devm_reset_control_get_optional_shared(&pdev->dev, "switch");
684 if (IS_ERR(reset))
685 return dev_err_probe(&pdev->dev, PTR_ERR(reset),
686 "Failed to get switch reset controller.\n");
687 reset_control_reset(reset);
688
689 /* Default values, some from DT */
690 sparx5->coreclock = SPX5_CORE_CLOCK_DEFAULT;
691
692 ports = of_get_child_by_name(np, "ethernet-ports");
693 if (!ports) {
694 dev_err(sparx5->dev, "no ethernet-ports child node found\n");
695 return -ENODEV;
696 }
697 sparx5->port_count = of_get_child_count(ports);
698
699 configs = kcalloc(sparx5->port_count,
700 sizeof(struct initial_port_config), GFP_KERNEL);
701 if (!configs) {
702 err = -ENOMEM;
703 goto cleanup_pnode;
704 }
705
706 for_each_available_child_of_node(ports, portnp) {
707 struct sparx5_port_config *conf;
708 struct phy *serdes;
709 u32 portno;
710
711 err = of_property_read_u32(portnp, "reg", &portno);
712 if (err) {
713 dev_err(sparx5->dev, "port reg property error\n");
714 continue;
715 }
716 config = &configs[idx];
717 conf = &config->conf;
718 conf->speed = SPEED_UNKNOWN;
719 conf->bandwidth = SPEED_UNKNOWN;
720 err = of_get_phy_mode(portnp, &conf->phy_mode);
721 if (err) {
722 dev_err(sparx5->dev, "port %u: missing phy-mode\n",
723 portno);
724 continue;
725 }
726 err = of_property_read_u32(portnp, "microchip,bandwidth",
727 &conf->bandwidth);
728 if (err) {
729 dev_err(sparx5->dev, "port %u: missing bandwidth\n",
730 portno);
731 continue;
732 }
733 err = of_property_read_u32(portnp, "microchip,sd-sgpio", &conf->sd_sgpio);
734 if (err)
735 conf->sd_sgpio = ~0;
736 else
737 sparx5->sd_sgpio_remapping = true;
738 serdes = devm_of_phy_get(sparx5->dev, portnp, NULL);
739 if (IS_ERR(serdes)) {
740 err = dev_err_probe(sparx5->dev, PTR_ERR(serdes),
741 "port %u: missing serdes\n",
742 portno);
743 goto cleanup_config;
744 }
745 config->portno = portno;
746 config->node = portnp;
747 config->serdes = serdes;
748
749 conf->media = PHY_MEDIA_DAC;
750 conf->serdes_reset = true;
751 conf->portmode = conf->phy_mode;
752 conf->power_down = true;
753 idx++;
754 }
755
756 err = sparx5_create_targets(sparx5);
757 if (err)
758 goto cleanup_config;
759
760 if (of_get_mac_address(np, mac_addr)) {
^^^^^^^^
Never initialized
761 dev_info(sparx5->dev, "MAC addr was not set, use random MAC\n");
762 eth_random_addr(sparx5->base_mac);
763 sparx5->base_mac[5] = 0;
764 } else {
765 ether_addr_copy(sparx5->base_mac, mac_addr);
^^^^^^^^
766 }
767
768 sparx5->xtr_irq = platform_get_irq_byname(sparx5->pdev, "xtr");
769
770 /* Read chip ID to check CPU interface */
771 sparx5->chip_id = spx5_rd(sparx5, GCB_CHIP_ID);
772
773 sparx5->target_ct = (enum spx5_target_chiptype)
774 GCB_CHIP_ID_PART_ID_GET(sparx5->chip_id);
775
776 /* Initialize Switchcore and internal RAMs */
777 err = sparx5_init_switchcore(sparx5);
778 if (err) {
779 dev_err(sparx5->dev, "Switchcore initialization error\n");
780 goto cleanup_config;
781 }
782
783 /* Initialize the LC-PLL (core clock) and set affected registers */
784 err = sparx5_init_coreclock(sparx5);
785 if (err) {
786 dev_err(sparx5->dev, "LC-PLL initialization error\n");
787 goto cleanup_config;
788 }
789
790 for (idx = 0; idx < sparx5->port_count; ++idx) {
791 config = &configs[idx];
792 if (!config->node)
793 continue;
794
regards,
dan carpenter
