[U-Boot] [PATCH 0/2] Add TOPIC Miami boards

[Michal Simek]

Hi Mike,

On 15.9.2016 08:02, Mike Looijmans wrote:
> These patches add support for the Miami range of boards from TOPIC.
> The boards are based on Xilinx Zynq SoCs, these two patches are for
> the 7-series, the Ultrascale MPSOC boards are to be added later.
> 
> Please note that the "ps7_init_gpl" files are largely generated by a
> tool and as a result of that, contain many style issues.
> 
> Mike Looijmans (2):
>   Add topic-miami board support
>   Add topic_miamiplus board
> 
>  arch/arm/dts/Makefile                              |    2 +
>  arch/arm/dts/zynq-topic-miami.dts                  |   98 +
>  arch/arm/dts/zynq-topic-miamiplus.dts              |   22 +
>  board/xilinx/zynq/zynq-topic-miami/ps7_init_gpl.c  | 4324 +++++++++++++++++++
>  board/xilinx/zynq/zynq-topic-miami/ps7_init_gpl.h  |  118 +
>  .../zynq/zynq-topic-miamiplus/ps7_init_gpl.c       | 4425 ++++++++++++++++++++
>  .../zynq/zynq-topic-miamiplus/ps7_init_gpl.h       |  150 +
>  configs/topic_miami_defconfig                      |   47 +
>  configs/topic_miamiplus_defconfig                  |   47 +
>  include/configs/topic_miami.h                      |  147 +
>  include/configs/topic_miamiplus.h                  |    2 +
>  11 files changed, 9382 insertions(+)
>  create mode 100644 arch/arm/dts/zynq-topic-miami.dts
>  create mode 100644 arch/arm/dts/zynq-topic-miamiplus.dts
>  create mode 100644 board/xilinx/zynq/zynq-topic-miami/ps7_init_gpl.c
>  create mode 100644 board/xilinx/zynq/zynq-topic-miami/ps7_init_gpl.h
>  create mode 100644 board/xilinx/zynq/zynq-topic-miamiplus/ps7_init_gpl.c
>  create mode 100644 board/xilinx/zynq/zynq-topic-miamiplus/ps7_init_gpl.h
>  create mode 100644 configs/topic_miami_defconfig
>  create mode 100644 configs/topic_miamiplus_defconfig
>  create mode 100644 include/configs/topic_miami.h
>  create mode 100644 include/configs/topic_miamiplus.h
> 

It is a question if this should go to board/xilinx folder.
I think you should create own folder just for your company.
Probably doesn't make sense to copy all Makefiles and so but I think
you can source them from Xilinx folder to be only at one location.

The next thing is that there needs to be additional steps to fix
ps7_init_gpl.* files to keep checkpatch happy.
I have created change request to fix this directly in tools but still
none has fixed it and the team which is responsible for it is keep
moving to the next release but I can do nothing with it.

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160915/69c20d32/attachment.sig>