linux-kernel - Re: [PATCH v3] mips: bcm63xx: add support for clk_get

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 14 Nov 2021 16:18:35 -0800
From:   Randy Dunlap <rdunlap@...radead.org>
To:     Andy Shevchenko <andy.shevchenko@...il.com>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        kernel test robot <lkp@...el.com>,
        Artur Rojek <contact@...ur-rojek.eu>,
        Paul Cercueil <paul@...pouillou.net>,
        "open list:BROADCOM NVRAM DRIVER" <linux-mips@...r.kernel.org>,
        Jonathan Cameron <jic23@...nel.org>,
        Lars-Peter Clausen <lars@...afoo.de>,
        linux-iio <linux-iio@...r.kernel.org>,
        Florian Fainelli <f.fainelli@...il.com>,
        Russell King <linux@...linux.org.uk>,
        bcm-kernel-feedback-list <bcm-kernel-feedback-list@...adcom.com>,
        Jonas Gorski <jonas.gorski@...il.com>
Subject: Re: [PATCH v3] mips: bcm63xx: add support for clk_get_parent()

On 11/14/21 8:40 AM, Andy Shevchenko wrote:
> On Sun, Nov 14, 2021 at 6:34 PM Randy Dunlap <rdunlap@...radead.org> wrote:
>>
>> BCM63XX selects HAVE_LEGACY_CLK but does not provide/support
>> clk_get_parent(), so add a simple implementation of that
>> function so that callers of it will build without errors.
>>
>> Fixes these build errors:
>>
>> mips-linux-ld: drivers/iio/adc/ingenic-adc.o: in function `jz4770_adc_init_clk_div':
>> ingenic-adc.c:(.text+0xe4): undefined reference to `clk_get_parent'
>> mips-linux-ld: drivers/iio/adc/ingenic-adc.o: in function `jz4725b_adc_init_clk_div':
>> ingenic-adc.c:(.text+0x1b8): undefined reference to `clk_get_parent'
> 
> Some nit-picks below.
> Otherwise looks good to me,
> Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>
> 
> Suggested-by?

Yes, I'll add Russell for that.

>> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
>> Reported-by: kernel test robot <lkp@...el.com>
>> Cc: Artur Rojek <contact@...ur-rojek.eu>
>> Cc: Paul Cercueil <paul@...pouillou.net>
>> Cc: linux-mips@...r.kernel.org
>> Cc: Jonathan Cameron <jic23@...nel.org>
>> Cc: Lars-Peter Clausen <lars@...afoo.de>
>> Cc: linux-iio@...r.kernel.org
>> Cc: Florian Fainelli <f.fainelli@...il.com>
>> Cc: Andy Shevchenko <andy.shevchenko@...il.com>
>> Cc: Russell King <linux@...linux.org.uk>
>> Cc: bcm-kernel-feedback-list@...adcom.com
>> Cc: Jonas Gorski <jonas.gorski@...il.com>
>> ---
>> v1 and v2 were:
>> [PATCH] iio/adc: ingenic: fix (MIPS) ingenic-adc build errors
> 
>> Fixes: 1a78daea107d ("IIO: add Ingenic JZ47xx ADC driver.")
> 
> Not sure why it's here. What does (the location of) this tag mean?

Yesterday I didn't see what Fixes: tag I should use,
but after looking again, it looks like I should use
Fixes: e7300d04bd08 ("MIPS: BCM63xx: Add support for the Broadcom BCM63xx family of SOCs."


> 
>>   arch/mips/bcm63xx/clk.c |    7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> --- linux-next-20211112.orig/arch/mips/bcm63xx/clk.c
>> +++ linux-next-20211112/arch/mips/bcm63xx/clk.c
>> @@ -381,6 +381,13 @@ void clk_disable(struct clk *clk)
>>
>>   EXPORT_SYMBOL(clk_disable);
>>
>> +struct clk *clk_get_parent(struct clk *clk)
>> +{
>> +       return NULL;
>> +}
> 
>> +
> 
> Perhaps it's not needed even if the rest have it (I mean blank line).

Sure, I'll drop it. It was for file consistency,
but it's not a big deal either way.

>> +EXPORT_SYMBOL(clk_get_parent);
>> +
>>   unsigned long clk_get_rate(struct clk *clk)
>>   {
>>          if (!clk)
> 

Thanks.
-- 
~Randy