search

facebook / fboss

Facebook Open Switching System Software for controlling network switches.
Other
860 stars 295 forks source link

Meru800bia: adds initial weutil config file #153

Closed DeanKalla closed 1 year ago

DeanKalla commented 1 year ago

Summary

Adds initial weutil config file for Meru800bia platform. This config file supports reading SCM EEPROM.

Testing

weutil --eeprom SCM --config_file weutil.json

facebook-github-bot commented 1 year ago

@joancaneus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

joancaneus commented 1 year ago

@DeanKalla we need to add the offset and idEepromFormatVer field for this PR just like the weutil json in this path https://github.com/facebook/fboss/blob/main/fboss/platform/configs/montblanc/weutil.json

Please make the change and send it again.

DeanKalla commented 1 year ago

@DeanKalla we need to add the offset and idEepromFormatVer field for this PR just like the weutil json in this path https://github.com/facebook/fboss/blob/main/fboss/platform/configs/montblanc/weutil.json

Please make the change and send it again.

what is "offset" here? @arajeev-ARISTA

arajeev-ARISTA commented 1 year ago

Hi @DeanKalla, the offset is 15360

DeanKalla commented 1 year ago

I will discard this PR and arajeev will post a new one

joancaneus commented 1 year ago

I will discard this PR and arajeev will post a new one

@DeanKalla @arajeev-ARISTA --- by the way, I added the offset: 0 field and tested it and it works. However, that's with your weutil change which is not using offset at all and reading the whole buffetsize. I put a comment there as well for you to take a look tomorrow.

Thanks Guys!