Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1422449 > unrolled thread
| Started by | Hoan Tran <hotran@apm.com> |
|---|---|
| First post | 2016-06-15 01:20 +0200 |
| Last post | 2016-06-16 00:20 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] mailbox: pcc: Add PCC request and free channel declarations Hoan Tran <hotran@apm.com> - 2016-06-15 01:20 +0200
Re: [PATCH] mailbox: pcc: Add PCC request and free channel declarations "Prakash, Prashanth" <pprakash@codeaurora.org> - 2016-06-15 18:00 +0200
Re: [PATCH] mailbox: pcc: Add PCC request and free channel declarations Hoan Tran <hotran@apm.com> - 2016-06-16 00:20 +0200
| From | Hoan Tran <hotran@apm.com> |
|---|---|
| Date | 2016-06-15 01:20 +0200 |
| Subject | [PATCH] mailbox: pcc: Add PCC request and free channel declarations |
| Message-ID | <rK5MK-2ZU-5@gated-at.bofh.it> |
As PCC will be used by other clients not only CPPC. This change exports pcc_mbox_request_channel() and pcc_mbox_free_channel() declarations Signed-off-by: Hoan Tran <hotran@apm.com> --- include/acpi/cppc_acpi.h | 4 ---- include/linux/mailbox_client.h | 4 ++++ 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/acpi/cppc_acpi.h b/include/acpi/cppc_acpi.h index dad8af3..819b4b9 100644 --- a/include/acpi/cppc_acpi.h +++ b/include/acpi/cppc_acpi.h @@ -130,8 +130,4 @@ extern int cppc_set_perf(int cpu, struct cppc_perf_ctrls *perf_ctrls); extern int cppc_get_perf_caps(int cpu, struct cppc_perf_caps *caps); extern int acpi_get_psd_map(struct cpudata **); -/* Methods to interact with the PCC mailbox controller. */ -extern struct mbox_chan * - pcc_mbox_request_channel(struct mbox_client *, unsigned int); - #endif /* _CPPC_ACPI_H*/ diff --git a/include/linux/mailbox_client.h b/include/linux/mailbox_client.h index 41e2af8..2758d18 100644 --- a/include/linux/mailbox_client.h +++ b/include/linux/mailbox_client.h @@ -50,4 +50,8 @@ void mbox_client_txdone(struct mbox_chan *chan, int r); /* atomic */ bool mbox_client_peek_data(struct mbox_chan *chan); /* atomic */ void mbox_free_channel(struct mbox_chan *chan); /* may sleep */ +/* Methods to interact with the PCC mailbox controller. */ +struct mbox_chan *pcc_mbox_request_channel(struct mbox_client *, int); +void pcc_mbox_free_channel(struct mbox_chan *chan); + #endif /* __MAILBOX_CLIENT_H */ -- 1.9.1
[toc] | [next] | [standalone]
| From | "Prakash, Prashanth" <pprakash@codeaurora.org> |
|---|---|
| Date | 2016-06-15 18:00 +0200 |
| Subject | Re: [PATCH] mailbox: pcc: Add PCC request and free channel declarations |
| Message-ID | <rKlot-4sY-1@gated-at.bofh.it> |
| In reply to | #1422449 |
Hi Hoan, On 6/14/2016 5:12 PM, Hoan Tran wrote: > As PCC will be used by other clients not only CPPC. > This change exports pcc_mbox_request_channel() and pcc_mbox_free_channel() > declarations > > Signed-off-by: Hoan Tran <hotran@apm.com> > --- > include/acpi/cppc_acpi.h | 4 ---- > include/linux/mailbox_client.h | 4 ++++ > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/include/acpi/cppc_acpi.h b/include/acpi/cppc_acpi.h > index dad8af3..819b4b9 100644 > --- a/include/acpi/cppc_acpi.h > +++ b/include/acpi/cppc_acpi.h > @@ -130,8 +130,4 @@ extern int cppc_set_perf(int cpu, struct cppc_perf_ctrls *perf_ctrls); > extern int cppc_get_perf_caps(int cpu, struct cppc_perf_caps *caps); > extern int acpi_get_psd_map(struct cpudata **); > > -/* Methods to interact with the PCC mailbox controller. */ > -extern struct mbox_chan * > - pcc_mbox_request_channel(struct mbox_client *, unsigned int); > - > #endif /* _CPPC_ACPI_H*/ > diff --git a/include/linux/mailbox_client.h b/include/linux/mailbox_client.h > index 41e2af8..2758d18 100644 > --- a/include/linux/mailbox_client.h > +++ b/include/linux/mailbox_client.h > @@ -50,4 +50,8 @@ void mbox_client_txdone(struct mbox_chan *chan, int r); /* atomic */ > bool mbox_client_peek_data(struct mbox_chan *chan); /* atomic */ > void mbox_free_channel(struct mbox_chan *chan); /* may sleep */ > > +/* Methods to interact with the PCC mailbox controller. */ > +struct mbox_chan *pcc_mbox_request_channel(struct mbox_client *, int); > +void pcc_mbox_free_channel(struct mbox_chan *chan); > + Since these APIs are specific to PCC, may be we should introduce a new header for pcc clients instead of adding to the common mailbox_client header. > #endif /* __MAILBOX_CLIENT_H */ Thanks, Prashanth
[toc] | [prev] | [next] | [standalone]
| From | Hoan Tran <hotran@apm.com> |
|---|---|
| Date | 2016-06-16 00:20 +0200 |
| Message-ID | <rKrkd-8p8-7@gated-at.bofh.it> |
| In reply to | #1423175 |
Hi Prashanth, On Wed, Jun 15, 2016 at 8:55 AM, Prakash, Prashanth <pprakash@codeaurora.org> wrote: > Hi Hoan, > > On 6/14/2016 5:12 PM, Hoan Tran wrote: >> As PCC will be used by other clients not only CPPC. >> This change exports pcc_mbox_request_channel() and pcc_mbox_free_channel() >> declarations >> >> Signed-off-by: Hoan Tran <hotran@apm.com> >> --- >> include/acpi/cppc_acpi.h | 4 ---- >> include/linux/mailbox_client.h | 4 ++++ >> 2 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/include/acpi/cppc_acpi.h b/include/acpi/cppc_acpi.h >> index dad8af3..819b4b9 100644 >> --- a/include/acpi/cppc_acpi.h >> +++ b/include/acpi/cppc_acpi.h >> @@ -130,8 +130,4 @@ extern int cppc_set_perf(int cpu, struct cppc_perf_ctrls *perf_ctrls); >> extern int cppc_get_perf_caps(int cpu, struct cppc_perf_caps *caps); >> extern int acpi_get_psd_map(struct cpudata **); >> >> -/* Methods to interact with the PCC mailbox controller. */ >> -extern struct mbox_chan * >> - pcc_mbox_request_channel(struct mbox_client *, unsigned int); >> - >> #endif /* _CPPC_ACPI_H*/ >> diff --git a/include/linux/mailbox_client.h b/include/linux/mailbox_client.h >> index 41e2af8..2758d18 100644 >> --- a/include/linux/mailbox_client.h >> +++ b/include/linux/mailbox_client.h >> @@ -50,4 +50,8 @@ void mbox_client_txdone(struct mbox_chan *chan, int r); /* atomic */ >> bool mbox_client_peek_data(struct mbox_chan *chan); /* atomic */ >> void mbox_free_channel(struct mbox_chan *chan); /* may sleep */ >> >> +/* Methods to interact with the PCC mailbox controller. */ >> +struct mbox_chan *pcc_mbox_request_channel(struct mbox_client *, int); >> +void pcc_mbox_free_channel(struct mbox_chan *chan); >> + > Since these APIs are specific to PCC, may be we should introduce a new header for pcc clients > instead of adding to the common mailbox_client header. Agreed, maybe I should create a "pcc.h" file inside "/include/acpi" folder. If you have any suggestion about the location of this file, please let me know. Thanks Hoan >> #endif /* __MAILBOX_CLIENT_H */ > > Thanks, > Prashanth
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web