Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-06-12 Thread Fabio Estevam
Hi Soeren,

On Thu, Jun 4, 2020 at 6:29 AM Soeren Moch  wrote:

> Maybe Fabio knows a better way how to reference NXP manuals? Otherwise

It seems that all the URLs for accessing the i.MX6 Reference Manual
require a login now, unfortunately.


Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-06-12 Thread Tom Rini
On Sat, May 30, 2020 at 05:24:25AM +0200, Denis 'GNUtoo' Carikli wrote:

> This documents the u-boot installation procedure and the
> hardware in order to get started.
> 
> Signed-off-by: Denis 'GNUtoo' Carikli 

I see there's some unresolved discussion about how to have a more
persistent link to (now move/removed) documentation but that can be
handled in a follow-up I believe.

Applied to u-boot/next, thanks!

-- 
Tom


signature.asc
Description: PGP signature


Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-06-04 Thread Soeren Moch
On 04.06.20 06:39, Denis 'GNUtoo' Carikli wrote:
> On Sat, 30 May 2020 19:18:33 +0200
> Soeren Moch  wrote:
>
>>> +Links:
>>> +--
>>> +  -
>>> https://www.tbsdtv.com/download/document/tbs2910/TBS2910-Matrix-ARM-mini-PC-SCH_rev2.1.pdf
>>> +- The schematics for the revision 2.1 of the TBS2910 Matrix
>>> ARM miniPC.
>>> +  -
>>> https://cache.freescale.com/files/32bit/doc/ref_manual/IMX6DQRM.pdf
>>> - The
>>> +SoC reference manual for additional details on the BOOT_CFG
>>> registers.
>> Unfortunately this link does not work for me. Is some sort of login
>> required? Or was this moved away?
> I didn't know it has moved.
>
> The old address has been archived by archive.org/web at the following
> address:
> https://web.archive.org/web/20190826025113/https://cache.freescale.com/files/32bit/doc/ref_manual/IMX6DQRM.pdf
>
> The Linux kernel has a unique link to this document at this URL:
> http://www.nxp.com/assets/documents/data/en/reference-manuals/IMX6DQRM.pdf
>
> And it's the same since since 2017, and it was introduced by this
> commit:
>> commit e130291212df5ce8160cd2e35387c96439863ad3
>> Author: Steve Longerbeam 
>> Date:   Sat Jun 10 16:00:29 2017 -0300
>>
>> [media] media: Add i.MX media core driver
>> 
>> Add the core media driver for i.MX SOC.
>> 
>> Switch from the v4l2_of_ APIs to the v4l2_fwnode_ APIs.
>> Add the bayer formats to imx-media's list of supported pixel and
>> bus formats.
>> 
>> Signed-off-by: Steve Longerbeam 
>> Signed-off-by: Philipp Zabel 
>> Signed-off-by: Russell King 
>> Signed-off-by: Hans Verkuil 
>> Signed-off-by: Mauro Carvalho Chehab 
> However that redirects to https://www.nxp.com/doc/IMX6DQRM which
> redirects to a login page.
AFAIK a login (free of charge, besides registering the e-mail address)
is required to download NXP documents. This way you always get the
latest document version, not a old version from some archive.

Maybe Fabio knows a better way how to reference NXP manuals? Otherwise
for this u-boot board documentation the link from linux would be fine
with me, maybe with the additional archive reference.

Soeren
> Archive.org/web has many links for that page, but I only found one of
> them to work, which is older than the archive of the link I had: 
> https://web.archive.org/web/20180829173128/https://www.nxp.com/docs/en/reference-manual/IMX6DQRM.pdf
>
> I'm not aware of any project tracking datasheet / reference manual URLs
> and versions though, but it would probably be something very useful to
> have.
>
> At some point, I plan to see if it's possible to something like that
> with Wikidata, but I'm not sure if it's fit to track datasheets URLs
> over time.
>
> Denis.




Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-06-03 Thread Denis 'GNUtoo' Carikli
On Sat, 30 May 2020 19:18:33 +0200
Soeren Moch  wrote:

