From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754011AbcGDXl7 (ORCPT ); Mon, 4 Jul 2016 19:41:59 -0400 Received: from relmlor3.renesas.com ([210.160.252.173]:58326 "EHLO relmlie2.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752371AbcGDXlz (ORCPT ); Mon, 4 Jul 2016 19:41:55 -0400 X-IronPort-AV: E=Sophos;i="5.22,559,1449500400"; d="scan'";a="214868370" Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Message-ID: <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto To: Rob Herring , Mark Brown CC: Laurent , Guennadi , Grant Likely , Frank Rowand , Linux-Kernel , Linux-DT , Linux-ALSA Subject: Re: [PATCH v2 0/7] of_graph: prepare for ALSA graph support In-Reply-To: <20160704160736.GA25708@rob-hp-laptop> References: <87k2h8g7rr.wl%kuninori.morimoto.gx@renesas.com> <20160704160736.GA25708@rob-hp-laptop> User-Agent: Wanderlust/2.15.9 Emacs/24.3 Mule/6.0 MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset="US-ASCII" Date: Mon, 4 Jul 2016 23:41:48 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: TY1PR06CA0023.apcprd06.prod.outlook.com (10.164.91.33) To PS1PR06MB1705.apcprd06.prod.outlook.com (10.167.53.15) X-MS-Office365-Filtering-Correlation-Id: 7e9dc950-2bed-4211-1bfa-08d3a464c4c8 X-Microsoft-Exchange-Diagnostics: 1;PS1PR06MB1705;2:gUV7b+KqP1s1qsSGIn9hwX9ViyBabxScknVxXF++DdBCr3JBV7ORd9yLTFiBITrfJ0Kh6Iq9WD4tvARvtowO4FhJbdqsnrxhgp3yZNqKYw04ksSGIhx4gGmUuve/C/VyLj/vZ6xN7vZlJzl5QJFl00wpnul6ayNJiJ18gYBQf/fJiVcLimhgrORUOl2C+Hoj;3:V2kKq6NOpmsWHEzORasfJ9najD9H3AE5jGF0vuGWovvE34w+Xr73tpm39aivN9pHcAcTEweyoE68TjXR5bHSuiIr0VuPj4HZ3zYdZ4r57qCL6+qBlliY4bFGjrAKfmWc;25:JbgdLW5JHogYX6AgfcfVd3rhOUYJ9hEYBNzPuXODzLWF915acJHfg++X6nWI7SGK5wOp+7usdkOOuVlrzPAdNBPoOVSKrtL6B+TbFGHz6CduTpVXRwWhdDtwGB0G9HHeA8pC4PffCDgHNmRn6etsM67CLlXSIuhZlJLs94GC1i5fvz2SmUHkbnKy4jSWfgu60FNAsIPSKqQ7Vn02YJFlxmsXtUfp9xEZVlWBQFeOVikFdq/F/6ejvxd/TGm8KEeNap8ieyRKYRkrxtOj7VL/iXePQxFHVB8KRsHy6hV6eBvuJ294MLo/eZjpODSCTHtiJTJayheuSvo7doASrbDnOxBX6gZm36HtS+axoKATKJeAfGsi43Lhpm0hyMZH79LOQq+dggE5+5mlxmMo3CvwdFMzRfSFtf4KB6uTpXO0NaI=;31:s5mWOM0dg0EzxugxBjQJqgBBEJ43h51MbP8LGxxxVH7C7RqH8kLL7V/x1gx8ja7znlsJPlESpD0zK23knZ2c9HXRM8zQLo6H84ytuv7TQfSWpiwm/AGdIotwlhaGzIyk5SopwLwJJLnGKK72WiDUOZIG01a9idCFvPtxEcdAWXGUX7n6p90WI5HOgtcEDQZlrjd9iRO0k5Mi7K/s3w7W FQ== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:PS1PR06MB1705; X-Microsoft-Exchange-Diagnostics: 1;PS1PR06MB1705;20:IiOFMuVf0BRvO5qppxoyC/TIhzcLxrtk+2mc6AGszYF+AHteA49EQhgFIk0igAbkNd3AwrzXH6kxXzvyNsrbCt8G0s293x9ItXeU8mhZyRCM7I1PosY/lLHxqXQ2FfR5Zmgd0uTq2Xw24mdwlaJviuH4h/RItjNO61x6X9wdLJCg2e9rMPH6z8AOWI+xJMAewfLG0ojRfgYgHNXXB1nEy7ck3d4JEzERfZp0iUVprbaa0Mv0i3ZdK29Mr9lFan5fITGTO+dfg5Eqxu0KGLYxqX1Z56DMhochpO/0voXrZ/GKv7vkoRV5vaNqp8azisCZFTgK4WvwR+jdQGaqy/EO90xfnPeYGhZA8z6J/yNk4BoxA0EZlMa2RRyA89j0pxkIrnrCkR6EUR7kgsJf4NiXt1CBXw8cH0rZX7TYiwY8dTTytdAPmp1OzOmqYtWfouan9FYf+O1d8lh3tvWe+MSXf9Gimg004xI1Giil/yPwuLaqFFBLdOhqPydP+swiS30v;4:qJotkKsANA4vv8R5E0XrRyZEnKhTPaz81TKRh+4eqiVIb5ysOym444VwowI8M/0wtUMTm8Tk2Udb6A7xGhkXAOr3qqrsTFNqKde1IHwWLsZys6s6bV5z5kvxJBKP5bzGloHW5KBcNRulVrnzCnxiLnZkZ3tuOvSMhUHPGa7itwDuAn4QgnCa3DHhbI49Ic3jLDXH7eVqeU+3iEx+Mk4rp+7EeNRGULb5K2CGIJRwNDEUUg4UjKYLhsUgrBqvUbJQgBsUNvhjANCe1CFSsy5ucapYhO9WMIE5h3gfP/iYMUI05wz8WSl6BPCZBo4ijRypgE9y13XZcTsXhpfllH+UgKC520Dt/d7Z10O1s+2ISxHQFucv1s5jC1fjsXVhn3B6HlyLCPH5womqXb3EPkMC1A== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026);SRVR:PS1PR06MB1705;BCL:0;PCL:0;RULEID:;SRVR:PS1PR06MB1705; X-Forefront-PRVS: 0993689CD1 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6009001)(7916002)(189002)(199003)(53416004)(4001350100001)(2950100001)(42186005)(81166006)(81156014)(66066001)(106356001)(54356999)(47776003)(586003)(97736004)(101416001)(8676002)(5001770100001)(36756003)(345774005)(189998001)(7846002)(33646002)(92566002)(68736007)(4326007)(7736002)(2906002)(305945005)(105586002)(46406003)(83506001)(77096005)(69596002)(3846002)(6116002)(76176999)(50986999)(8666005)(50466002)(23726003)(86362001)(7059030)(16060500001);DIR:OUT;SFP:1102;SCL:1;SRVR:PS1PR06MB1705;H:morimoto-PC.renesas.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;PS1PR06MB1705;23:d3buEiebnv2OsznOWVkPRNs7SMmZ7Ks3cSvryeUkX?= =?us-ascii?Q?YlHYza0LtzgLeNgMLJwmGvjMo1Q4T83Eq825hYKhIHhN7dkP2Yde0xRv9gH/?= =?us-ascii?Q?3yJBvikI5B2tAzTvwCMIuczpaxXr4I9vm7m8BqsVwMyU+/F1j2TEdKlusucP?= =?us-ascii?Q?/Qe8ygAaPnxCVqwOEgkZt2Z4n2fJjk9txbnPT00rZVlzD6GrQTO3NsaUeJBX?= =?us-ascii?Q?e/tJowQUbakXqSxj2n+wdZrUL5+usdN6mCVexck/dcSzXwmF/9xNwkItl4k7?= =?us-ascii?Q?YD1yskudUHEl+rvzwhbnWaEfoQoKyNheH9hJR3PniTzX1sqvH87VeFiKc4Hj?= =?us-ascii?Q?mg0+DYCEb02rEomOiEcfwbd30DMGNmgZYoBygYsHW1uJzgcD15l7w6ukBMYE?= =?us-ascii?Q?15JPH9ECwklTUxcBeBo49fR0nNjxC6/WL8vln77yeB7uNXxSZladJgt7/gYA?= =?us-ascii?Q?HBCzyTdUZ6QgX0w4FyqyiELGgDtGzYZUbISu8wpQ8l2MwghC6eV+1IOdGkJ2?= =?us-ascii?Q?mxRJ4Ch0tjpSFT6RmbZyAtO7GkWPwKtc3DweVpPOyRHXJzTjSMw3I345QmXl?= =?us-ascii?Q?2TsgNjTbrSIpZptr21I+WXvxJlXkgdGCUpjwqQjBL74/13P1y84612heO28S?= =?us-ascii?Q?9Z7W/hRJRP/tUakdyD6GXwH99iBkF43pQGztLjNG1yMto2953ZkYZScbsley?= =?us-ascii?Q?BUaf0pg7FkZJaxElT+uCZzzIkFj4h+0/Wy+6aGER7iNvuQ059HIUGvnTdJxm?= =?us-ascii?Q?U0GloeCz177YYmwbjF3niInxRVA/N2kgN0Z6o4GS+TOilcADMKt4kWm9XAXG?= =?us-ascii?Q?TIowzcho1B1MTKlQK6J3Nw82EbGBW862A85LuckWMcv5DEKemg6RvfQYk9C3?= =?us-ascii?Q?LwgHB0KQpw7aRXFKwJ+eNwSAJPmos/x7bBtZ95W/0LrV+oy4p9/isnK1Omdb?= =?us-ascii?Q?E7tNjlRY+y/MBRxiIpPRxGdZTM3xgXlELehlxLLAQwoKK72Kvju17nBVQhnI?= =?us-ascii?Q?K2+9bi29LRwpgGQ3EV35d8QulCHOW2qJuVmSVlgXQajCVD5tUXwtFJtgHT6D?= =?us-ascii?Q?fx/1kQMcrZOmXOQWB5GU7quTAiwuxLAZIDGLn3UHJ9TJ9/zRa0wFc4DgHz8p?= =?us-ascii?Q?FmcTMEYqVt7wT5IhJDGrwGDtpJoaVSJJOpic/d0NT6KjVQ1xTkYJw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;PS1PR06MB1705;6:ToInEqKNESDNQriv0tWjYSK14webhYbYY55vxIPgdpbSf3fSEQSdekDLPoIscsL5y78GG/0w1YwIM/MfMyiJIDbKpLscRKLetYpCohcrogUNjL4EK3zIpUuYE8GrZWlzOELXEOlqG9nSLt+GB5H7y0g/lrAAUZh62mN4ZtY46+TpGfKaMW3gUBjEMc7OtWGtBzKq3ERbzZVgYZPEmErq73mvIEkg6mX6hhp1XZw+z5dROsF5Y1XsiVrej47rZuQt1jIBOJ/q0/7v2N9TzZetsDeZN2oUT0cLaBFUHOQXWTGd9FQcB4UFaDoZjHEe/ReP3HbCbM/kjD8ZWbtUQrZn9w==;5:2BjBsBjn325vu+ubunu1IJKJ6k8I6gzhhzOsPZtpElqyIIeByImbnme60IclEJL1ce8cGTbV4Tl5XKzlKWb4Tpg5Yk/xSRgdbLU/76Tc1aRhsdos0kxwlmRGokpnZ72X+7YRPeV+k/ayWOmkjPKTJA==;24:H6M6STs2Wur1TpFt88X0fKd0opYOt3M3bJg/CKBqZ1DCg0L5lsRC2OV9J6/5nCs/bA510wnuPkifERp4wUhd5to68LNqcu8fbbHt9jeTJwg=;7:kztJ+qYoiZJtWL4+4TS9FI1e/At8vzbnH1mkehhXB6BqTrBgZl4zLooqJ5zqgYnXAgfkJ/WFIODFI6wPt/ut4pVfXU9bX9lY1dSrggunpvBRKa5WlXOnp2yS8LJlYvmSiiZOBKYB/bc83a4TiNm9/UagNPlx1C+KKIbfSTJsk/PeHHxoawmOwQpgY8SoVgbofzB8GURDUHVs7eMP+3n1Exp/2VN/pXU/tLJyX/9bNbFeXOhMCvz3P3SpZagM0Xsx SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;PS1PR06MB1705;20:DptjhJPpziA7GiN/YZ89qlpdbAN9MZtkiEvRnSq5o9gNR5gktn7rp5MK/6QwdXLGOtGfv6LMs4PVhMTkdmwrr87GtDqmQ+X95kckwRMgqToFtRNiTB6HyY7mQHdvGzxOpog42SO6FUvdnQhPpb1y4Drd4vxYxhR3Lm3izl9WPIw= X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jul 2016 23:41:48.8480 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: PS1PR06MB1705 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, Mark Thank you for your feedback > I need to see how you want to use this. I'm not completely convinced > this is necessary as the port number should be meaningful. For example, > port 0 is video and port 1 is audio. This information is specific to the > local parent node which could be problematic if you have generic code > parsing the local node. However, if you know which local port is audio, > then you know the remote endpoint's port is also the audio port. Yes, as you pointed, sound side (= ALSA SoC) will use generic driver for sound card which needs to know its total port number. Then, these patches are needed. I posted OF graph part only this time, but of course I have total full-set in my local environment. But these are ... 1) cleanup current generic sound driver (= almost 30 patch) 2) OF graph new feature (= this patch-set) (= almost 10 patch) 3) OF graph base generic sound driver (= almost 30 patch) - ... I'm posting 1) part to ALSA SoC ML, 2) part to this ML. 1) will take more long term >> Mark ?? If you want to see this patch-set together with use case, then 3) is needed. If so, I will merge 2) and 3), and post these to this ML and ALSA SoC ML. Is it OK for you ? >> Rob, Mark I don't know how to handle it, but I can follow your opinion > In general, all these helpers look okay if you have a user for them. Thanks. As I explained above, it is based on 1) part, but it will takes more long term. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kuninori Morimoto Subject: Re: [PATCH v2 0/7] of_graph: prepare for ALSA graph support Date: Mon, 4 Jul 2016 23:41:48 +0000 Message-ID: <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> References: <87k2h8g7rr.wl%kuninori.morimoto.gx@renesas.com> <20160704160736.GA25708@rob-hp-laptop> Mime-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset="US-ASCII" Return-path: In-Reply-To: <20160704160736.GA25708@rob-hp-laptop> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring , Mark Brown Cc: Laurent , Guennadi , Grant Likely , Frank Rowand , Linux-Kernel , Linux-DT , Linux-ALSA List-Id: devicetree@vger.kernel.org Hi Rob, Mark Thank you for your feedback > I need to see how you want to use this. I'm not completely convinced > this is necessary as the port number should be meaningful. For example, > port 0 is video and port 1 is audio. This information is specific to the > local parent node which could be problematic if you have generic code > parsing the local node. However, if you know which local port is audio, > then you know the remote endpoint's port is also the audio port. Yes, as you pointed, sound side (= ALSA SoC) will use generic driver for sound card which needs to know its total port number. Then, these patches are needed. I posted OF graph part only this time, but of course I have total full-set in my local environment. But these are ... 1) cleanup current generic sound driver (= almost 30 patch) 2) OF graph new feature (= this patch-set) (= almost 10 patch) 3) OF graph base generic sound driver (= almost 30 patch) - ... I'm posting 1) part to ALSA SoC ML, 2) part to this ML. 1) will take more long term >> Mark ?? If you want to see this patch-set together with use case, then 3) is needed. If so, I will merge 2) and 3), and post these to this ML and ALSA SoC ML. Is it OK for you ? >> Rob, Mark I don't know how to handle it, but I can follow your opinion > In general, all these helpers look okay if you have a user for them. Thanks. As I explained above, it is based on 1) part, but it will takes more long term. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html