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] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230729125516.68a29852@jic23-huawei>
Date:   Sat, 29 Jul 2023 12:55:16 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     "Miclaus, Antoniu" <Antoniu.Miclaus@...log.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "conor+dt@...nel.org" <conor+dt@...nel.org>,
        "linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dt-bindings: iio: admv1014: make all regs required

On Thu, 27 Jul 2023 12:02:04 +0000
"Miclaus, Antoniu" <Antoniu.Miclaus@...log.com> wrote:

> > -----Original Message-----
> > From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> > Sent: Thursday, July 27, 2023 2:47 PM
> > To: Miclaus, Antoniu <Antoniu.Miclaus@...log.com>; jic23@...nel.org;
> > robh+dt@...nel.org; krzysztof.kozlowski+dt@...aro.org;
> > conor+dt@...nel.org; linux-iio@...r.kernel.org;
> > devicetree@...r.kernel.org; linux-kernel@...r.kernel.org
> > Subject: Re: [PATCH] dt-bindings: iio: admv1014: make all regs required
> > 
> > [External]
> > 
> > On 27/07/2023 13:31, Antoniu Miclaus wrote:  
> > > Since the regulators are required in the driver implementation, make
> > > them required also in the bindings.

This bit is probably not strictly true.  Try no providing them and you will
probably find stub regulators created on assumption they are fixed regs
that we aren't interested in controlling.

> > >  
> > 
> > The true reason should be whether the hardware requires them. Because if
> > hardware does not need some, the driver should be fixed.  
> The datasheet is not very explicit on this topic, but all the specifications of the
> part are built around these pins being supplied.
> Moreover, the evaluation board or the part comes with all VCC pins connected
> to fixed supplies.

Agreed - these should be required in the binding.

There was an old disagreement on this in which it they were not marked
required in some bindings because the regulator subsystem would assume they
were fixed supplies that were just missing in the DT and so provides stub regulators.
My understanding at least has changed and now we mark the required even if
the driver works fine on some boards without them being supplied.

Note though that this means the whole of trivial-devices.yaml is garbage
as very few devices actually work without any power :)

Jonathan



> > Best regards,
> > Krzysztof  
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