> > +Links:
> > +--
> > +  -
> > https://www.tbsdtv.com/download/document/tbs2910/TBS2910-Matrix-ARM-mini-PC-SCH_rev2.1.pdf
> > +- The schematics for the revision 2.1 of the TBS2910 Matrix
> > ARM miniPC.
> > +  -
> > https://cache.freescale.com/files/32bit/doc/ref_manual/IMX6DQRM.pdf
> > - The
> > +SoC reference manual for additional details on the BOOT_CFG
> > registers.
> Unfortunately this link does not work for me. Is some sort of login
> required? Or was this moved away?
I didn't know it has moved.

The old address has been archived by archive.org/web at the following
address:
https://web.archive.org/web/20190826025113/https://cache.freescale.com/files/32bit/doc/ref_manual/IMX6DQRM.pdf

The Linux kernel has a unique link to this document at this URL:
http://www.nxp.com/assets/documents/data/en/reference-manuals/IMX6DQRM.pdf

And it's the same since since 2017, and it was introduced by this
commit:
> commit e130291212df5ce8160cd2e35387c96439863ad3
> Author: Steve Longerbeam 
> Date:   Sat Jun 10 16:00:29 2017 -0300
> 
> [media] media: Add i.MX media core driver
> 
> Add the core media driver for i.MX SOC.
> 
> Switch from the v4l2_of_ APIs to the v4l2_fwnode_ APIs.
> Add the bayer formats to imx-media's list of supported pixel and
> bus formats.
> 
> Signed-off-by: Steve Longerbeam 
> Signed-off-by: Philipp Zabel 
> Signed-off-by: Russell King 
> Signed-off-by: Hans Verkuil 
> Signed-off-by: Mauro Carvalho Chehab 

However that redirects to https://www.nxp.com/doc/IMX6DQRM which
redirects to a login page.

Archive.org/web has many links for that page, but I only found one of
them to work, which is older than the archive of the link I had: 
https://web.archive.org/web/20180829173128/https://www.nxp.com/docs/en/reference-manual/IMX6DQRM.pdf

I'm not aware of any project tracking datasheet / reference manual URLs
and versions though, but it would probably be something very useful to
have.

At some point, I plan to see if it's possible to something like that
with Wikidata, but I'm not sure if it's fit to track datasheets URLs
over time.

Denis.


pgpf5cOTaKtui.pgp
Description: OpenPGP digital signature


Re: [PATCHv5][ 7/7] board: tbs2910: add documentation

2020-05-30 Thread Soeren Moch
On 30.05.20 05:24, Denis 'GNUtoo' Carikli wrote:
> This documents the u-boot installation procedure and the
> hardware in order to get started.
>
> Signed-off-by: Denis 'GNUtoo' Carikli 
> ---
> Changelog since v4:
> ---
> - Added dded information about SYSBOOT boot support as requested.
> - Fixed the switch positions as suggested.
> ---
>  doc/board/index.rst   |   1 +
>  doc/board/tbs/index.rst   |   9 ++
>  doc/board/tbs/tbs2910.rst | 191 ++
>  3 files changed, 201 insertions(+)
>  create mode 100644 doc/board/tbs/index.rst
>  create mode 100644 doc/board/tbs/tbs2910.rst
Denis,

thanks for providing this documentation. 2 remarks below.
Since this already is v5, do you want to resend a fixed version of this
series, or should we fix this later in a separate patch?

If you want to send a new version of this patch, please also add
---8<---
diff --git a/board/tbs/tbs2910/MAINTAINERS b/board/tbs/tbs2910/MAINTAINERS
index a3ad2f712a..1e3c0d0ece 100644
--- a/board/tbs/tbs2910/MAINTAINERS
+++ b/board/tbs/tbs2910/MAINTAINERS
@@ -4,4 +4,5 @@ S:  Maintained
 F: arch/arm/dts/imx6q-tbs2910.dts
 F: board/tbs/tbs2910/
 F: configs/tbs2910_defconfig
+F: doc/board/tbs/
 F: include/configs/tbs2910.h
---8<---
(whitespace may be broken in this snippet)

