- Jul 22, 2022
-
-
Krzysztof Kozlowski authored
The multi-led node defined address/size cells, so it is intended to have children with unit addresses. The second multi-led's reg property defined three LED indexes within one reg item, which is not correct - these are three separate items. Signed-off-by:
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by:
Rob Herring <robh@kernel.org> Signed-off-by:
-
- Jul 20, 2022
-
-
Geert Uytterhoeven authored
It does not hurt to fill in the changeset id while the mutex is still held. After doing so, the function tails for the success and failure cases become identical, so they can be unified. Signed-off-by:
Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by:
Frank Rowand <frank.rowand@sony.com> Tested-by:
-
Geert Uytterhoeven authored
There is no point in doing several preparatory steps in of_overlay_fdt_apply(), only to see of_overlay_apply() return early because of a corrupt device tree. Move the check for a corrupt device tree from of_overlay_apply() to of_overlay_fdt_apply(), to check for this as early as possible. Signed-off-by:
-
- Jul 19, 2022
-
-
Krzysztof Kozlowski authored
There is no need to specify "minItems" if it equals to "maxItems". On the other hand number of items in an array can be specified via describing items, which might bring some additional information. This simplifies a bit the binding. Signed-off-by:
-
- Jul 13, 2022
-
-
Liang He authored
We should use of_node_put() for the reference 'node' returned by of_parse_phandle() which will increase the refcount. Fixes: fec9b625 ("of: Add plumbing for restricted DMA pool") Co-authored-by:
Miaoqian Lin <linmq006@gmail.com> Signed-off-by:
Liang He <windhl@126.com> Signed-off-by:
-
- Jul 12, 2022
-
-
Andy Shevchenko authored
On the ->remove() stage the callback uses physical device node instead of one from GPIO chip and the variable name which is different to one used in unittest_gpio_probe(). Make these consistent with unittest_gpio_probe(). Signed-off-by:
Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by:
-
Andy Shevchenko authored
The OF node in the GPIO library is deprecated and soon will be removed. GPIO library now accepts fwnode as a firmware node, so switch the module to use it instead. Signed-off-by:
Bartosz Golaszewski <brgl@bgdev.pl> Signed-off-by:
-
Rob Herring authored
Convert the arm,arm-firmware-suite partition binding to DT schema format. Simple conversion as there's only a compatible property. Reviewed-by:
Linus Walleij <linus.walleij@linaro.org> Signed-off-by:
Miquel Raynal <miquel.raynal@bootlin.com> Link: https://lore.kernel.org/r/20220629185031.23826-1-robh@kernel.org
-
Chanho Park authored
Cortex A78AE's MPIDR has been added since commit 83bea32a ("arm64: Add part number for Arm Cortex-A78AE") We also need to add the compatible. Signed-off-by:
Chanho Park <chanho61.park@samsung.com> Signed-off-by:
-
Rob Herring authored
Fixed string property names should be under 'properties' rather than 'patternProperties'. Additionally, without beginning and end of line anchors, any prefix or suffix is allowed on the specified property names. As all the nvidia,tegra20-pmc powergates child node properties are fixed strings, change 'patternProperties' to 'properties'. Signed-off-by:
Thierry Reding <treding@nvidia.com> Link: https://lore.kernel.org/r/20220706212034.568861-1-robh@kernel.org
-
- Jul 06, 2022
-
-
Krzysztof Kozlowski authored
Document already used S5PV210 ChipID block. Signed-off-by:
-
Krzysztof Kozlowski authored
Group devices like Chip ID or SoC information under "hwinfo" directory. Signed-off-by:
-
- Jul 02, 2022
-
-
Rob Herring authored
The 'unevaluatedProperties' schema checks is not fully working and doesn't catch some cases where there's a $ref to another schema. A fix is pending, but results in new warnings in examples. The correct common watchdog property is 'timeout-sec', not 'timeout-secs'. Signed-off-by:
Guenter Roeck <linux@roeck-us.net> Link: https://lore.kernel.org/r/20220526014127.2872254-1-robh@kernel.org
-
Jason A. Donenfeld authored
chosen.txt has been replaced by a schema in dtschema[1] and is now out of date as well. Remove it to avoid confusion. [1] https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/chosen.yaml Link: https://lore.kernel.org/lkml/c8dddfe6-6385-ed34-e789-9f845c8a32bd@linaro.org/ Link: https://lore.kernel.org/lkml/CAL_Jsq+uSdk9YNbUW35yjN3q8-3FDobrxHmBpy=4RKmCfnB0KQ@mail.gmail.com/ Signed-off-by:
Jason A. Donenfeld <Jason@zx2c4.com> [robh: Improve commmit msg] Signed-off-by:
-
- Jul 01, 2022
-
-
Krzysztof Kozlowski authored
One DTS example was using 8-space indentation, while others 2-space. For complex DTS with multiple levels, 2-space is not that readable and in DTS examples 4-space is preferred. Signed-off-by:
-
Paul Cercueil authored
The previous URL was giving a 404 error. Signed-off-by:
Paul Cercueil <paul@crapouillou.net> Reviewed-by:
-
- Jun 30, 2022
-
-
Rob Herring authored
Pull in the binding fixes so we don't have all the warnings.
-
Rob Herring authored
Convert the gpio-ir-receiver binding to DT schema format. Signed-off-by:
-
Rob Herring authored
Most existing 'gpio-ir-receiver' nodes use 'ir-receiver' for their node name, so add it as an allowed node name. Signed-off-by:
-
- Jun 29, 2022
-
-
Michael Walle authored
Mails to Charles get an auto reply, that he is no longer working at Eff'Innov technologies. Remove the entry. Signed-off-by:
Michael Walle <michael@walle.cc> Reviewed-by:
-
Piyush Mehta authored
Convert the ahci-ceva doc to yaml. Signed-off-by:
Piyush Mehta <piyush.mehta@xilinx.com> Signed-off-by:
-
Vincent Whitchurch authored
If the allocation/reservation of reserved-memory fails, it is normally an error, so print it as an error so that it doesn't get hidden from the console due to the loglevel. Also make the allocation failure include the size just like the reservation failure. Signed-off-by:
Vincent Whitchurch <vincent.whitchurch@axis.com> Signed-off-by:
-
- Jun 28, 2022
-
-
Deming Wang authored
Delete duplicate words of "the". Signed-off-by:
Deming Wang <wangdeming@inspur.com> Signed-off-by:
-
- Jun 18, 2022
-
-
Rob Herring authored
It is valid for arm,vexpress-sysreg to have no child nodes when either the deprecated binding without child nodes is used or a platform has no GPIOs. If there are no child nodes, then "#address-cells" and "#size-cells" should not be required. However, "reg" should be present in all cases, so let's add it now. Cc: Andre Przywara <andre.przywara@arm.com> Signed-off-by:
-
Rob Herring authored
While the current binding expects a child GPIO node, the old style binding with GPIO provider properties in the parent vexpress-sysreg node is still in use. Add the GPIO provider properties as deprecated. Cc: Andre Przywara <andre.przywara@arm.com> Signed-off-by:
-
Rob Herring authored
Convert the Arm CCN performance monitors binding to DT schema format. Signed-off-by:
Robin Murphy <robin.murphy@arm.com> Link: https://lore.kernel.org/r/20220613145554.3727354-1-robh@kernel.org
-
Rob Herring authored
'arm,rtsm-display' is a panel for Arm, Ltd. virtual platforms (e.g. FVP). The binding has been in use for a long time, but was never documented. Some users and an example have a 'panel-dpi' compatible, but that's not needed without a 'panel-timing' node which none of the users have since commit 928faf5e ("arm64: dts: fvp: Remove panel timings"). The example does have a 'panel-timing' node, but it should not for the same reasons the node was removed in the dts files. So update the example in arm,pl11x.yaml to match the schema. Cc: Linus Walleij <linus.walleij@linaro.org> Cc: Robin Murphy <robin.murphy@arm.com> Cc: Andre Przywara <andre.przywara@arm.com> Signed-off-by:
Sam Ravnborg <sam@ravnborg.org> Reviewed-by:
-
- Jun 17, 2022
-
-
Wolfram Sang authored
Commit cc611137 ("ARM: drop efm32 platform") removed the platform, so no need to still carry the bindings. Signed-off-by:
Wolfram Sang <wsa@kernel.org> Acked-by:
Mark Brown <broonie@kernel.org> Acked-by:
Stephen Boyd <sboyd@kernel.org> Signed-off-by:
-
Wolfram Sang authored
It accidently ended up in i2c, but it should be in the hwmon folder. Signed-off-by:
-
Stanislav Jakubek authored
Convert RDA Micro interrupt controller bindings to DT schema format. Signed-off-by:
Stanislav Jakubek <stano.jakubek@gmail.com> Signed-off-by:
-
- Jun 11, 2022
-
-
Andre Przywara authored
As Liviu pointed out, the arm,malidp-arqos-high-level property mentioned in the original .txt binding was a mistake, and arm,malidp-arqos-value needs to take its place. The binding commit ce6eb025 ("dt/bindings: display: Add optional property node define for Mali DP500") mentions the right name in the commit message, but has the wrong name in the diff. Commit d298e6a2 ("drm/arm/mali-dp: Add display QoS interface configuration for Mali DP500") uses the property in the driver, but uses the shorter name. Remove the wrong property from the binding, and use the proper name in the example. The actual property was already documented properly. Fixes: 2c8b082a ("dt-bindings: display: convert Arm Mali-DP to DT schema") Link: https://lore.kernel.org/linux-arm-kernel/YnumGEilUblhBx8E@e110455-lin.cambridge.arm.com/ Signed-off-by:
Andre Przywara <andre.przywara@arm.com> Reported-by:
Liviu Dudau <liviu.dudau@arm.com> Reviewed-by:
-
Rob Herring authored
There's no reason to list the same value twice in an 'enum'. This was fixed treewide in commit c3b00681 ("dt-bindings: Fix 'enum' lists with duplicate entries"), but this one got added in the merge window. A meta-schema change will catch future cases. Signed-off-by:
Sergio Paracuellos <sergio.paracuellos@gmail.com> Link: https://lore.kernel.org/r/20220606212239.1360877-1-robh@kernel.org
-
- Jun 10, 2022
-
-
Krzysztof Kozlowski authored
Add bindings for IMEM memory region (like SRAM) used on several Qualcomm SoCs. In the downstream (vendor) tree, it comes with several children used for debugging purposes. The upstreamed parts are much smaller - just image loader and syscon reboot. Signed-off-by:
-
Krzysztof Kozlowski authored
Fix the indentation in the OCMEM example and use generic node name - sram - to reflect the type of the device. Signed-off-by:
-
- Jun 08, 2022
-
-
Krzysztof Kozlowski authored
Add vendor prefixes for manufacturers of Aspeed SoC based BMC boards: ASrock, ByteDance, Ingrasys, Inventec and Quanta. Move also bticino to proper alphabetical place. Signed-off-by:
-
Vaibhav Jain authored
Presently ima_get_kexec_buffer() doesn't check if the previous kernel's ima-kexec-buffer lies outside the addressable memory range. This can result in a kernel panic if the new kernel is booted with 'mem=X' arg and the ima-kexec-buffer was allocated beyond that range by the previous kernel. The panic is usually of the form below: $ sudo kexec --initrd initrd vmlinux --append='mem=16G' <snip> BUG: Unable to handle kernel data access on read at 0xc000c01fff7f0000 Faulting instruction address: 0xc000000000837974 Oops: Kernel access of bad area, sig: 11 [#1] <snip> NIP [c000000000837974] ima_restore_measurement_list+0x94/0x6c0 LR [c00000000083b55c] ima_load_kexec_buffer+0xac/0x160 Call Trace: [c00000000371fa80] [c00000000083b55c] ima_load_kexec_buffer+0xac/0x160 [c00000000371fb00] [c0000000020512c4] ima_init+0x80/0x108 [c00000000371fb70] [c0000000020514dc] init_ima+0x4c/0x120 [c00000000371fbf0] [c000000000012240] do_one_initcall+0x60/0x2c0 [c00000000371fcc0] [c000000002004ad0] kernel_init_freeable+0x344/0x3ec [c00000000371fda0] [c0000000000128a4] kernel_init+0x34/0x1b0 [c00000000371fe10] [c00000000000ce64] ret_from_kernel_thread+0x5c/0x64 Instruction dump: f92100b8 f92100c0 90e10090 910100a0 4182050c 282a0017 3bc00000 40810330 7c0802a6 fb610198 7c9b2378 f80101d0 <a1240000> 2c090001 40820614 e9240010 ---[ end trace 0000000000000000 ]--- Fix this issue by checking returned PFN range of previous kernel's ima-kexec-buffer with page_is_ram() to ensure correct memory bounds. Fixes: 467d2782 ("powerpc: ima: get the kexec buffer passed by the previous kernel") Cc: Frank Rowand <frowand.list@gmail.com> Cc: Prakhar Srivastava <prsriva@linux.microsoft.com> Cc: Lakshmi Ramasubramanian <nramas@linux.microsoft.com> Cc: Thiago Jung Bauermann <bauerman@linux.ibm.com> Cc: Rob Herring <robh@kernel.org> Cc: Ritesh Harjani <ritesh.list@gmail.com> Cc: Robin Murphy <robin.murphy@arm.com> Signed-off-by:
Vaibhav Jain <vaibhav@linux.ibm.com> Signed-off-by:
-
Rob Herring authored
Another round from new cases in 5.19-rc of removing redundant minItems/maxItems when 'items' list is specified. This time it is in if/then schemas as the meta-schema was failing to check this case. If a property has an 'items' list, then a 'minItems' or 'maxItems' with the same size as the list is redundant and can be dropped. Note that is DT schema specific behavior and not standard json-schema behavior. The tooling will fixup the final schema adding any unspecified minItems/maxItems. Signed-off-by:
Ulf Hansson <ulf.hansson@linaro.org> Link: https://lore.kernel.org/r/20220606225137.1536010-1-robh@kernel.org
-
Rob Herring authored
A 'phandle' type is always a single cell, so 'maxItems: 1' is redundant. Fixes: 82b96552 ("dt-bindings: nvme: Add Apple ANS NVMe") Signed-off-by:
Sven Peter <sven@svenpeter.dev> Link: https://lore.kernel.org/r/20220606212230.1360617-1-robh@kernel.org
-
- Jun 07, 2022
-
-
Geert Uytterhoeven authored
Convert the Silicon Image sii902x HDMI bridge Device Tree binding documentation to json-schema. Add missing sil,sii9022-cpi and sil,sii9022-tpi compatible values. Signed-off-by:
-
Danilo Krummrich authored
The PS/2 bus defines data and clock line to be open drain, this should be reflected in the gpio flags set in the binding. Especially, this is important since the clock line sometimes is driven by the host while being used as interrupt source. Reviewed-by:
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Signed-off-by:
Danilo Krummrich <danilokrummrich@dk-develop.de> Reviewed-by:
-
