Date:   Tue, 7 Mar 2023 15:33:20 +0100
From:   Andrew Lunn <andrew@lunn.ch>
To:     Vladimir Oltean <olteanv@gmail.com>
Cc:     Michael Walle <michael@walle.cc>, sean.anderson@seco.com,
        davem@davemloft.net, edumazet@google.com, f.fainelli@gmail.com,
        hkallweit1@gmail.com, kuba@kernel.org,
        linux-kernel@vger.kernel.org, linux@armlinux.org.uk,
        netdev@vger.kernel.org, pabeni@redhat.com, tobias@waldekranz.com
Subject: Re: [PATCH net-next] net: mdio: Add netlink interface
Message-ID: <7013dea3-a026-4a0c-81e0-7ebe6f708e39@lunn.ch>
References: <20230306204517.1953122-1-sean.anderson@seco.com>
 <20230307112307.777207-1-michael@walle.cc>
 <684c859a-02e2-4652-9a40-9607410f95e6@lunn.ch>
 <20230307140535.32ldprkyblpmicjg@skbuf>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230307140535.32ldprkyblpmicjg@skbuf>
Precedence: bulk

> - Atomic (why only atomic?) (read) access to paged registers

I would say 'atomic' is wrong, you cannot access paged registers at
all.

> are we ok with the implications?

I am. Anybody doing this level of debugging should be able to
recompile the kernel to enable write support. It does limit debugging
in field, where maybe you cannot recompile the kernel, but to me, that
is a reasonable trade off.

   Andrew