1691668feSPatrick Williams /* SPDX-License-Identifier: Apache-2.0 OR GPL-2.0-or-later */ 27992eb84SAndrew Jeffery #ifndef PLDM_MSGBUF_PLATFORM_H 37992eb84SAndrew Jeffery #define PLDM_MSGBUF_PLATFORM_H 47992eb84SAndrew Jeffery 57992eb84SAndrew Jeffery #include "../msgbuf.h" 67992eb84SAndrew Jeffery #include <libpldm/base.h> 77992eb84SAndrew Jeffery #include <libpldm/platform.h> 87992eb84SAndrew Jeffery 97992eb84SAndrew Jeffery static inline int 107992eb84SAndrew Jeffery pldm_msgbuf_extract_value_pdr_hdr(struct pldm_msgbuf *ctx, 117992eb84SAndrew Jeffery struct pldm_value_pdr_hdr *hdr) 127992eb84SAndrew Jeffery { 137992eb84SAndrew Jeffery pldm_msgbuf_extract(ctx, &hdr->record_handle); 147992eb84SAndrew Jeffery pldm_msgbuf_extract(ctx, &hdr->version); 157992eb84SAndrew Jeffery pldm_msgbuf_extract(ctx, &hdr->type); 167992eb84SAndrew Jeffery pldm_msgbuf_extract(ctx, &hdr->record_change_num); 177992eb84SAndrew Jeffery pldm_msgbuf_extract(ctx, &hdr->length); 187992eb84SAndrew Jeffery 197992eb84SAndrew Jeffery return pldm_msgbuf_validate(ctx); 207992eb84SAndrew Jeffery } 217992eb84SAndrew Jeffery 227992eb84SAndrew Jeffery /* 237992eb84SAndrew Jeffery * We use __attribute__((always_inline)) below so the compiler has visibility of 247992eb84SAndrew Jeffery * the switch() at the call site. It is often the case that the size of multiple 257992eb84SAndrew Jeffery * fields depends on the tag. Inlining thus gives the compiler visibility to 267992eb84SAndrew Jeffery * hoist one tag-based code-path condition to cover all invocations. 277992eb84SAndrew Jeffery */ 287992eb84SAndrew Jeffery 297992eb84SAndrew Jeffery __attribute__((always_inline)) static inline int 307992eb84SAndrew Jeffery pldm_msgbuf_extract_sensor_data(struct pldm_msgbuf *ctx, 317992eb84SAndrew Jeffery enum pldm_sensor_readings_data_type tag, 327992eb84SAndrew Jeffery union_sensor_data_size *dst) 337992eb84SAndrew Jeffery { 347992eb84SAndrew Jeffery switch (tag) { 357992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT8: 367992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u8); 377992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT8: 387992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s8); 397992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT16: 407992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u16); 417992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT16: 427992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s16); 437992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT32: 447992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u32); 457992eb84SAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT32: 467992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s32); 477992eb84SAndrew Jeffery } 487992eb84SAndrew Jeffery 497992eb84SAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 507992eb84SAndrew Jeffery } 517992eb84SAndrew Jeffery 52840b140aSAndrew Jeffery /* 53840b140aSAndrew Jeffery * This API is bad, but it's because the caller's APIs are also bad. They should 54840b140aSAndrew Jeffery * have used the approach used by callers of pldm_msgbuf_extract_sensor_data() 55840b140aSAndrew Jeffery * above 56840b140aSAndrew Jeffery */ 57840b140aSAndrew Jeffery __attribute__((always_inline)) static inline int 58840b140aSAndrew Jeffery pldm_msgbuf_extract_sensor_value(struct pldm_msgbuf *ctx, 59*cd07fec7SThu Nguyen enum pldm_sensor_readings_data_type tag, 60*cd07fec7SThu Nguyen uint8_t *val) 61840b140aSAndrew Jeffery { 62840b140aSAndrew Jeffery switch (tag) { 63840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT8: 64840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val); 65840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT8: 66840b140aSAndrew Jeffery return pldm_msgbuf_extract_int8(ctx, (int8_t *)val); 67840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT16: 68840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val); 69840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT16: 70840b140aSAndrew Jeffery return pldm_msgbuf_extract_int16(ctx, (int16_t *)val); 71840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT32: 72840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val); 73840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT32: 74840b140aSAndrew Jeffery return pldm_msgbuf_extract_int32(ctx, (int32_t *)val); 75840b140aSAndrew Jeffery } 76840b140aSAndrew Jeffery 77840b140aSAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 78840b140aSAndrew Jeffery } 79840b140aSAndrew Jeffery 807992eb84SAndrew Jeffery __attribute__((always_inline)) static inline int 817992eb84SAndrew Jeffery pldm_msgbuf_extract_range_field_format(struct pldm_msgbuf *ctx, 827992eb84SAndrew Jeffery enum pldm_range_field_format tag, 837992eb84SAndrew Jeffery union_range_field_format *dst) 847992eb84SAndrew Jeffery { 857992eb84SAndrew Jeffery switch (tag) { 867992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT8: 877992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u8); 887992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT8: 897992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s8); 907992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT16: 917992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u16); 927992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT16: 937992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s16); 947992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT32: 957992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u32); 967992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT32: 977992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s32); 987992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_REAL32: 997992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_f32); 1007992eb84SAndrew Jeffery } 1017992eb84SAndrew Jeffery 1027992eb84SAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 1037992eb84SAndrew Jeffery } 1047992eb84SAndrew Jeffery 1053884c44dSAndrew Jeffery /* This API is bad, but it's because the caller's APIs are also bad */ 1063884c44dSAndrew Jeffery __attribute__((always_inline)) static inline int 1073884c44dSAndrew Jeffery pldm_msgbuf_extract_effecter_value(struct pldm_msgbuf *ctx, 1083884c44dSAndrew Jeffery enum pldm_effecter_data_size tag, 1093884c44dSAndrew Jeffery uint8_t *val) 1103884c44dSAndrew Jeffery { 1113884c44dSAndrew Jeffery switch (tag) { 1123884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT8: 1133884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val); 1143884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT8: 1153884c44dSAndrew Jeffery return pldm_msgbuf_extract_int8(ctx, (int8_t *)val); 1163884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT16: 1173884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val); 1183884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT16: 1193884c44dSAndrew Jeffery return pldm_msgbuf_extract_int16(ctx, (int16_t *)val); 1203884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT32: 1213884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val); 1223884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT32: 1233884c44dSAndrew Jeffery return pldm_msgbuf_extract_int32(ctx, (int32_t *)val); 1243884c44dSAndrew Jeffery } 1253884c44dSAndrew Jeffery 1263884c44dSAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 1273884c44dSAndrew Jeffery } 1283884c44dSAndrew Jeffery 1297992eb84SAndrew Jeffery #endif 130