Thanks,
Soeren
>
> diff --git a/doc/board/index.rst b/doc/board/index.rst
> index 01b233f737..bb4473152a 100644
> --- a/doc/board/index.rst
> +++ b/doc/board/index.rst
> @@ -18,5 +18,6 @@ Board-specific doc
> rockchip/index
> sifive/index
> st/index
> +   tbs/index
> toradex/index
> xilinx/index
> diff --git a/doc/board/tbs/index.rst b/doc/board/tbs/index.rst
> new file mode 100644
> index 00..b677bc624f
> --- /dev/null
> +++ b/doc/board/tbs/index.rst
> @@ -0,0 +1,9 @@
> +.. SPDX-License-Identifier: GPL-2.0+
> +
> +TBS
> +===
> +
> +.. toctree::
> +   :maxdepth: 2
> +
> +   tbs2910
> diff --git a/doc/board/tbs/tbs2910.rst b/doc/board/tbs/tbs2910.rst
> new file mode 100644
> index 00..e97f2b6e61
> --- /dev/null
> +++ b/doc/board/tbs/tbs2910.rst
> @@ -0,0 +1,191 @@
> +TBS2910 Matrix ARM miniPC
> +=
> +
> +Building
> +
> +To build u-boot for the TBS2910 Matrix ARM miniPC, you can use the following
> +procedure:
> +
> +First add the ARM toolchain to your PATH
> +
> +Then setup the ARCH and cross compilation environment variables.
> +
> +When this is done you can then build u-boot for the TBS2910 Matrix ARM miniPC
> +with the following commands:
> +
> +.. code-block:: none
> +
> +   make mrproper
> +   make tbs2910_defconfig
> +   make
> +
> +Once the build is complete, you can find the resulting image as u-boot.imx in
> +the current directory.
> +
> +UART
> +
> +The UART voltage is at 3.3V and its settings are 115200bps 8N1
> +
> +BOOT/UPDATE boot switch:
> +
> +The BOOT/UPDATE switch (SW11) is connected to the BOOT_MODE0 and
> +BOOT_MODE1 SoC pins. It has "BOOT" and "UPDATE" markings both on
> +the PCB and on the plastic case.
> +
> +When set to the "UPDATE" position, the SoC will use the "Boot From Fuses"
> +configuration, and since BT_FUSE_SEL is 0, this makes the SOC jump to serial
> +downloader.
> +
> +When set in the "BOOT" position, the SoC will use the "Internal boot"
> +configuration, and since BT_FUSE_SEL is 0, it will then use the GPIO pins
> +for the boot configuration.
> +
> +SW6 binary DIP switch array on the PCB revision 2.1:
> +
> +On that PCB revision, SW6 has 8 positions.
> +
> +Switching a position to ON sets the corresponding
> +register to 1.
> +
> +See the following table for a correspondence between the switch positions and
> +registers:
> +
> +===
> +Switch positionRegister
> +===
> +1  BOOT_CFG2[3]
> +2  BOOT_CFG2[4]
> +3  BOOT_CFG2[5]
> +4  BOOT_CFG2[6]
> +5  BOOT_CFG1[4]
> +6  BOOT_CFG1[5]
> +7  BOOT_CFG1[6]
> +8  BOOT_CFG1[7]
> +===
> +
> +For example:
> +
> +  - To boot from the eMMC: 1:ON , 2:ON, 3:ON, 4:OFF, 5:OFF, 6:ON, 7:ON, 8:OFF
> +  - To boot from the microSD slot: 1: ON, 2: OFF, 3: OFF, 4: OFF, 5:OFF, 
> 6:OFF,
> +7:ON, 8:OFF
> +  - To boot from the SD slot: 1: OFF, 2: ON, 3: OFF, 4: OFF, 5:OFF, 6:OFF, 
> 7:ON,
> +8:OFF
> +  - To boot from SATA: 1: OFF, 2: OFF, 3: OFF, 4: OFF, 5:OFF, 6:ON, 7:OFF, 
> 8:OFF
> +
> +You can refer to the BOOT_CFG registers in the I.MX6Q reference manual for
> +additional details.
> +
> +SW6 binary DIP switch array on the PCB revision 2.3:
> +
> +On that PCB revision, SW6 has only 4 positions.
> +
> +Switching a position to ON sets the corresponding
> +register to 1.
> +
> +See the following table for a correspondence