diff mbox

[LEDE-DEV,4/5] base-files: add generic board detection preinit script

Message ID 1493384248-8006-1-git-send-email-roman@advem.lv
State Changes Requested
Headers show

Commit Message

Roman Yeryomin April 28, 2017, 12:57 p.m. UTC
Signed-off-by: Roman Yeryomin <roman@advem.lv>
---
 package/base-files/files/lib/preinit/03_preinit_board_detect | 11 +++++++++++
 1 file changed, 11 insertions(+)
 create mode 100644 package/base-files/files/lib/preinit/03_preinit_board_detect

Comments

Felix Fietkau April 30, 2017, 9:53 a.m. UTC | #1
On 2017-04-28 14:57, Roman Yeryomin wrote:
> Signed-off-by: Roman Yeryomin <roman@advem.lv>
> ---
>  package/base-files/files/lib/preinit/03_preinit_board_detect | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>  create mode 100644 package/base-files/files/lib/preinit/03_preinit_board_detect
> 
> diff --git a/package/base-files/files/lib/preinit/03_preinit_board_detect b/package/base-files/files/lib/preinit/03_preinit_board_detect
> new file mode 100644
> index 0000000..2d64ca6
> --- /dev/null
> +++ b/package/base-files/files/lib/preinit/03_preinit_board_detect
> @@ -0,0 +1,11 @@
> +#!/bin/sh
> +#
> +# Copyright (c) 2017 The Linux Foundation. All rights reserved.
> +#
> +
> +do_board_detect()
> +{
> +	include_file /lib/board_detect.sh && board_detect
> +}
> +
> +boot_hook_add preinit_main do_board_detect
If you add something like this, please convert the existing sysinfo
script as well. Then you won't need the include_file() function, because
board_detect.sh will always be present (and overwritten by target
base-files if necessary).

- Felix
Roman Yeryomin May 1, 2017, 10:55 p.m. UTC | #2
On 30 April 2017 at 12:53, Felix Fietkau <nbd@nbd.name> wrote:
> On 2017-04-28 14:57, Roman Yeryomin wrote:
>> Signed-off-by: Roman Yeryomin <roman@advem.lv>
>> ---
>>  package/base-files/files/lib/preinit/03_preinit_board_detect | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>  create mode 100644 package/base-files/files/lib/preinit/03_preinit_board_detect
>>
>> diff --git a/package/base-files/files/lib/preinit/03_preinit_board_detect b/package/base-files/files/lib/preinit/03_preinit_board_detect
>> new file mode 100644
>> index 0000000..2d64ca6
>> --- /dev/null
>> +++ b/package/base-files/files/lib/preinit/03_preinit_board_detect
>> @@ -0,0 +1,11 @@
>> +#!/bin/sh
>> +#
>> +# Copyright (c) 2017 The Linux Foundation. All rights reserved.
>> +#
>> +
>> +do_board_detect()
>> +{
>> +     include_file /lib/board_detect.sh && board_detect
>> +}
>> +
>> +boot_hook_add preinit_main do_board_detect
> If you add something like this, please convert the existing sysinfo
> script as well. Then you won't need the include_file() function, because
> board_detect.sh will always be present (and overwritten by target
> base-files if necessary).
>

You mean convert /lib/preinit/10_sysinfo into default board_detect.sh ?
Ok, that sounds reasonable.

Regards,
Roman
diff mbox

Patch

diff --git a/package/base-files/files/lib/preinit/03_preinit_board_detect b/package/base-files/files/lib/preinit/03_preinit_board_detect
new file mode 100644
index 0000000..2d64ca6
--- /dev/null
+++ b/package/base-files/files/lib/preinit/03_preinit_board_detect
@@ -0,0 +1,11 @@ 
+#!/bin/sh
+#
+# Copyright (c) 2017 The Linux Foundation. All rights reserved.
+#
+
+do_board_detect()
+{
+	include_file /lib/board_detect.sh && board_detect
+}
+
+boot_hook_add preinit_main do_board_detect