driver: hwinfo: npcx: Add hwinfo driver support in npcx family.

[MulinChao]

Add hwinfo driver support in npcx family.

Signed-off-by: Mulin Chao [email protected]

[alexanderwachter]

@MulinChao Thanks for this addition. Do you have a datasheet of this SoC. Do those registers provide unique values? From the naming, it sounds like they identify the chip and are the same for each chip.
The device ID should be a unique ID.

[MulinChao]

@MulinChao Thanks for this addition. Do you have a datasheet of this SoC. Do those registers provide unique values? From the naming, it sounds like they identify the chip and are the same for each chip.
The device ID should be a unique ID.

Ping our PM, @Michal-Sch, for further actions.
Hi @alexanderwachter, thanks for your comment. Regarding the datasheet, it is provided under NDA. Please refer to the discussion in PR27578

Related to the unique values of these registers, yes, they are unique that rely on the device's family, group, revision, and so on. They help identifying the chip's family, group, and revision.

[alexanderwachter]

So as I understand, the numbers are unique for each series.
The device id is meant to be unique for each piece. So if you take two pieces of the same type, they have to be different.
For me, as the maintainer of hwinfo, it is hard to give a +1 without having a look at the datasheet.

[MulinChao]

So as I understand, the numbers are unique for each series.
The device id is meant to be unique for each piece. So if you take two pieces of the same type, they have to be different.
For me, as the maintainer of hwinfo, it is hard to give a +1 without having a look at the datasheet.

Understood. Sorry for the inconvenience.

[alexanderwachter]

Does your MCU support unique IDs?
If not, we should consider closing this PR

[alexanderwachter]

This driver does not implement a unique id, but an identifier for a series.

[MulinChao]

Does your MCU support unique IDs?
If not, we should consider closing this PR

If I understand correctly, the unique ID means each different chip in the same series should provide a unique number. If so, I misunderstood 'hwinfo' driver and npcx MCU doesn't support this feature.

[alexanderwachter]

As a background:
We are using this API for USB to provide a unique ID. I think updatehub uses it to identify their nodes.

[MulinChao]

As a background:
We are using this API for USB to provide a unique ID. I think updatehub uses it to identify their nodes.

Got it. Thanks for sharing the background usage. I will close this PR.

[MulinChao]

Hi All:
Since the interface of 'hwinfo' is for USB application. I will move this PR into the other repo and close it.

[MulinChao]

Hi All:
Since the interface of 'hwinfo' is for USB application. I will move it into the other repo and close it.