Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1309475 > unrolled thread

Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c Jassi Brar <jassisinghbrar@gmail.com> - 2016-01-14 17:40 +0100
    Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c Ashwin Chaugule <ashwin.chaugule@linaro.org> - 2016-01-14 18:10 +0100
      Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c Valdis.Kletnieks@vt.edu - 2016-01-14 18:40 +0100
        Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c Valdis.Kletnieks@vt.edu - 2016-01-14 21:40 +0100

#1309475 — Re: [PATCH RESEND] maibox:Remove unused functions in the file pcc.c

On Tue, Jan 12, 2016 at 11:11 AM, Nicholas Krause <xerofoify@gmail.com> wrote:
> This removes the unused function pcc_mbox_request_channel
> due to having no more callers in this driver or else where
> in the kernel. In addition remove the function get_pcc_channel
> now due to this function's only caller pcc_mbox_request_channel
> being removed from the file pcc.c in this patch too.
>
CC'ing Ashwin, the author of this api.

[toc] | [next] | [standalone]

#1309490

On 14 January 2016 at 11:37, Jassi Brar <jassisinghbrar@gmail.com> wrote:
> On Tue, Jan 12, 2016 at 11:11 AM, Nicholas Krause <xerofoify@gmail.com> wrote:
>> This removes the unused function pcc_mbox_request_channel
>> due to having no more callers in this driver or else where
>> in the kernel. In addition remove the function get_pcc_channel
>> now due to this function's only caller pcc_mbox_request_channel
>> being removed from the file pcc.c in this patch too.
>>
> CC'ing Ashwin, the author of this api.

Have you looked in -
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/acpi/cppc_acpi.c
?

Ashwin

[toc] | [prev] | [next] | [standalone]

#1309535

[Multipart message — attachments visible in raw view] — view raw

On Thu, 14 Jan 2016 12:03:28 -0500, Ashwin Chaugule said:
> On 14 January 2016 at 11:37, Jassi Brar <jassisinghbrar@gmail.com> wrote:
> > On Tue, Jan 12, 2016 at 11:11 AM, Nicholas Krause <xerofoify@gmail.com> wrote:
> >> This removes the unused function pcc_mbox_request_channel
> >> due to having no more callers in this driver or else where
> >> in the kernel. In addition remove the function get_pcc_channel
> >> now due to this function's only caller pcc_mbox_request_channel
> >> being removed from the file pcc.c in this patch too.
> >>
> > CC'ing Ashwin, the author of this api.
>
> Have you looked in -
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/acpi/cppc_acpi.c
> ?

Of course he didn't.

[toc] | [prev] | [next] | [standalone]

#1309642

[Multipart message — attachments visible in raw view] — view raw

On Thu, 14 Jan 2016 14:26:59 -0500, nick said:
>
>
> On 2016-01-14 12:32 PM, Valdis.Kletnieks@vt.edu wrote:

> >> Have you looked in -
> >> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/acpi/cppc_acpi.c
> >> ?
> >
> > Of course he didn't.
> >
> Or maybe he did a allmodconfig build on x86_64 bit and either that file builds correctly on x86 with

That doesn't excuse not doing a 'git grep pcc_mbox_request_channel' or a
'grep -r pcc_mbox_request_channel .', both of which would have find the caller.

> the patch applied due to it never being use on x86 or Kconfig files incorrectly set for allmodconfig
> builds on x86 for this file, or maybe my compiler is broken but I doubt that as it works for all other
> build errors similar to this.

Or you don't have the foggiest clue how Kconfig works, and don't understand
why in a 'allmodconfig' this file is *NOT* built.

Hint 1:

grep cppc drivers/acpi/Makefile
obj-$(CONFIG_ACPI_CPPC_LIB)     += cppc_acpi.o

Hint 2: In the Kconfig:

config ACPI_CPPC_LIB
        bool
        depends on ACPI_PROCESSOR
        depends on !ACPI_CPU_FREQ_PSS

config ACPI_PROCESSOR
        tristate "Processor"
        depends on X86 || IA64 || ARM64
        select ACPI_PROCESSOR_IDLE if X86 || IA64
        select ACPI_CPU_FREQ_PSS if X86 || IA64

It's left as an exercise for the reader to figure out what happens
to ACPI_CPPC_LIB when 'make allmodconfig' sets ACPI_PROCESSOR=m

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web