From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751961AbcFVHHK (ORCPT ); Wed, 22 Jun 2016 03:07:10 -0400 Received: from mail-db5eur01on0107.outbound.protection.outlook.com ([104.47.2.107]:12017 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751311AbcFVHHF (ORCPT ); Wed, 22 Jun 2016 03:07:05 -0400 X-Greylist: delayed 67707 seconds by postgrey-1.27 at vger.kernel.org; Wed, 22 Jun 2016 03:07:04 EDT Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=peda@axentia.se; Subject: Re: [PATCH 3/3] iio: potentiometer: mcp4531: Add device tree binding To: Florian Vaussard , , Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler References: <1466492137-32683-1-git-send-email-florian.vaussard@heig-vd.ch> <1466492137-32683-4-git-send-email-florian.vaussard@heig-vd.ch> <829a4507-39d7-e04a-9672-6309def89ee2@heig-vd.ch> CC: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Slawomir Stepien , Joachim Eastwood , Matt Ranostay , Cristina Moraru , , From: Peter Rosin Organization: Axentia Technologies AB Message-ID: Date: Wed, 22 Jun 2016 09:06:16 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <829a4507-39d7-e04a-9672-6309def89ee2@heig-vd.ch> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit X-Originating-IP: [217.210.101.82] X-ClientProxiedBy: HE1PR05CA0004.eurprd05.prod.outlook.com (10.162.181.14) To DB5PR02MB1301.eurprd02.prod.outlook.com (10.164.177.147) X-MS-Office365-Filtering-Correlation-Id: 7cc74165-59e9-4918-a1f7-08d39a6bb864 X-Microsoft-Exchange-Diagnostics: 1;DB5PR02MB1301;2:dmtJz75CUgY20S8nxveBxWlAqj2x/YMdyf5dYYdyU4nw3mWRt8ztxaJAA5h1J4gogeVxjv7IJwCD9QexUDELkWWQc6dcSJqzJCwMWAhfGn1mT4TuzqyCJFCoOSFaS/Q2v309mns8HWjTBxP7rBIDEFnqulCV0lnPjYJR1nLuk1y0iw7+Hk166urFUhlGQz9u;3:w8vyOvefkHhxjgruslcPMPGd2XKTeH+5w6HuqD/+I6r+xRri/rSDEzxBb16FnuGmWp+KCAoVXUcGe8wMwPs6RPcsobuHXDv1T0nLY7CLAEvVqfWZRXPxvZ85M8agsAx+;25:iTMJ4qfaFuoRjVlxE1Am9P+bEGfRk6aW3WhqW9qfB/7SYjvXeDk+GFov9FYDYF6bM/Rs1BHbhRnfsHilxMSssdoxQjVBAU1A+DHplVxnx+yi68UKHtDcolEZolwORYRBTcUtOyJ0ZtBsq7wzz6gu/alLryTluhlpkyzptZDqCZ7Ig6c5st8QWrXS9L1rYlp2M9VpzT9KuCUKERC0iWh5hLHQH4YddaJ3/qzi6ukVwxtCxnGcetSFQeTLclU5o6Ea0xOMjb3xjrAtmCh9yMouP4hqxhdKWsNg77hIAl6qMaHNynpSxgPnZ7qn7Y6KbRvApFVO+Zp7DYRQWc52I2qH43Ya0SAvzZ4eU7FijliRO1qTLrP+sIr0I1LRcQB0fDTU7izzzhLe6Nxbs3YJz0DeZ6ss64wlDC8EDNbxhZ0JeOg= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB5PR02MB1301; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040130)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041072)(6043046);SRVR:DB5PR02MB1301;BCL:0;PCL:0;RULEID:;SRVR:DB5PR02MB1301; X-Microsoft-Exchange-Diagnostics: 1;DB5PR02MB1301;4:fdJ1Yh+SCFwoOIji656axviXPMTrNjz6TPbhjAcb3gd7lMRVO9ArXAofrstKuBusUVspjiE/nX+WXzWjNVeDe+PTZWalzpauml4nBm2/aorR8gtsOKXWWbq5KklXP7/ulWlIex4lJARs2qoPUtGInrAJ6Yew/mownHbc73wiWngXNPZQqWsm8+dpCMHWInFQ8wJQOtk4o1Qw4Vqng6ySc6yCM4HEy8B86M6HMvg6Yvk2rqVaRjZOWursVuKzGyujS/I7Ck/YvICsv275mOGWk1to1UXmcfVUzTUpAooVs7UXzR9AMlsbtApQxgfw+dzapIMk6W8/ABpx9Fyu35/Yajc5izCdxmzIM8eIWZR7bnliNjxL1VBAmMLh3CszuKgbI++zIgD1yKwy6oll2gjQ4OgrLVhq7PSegn69/lxnF5I= X-Forefront-PRVS: 0981815F2F X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6009001)(6049001)(7916002)(199003)(51444003)(24454002)(377424004)(189002)(50466002)(19580405001)(92566002)(7846002)(106356001)(5001770100001)(83506001)(64126003)(8676002)(7736002)(81156014)(81166006)(42186005)(33646002)(31686004)(36756003)(68736007)(105586002)(97736004)(2870700001)(2906002)(76176999)(101416001)(189998001)(54356999)(3846002)(77096005)(74482002)(4326007)(2950100001)(6116002)(586003)(50986999)(31696002)(4001350100001)(23746002)(8666005)(117156001)(65956001)(66066001)(86362001)(65806001)(47776003)(93886004)(65826006)(7059030)(42262002);DIR:OUT;SFP:1102;SCL:1;SRVR:DB5PR02MB1301;H:[192.168.0.125];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;DB5PR02MB1301;23:5/E7UCJSjm968/QALfnhRRoOhp40Rh0RehPEB?= =?Windows-1252?Q?aSF0bIM2e+fiRgQDHc9ejvBy+Kvs0NdHVaZx4DL4qP8hQ+9+i0y1p5JE?= =?Windows-1252?Q?hjzvWZcZHzEuLg5ZjTS0JQi38iHkRMKydcpw3JZT7R75/dkD3Tja2L2O?= =?Windows-1252?Q?YjMxVUex8jJkjH9i7KnLt68DnAyAsADoFlA6R9CXxJQwra92q7rGfUeu?= =?Windows-1252?Q?KDs7LKkpmhIZh6Gzq6Qkm1ZFTHXR1zUNch8V9g3+TavvdO5n1DM/MVYX?= =?Windows-1252?Q?R6TIYgeTCZ6QLY5BxpTqd1uxOAtn6VuGPtztH6o976wTZcyuCdhWKMPS?= =?Windows-1252?Q?+uuCr9xsxU52H7cQjyNC+1FUPK6P7B75h1rQFbPuXwLryeMNSoq0dmrF?= =?Windows-1252?Q?3e/55OPsMrboaFIDPXaElhKIFNmcz6dyuI8dywtLnhn0u0pQ91FizNlz?= =?Windows-1252?Q?2piq4vkbvum2pvzBV+cGt1O+rNKoe1QrWe1s03nLLbfNtKBxiJGHifGh?= =?Windows-1252?Q?HZPgCZpPuAXm7XTY7d13yEosOCbiaBgtHf0tx1/jv3ftmfrjA+lBtUyr?= =?Windows-1252?Q?LaNZS1jWgT632AsRadhMC1/rjL3qJWeoUsORUPO1Y9GB0b3QOZRpIDc6?= =?Windows-1252?Q?yOlVFW3N4UgDa7gbQuBleKniepEqFlIW2FhtlEdh/hZJWMyYKBo3dSKX?= =?Windows-1252?Q?j6jOwX64++A/rldqF4i5bjlTZ6HKfJpRVOzvcuRhAJb+l0u974EY98ol?= =?Windows-1252?Q?gVIqI/CALwO+U2MLgCG5zxTDFyeX+1BdXCjJuoAsUgNPIVy3A2+sB+7c?= =?Windows-1252?Q?K1D2esFDpIE1zRJs1sprbtH4+mtMVTzj4A7FQe2Y7krY7xf320860Eha?= =?Windows-1252?Q?UITSaKGy9kKZDx6Qj19JIkB7EmWUPP6sYS+wGquPNYx5JFtWVTOfQoaS?= =?Windows-1252?Q?Lz0FxFwfoGU5WMuinXPaAv7frCXr0URnGaO++2XwQY4D8QG6SaRRQLUW?= =?Windows-1252?Q?l1w4faXKpTclYeUufHvyf/cA7Pn7ciPrIzEVp628OBtqJLY3NcnYQ8vJ?= =?Windows-1252?Q?XWvucpn/gM60Zbr8RHw00OCmbEpZVV3iwrlXPhGZ28YaetDm6pjfIfKR?= =?Windows-1252?Q?zpMasUA/IMkvG2ITDSs71WRXKgQLACogT7P8jSS4GGfM2tkb1d1RVtzO?= =?Windows-1252?Q?9CebgEwj1ilbAOqAruaFP7P7LK2g0ydjwpokUvPelCWfJDUpQSAvxCmX?= =?Windows-1252?Q?1KfJ/b75roBYLYAqjWnjiTol8HiyIzh7fRpzrO5JycDpgWAR92W36BZ4?= =?Windows-1252?Q?W8g7DWjJ7PmcNiFQHZGZF8C52R7KoJuaqAnz+kEhP9fiYT4z5Cvq9oUF?= =?Windows-1252?Q?Bp3ZAV/jOScEUujI+yn/hwV9XMFZcU+qxjnIVMSbhvR9Z02onTHZ+BFI?= =?Windows-1252?Q?umL02mz4K35Qvy/J20h3NHrJ2kC6ffvMXxwP+F74vj3wfVR5pTBigDUz?= =?Windows-1252?Q?HSmEfg=3D?= X-Microsoft-Exchange-Diagnostics: 1;DB5PR02MB1301;6:gJbVfP4VgZYS7Ow+HSeGhLOqYgApMFlGWxXZGdVNnlGYVsxv3DFG7uSl2YBQfYv0pNN4YJyILkvuJKQr81JbEaOWjp/hfjBFMhF1vf1khm39gGJhkdqdnVZprc+iG27PPL5a4/WumV/HM2pCpVHqz73il22qRxvVeYahosu/cohNg/vbVuyqaGSLFMEAw9Yce3/VKZ6L8grOhixpJF5j1m3I9X66yKWQ0XQpm/ulEaPlhtOn1oLHUSBSBVHJ6LHcNGqSD5zPww0tnTNGwx11CKF2pBRFrirH/OSQICfrQXivrS20QrCCsRlagDohzTrFv0Cma6gQXihQ6KTlrsJi9A==;5:ToS8Gz5nL7+HTppLQUcbWVxSfW4m6i+bynAqyPjef4EBjt5t1Rb8vsYzSyulxRhSm3C1/AmByKJfysiVZaCiR6DZ49tKKJQpY1+LtmllXqBdN2fHvFWLKE0XFCqGnbNthf2s+7D9HchUkpJGPnoWMw==;24:AVfZtFR5/jdMO0LOZPw57QkRk1bguQ6qTdwOspAIziZpCZzatuMIWLlV/u+zRIEBOlicHvsVIEMK19bVE+KX84XuIAujqbfXw5cOGJKar7Q=;7:vpTNshTwq95Q2qIn6UCqiBln2U3phgz5KA041SjeTzzWMOcTdu0G4HL60pxjIWNLWH09DNBD0F6rrDvdYYnJ4SpgKLW8EbYrKuLpBhv+lbQ3dtgM4FFdLRyHgcJ/DZ5G9npfaWjSxSOeExyyw0c3cQlP0RSz2iGKH3z/FM2qTqydX4CW/yQwSCi+SAw0Xx/vc+yzOyzcMNOQeThGc75guJf4LUJvmGsruZ6NR324eOr1LbiUrE/jRqUBuwJZoyTPQNNbxd/FK0G93Bf4QS6WWg== SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jun 2016 07:06:21.4896 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR02MB1301 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016-06-22 08:22, Florian Vaussard wrote: > Hello Peter, > > Le 21. 06. 16 à 09:51, Peter Rosin a écrit : >> That is, if you need this patch at all, see my reply to 2/3... >> > > This seems necessary in order to have the vendor ID in the compatible string. Hmm, I don't think so. The way I read the response from Rob was that *my* device tree snippet should not assume that the i2c subsystem ignores the vendor. So, I think that even w/o this patch a DT entry like mcp4651-104@28 { compatible = "microchip,mcp4651-104"; reg = <0x28>; }; will work, precisely since i2c ignores the microchip, part (and thus allows you to omit/misspell it). I.e. I think that Rob is concerned with how the DT is documented/defined, and not so much about how it is then implemented in Linux. I haven't actually tested that though, so I may be completely wrong in all of the above statements... Cheers, Peter