Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45362 )
Change subject: nb/intel/ironlake: Drop some unused function parameters ......................................................................
nb/intel/ironlake: Drop some unused function parameters
Some of the HECI functions do not need raminfo at all.
Change-Id: If0720fa87e5e18820db77a1b61bcdb42ecc538fb Signed-off-by: Angel Pons th3fanbus@gmail.com --- M src/northbridge/intel/ironlake/raminit.c 1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/45362/1
diff --git a/src/northbridge/intel/ironlake/raminit.c b/src/northbridge/intel/ironlake/raminit.c index 6905cae..426ef87 100644 --- a/src/northbridge/intel/ironlake/raminit.c +++ b/src/northbridge/intel/ironlake/raminit.c @@ -1691,7 +1691,7 @@
/* FIXME: Add timeout. */ static int -recv_heci_packet(struct raminfo *info, struct mei_header *head, u32 *packet, +recv_heci_packet(struct mei_header *head, u32 *packet, u32 *packet_size) { union { @@ -1736,7 +1736,7 @@
/* FIXME: Add timeout. */ static int -recv_heci_message(struct raminfo *info, u32 *message, u32 *message_size) +recv_heci_message(u32 *message, u32 *message_size) { struct mei_header head; int current_position; @@ -1746,7 +1746,7 @@ u32 current_size; current_size = *message_size - current_position; if (recv_heci_packet - (info, &head, message + (current_position >> 2), + (&head, message + (current_position >> 2), ¤t_size) == -1) break; if (!current_size) @@ -1796,7 +1796,7 @@ send_heci_message((u8 *) & msg, sizeof(msg), 0, 7);
reply_size = sizeof(reply); - if (recv_heci_message(info, (u32 *) & reply, &reply_size) == -1) + if (recv_heci_message((u32 *) & reply, &reply_size) == -1) return;
if (reply.command != (MKHI_SET_UMA | (1 << 7)))