On Wed, Nov 18, 2020 at 09:30:07PM +0100, Christian Eggers wrote:
> Interrupts are required for TX time stamping. Probably they could also
> be used for PHY connection status.
> 
> This patch only adds the basic infrastructure for interrupts, no
> interrupts are finally enabled nor handled.
> 
> Signed-off-by: Christian Eggers <cegg...@arri.de>
> ---

Reviewed-by: Vladimir Oltean <olte...@gmail.com>

> +static int ksz9477_enable_port_interrupts(struct ksz_device *dev, bool 
> enable)
> +{
> +     u32 data, mask = GENMASK(dev->port_cnt - 1, 0);
> +     int ret;
> +
> +     ret = ksz_read32(dev, REG_SW_PORT_INT_MASK__4, &data);
> +     if (ret)
> +             return ret;
> +
> +     /* bits in REG_SW_PORT_INT_MASK__4 are low active */

s/low active/active low/

> +     if (enable)
> +             data &= ~mask;
> +     else
> +             data |= mask;
> +
> +     return ksz_write32(dev, REG_SW_PORT_INT_MASK__4, data);
> +}

Reply via email to