lkml.org 
[lkml]   [2006]   [Feb]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 03/22] pHype specific stuff

Hi,

> +inline static u32 getLongBusyTimeSecs(int longBusyRetCode)
> +{
> + switch (longBusyRetCode) {
> + case H_LongBusyOrder1msec:
> + return 1;
> + case H_LongBusyOrder10msec:
> + return 10;
> + case H_LongBusyOrder100msec:
> + return 100;
> + case H_LongBusyOrder1sec:
> + return 1000;
> + case H_LongBusyOrder10sec:
> + return 10000;
> + case H_LongBusyOrder100sec:
> + return 100000;
> + default:
> + return 1;
> + } /* eof switch */
> +}

Since this actually returns milliseconds it might be worth making it
obvious in the function name. Also no need to use studly caps for the
function name and variable. We will fix the studly caps H_LongBusy*
stuff another day :)

> +inline static long plpar_hcall_7arg_7ret(unsigned long opcode,
> +inline static long plpar_hcall_9arg_9ret(unsigned long opcode,

These belong in arch/powerpc/platforms/pseries/hvCall.S

Anton
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-20 16:16    [W:0.175 / U:1.420 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site