Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 25 Apr 2022 18:23:21 -0700
From:   Zev Weiss <zev@bewilderbeest.net>
To:     Guenter Roeck <linux@roeck-us.net>
Cc:     Jean Delvare <jdelvare@suse.com>, linux-hwmon@vger.kernel.org,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org,
        Renze Nicolai <renze@rnplus.nl>,
        Oleksandr Natalenko <oleksandr@natalenko.name>,
        openbmc@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Subject: Re: [PATCH v2 1/6] dt-bindings: hwmon: Add nuvoton,nct6775
Message-ID: <YmdJibUk2cfOamMk@hatter.bewilderbeest.net>
References: <20220309005047.5107-1-zev@bewilderbeest.net>
 <20220309005047.5107-2-zev@bewilderbeest.net>
 <20220425150748.GA4165124@roeck-us.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <20220425150748.GA4165124@roeck-us.net>
Precedence: bulk

On Mon, Apr 25, 2022 at 08:07:48AM PDT, Guenter Roeck wrote:
>On Tue, Mar 08, 2022 at 04:50:42PM -0800, Zev Weiss wrote:
>> These Super I/O chips have an i2c interface that some systems expose
>> to a BMC; the BMC's device tree can now describe that via this
>> binding.
>>
>> Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>
>This is still waiting for DT maintainer approval.
>
>Do you expect to add further properties along the line ?
>If not, you might consider adding the chips to trivial devices instead.
>

Additional properties seem possible, though at the moment I don't know 
exactly what they might be, so trivial-devices.yml probably makes more 
sense.

I'll send a v3 soon with that change and the Kconfig fixes in the other 
patches.


Thanks,
Zev