netdev - Re: [PATCHv3 2/3] net: dsa: mt7530: support the 7530 switch on the Mediatek MT7621 SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20190122211307.Horde.V6XcMCE3ZFBZOJMYl12Uxex@www.vdorst.com>
Date:   Tue, 22 Jan 2019 21:13:07 +0000
From:   René van Dorst <opensource@...rst.com>
To:     Greg Ungerer <gerg@...nel.org>
Cc:     Sean Wang <sean.wang@...iatek.com>, bjorn@...k.no, andrew@...n.ch,
        vivien.didelot@...oirfairelinux.com, f.fainelli@...il.com,
        netdev@...r.kernel.org, john@...ozen.org, neil@...wn.name,
        linux-mediatek@...ts.infradead.org
Subject: Re: [PATCHv3 2/3] net: dsa: mt7530: support the 7530 switch on the
 Mediatek MT7621 SoC

Quoting Greg Ungerer <gerg@...nel.org>:

<snip>

>>>    	/* Reset whole chip through gpio pin or memory-mapped  
>>> registers for @@ -1326,9 +1335,17 @@ static const struct  
>>> dsa_switch_ops mt7530_switch_ops = {
>>>  	.port_vlan_del		= mt7530_port_vlan_del,
>>>  };
>>>  +static const struct of_device_id mt7530_of_match[] = {
>>> +	{ .compatible = "mediatek,mt7621", .data = (void *)ID_MT7621, },
>>> +	{ .compatible = "mediatek,mt7530", .data = (void *)ID_MT7530, },
>>> +	{ /* sentinel */ },
>>> +};
>>> +MODULE_DEVICE_TABLE(of, mt7530_of_match);
>>> +
>>>  static int
>>>  mt7530_probe(struct mdio_device *mdiodev)  {
>>> +	const struct of_device_id *of_id;
>>>  	struct mt7530_priv *priv;
>>>  	struct device_node *dn;
>>>  @@ -1356,13 +1373,21 @@ mt7530_probe(struct mdio_device *mdiodev)
>>>  		}
>>>  	}
>>>  -	priv->core_pwr = devm_regulator_get(&mdiodev->dev, "core");
>>> -	if (IS_ERR(priv->core_pwr))
>>> -		return PTR_ERR(priv->core_pwr);
>>> +	/* Get the hardware identifier from the devicetree node.
>>> +	 * We will need it for some of the clock and regulator setup.
>>> +	 */
>>> +	of_id = of_match_node(mt7530_of_match, dn);
>>> +	priv->id = (unsigned int)of_id->data;
>>>
>>
>> We could use of_device_get_match_data instead of the above two lines.
>
> Yep, good idea. Will change. I will add in some extra
> casting to take care of the kbuild warning too.
>
>
>> and could you help Cc linux-mediatek@...ts.infradead.org if you have the
>> next version for the series ?
>
> Yep, sure thing.
>
> Regards
> Greg

Hi Greg,

What about enabling mips mt7621/arm mt7623 code at compile time?
Same binary can't be used on both platforms so it saves some space.
This also solved/eliminates the need for CPU detection which was also  
needed if we want to support trgmii mode.
Device tree bindings are now similar between the two platforms, no  
extra compatible id.

I think it should look something like this:  
https://github.com/vDorst/linux-1/commit/93dc90ce56baeb3e71cae0f5339b811bbda57a97

Greats,

René