From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS,UNPARSEABLE_RELAY,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 979CAC433DF for ; Thu, 15 Oct 2020 08:25:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 47BE022249 for ; Thu, 15 Oct 2020 08:25:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726054AbgJOIZq (ORCPT ); Thu, 15 Oct 2020 04:25:46 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:56726 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725959AbgJOIZq (ORCPT ); Thu, 15 Oct 2020 04:25:46 -0400 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: eballetbo) with ESMTPSA id 87F471F4591F Subject: Re: [PATCH v2 3/3] mfd: google,cros-ec: add missing properties To: =?UTF-8?Q?Ricardo_Ca=c3=b1uelo?= , Tzung-Bi Shih Cc: Rob Herring , Collabora Kernel ML , Benson Leung , Guenter Roeck , Simon Glass , Doug Anderson , devicetree@vger.kernel.org, Dmitry Torokhov , Cheng-Yi Chiang References: <20201008102825.3812-1-ricardo.canuelo@collabora.com> <20201008102825.3812-4-ricardo.canuelo@collabora.com> <20201008183818.GB2395464@bogus> <20201009054819.di4dlfljadsfs6cw@rcn-XPS-13-9360> <20201013064609.udss7hbdek2of4sw@rcn-XPS-13-9360> <0578a7e56430da42daee95aabc05850da3dcc294.camel@collabora.com> From: Enric Balletbo i Serra Message-ID: <8c5c7a2b-667b-cb15-1fbd-c7015f5b97dd@collabora.com> Date: Thu, 15 Oct 2020 10:25:41 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <0578a7e56430da42daee95aabc05850da3dcc294.camel@collabora.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Tzung-Bi, On 15/10/20 8:52, Ricardo CaƱuelo wrote: > On Wed, 2020-10-14 at 10:18 +0800, Tzung-Bi Shih wrote: >> The intermediate layer (i.e. codecs { ... }) breaks current code as >> you already discussed about that in previous threads: >> - of_platform_populate only creates immediate child devices. >> - the codec driver expects its parent is EC node. > > Thanks for your input, I'll try to address these issues in the drivers in v3. > Can you remind us which platforms support cros-ec-codec and why there is no an EC_FEATURE for that? Thanks, Enric > Cheers, > Ricardo >