1*691668feSPatrick 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, 59840b140aSAndrew Jeffery enum pldm_effecter_data_size tag, uint8_t *val) 60840b140aSAndrew Jeffery { 61840b140aSAndrew Jeffery switch (tag) { 62840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT8: 63840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val); 64840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT8: 65840b140aSAndrew Jeffery return pldm_msgbuf_extract_int8(ctx, (int8_t *)val); 66840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT16: 67840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val); 68840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT16: 69840b140aSAndrew Jeffery return pldm_msgbuf_extract_int16(ctx, (int16_t *)val); 70840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_UINT32: 71840b140aSAndrew Jeffery return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val); 72840b140aSAndrew Jeffery case PLDM_SENSOR_DATA_SIZE_SINT32: 73840b140aSAndrew Jeffery return pldm_msgbuf_extract_int32(ctx, (int32_t *)val); 74840b140aSAndrew Jeffery } 75840b140aSAndrew Jeffery 76840b140aSAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 77840b140aSAndrew Jeffery } 78840b140aSAndrew Jeffery 797992eb84SAndrew Jeffery __attribute__((always_inline)) static inline int 807992eb84SAndrew Jeffery pldm_msgbuf_extract_range_field_format(struct pldm_msgbuf *ctx, 817992eb84SAndrew Jeffery enum pldm_range_field_format tag, 827992eb84SAndrew Jeffery union_range_field_format *dst) 837992eb84SAndrew Jeffery { 847992eb84SAndrew Jeffery switch (tag) { 857992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT8: 867992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u8); 877992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT8: 887992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s8); 897992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT16: 907992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u16); 917992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT16: 927992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s16); 937992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_UINT32: 947992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_u32); 957992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_SINT32: 967992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_s32); 977992eb84SAndrew Jeffery case PLDM_RANGE_FIELD_FORMAT_REAL32: 987992eb84SAndrew Jeffery return pldm_msgbuf_extract(ctx, &dst->value_f32); 997992eb84SAndrew Jeffery } 1007992eb84SAndrew Jeffery 1017992eb84SAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 1027992eb84SAndrew Jeffery } 1037992eb84SAndrew Jeffery 1043884c44dSAndrew Jeffery /* This API is bad, but it's because the caller's APIs are also bad */ 1053884c44dSAndrew Jeffery __attribute__((always_inline)) static inline int 1063884c44dSAndrew Jeffery pldm_msgbuf_extract_effecter_value(struct pldm_msgbuf *ctx, 1073884c44dSAndrew Jeffery enum pldm_effecter_data_size tag, 1083884c44dSAndrew Jeffery uint8_t *val) 1093884c44dSAndrew Jeffery { 1103884c44dSAndrew Jeffery switch (tag) { 1113884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT8: 1123884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val); 1133884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT8: 1143884c44dSAndrew Jeffery return pldm_msgbuf_extract_int8(ctx, (int8_t *)val); 1153884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT16: 1163884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val); 1173884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT16: 1183884c44dSAndrew Jeffery return pldm_msgbuf_extract_int16(ctx, (int16_t *)val); 1193884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_UINT32: 1203884c44dSAndrew Jeffery return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val); 1213884c44dSAndrew Jeffery case PLDM_EFFECTER_DATA_SIZE_SINT32: 1223884c44dSAndrew Jeffery return pldm_msgbuf_extract_int32(ctx, (int32_t *)val); 1233884c44dSAndrew Jeffery } 1243884c44dSAndrew Jeffery 1253884c44dSAndrew Jeffery return -PLDM_ERROR_INVALID_DATA; 1263884c44dSAndrew Jeffery } 1273884c44dSAndrew Jeffery 1287992eb84SAndrew Jeffery #endif 129