From patchwork Tue Sep 20 08:25:42 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michal Simek X-Patchwork-Id: 672151 X-Patchwork-Delegate: monstr@monstr.eu Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 3sdbS62BF2z9s5w for ; Tue, 20 Sep 2016 18:25:58 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=monstr-eu.20150623.gappssmtp.com header.i=@monstr-eu.20150623.gappssmtp.com header.b=Hf1D6WQC; dkim-atps=neutral Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 468CF4B98B; Tue, 20 Sep 2016 10:25:56 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1ZG1jptNVtg3; Tue, 20 Sep 2016 10:25:56 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id B3B3C4B77D; Tue, 20 Sep 2016 10:25:55 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 76BE44B77D for ; Tue, 20 Sep 2016 10:25:52 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1JJqzBHF5cD9 for ; Tue, 20 Sep 2016 10:25:52 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-wm0-f66.google.com (mail-wm0-f66.google.com [74.125.82.66]) by theia.denx.de (Postfix) with ESMTPS id 2ACC64A01C for ; Tue, 20 Sep 2016 10:25:48 +0200 (CEST) Received: by mail-wm0-f66.google.com with SMTP id b184so2049998wma.3 for ; Tue, 20 Sep 2016 01:25:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monstr-eu.20150623.gappssmtp.com; s=20150623; h=reply-to:subject:references:to:from:message-id:date:user-agent :mime-version:in-reply-to; bh=5Ote4zu+TwlqJu/WkYwY5YWCrGUyO3V4EbEpHFrTJ7c=; b=Hf1D6WQCohvRX10Cb5QzxUJ8s8Cz+VHZJEfl8ppIXl3qG+tamwFM+kOdX0V1hFnnsa 3fvxclO4INuzar9K5GzwBARR8CgphJBIoPE4MD8c3vmvRja7i1KtguMiBdKLg1Mb53oJ +BRUQ+dUuUFS8vDgLbEeq81yyHlG1BQPO9yRf30gMrlNdm8R7eJ3Ch/LTQiRYBH5L6rt L95VUJ9Clhx/UcHU6D+Uf0ygnBYW5gHIB7I8o8/6qvaj9f+cHdIxyj4emJrTbi7NarRn yygtnxOy06Yu3gvNFZD2PCtPemLUvIVjUbtBSqtHgghW7jEqXpSYve7ZxdRnRCjrNXzE Ztqg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:reply-to:subject:references:to:from:message-id :date:user-agent:mime-version:in-reply-to; bh=5Ote4zu+TwlqJu/WkYwY5YWCrGUyO3V4EbEpHFrTJ7c=; b=a5KrCqesAjKtGsy9c3MrzgKDfLOjmrtBJvw8/35sal0bwgWMHnyDr0splh2cN/aUOu fx6mJadpE524ET8cDymWzDwASAnl5nriiE/oNT54XYSiKYhewHhmFUHIfFinY4p5d3O1 hoSZIhdU3fpwWXF6+nAyXPb/E/FUGPR+IMcC3IxPJiktxk9RF3gK3+hwWUzW3JjL+pQn TqYZat3x3EaVLNM3vKsTPhAXRNoQfQCg8Gy6DLVEVDMIFAdMp55XoXsUEpJwXKGLI1ah E5hjx/fPBVLriMjVwGC2nJxaVbc7Yhj2XAJXej1tpZNj0uDqFk+eKxMuLMCghJLlmzZF 82sQ== X-Gm-Message-State: AE9vXwOLeB/yKvY8poZN/g4NoxqXm8KY6VKIzTfVE2PhQUr1MhCt9mgKRqLRTKSWD73kyg== X-Received: by 10.28.151.72 with SMTP id z69mr2183017wmd.61.1474359948072; Tue, 20 Sep 2016 01:25:48 -0700 (PDT) Received: from [192.168.0.105] (nat-35.starnet.cz. [178.255.168.35]) by smtp.gmail.com with ESMTPSA id md9sm27125909wjb.20.2016.09.20.01.25.47 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 20 Sep 2016 01:25:47 -0700 (PDT) References: <1473919370-3717-1-git-send-email-mike.looijmans@topic.nl> <57DA508C.8070704@topic.nl> <57E0DC88.2080903@topic.nl> To: Mike Looijmans , u-boot@lists.denx.de From: Michal Simek Message-ID: <430759df-42df-36e7-3ea6-f0dfef8120a5@monstr.eu> Date: Tue, 20 Sep 2016 10:25:42 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <57E0DC88.2080903@topic.nl> X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [U-Boot] [PATCH 0/2] Add TOPIC Miami boards X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list Reply-To: monstr@monstr.eu List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" On 20.9.2016 08:51, Mike Looijmans wrote: > On 15-09-16 09:41, Mike Looijmans wrote: >> On 15-09-16 08:54, Michal Simek wrote: >>> 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. >> >> I think a "topic" folder would be better, if I can avoid duplicating >> stuff. >> It'd also solve the merge conflict between the u-boot-xlnx tree and >> mainline, >> which have different makefiles. > > Tried that, but leads to needless complication and duplication. Xilinx > does not produce the "zed" or "zybo" boards either, so there's probably > no "digilent" folder for the same reasons. It is started with zedboard which was made by Avnet and Digilent together with Xilinx help that's why it was added there. There is cooperation between Xilinx and Digilent on these boards that's why they are added there. For your case this is private board which I expect none can buy as a kit that's why it should be in separate folder. I did play with it and it is not that hard. I did just these changes and move stuff to your folder. diff --git a/board/topic/zynq/zynq-topic-miami/ps7_init_gpl.c b/board/topic/zynq/zynq-topic-miami/ps7_init_gpl.c new file mode 100644 index 000000000000..0b4735a2b739 --- /dev/null +++ b/board/topic/zynq/zynq-topic-miami/ps7_init_gpl.c >>> 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. >> >> A bit of scripting would take care of most of the problems I guess. Just >> removing all those C++ comment lines would be a good start. > > Just wonder why other boards can include these messy files without > anyone caring? We did that in past but for example if you look at zybo which was added the last coding style is fixed there. >> Currently I just include zynq-common.h and then have to #undef a lot of >> things. Maybe I'd better just copy the relevant parts and avoid using >> zynq-common.h. The alternative would be to make zynq-common more >> common and >> less specific to the evaluation boards. > > Any thoughts on that? As you see all stuff is coming to Kconfig that's why I hope that we can reduce number of configurations files for every board. Also currently as you see buildman is reporting this as a bug now. Thanks, Michal diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig index a98232097b28..525f0d3e6b2e 100644 --- a/arch/arm/mach-zynq/Kconfig +++ b/arch/arm/mach-zynq/Kconfig @@ -4,6 +4,7 @@ config SYS_BOARD default "zynq" config SYS_VENDOR + string "Vendor name" default "xilinx" config SYS_SOC diff --git a/board/topic/zynq/Makefile b/board/topic/zynq/Makefile new file mode 100644 index 000000000000..5653b9928a3c --- /dev/null +++ b/board/topic/zynq/Makefile @@ -0,0 +1,5 @@ +# +# SPDX-License-Identifier: GPL-2.0+ +# + +include $(srctree)/board/xilinx/zynq/Makefile diff --git a/board/topic/zynq/board.c b/board/topic/zynq/board.c new file mode 120000 index 000000000000..75964550f37a --- /dev/null +++ b/board/topic/zynq/board.c @@ -0,0 +1 @@ +../../xilinx/zynq/board.c \ No newline at end of file diff --git a/board/topic/zynq/xil_io.h b/board/topic/zynq/xil_io.h new file mode 120000 index 000000000000..c0f3a8a26321 --- /dev/null +++ b/board/topic/zynq/xil_io.h @@ -0,0 +1 @@ +../../xilinx/zynq/xil_io.h \ No newline at end of file