Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp814290imm; Sat, 14 Jul 2018 13:16:53 -0700 (PDT) X-Google-Smtp-Source: AAOMgpdwAf5N7neq/51pxotkTNS3mHiQQ/rVDhd/k8pLTiBfZgTIUdT7FDYe1+96Zc93L0qY+9/T X-Received: by 2002:a63:3c4a:: with SMTP id i10-v6mr10447464pgn.415.1531599413371; Sat, 14 Jul 2018 13:16:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531599413; cv=none; d=google.com; s=arc-20160816; b=O6RVT/PoFK25jyh4blS+XZy3/bXersKsB4EOLY+0vtmChUSV1vhj1Oog2KKHO3cTrk tHaVq4xisNfMmkuDs2vD0D+cuhhdtuqdMQaElg5wOit9+DXAIlOuryEttkQdlVjsODU+ fTh5dNau78bLeoSEICnIGXuMpnMafHVw6ZfJRoBbxSD0Q3uRMl62kEM7LOd6tfwIBK0l 2rRY8wyaA3TyetRPDLgLpB9HSbWUWyj8yr8Wq1rrkHYpHZpFx11ABZQEC9trhN3MISrW ynGNVGLROK4C8qEZNH5aMcfkj+ELyOduSx5TJJ8ViIzbwGzjlyRmb83YgyjUAVbp4dCk 0UPg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:user-agent:message-id :in-reply-to:date:references:subject:cc:to:from :arc-authentication-results; bh=N7kMyCR2Vq+47Wh9ywr8e82uiZ7veYk9AfsBQJPiDM0=; b=k8YiUF9G+4i4Aa89I78tYtvUhJmNLGmm64jOlXPYxJoRhV7xZV9vX5hXPe6NjOnTFg Mlwg+4vnP8QZlCKhTkLvRaT9TeQJcydA70i1ifO5xkA6Ge7oj1h6mJP0H6EZ5kNhgoKd GS9mt0xPHNXEIN7iz6OR7OU/nUaa+wOVfgVWcXkfMb5IKxRqHZZuQfC7w5kXtodoxStr s8aprxsk4f173z2Slpo6kmKjSYF0NwZsIUtDg+ghgJE0thbK8LwYKKsG/S3aePpW+809 voUtYQdnhTb7H5rMqwJ7X+FIN8n0xgjtty6vOeiDhmDbumzDyCPnXltW4aeJFgJCKKTZ DRvg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id i12-v6si10262812pgr.266.2018.07.14.13.16.37; Sat, 14 Jul 2018 13:16:53 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731175AbeGNUgO (ORCPT + 99 others); Sat, 14 Jul 2018 16:36:14 -0400 Received: from smtp01.smtpout.orange.fr ([80.12.242.123]:21959 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727955AbeGNUgN (ORCPT ); Sat, 14 Jul 2018 16:36:13 -0400 Received: from belgarion ([90.76.60.200]) by mwinf5d36 with ME id AkFm1y0044KClF603kFmY7; Sat, 14 Jul 2018 22:16:00 +0200 X-ME-Helo: belgarion X-ME-Auth: amFyem1pay5yb2JlcnRAb3JhbmdlLmZy X-ME-Date: Sat, 14 Jul 2018 22:16:00 +0200 X-ME-IP: 90.76.60.200 From: Robert Jarzmik To: Rob Herring Cc: Mark Rutland , Daniel Mack , Haojian Zhuang , Kees Cook , Anton Vorontsov , Colin Cross , Tony Luck , devicetree@vger.kernel.org, "linux-kernel\@vger.kernel.org" , "moderated list\:ARM\/FREESCALE IMX \/ MXC ARM ARCHITECTURE" Subject: Re: [PATCH 2/2] ARM: dts: pxa: add mioa701 board description References: <20180702055733.18578-1-robert.jarzmik@free.fr> <20180702055733.18578-2-robert.jarzmik@free.fr> X-URL: http://belgarath.falguerolles.org/ Date: Sat, 14 Jul 2018 22:15:45 +0200 In-Reply-To: (Rob Herring's message of "Mon, 9 Jul 2018 12:00:26 -0600") Message-ID: <87lgada80e.fsf@belgarion.home> User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Rob Herring writes: Hi Rob, >> + /* compatible = "mitac,mioa701"; */ >> + compatible = "marvell,pxa270"; > > Why the comment? Leftover, I'll remove it. >> + pstore_region:region@0xa2000000 { > > Drop the 0x Done. >> + compatible = "linux,contiguous-memory-region"; >> + reg = <0xa2000000 1048576>; > > Use hex for the size. Ok. >> + ffuart: uart@40100000 { > > Should be "serial@...". You'll have to fix in the base dtsi. Ok for all of these. >> + pxai2c1: i2c@40301680 { >> + mrvl,i2c-fast-mode; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_i2c_default>; >> + status = "okay"; >> + >> + mt9m111: camera@5d { >> + compatible = "micron,mt9m111"; >> + reg = <0x5d>; >> + gpios = <&gpio 56 GPIO_ACTIVE_HIGH>; >> + >> + remote = <&pxa_camera>; > > Not needed with the graph: I don't understand that, do you mean the port node is not needed ? > >> + port { >> + mt9m111_1: endpoint { >> + bus-width = <8>; >> + remote-endpoint = <&pxa_camera>; >> + }; >> + }; >> + }; >> + }; >> + gpio-keys { >> + compatible = "gpio-keys"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + autorepeat; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_gpiokeys_default>; >> + status = "okay"; > > Don't need status here? It shouldn't be defined in the base dtsi. Heuh what ? The status is just above, and gpio-keys is not defined in any dtsi. >> + >> + button@0 { > > If you have unit-address there should also be a reg property. Just > drop and name the node 'power' or 'power-button'. And similarly for > the rest. Ok. >> + >> + regulators { >> + compatible = "simple-bus"; > > Drop this and move children to the top level. This is not an mmio bus. Ok. >> + docg3: flash@0 { > > This probably should be a child of the bus controller. Most certainly yes. > >> + compatible = "m-systems,diskonchip-g3"; >> + reg = <0x0 0x2000>; >> + }; >> + >> + panel { >> + compatible = "toshiba,ltm0305a776"; >> + lcd-type = "color-tft"; > > This should be implied by the compatible. Mmm no. Actually, this is used by the framebuffer controller, ie. pxafb to set it up. And I don't think adding a list of panel compatibles in pxafb is a good idea. The corresponding code is in : drivers/video/fbdev/pxafb.c, see "static const char * const lcd_types[]". >> + display-timings { >> + native-mode = <&timing0>; > > We generally imply this from the compatible (i.e. use simple-panel). But you suppose there is an actual driver for the panel, right ? But there is none. > Could use another level of indentation. Sure. >> + ac_charger: charger@1 { > > Drop the unit-address. Ok. Thanks for the review. -- Robert