Le 11/28/16 à 18:50, Andrew Lunn a écrit :
>> +struct bcm7xxx_phy_priv {
>> + u64 *stats;
>> +};
>
>> +static int bcm7xxx_28nm_probe(struct phy_device *phydev)
>> +{
>> + struct bcm7xxx_phy_priv *priv;
>> +
>> + priv = devm_kzalloc(&phydev->mdio.dev, sizeof(*priv), GFP_KERNEL);
>> + if (!priv)
>> + return -ENOMEM;
>> +
>> + phydev->priv = priv;
>> +
>> + priv->stats = devm_kzalloc(&phydev->mdio.dev,
>> + bcm_phy_get_sset_count(phydev), GFP_KERNEL);
>
> Hi Florian
>
> Should there be a * sizeof(u64) in there?
It should thanks for noticing!
--
Florian
