Remove the unnecessary #address-cells and #size-cells nodes from
the fan-controller. These are not needed as the fan controller does not
have any children.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20230612162444.3936302-1-tharvey@gateworks.com
Signed-off-by: Lee Jones <lee@kernel.org>
In order to ensure only documented properties are present, node schemas
must have unevaluatedProperties or additionalProperties set to false
(typically).
Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Heiko Stuebner <heiko@sntech.de>
Acked-by: Alistair Francis <alistair@alistair23.me>
Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Lee Jones <lee@kernel.org>
Link: https://lore.kernel.org/r/20220823145649.3118479-4-robh@kernel.org
Emails to Robert Jones bounce ("550 5.2.1 The email account that you
tried to reach is disabled").
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Acked-by: Lee Jones <lee@kernel.org>
Signed-off-by: Rob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20220809162752.10186-5-krzysztof.kozlowski@linaro.org
In 7497d4a66c ("hwmon: (gsc-hwmon) add fan sensor") a mode
was added to report RPM's from a fan tach input.
Add this mode to the dt-bindings for the Gateworks System Controller.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
GPIO_ACTIVE_x flags are not correct in the context of interrupt flags.
These are simple defines so they could be used in DTS but they will not
have the same meaning:
1. GPIO_ACTIVE_HIGH = 0 = IRQ_TYPE_NONE
2. GPIO_ACTIVE_LOW = 1 = IRQ_TYPE_EDGE_RISING
Correct the interrupt flags, assuming the author of the code wanted same
logical behavior behind the name "ACTIVE_xxx", this is:
ACTIVE_LOW => IRQ_TYPE_LEVEL_LOW
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com> # for BD70528
Acked-by: Tim Harvey <tharvey@gateworks.com> # for gateworks-gsc.yaml
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Another round of 'allOf' removals and whitespace clean-ups of schemas.
-----BEGIN PGP SIGNATURE-----
iQJEBAABCgAuFiEEktVUI4SxYhzZyEuo+vtdtY28YcMFAl825dIQHHJvYmhAa2Vy
bmVsLm9yZwAKCRD6+121jbxhw793EACDXwtLbcIzcJgM1r7es23hSlVMvHoNHFaV
Hkg7GJ8ZCuaHXtXFiEzPvxKTb9CmEgDEDrGRXZDmv2MsaQ1wS7YjG1TW/jL+JVP+
KsAJNwOklK6qyJ3AeX01T6pG1hZf5+2OVyuCKryE2PY8EE+2ZweW+Yoz9ivuy1L0
PokTc9SPc3sm85Ex3BhzzXNTMa1efzWcj6lrxTFsLywuc5msEhLkYRR76jx21xBP
Rs49KxlPozVFTtk9dIWhybXu6VHJ/jCPn9ubv/bKPAZZbqMjVc1KRpTuHxzUW0Ew
2xNEhYdp76GfhaTHrTe9v65DNe6a2xpsQbmlRnZC7ASovjqM1JOb44tt1QFEyI/M
0hIf+nqhgiUpm2+UZcLmJaDmoIbmc10DROsIrtavSCBjPFjLS3ocN2LbKiLJj1rT
lc9XvvXGHFKB8O/RBqTKrmv/crNwQYDLirHvlcvJTAa3LBOCLrnQL/d5aRP9ShJ3
S/gORXg0ggCdi0WFSpQxpRKInJu1GFkzDqRRSXMFADnnHZtJBYWXihE6L48XIqXP
9XREitC6BOgQrc4nZzZnEwgt3TkVw7kZJvOBS9Y2yk+EK4zMTwaPdZ/sa5Xwx/J7
JEznSK27PHr3QVRmwPQPBFu+B8c7S2TuY/Z4k5744dn4T9d7cG0BOSzABOyuA1cx
lPRW0frbjw==
=G4/5
-----END PGP SIGNATURE-----
Merge tag 'devicetree-fixes-for-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux
Pull devicetree fixes from Rob Herring:
"Another round of 'allOf' removals and whitespace clean-ups of schemas"
* tag 'devicetree-fixes-for-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux:
dt-bindings: Remove more cases of 'allOf' containing a '$ref'
dt-bindings: Whitespace clean-ups in schema files
Add a 16-bit pre-scaled voltage mode to ADC and clarify that existing
pre-scaled mode is 24bit.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
This patch adds documentation of device-tree bindings for the
Gateworks System Controller (GSC).
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
