public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Giulio Benetti <giulio.benetti@benettiengineering.com>
To: u-boot@lists.denx.de
Subject: [PATCH] clk: uclass: clk_get_by_name() must not be available if CONFIG_OF_PLATDATA is enabled
Date: Sun, 15 Dec 2019 16:09:39 +0100	[thread overview]
Message-ID: <e08527d3-17f2-a070-6ce1-534dfb368dfa@benettiengineering.com> (raw)
In-Reply-To: <20191215145036.0a223a46@jawa>

Hi Lukasz,

On 12/15/19 2:50 PM, Lukasz Majewski wrote:
> Hi Giulio,
> 
>> On 12/13/19 11:09 AM, Lukasz Majewski wrote:
>>> On Fri, 13 Dec 2019 00:02:25 +0100
>>> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
>>>    
>>>> Hi Lukasz,
>>>>
>>>> On 12/12/19 11:53 PM, Giulio Benetti wrote:
>>>>> clk_get_by_name() requires clk_get_by_id() that is not available
>>>>> if CONFIG_OF_PLATDATA is defined, so move clk_get_by_name() into
>>>>> #else condition of #if CONFIG_IS_ENABLED(OF_PLATDATA).
>>>>>
>>>>> Signed-off-by: Giulio Benetti
>>>>> <giulio.benetti@benettiengineering.com>
>>>>
>>>> I've already successfully executed a complete Travis with master
>>>> and this patch applied:
>>>> https://travis-ci.org/giuliobenetti/u-boot-imxrt/builds/624256888
>>>
>>> Is this a preparatory patch for some other work? Or is it just a
>>> standalone fix?
>>
>> This is a standalone fix.
> 
> As we are really close to final U-Boot release - I'm more keen to add
> this patch to u-boot-clk -next repository.
> 
> Do you mind ?

No problem at all, we'll see it "next"!

Best regards

-- 
Giulio Benetti
Benetti Engineering sas

>>
>> Best regards
> 
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
> 

  reply	other threads:[~2019-12-15 15:09 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-12 22:53 [PATCH] clk: uclass: clk_get_by_name() must not be available if CONFIG_OF_PLATDATA is enabled Giulio Benetti
2019-12-12 23:02 ` Giulio Benetti
2019-12-13 10:09   ` Lukasz Majewski
2019-12-13 11:38     ` Giulio Benetti
2019-12-13 13:42       ` Lukasz Majewski
2019-12-15 13:50       ` Lukasz Majewski
2019-12-15 15:09         ` Giulio Benetti [this message]
2020-01-17  9:44           ` Giulio Benetti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e08527d3-17f2-a070-6ce1-534dfb368dfa@benettiengineering.com \
    --to=giulio.benetti@benettiengineering.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox