u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
* [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
@ 2023-03-31 20:36 Judith Mendez
  2023-03-31 20:37 ` Tom Rini
  2023-04-04 13:59 ` Heinrich Schuchardt
  0 siblings, 2 replies; 9+ messages in thread
From: Judith Mendez @ 2023-03-31 20:36 UTC (permalink / raw)
  To: Tom Rini, u-boot
  Cc: Andrew Davis, Bryan Brattlof, Heinrich Schuchardt, Judith Mendez

List some common boot modes and their corresponding switch
settings for AM62 SK.

Signed-off-by: Judith Mendez <jm@ti.com>
---

Changes from v1: Change table format from ascii-art to list-table

 doc/board/ti/am62x_sk.rst | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
index b1b7d99befb..3d85400b1a1 100644
--- a/doc/board/ti/am62x_sk.rst
+++ b/doc/board/ti/am62x_sk.rst
@@ -229,3 +229,39 @@ Image formats:
                 | |   SPL DTB 1...N   | |
                 | +-------------------+ |
                 +-----------------------+
+
+Switch Setting for Boot Mode
+----------------------------
+
+Boot Mode pins provide means to select the boot mode and options before the
+device is powered up. After every POR, they are the main source to populate
+the Boot Parameter Tables.
+
+The following table shows some common boot modes used on AM62 platform. More
+details can be found in the Technical Reference Manual:
+https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
+
+.. list-table:: Boot Modes
+   :widths: 32 32 32
+   :header-rows: 1
+
+   * - Switch Label
+     - SW2: 12345678
+     - SW3: 12345678
+   * - SD
+     - 01000000
+     - 11000010
+   * - OSPI
+     - 00000000
+     - 11001110
+   * - EMMC
+     - 00000000
+     - 11010010
+   * - UART
+     - 00000000
+     - 11011100
+   * - USB DFU
+     - 00000000
+     - 11001010
+
+For SW2 and SW1, the switch state in the "ON" position = 1.
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-03-31 20:36 [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK Judith Mendez
@ 2023-03-31 20:37 ` Tom Rini
  2023-04-04 13:59 ` Heinrich Schuchardt
  1 sibling, 0 replies; 9+ messages in thread
From: Tom Rini @ 2023-03-31 20:37 UTC (permalink / raw)
  To: Judith Mendez; +Cc: u-boot, Andrew Davis, Bryan Brattlof, Heinrich Schuchardt

[-- Attachment #1: Type: text/plain, Size: 284 bytes --]

On Fri, Mar 31, 2023 at 03:36:15PM -0500, Judith Mendez wrote:

> List some common boot modes and their corresponding switch
> settings for AM62 SK.
> 
> Signed-off-by: Judith Mendez <jm@ti.com>

Thanks for reworking.

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-03-31 20:36 [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK Judith Mendez
  2023-03-31 20:37 ` Tom Rini
@ 2023-04-04 13:59 ` Heinrich Schuchardt
  2023-04-04 14:22   ` Tom Rini
  1 sibling, 1 reply; 9+ messages in thread
From: Heinrich Schuchardt @ 2023-04-04 13:59 UTC (permalink / raw)
  To: Judith Mendez; +Cc: Andrew Davis, Bryan Brattlof, u-boot, Tom Rini

On 3/31/23 22:36, Judith Mendez wrote:
> List some common boot modes and their corresponding switch
> settings for AM62 SK.
>
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
>
> Changes from v1: Change table format from ascii-art to list-table
>
>   doc/board/ti/am62x_sk.rst | 36 ++++++++++++++++++++++++++++++++++++
>   1 file changed, 36 insertions(+)
>
> diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
> index b1b7d99befb..3d85400b1a1 100644
> --- a/doc/board/ti/am62x_sk.rst
> +++ b/doc/board/ti/am62x_sk.rst
> @@ -229,3 +229,39 @@ Image formats:
>                   | |   SPL DTB 1...N   | |
>                   | +-------------------+ |
>                   +-----------------------+
> +
> +Switch Setting for Boot Mode
> +----------------------------
> +
> +Boot Mode pins provide means to select the boot mode and options before the
> +device is powered up. After every POR, they are the main source to populate
> +the Boot Parameter Tables.
> +
> +The following table shows some common boot modes used on AM62 platform. More
> +details can be found in the Technical Reference Manual:
> +https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
> +
> +.. list-table:: Boot Modes
> +   :widths: 32 32 32
> +   :header-rows: 1
> +
> +   * - Switch Label
> +     - SW2: 12345678
> +     - SW3: 12345678
> +   * - SD
> +     - 01000000
> +     - 11000010
> +   * - OSPI
> +     - 00000000
> +     - 11001110
> +   * - EMMC
> +     - 00000000
> +     - 11010010
> +   * - UART
> +     - 00000000
> +     - 11011100
> +   * - USB DFU
> +     - 00000000
> +     - 11001010

Hello Judith,

ReStructured text allows different flavors of formatting tables.

The HTML output for the table above is correct but the .rst file is hard
to read in a text-editor. I would prefer:

*Boot Modes*

============ ============= =============
Switch Label SW2: 12345678 SW3: 12345678
============ ============= =============
SD           01000000      11000010
OSPI         00000000      11001110
EMMC         00000000      11010010
UART         00000000      11011100
USB DFU      00000000      11001010
============ ============= =============

Best regards

Heinrich


> +
> +For SW2 and SW1, the switch state in the "ON" position = 1.


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-04-04 13:59 ` Heinrich Schuchardt
@ 2023-04-04 14:22   ` Tom Rini
  2023-04-06 17:54     ` Mendez, Judith
  0 siblings, 1 reply; 9+ messages in thread
From: Tom Rini @ 2023-04-04 14:22 UTC (permalink / raw)
  To: Heinrich Schuchardt; +Cc: Judith Mendez, Andrew Davis, Bryan Brattlof, u-boot

[-- Attachment #1: Type: text/plain, Size: 2678 bytes --]

On Tue, Apr 04, 2023 at 03:59:56PM +0200, Heinrich Schuchardt wrote:
> On 3/31/23 22:36, Judith Mendez wrote:
> > List some common boot modes and their corresponding switch
> > settings for AM62 SK.
> > 
> > Signed-off-by: Judith Mendez <jm@ti.com>
> > ---
> > 
> > Changes from v1: Change table format from ascii-art to list-table
> > 
> >   doc/board/ti/am62x_sk.rst | 36 ++++++++++++++++++++++++++++++++++++
> >   1 file changed, 36 insertions(+)
> > 
> > diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
> > index b1b7d99befb..3d85400b1a1 100644
> > --- a/doc/board/ti/am62x_sk.rst
> > +++ b/doc/board/ti/am62x_sk.rst
> > @@ -229,3 +229,39 @@ Image formats:
> >                   | |   SPL DTB 1...N   | |
> >                   | +-------------------+ |
> >                   +-----------------------+
> > +
> > +Switch Setting for Boot Mode
> > +----------------------------
> > +
> > +Boot Mode pins provide means to select the boot mode and options before the
> > +device is powered up. After every POR, they are the main source to populate
> > +the Boot Parameter Tables.
> > +
> > +The following table shows some common boot modes used on AM62 platform. More
> > +details can be found in the Technical Reference Manual:
> > +https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
> > +
> > +.. list-table:: Boot Modes
> > +   :widths: 32 32 32
> > +   :header-rows: 1
> > +
> > +   * - Switch Label
> > +     - SW2: 12345678
> > +     - SW3: 12345678
> > +   * - SD
> > +     - 01000000
> > +     - 11000010
> > +   * - OSPI
> > +     - 00000000
> > +     - 11001110
> > +   * - EMMC
> > +     - 00000000
> > +     - 11010010
> > +   * - UART
> > +     - 00000000
> > +     - 11011100
> > +   * - USB DFU
> > +     - 00000000
> > +     - 11001010
> 
> Hello Judith,
> 
> ReStructured text allows different flavors of formatting tables.
> 
> The HTML output for the table above is correct but the .rst file is hard
> to read in a text-editor. I would prefer:
> 
> *Boot Modes*
> 
> ============ ============= =============
> Switch Label SW2: 12345678 SW3: 12345678
> ============ ============= =============
> SD           01000000      11000010
> OSPI         00000000      11001110
> EMMC         00000000      11010010
> UART         00000000      11011100
> USB DFU      00000000      11001010
> ============ ============= =============

There's other list-table entires in the docs today. I'm fine either way,
but we need to be consistent in the docs, and should update perhaps
doc/develop/codingstyle.rst should get a section too on docs?

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-04-04 14:22   ` Tom Rini
@ 2023-04-06 17:54     ` Mendez, Judith
  0 siblings, 0 replies; 9+ messages in thread
From: Mendez, Judith @ 2023-04-06 17:54 UTC (permalink / raw)
  To: Tom Rini, Heinrich Schuchardt; +Cc: Davis, Andrew, Brattlof, Bryan, u-boot

Hello,

On 4/4/2023 9:22 AM, Tom Rini wrote:
> On Tue, Apr 04, 2023 at 03:59:56PM +0200, Heinrich Schuchardt wrote:
>> On 3/31/23 22:36, Judith Mendez wrote:
>>> List some common boot modes and their corresponding switch
>>> settings for AM62 SK.
>>>
>>> Signed-off-by: Judith Mendez <jm@ti.com>
>>> ---
>>>
>>> Changes from v1: Change table format from ascii-art to list-table
>>>
>>>    doc/board/ti/am62x_sk.rst | 36 ++++++++++++++++++++++++++++++++++++
>>>    1 file changed, 36 insertions(+)
>>>
>>> diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
>>> index b1b7d99befb..3d85400b1a1 100644
>>> --- a/doc/board/ti/am62x_sk.rst
>>> +++ b/doc/board/ti/am62x_sk.rst
>>> @@ -229,3 +229,39 @@ Image formats:
>>>                    | |   SPL DTB 1...N   | |
>>>                    | +-------------------+ |
>>>                    +-----------------------+
>>> +
>>> +Switch Setting for Boot Mode
>>> +----------------------------
>>> +
>>> +Boot Mode pins provide means to select the boot mode and options before the
>>> +device is powered up. After every POR, they are the main source to populate
>>> +the Boot Parameter Tables.
>>> +
>>> +The following table shows some common boot modes used on AM62 platform. More
>>> +details can be found in the Technical Reference Manual:
>>> +https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
>>> +
>>> +.. list-table:: Boot Modes
>>> +   :widths: 32 32 32
>>> +   :header-rows: 1
>>> +
>>> +   * - Switch Label
>>> +     - SW2: 12345678
>>> +     - SW3: 12345678
>>> +   * - SD
>>> +     - 01000000
>>> +     - 11000010
>>> +   * - OSPI
>>> +     - 00000000
>>> +     - 11001110
>>> +   * - EMMC
>>> +     - 00000000
>>> +     - 11010010
>>> +   * - UART
>>> +     - 00000000
>>> +     - 11011100
>>> +   * - USB DFU
>>> +     - 00000000
>>> +     - 11001010
>>
>> Hello Judith,
>>
>> ReStructured text allows different flavors of formatting tables.
>>
>> The HTML output for the table above is correct but the .rst file is hard
>> to read in a text-editor. I would prefer:
>>
>> *Boot Modes*
>>
>> ============ ============= =============
>> Switch Label SW2: 12345678 SW3: 12345678
>> ============ ============= =============
>> SD           01000000      11000010
>> OSPI         00000000      11001110
>> EMMC         00000000      11010010
>> UART         00000000      11011100
>> USB DFU      00000000      11001010
>> ============ ============= =============
> 
> There's other list-table entires in the docs today. I'm fine either way,
> but we need to be consistent in the docs, and should update perhaps
> doc/develop/codingstyle.rst should get a section too on docs?
> 

I will send again with the ASCII-style so y'all have the option.

regards,
Judith

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-03-27 16:15 Judith Mendez
@ 2023-03-27 16:40 ` Tom Rini
  0 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2023-03-27 16:40 UTC (permalink / raw)
  To: Judith Mendez; +Cc: u-boot, Andrew Davis, Bryan Brattlof, Heinrich Schuchardt

[-- Attachment #1: Type: text/plain, Size: 545 bytes --]

On Mon, Mar 27, 2023 at 11:15:21AM -0500, Judith Mendez wrote:

> List some common boot modes and their corresponding switch
> settings for AM62 SK.
> 
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
>  doc/board/ti/am62x_sk.rst | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)

Please see doc/board/apple/m1.rst for an example of list-table rather
than ascii-art tables. We should be adding this information via
list-table and then cleaning up existing parts of the docs to also use
it, thanks.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
@ 2023-03-27 16:15 Judith Mendez
  2023-03-27 16:40 ` Tom Rini
  0 siblings, 1 reply; 9+ messages in thread
From: Judith Mendez @ 2023-03-27 16:15 UTC (permalink / raw)
  To: Tom Rini, u-boot
  Cc: Andrew Davis, Bryan Brattlof, Heinrich Schuchardt, Judith Mendez

List some common boot modes and their corresponding switch
settings for AM62 SK.

Signed-off-by: Judith Mendez <jm@ti.com>
---
 doc/board/ti/am62x_sk.rst | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
index b1b7d99befb..40d58ea8bfa 100644
--- a/doc/board/ti/am62x_sk.rst
+++ b/doc/board/ti/am62x_sk.rst
@@ -229,3 +229,32 @@ Image formats:
                 | |   SPL DTB 1...N   | |
                 | +-------------------+ |
                 +-----------------------+
+
+Switch Setting for Boot Mode
+----------------------------
+
+Boot Mode pins provide means to select the boot mode and options before the
+device is powered up. After every POR, they are the main source to populate
+the Boot Parameter Tables.
+
+The following table shows some common boot modes used on AM62 platform. More
+details can be found in the Technical Reference Manual:
+https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
+
++--------------+------------+------------+
+|*Boot Mode*   | *SW2*      | *SW1*      |
++--------------+------------+------------+
+| Switch label | 12345678   | 12345678   |
++==============+============+============+
+| SD           | 01000000   | 11000010   |
++--------------+------------+------------+
+| OSPI         | 00000000   | 11001110   |
++--------------+------------+------------+
+| EMMC         | 00000000   | 11010010   |
++--------------+------------+------------+
+| UART         | 00000000   | 11011100   |
++--------------+------------+------------+
+| USB DFU      | 00000000   | 11001010   |
++--------------+------------+------------+
+
+For SW2 and SW1, the switch state in the "ON" position = 1.
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
  2023-01-10 22:37 ` Andrew Davis
@ 2023-01-11 19:58   ` Bryan Brattlof
  0 siblings, 0 replies; 9+ messages in thread
From: Bryan Brattlof @ 2023-01-11 19:58 UTC (permalink / raw)
  To: Andrew Davis
  Cc: Judith Mendez, Tom Rini, Heinrich Schuchardt,
	Vignesh Raghavendra, u-boot

Hi Judith

Wild... I'm not seeing this on the list? did you get a bounce?

On January 10, 2023 thus sayeth Andrew Davis:
> On 1/10/23 3:25 PM, Judith Mendez wrote:
> > List some common boot modes and their corresponding switch
> > settings for AM62 SK.
> > 
> > Signed-off-by: Judith Mendez <jm@ti.com>
> > ---
> 
> Acked-by: Andrew Davis <afd@ti.com>

Reviewed-by: Bryan Brattlof <bb@ti.com> 

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK
       [not found] <20230110212550.22844-1-jm@ti.com>
@ 2023-01-10 22:37 ` Andrew Davis
  2023-01-11 19:58   ` Bryan Brattlof
  0 siblings, 1 reply; 9+ messages in thread
From: Andrew Davis @ 2023-01-10 22:37 UTC (permalink / raw)
  To: Judith Mendez, Tom Rini
  Cc: Bryan Brattlof, Heinrich Schuchardt, Vignesh Raghavendra, u-boot

On 1/10/23 3:25 PM, Judith Mendez wrote:
> List some common boot modes and their corresponding switch
> settings for AM62 SK.
> 
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---

Acked-by: Andrew Davis <afd@ti.com>

>   doc/board/ti/am62x_sk.rst | 29 +++++++++++++++++++++++++++++
>   1 file changed, 29 insertions(+)
> 
> diff --git a/doc/board/ti/am62x_sk.rst b/doc/board/ti/am62x_sk.rst
> index b1b7d99befb..40d58ea8bfa 100644
> --- a/doc/board/ti/am62x_sk.rst
> +++ b/doc/board/ti/am62x_sk.rst
> @@ -229,3 +229,32 @@ Image formats:
>                   | |   SPL DTB 1...N   | |
>                   | +-------------------+ |
>                   +-----------------------+
> +
> +Switch Setting for Boot Mode
> +----------------------------
> +
> +Boot Mode pins provide means to select the boot mode and options before the
> +device is powered up. After every POR, they are the main source to populate
> +the Boot Parameter Tables.
> +
> +The following table shows some common boot modes used on AM62 platform. More
> +details can be found in the Technical Reference Manual:
> +https://www.ti.com/lit/pdf/spruiv7 under the `Boot Mode Pins` section.
> +
> ++--------------+------------+------------+
> +|*Boot Mode*   | *SW2*      | *SW1*      |
> ++--------------+------------+------------+
> +| Switch label | 12345678   | 12345678   |
> ++==============+============+============+
> +| SD           | 01000000   | 11000010   |
> ++--------------+------------+------------+
> +| OSPI         | 00000000   | 11001110   |
> ++--------------+------------+------------+
> +| EMMC         | 00000000   | 11010010   |
> ++--------------+------------+------------+
> +| UART         | 00000000   | 11011100   |
> ++--------------+------------+------------+
> +| USB DFU      | 00000000   | 11001010   |
> ++--------------+------------+------------+
> +
> +For SW2 and SW1, the switch state in the "ON" position = 1.

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2023-04-06 17:54 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-31 20:36 [PATCH] doc: ti: Add switch setting for boot modes on AM62 SK Judith Mendez
2023-03-31 20:37 ` Tom Rini
2023-04-04 13:59 ` Heinrich Schuchardt
2023-04-04 14:22   ` Tom Rini
2023-04-06 17:54     ` Mendez, Judith
  -- strict thread matches above, loose matches on Subject: below --
2023-03-27 16:15 Judith Mendez
2023-03-27 16:40 ` Tom Rini
     [not found] <20230110212550.22844-1-jm@ti.com>
2023-01-10 22:37 ` Andrew Davis
2023-01-11 19:58   ` Bryan Brattlof

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).