linux-kernel - Re: [PATCH v3 3/3] regulator: da9121: Add DA914x support

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: <YaTuUS4GwZ4ON7fz@sirena.org.uk>
Date:   Mon, 29 Nov 2021 15:25:26 +0000
From:   Mark Brown <broonie@...nel.org>
To:     Adam Ward <Adam.Ward.opensource@...semi.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
        Support Opensource <support.opensource@...semi.com>
Subject: Re: [PATCH v3 3/3] regulator: da9121: Add DA914x support

On Tue, Nov 23, 2021 at 11:27:58PM +0000, Adam Ward wrote:
> Signed-off-by: Adam Ward <Adam.Ward.opensource@...semi.com>

This doesn't build:

/mnt/kernel/drivers/regulator/da9121-regulator.c:571:8: error: 'DA9141_IDX_BUCK1' undeclared here (not in a function); did you mean 'DA9121_IDX_BUCK1'?
  571 |  .id = DA9141_IDX_BUCK1,
      |        ^~~~~~~~~~~~~~~~
      |        DA9121_IDX_BUCK1
/mnt/kernel/drivers/regulator/da9121-regulator.c:595:8: error: 'DA914X_IDX_BUCK1' undeclared here (not in a function); did you mean 'DA9121_IDX_BUCK1'?
  595 |  .id = DA914X_IDX_BUCK1,
      |        ^~~~~~~~~~~~~~~~
      |        DA9121_IDX_BUCK1
/mnt/kernel/drivers/regulator/da9121-regulator.c: In function 'da9121_check_device_type':
/mnt/kernel/drivers/regulator/da9121-regulator.c:915:55: error: 'DA914X_DEVICE_ID' undeclared (first use in this function); did you mean 'DA914x_DEVICE_ID'?
  915 |  if ((device_id != DA9121_DEVICE_ID) && (device_id != DA914X_DEVICE_ID) {
      |                                                       ^~~~~~~~~~~~~~~~
      |                                                       DA914x_DEVICE_ID
/mnt/kernel/drivers/regulator/da9121-regulator.c:915:55: note: each undeclared identifier is reported only once for each function it appears in
/mnt/kernel/drivers/regulator/da9121-regulator.c:915:72: error: expected ')' before '{' token
  915 |  if ((device_id != DA9121_DEVICE_ID) && (device_id != DA914X_DEVICE_ID) {
      |     ~                                                                  ^~
      |                                                                        )
/mnt/kernel/drivers/regulator/da9121-regulator.c:996:1: error: expected expression before '}' token
  996 | }
      | ^
/mnt/kernel/drivers/regulator/da9121-regulator.c:912:3: error: label 'error' used but not defined
  912 |   goto error;
      |   ^~~~
/mnt/kernel/drivers/regulator/da9121-regulator.c:900:7: error: unused variable 'config_match' [-Werror=unused-variable]
  900 |  bool config_match = false;
      |       ^~~~~~~~~~~~
/mnt/kernel/drivers/regulator/da9121-regulator.c:899:8: error: unused variable 'type' [-Werror=unused-variable]
  899 |  char *type;
      |        ^~~~
/mnt/kernel/drivers/regulator/da9121-regulator.c:898:18: error: unused variable 'variant_vrc' [-Werror=unused-variable]
  898 |  u8 variant_mrc, variant_vrc;
      |                  ^~~~~~~~~~~
/mnt/kernel/drivers/regulator/da9121-regulator.c:898:5: error: unused variable 'variant_mrc' [-Werror=unused-variable]
  898 |  u8 variant_mrc, variant_vrc;
      |     ^~~~~~~~~~~
/mnt/kernel/drivers/regulator/da9121-regulator.c:996:1: error: no return statement in function returning non-void [-Werror=return-type]
  996 | }
      | ^


Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)