summaryrefslogtreecommitdiffstats
path: root/commands
Commit message (Collapse)AuthorAgeFilesLines
* commands: of_property: copy value instead of resizingOleg.Karfich@wago.com2018-08-291-2/+1
| | | | | | | | | | | | | | Commit 748d1b8 adds the fixup feature to the above command. With this commit an unwanted free(data) on the freshly set property value is done if someone tries to set the value of an already present property (not the case for a new property). Furthermore in fixup mode the pointer fixup->data is freed when the fixup function is called more than once. This leads to a hanging system after calling of_dump -F many times. Fix this issues by copying the data to a new allocated memory area like a new property is created. Signed-off-by: Oleg Karfich <oleg.karfich@wago.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/misc'Sascha Hauer2018-08-131-4/+3
|\
| * of_node: clarify help text and do stricter commandline parsingUwe Kleine-König2018-08-081-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To create an oftree node /path/node I tried after reading the help text of of_node: of_node -c /path node similar to the syntax of of_property. This didnt result in any output but still failed to do what I meant. So clarify the help text to not suggest there are two parameters and return an usage error if still two (or more) names are passed. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands: teach commands that write to files to also create themUwe Kleine-König2018-08-103-3/+3
|/ | | | | | | | | | This allows to use the adapted commands on non-existing files which failed before with open: No such file or directory Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/misc'Sascha Hauer2018-07-092-10/+26
|\
| * commands: mmc_extcsd: Make use of devpath_to_name()Andrey Smirnov2018-06-251-3/+2
| | | | | | | | | | Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * commands: mount: Make use of devpath_to_name()Andrey Smirnov2018-06-251-4/+1
| | | | | | | | | | Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * mmc_extcsd command: Add cmd option to set bkops-enableChristoph Fritz2018-06-201-3/+23
| | | | | | | | | | | | | | | | This patch is for convenience to set bkops-enable (EXT_CSD_BKOPS_EN[163]) on devices which have it not yet activated. Signed-off-by: Christoph Fritz <chf.fritz@googlemail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands: oftree: kill oftree -fSascha Hauer2018-06-111-16/+1
|/ | | | | | | | | | Being able to free the live device tree seemed like a good idea in the early days of device tree support when the first boards were brought up. It turned out to be illusory that noone stores pointers to the live tree which become invalid when oftree -f is executed. Enough people stumbled upon crashed boards with this option, so remove it finally. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* commands: mmc_extcsd: Add support to write multibyte registersTeresa Remmet2018-06-081-17/+27
| | | | | | | | | It has been possible to write only the first byte of a multibyte register. Updated the command to write the complete value of a multibyte register at once. Signed-off-by: Teresa Remmet <t.remmet@phytec.de> Reviewed-by: Bastian Stender <bst@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* commands/of_node: add option to apply changes as fixupBaeuerle, Florian2018-05-281-31/+75
| | | | | | | | | | Analogous to the of_property command, add an -f option to the of_node command. This option defers the device tree modification until boot time and thus applies the modification as a fixup on the linux device tree. Signed-off-by: Florian Bäuerle <florian.baeuerle@allegion.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* commands/of_property: add option to apply changes as fixupBaeuerle, Florian2018-05-281-53/+138
| | | | | | | | | Add an -f option to the of_property command, which defers the device tree modification until boot time and thus applies the modification as a fixup on the linux device tree. Signed-off-by: Florian Bäuerle <florian.baeuerle@allegion.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* mw command: add `-x` option descriptionPeter Mamonov2018-05-241-1/+2
| | | | | Signed-off-by: Peter Mamonov <pmamonov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* commands: i2c_write: enable reg == 0 tooAntony Pavlov2018-04-171-1/+1
| | | | | | | | | | | | | This commit fixes the mistake introduced in the commit 34fadb685905 ('commands: i2c_write: enable raw write to address'). The intended behaviour for i2c_write command is "use raw i2c write only if reg address is not defined". Before the commit 34fadb685905 reg address == 0 was acceptable. After the commit 34fadb685905 reg address == 0 became unacceptable ("undefined") and will lead to raw i2c write. Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/fs'Sascha Hauer2018-04-064-16/+9
|\
| * loadenv: Do not depend on normalise_path()Sascha Hauer2018-04-061-5/+1
| | | | | | | | | | | | normalise_path() will go away, so do without it. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * ls: Do not depend on normalise_path()Sascha Hauer2018-04-061-6/+3
| | | | | | | | | | | | | | When we want to show a directory we want to show the path ls is passed, not the normalised one. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * rename file_operations -> cdev_operationsSascha Hauer2018-04-062-5/+5
| | | | | | | | | | | | | | | | Linux also has struct file_operations which are something different. Rename our file_operations to cdev_operations which better matches what we have. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | ARM: aarch64: disable 32bit boot commandsSascha Hauer2018-04-041-2/+2
|/ | | | | | | | The 32bit boot support is full of legacy cruft. While it should be possible to merge the aarch64 support there, a fresh start looks more promising. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* commands: i2c_write: enable raw write to addressAntony Pavlov2018-02-091-2/+6
| | | | | | | | | | | | | | | | | | | | Sometimes for communication with a simple I2C devices (e.g. PCF8574 or TM1650) it's necessary to send only one data byte into the I2C device. Current i2c_write command makes this impossible because you can't just pass 'device address' and 'register number' (or 'device address' and 'one data byte') to the command. You always have to pass all three parameters: 'device address', 'register number' and 'data'. This commit fixes the problem. Sample usage: barebox@barebox sandbox:/ i2c_write -a 0x24 0x01 Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/of'Sascha Hauer2018-02-081-0/+1
|\
| * of: Allow for const propertiesSascha Hauer2018-01-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds support for const properties. We add a const void *value_const to struct property which will not be modified. The value of a property should no longer be used directly, but with the of_property_get_value() accessor which picks the right value. With this we can unflatten dtbs and use the property values directly from the dtb instead of copying them. This is useful for device trees which (ab)use properties to store huge data files, aka FIT images. To create a property whose value is not copied but used from the original input data we introduce of_new_property_const(). Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | Merge branch 'for-next/misc'Sascha Hauer2018-02-081-1/+1
|\ \
| * | commands: ubi: ubiupdatevol: Reduce error code to 0 or 1Daniel Schultz2018-01-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | do_ubiupdatevol can either return 0 or the ioctl return value. This is not in conformity with the other ubi comannds return values. Signed-off-by: Daniel Schultz <d.schultz@phytec.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | | bbu: command: create temporary variable holding the pointer to the imageSascha Hauer2018-01-301-3/+5
| |/ |/| | | | | | | | | | | Create variable to hold the pointer to the image so that we can make the data.image pointer const later. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands/Kconfig: fix copy'n'paste error in CMD_GPIO helpUlrich Ölmann2018-01-251-1/+1
|/ | | | | Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/misc'Sascha Hauer2018-01-222-8/+8
|\
| * mci: drop unused parameter from mci_switch()Uwe Kleine-König2018-01-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The SWITCH command has two purposes: a) switch the command set b) Write to the EXT_CSD register If the access field (bits [25:24]) in the argument are b00, we're in case a), otherwise in b). As mci_switch() always passes MMC_SWITCH_MODE_WRITE_BYTE (0b3) in the access field, only case b) is relevant here. According to the eMMC specification[1] the command set field is ignored in case b) and so the respective parameter (that is unused already now) can be dropped. [1] Embedded Multi-Media Card (e•MMC) Electrical Standard (5.1), February 2015; paragraph 6.6.1 Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * usbgadget: Update command help textSascha Hauer2018-01-111-7/+7
| | | | | | | | | | | | | | | | | | - Do not use BAREBOX_CMD_HELP_OPT with an empty option to extend the previous option text. It confuses the rst converter - Add a '\t' to the options to align the option help texts in the barebox command help Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands/Kconfig: Fix tpyoUwe Kleine-König2018-01-221-1/+1
|/ | | | | | | s/Pring/Print/ Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/net'Sascha Hauer2018-01-056-92/+132
|\
| * commands: Add ip_route_add commandSascha Hauer2017-12-153-0/+107
| | | | | | | | | | | | | | | | | | | | The ip_route_get command is used to retrieve the network device used to reach a given IP address. This is useful for informational purposes and also for the network boot which wants to get the linux.bootargs parameter of the network device used for nfsroot. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * net: remove "current" network deviceSascha Hauer2017-12-142-67/+0
| | | | | | | | | | | | | | Now that we can do routing we no longer need a "current" network device. Remove it. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * net: dhcp: reworkSascha Hauer2017-12-141-12/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DHCP code is a mess. It is not clear which options are sent to the server and which options are returned from the server. Also environment variables are read from and written to all over the place. This patch cleans this up. There now is struct dhcp_req_param which is used for options sent to the server and struct dhcp_result which contains the values sent from the server. The values from the server are written to the barebox variables in a single place. Also it's now possible to call the dhcp code without modifying barebox variables at all, storing the result only in the dhcp result struct. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * net: dhcp: Allow to specify network deviceSascha Hauer2017-12-141-3/+17
| | | | | | | | | | | | | | | | | | Instead of allowing to DHCP only on the "current" network device, allow to specify the desired network device. This is a first step to get rid of the concept of a "current" network device. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * driver: Add device_detect_all() functionSascha Hauer2017-12-011-3/+1
| | | | | | | | | | | | | | | | Add a device_detect_all function to detect all devices and use it in the detect command. This makes the functionality reusable in other code. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * detect command: Drop -e optionSascha Hauer2017-12-011-13/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The -e option is not that useful. Normally, when a device cannot be detected, then there should be no reason to not try to detect other devices. When a user is interested in a particular detect result, detect can still be called with that (and only that) device. We could also argue that the user should not care about the result of the detect call, but instead check if the desired device is present afterwards. Drop handling of the -e option, but still ignore it for compatibility reasons. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | Merge branch 'for-next/misc'Sascha Hauer2018-01-052-2/+2
|\ \
| * | nv: add device parameter overwrites to completion listSascha Hauer2017-11-232-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | nv variables in the form dev.<devname>.* can be used to make device parameters persistent. Add these to the completion list to make setting these variables more convenient. Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | | commands/miitool: Allow specifying PHY by their full pathAndrey Smirnov2017-12-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make -s option to be a bit more consistent with how it is used in other commands by allowing PHY device to be specified by either its name of full path in /dev. This change also allows us to leverage autocompletion since it will work as expected for paths in /dev. Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | | commands: miitool: Constify 'phydevname' in mdiobus_show()Andrey Smirnov2017-12-061-1/+2
|/ / | | | | | | | | | | | | | | It seems like mdiobus_show() doesn't need more than read-only to 'phydevname', so constify it to allow passing const stings in. Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | Merge branch 'for-next/nv'Sascha Hauer2017-11-131-16/+29
|\ \
| * | command: nv: rewrap help text linesEnrico Jorns2017-11-031-3/+3
| | | | | | | | | | | | | | | | | | | | | Just let the help text appear a bit more aligned. Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * | commands: nv: adjust command helpEnrico Jorns2017-11-031-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Help did not point out that -r option requires n arguments and did not mention support for removing variables in help text. Also introduce and consistently reuse 'NV' as abbreviation for 'non volatile'. Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * | commands: nv: assure error code will be returned when an error occurredEnrico Jorns2017-11-031-4/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Due to iteration over possibly multiple variables, the return value of an individual call to nvvar_remove() / nvvar_add() gets lost. This will result in do_nv() returning success (0) if any variable processing failed as long as processing of the last variable succeeded. Processing will not be aborted on individual errors, such as the 'cp' handles it for invalid files. Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * | commands: nv: call nvvar_print() only if no argument is givenEnrico Jorns2017-11-031-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | Printing the nv variables (before performing any modification!) is useless and irritating. Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| * | commands: nv: fail with verbose message if invoked without argsEnrico Jorns2017-11-031-1/+3
| |/ | | | | | | | | Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands: spi: do not restrict spi's data to be hex-formattedUlrich Ölmann2017-10-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | Change the default base to 0 to be consistent with the "mw" command and particularly allow decimally formatted data. The price to pay is needing the usual prefix "0x" for each hex-formatted number. Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* | commands: i2c: do not restrict i2c_write's data to be hex-formattedUlrich Ölmann2017-10-231-1/+1
|/ | | | | | | | | | | Change the default base to 0 to be consistent with the "mw" command and particularly allow decimally formatted data. The price to pay is needing the usual prefix "0x" for each hex-formatted number. Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
* Merge branch 'for-next/usbgadget'Sascha Hauer2017-10-191-2/+18
|\
generated by cgit v1.2.3 (git 2.39.1) at 2024-04-19 00:33:53 +0000