Add Cameo platform ESC601-32Q

[Hanly-Cameo]

Switch Vendor: Cameo
Switch SKU: ESC601-32Q
CPU: BROADWELL-DE
ASIC Vendor: Innovium
Switch ASIC: TL7
Port Configuration: 32x200G

What I did
Add a new Cameo platform ESC601-32Q.

How I did it
Add files by following SONiC Porting Guide.

How to verify it
a)with "make target/sonic-innovium.bin"
b)install image and check the following commands
decode-syseeprom
sensors
psuutil
sfputil

- Which release branch to backport (provide reason below if seleted)

• 201811
• 201911
• 202006

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

[lguohan]

retest vsimage please

[jleveque]

Why not use the at24.c which is part of the Linux kernel?

[Hanly-Cameo]

Intel I801 works on SMBus mode. We are using "24c64"(16bit address) for system eeprom.
in at24.c:

	if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C)) {
		if (chip.flags & AT24_FLAG_ADDR16)
			return -EPFNOSUPPORT;

So, we made a modification in it.

[maxliu37]

Any further comment ?

[jleveque]

@lguohan to comment.

[jleveque]

Why not use the mcp3422.c which is part of the Linux kernel?

[Hanly-Cameo]

Because of I2C bus controller is only SMbus(I801) mode on our board. In mcp3422 linux native driver will call "i2c_check_functionality" and return "EOPNOTSUPP", so we do a work around in it.

[maxliu37]

The mcp3425_smbus.c is based on Linux mcp3422.c to fit our dedicated SMbus requirement. Any further comment on this issue?

[jleveque]

@lguohan to comment.

[maxliu37]

retest vsimage please

I don't understand the meaning to test with vsimage. As we know, the vs is a Virtual Switch independent from any platform/device, but the platform driver is dedicated to our device. Furthermore, we also followed the "platform test plan" to verify the driver package with SONiC. Please kind advise the purpose and the test cases to be processed. Thanks.

[jleveque]

@maxliu37: That comment wasn't directed at you. By entering the phrase "retest this please" into a comment, GitHub triggers our Jenkins server to re-run all of the check builds, and "retest <test_name> please" will trigger a specific check build to re-run.

[maxliu37]

@maxliu37: That comment wasn't directed at you. By entering the phrase "retest this please" into a comment, GitHub triggers our Jenkins server to re-run all of the check builds, and "retest <test_name> please" will trigger a specific check build to re-run.

Thanks for the clarification. This is our first merge request, not knowing well on all activities.

[jleveque]

Retest vsimage please

[lgtm-com]

This pull request introduces 13 alerts when merging 9a5be5627a4eb6d341840d03b9cfe668d3bc71cc into 895f4e0 - view on LGTM.com

new alerts:

• 6 for Unused local variable
• 4 for Variable defined multiple times
• 2 for Use of the return value of a procedure
• 1 for Unused import