Is it platform specific

generic

Importance or Severity

low

Description of the enhancement

As of now, sonic_platform_pddf_base.PddfFan does not implement get_fault(). So, the fan{idx}_fault attributes aren't utilized.

Current Behavior

get_fault() is not implemented

Proposed Behavior

Implement PddfFan.get_fault() by reading a fan{idx}_fault attribute from FAN-CTRL pddf object.

Also, introduce FAN's fault to pd-plugin.json like:

    "FAN": {
        ...
        "fault": {
            "i2c": {
                "valmap": {
                    "1": true,
                    "0": false
                }
            }
        },
        ...
    }

so each vendor can specify what raw value maps to