On Mon, Apr 09, 2018 at 03:38:43PM +0900, Kunihiko Hayashi wrote: > When the link is becoming up for Pro4 SoC, the kernel is stalled > due to some missing clocks and resets. > > The AVE block for Pro4 is connected to the GIO bus in the SoC. > Without its clock/reset, the access to the AVE register makes the > system stall. > > In the same way, another MAC clock for Giga-bit Connection and > the PHY clock are also required for Pro4 to activate the Giga-bit feature > and to recognize the PHY. > > To satisfy these requirements, this patch adds support for multiple clocks > and resets, and adds the clock-names and reset-names to the binding because > we need to distinguish clock/reset for the AVE main block and the others. > > Also, make the resets a required property. Currently, "reset is > optional" relies on that the bootloader or firmware has deasserted > the reset before booting the kernel. Drivers should work without > such expectation. > > Fixes: 4c270b55a5af ("net: ethernet: socionext: add AVE ethernet driver") > Suggested-by: Masahiro Yamada <yamada.masah...@socionext.com> > Signed-off-by: Kunihiko Hayashi <hayashi.kunih...@socionext.com> > --- > .../bindings/net/socionext,uniphier-ave4.txt | 13 ++-
Reviewed-by: Rob Herring <r...@kernel.org> > drivers/net/ethernet/socionext/sni_ave.c | 108 > ++++++++++++++++----- > 2 files changed, 96 insertions(+), 25 deletions(-)