zephyr icon indicating copy to clipboard operation
zephyr copied to clipboard

drivers: hwinfo: add Andes hwinfo driver

Open wtlee1995 opened this issue 2 years ago • 0 comments

Added hwinfo driver that supported reset cause on adp_xc7k_ae350.

Signed-off-by: Wei-Tai Lee [email protected]

wtlee1995 avatar Aug 03 '22 05:08 wtlee1995

Hi @alexanderwachter @carlocaione , are there any results from the architecture meeting?

In our internal discussions, it's better to have a syscon driver for adp_xc7k_ae350. Because we may have pinmux and power control function in the future. I'll have another PR that adds the generic syscon driver and use the driver in cache driver recently.

wtlee1995 avatar Aug 17 '22 09:08 wtlee1995

Hi @alexanderwachter @carlocaione , are there any results from the architecture meeting?

This was not put in the agenda AFAIK.

In our internal discussions, it's better to have a syscon driver for adp_xc7k_ae350. Because we may have pinmux and power control function in the future. I'll have another PR that adds the generic syscon driver and use the driver in cache driver recently.

I agree with this decision, thanks.

carlocaione avatar Aug 17 '22 09:08 carlocaione

@alexanderwachter I'll update hwinfo driver that uses needed macros directly instead of including "smu.h".

wtlee1995 avatar Aug 17 '22 10:08 wtlee1995

@wtlee1995 we removed it from the agenda because I got sick. Awaiting you new implementation then!

alexanderwachter avatar Aug 17 '22 10:08 alexanderwachter

@alexanderwachter Hope you get well soon!

Updated a temporary version for using macros from smu.h, because we'll remove smu.h in PR #49274

If there are any descisions from the meeting, we'll update the driver accordingly

wtlee1995 avatar Aug 19 '22 10:08 wtlee1995

API meeting:

  • No objections to using syscon_ to access the hardware
  • @wtlee1995 please rewrite as agreed with @carlocaione

carlescufi avatar Aug 23 '22 15:08 carlescufi