From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754595AbcGEGYX (ORCPT ); Tue, 5 Jul 2016 02:24:23 -0400 Received: from relmlor1.renesas.com ([210.160.252.171]:27469 "EHLO relmlie4.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753394AbcGEGYI (ORCPT ); Tue, 5 Jul 2016 02:24:08 -0400 X-IronPort-AV: E=Sophos;i="5.22,559,1449500400"; d="scan'";a="214414727" Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Message-ID: <87mvlw4nje.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: <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> References: <87k2h8g7rr.wl%kuninori.morimoto.gx@renesas.com> <20160704160736.GA25708@rob-hp-laptop> <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> 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: Tue, 5 Jul 2016 06:24:01 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: KAWPR01CA0025.jpnprd01.prod.outlook.com (10.161.24.35) To SG2PR06MB1709.apcprd06.prod.outlook.com (10.167.82.139) X-MS-Office365-Filtering-Correlation-Id: 10fc1f9c-6a66-4d35-645e-08d3a49cf557 X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1709;2:F/Nrm7Gx7bHCduBT6k0ywp3iXKOz2T9HpQYWjdhXcOq6fuizLMWSkW2+iYgpjbOQfZX+cr3qf/b5HZ6MKYoYv6YKufIcRBIznWjYRSX5ugRI52v9HzqjAeS42Hg6oCUV+Tt5FgaAtAJit6t3UxKaBJ/KkvUWV+jCsg8w01JzlYuGjgFk7v5d9bvZHqK7SKRn;3:jbaOGK1yTDWIUhrsEviakiRtqizzIxxsc9ZPNAHclcIjxFSCi6jgyu9M+9fW0/03Mi26t2SvDIBXgUJFwm2c8BBs1080QzJqTOvrmtVy2L2o97rmrG5qnnepKZZ4Fzta;25:6gwZrt2XQMTHlODGbaCnuM1eWaZEQmylgMFi1R8BI/UKE/jZ+oW3xfaN+1kiFJLyuodiFBGOK7/mYZHrtUkwfPFCOyXDpgfX2TOpcVJEzxFzudt/tDZbyJx+mJy9j0v9V7+g2IlywDUe+tvE6C/nV5CkH6oa4ZF4cPq6HhlDaV3Xdu957bMqKRQJntFsfQzOKU3xlms9RRnNDUMvIAlYFPX1McRzR0jlqH1bJ5iqO63dO8O7gfXdNPF3nMjotT7y6jZeXasjft/AjF3AGyoLXpBgTccVKoZlz908HXq0TH48m70xT+3p2J9KhhKHpS5LT+QGgqXiV+04pIpoRFJJbiMv5c2KLvz0S+EWTV8MpneNd9wPQL1VF9XYNslk6ZuofP0kVY4g8tVsw6EK8irEkCTEpXHURr9zybf5hvzKxGI=;31:6Omh4kRDu58V96Bo7ubvnOancd1Tgqww8lL5LcpmPcZJbWKrlE1n1ooK/lUt0R85dQUymEOaaWCD7th+fOqk2K21R0XXKHxxybnOw659xITleIppVJBcgpI1N3j5i41+4priuqaIzTjJ8HpXRZGfEbI+1urKn92A369h/QEDMb8b/s13OuVh0zr5iJcNvk6eYO5jGMZ89o+cYjb9f6Jc Gw== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SG2PR06MB1709; X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1709;20:V7GKvHwMgH/bIFae2993fgfwIkFv/tQMWEcJycqjkLqL7fVE6UOCg3bOkc38nm4Rxqkx5xuEvcB7XxYBpfoBqEaSXHbCvqdsLRutLKMQfJ0lJ3LXQh/HNIfE1dM4i2gUoz3c/HQIcN0qXwe0sExBdbhQzQYua/6jQoDIgA9NwBmEvptgh/4rbcnoKFY4nlcpq5Nq0hVH4xqzymC4A05ZPGYktEoDAh0uejeJhhlVU0aDiJ8K9i2KzaLMIWcObsL+XvuX0Ajoo4V7bGn0Wq/1EtffOVMjUcVh5BPE+DuvvWQ6Iw5Z6H4tOC3mxC6SdF6F/VzBS1CYyVWqA1fxCjRQ1FxaP4aPl412yep4agmoin9X6DZoPaF/sx/Lzw+RGxBiCb5NqrENS/yIZ5nCZFtPTDZje1wAJ8Da593llqk3pqLstEYQEareS56qYOFd6cYU/Hdjz4aun1ZRGJSLdE5+EuDeq9yI2jOCV+UtUcgRS/QvEDmYKzT53ShhYXgGF0Xq;4:nGlNZkhMjDHmf67zpOOXUy34/0ctC5aZllahDVqunzJsKH+LrfACqsNrO/9DFqx6i3uxmGvQqLSJJ/0e38Z6cSLlPkNrxIP20tO9hrE3CXHpKCzxab6Wf8zLvmewT0HOHnZXiX3qMm2bTeRt1R8aeIyZknCcPkOgXxvJUA8k+Ah7dpa21qNukgkZNT5Grf5Bz64fvYckNxdU2DoeBKp5qeraCwrtGQipsisaC3mREyxEef/THugtWIPJKR6w4ZzUQVtNpdNO+NhmNLuOkFecaIOF7BmmiN4dNsaHH0I8JbLoFbCKYiP7fUDK30NRtWM7atx1uJqBs0/LjUQJGZPXtAKNuZd6U11TW4CMUhuTgQNzIM6WnE/AYM0TwXDic6OXsNWdzZG6p/SZ02uJP/3Wfg== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6055026);SRVR:SG2PR06MB1709;BCL:0;PCL:0;RULEID:;SRVR:SG2PR06MB1709; X-Forefront-PRVS: 0994F5E0C5 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(979002)(6009001)(7916002)(199003)(189002)(2950100001)(54356999)(69596002)(50986999)(76176999)(92566002)(86362001)(46406003)(33646002)(66066001)(47776003)(77096005)(53416004)(6116002)(3846002)(97736004)(23726003)(5001770100001)(586003)(42186005)(305945005)(4326007)(189998001)(2906002)(68736007)(50466002)(8666005)(7846002)(7736002)(81166006)(101416001)(105586002)(8676002)(106356001)(4001350100001)(83506001)(81156014)(36756003)(7059030)(16060500001)(969003)(989001)(999001)(1009001)(1019001);DIR:OUT;SFP:1102;SCL:1;SRVR:SG2PR06MB1709;H:morimoto-PC.renesas.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;SG2PR06MB1709;23:tGSHQc3upPc+KAxvVsZj02O4KNIFchTl7lNKARXXl?= =?us-ascii?Q?QC1uxID0QEMXd9H0qnw9So92V2VY4bYJuzk3CK3pQUuvtEvZKUSV9nuMQDJj?= =?us-ascii?Q?REsvyJDkVzcr72YG+CMyRM2MRYRP9SmP7PSldHGBsLSmkopL4vqewp58hBh6?= =?us-ascii?Q?oRa219+fWeq18J/P1g/6FoUf/OZLOqtRUchJ/156Z8MnFVQXJ5VEAogXAJbU?= =?us-ascii?Q?21OjZrtNMm+3X/kjDj69I/rxiRdu9h2QyZJWRC3tu7FZNRmCJ3vIn9XdUMPM?= =?us-ascii?Q?f1oBqlgcswASBIDGejiVbiXskkNA9axipbWdjhecLE+T7Krf+bhhNl/Wz3Vz?= =?us-ascii?Q?NdAMRNlJ/dSyWy1Uz9Kbln+3vNf2HknNa/nOZ0q18vX7EFeoku3WkdYk93Q/?= =?us-ascii?Q?3N/Myd0HjvQ2BPpUrvaJHTIlQoJIs9a74zSLo7ALacsCrxPy8IpAw0t9lm77?= =?us-ascii?Q?fMDiqXLPbisCIrBZeAdargaXe4h22i8Qhp5WuF7tgV7U7ApID4GtasslVMb/?= =?us-ascii?Q?WWSxvf+93c8sxMhfHtMCLnbrz6owP/1vP4Y1YtFXO6EZMBrUFQxqAyafiO3r?= =?us-ascii?Q?dQEsktg/zIshmj2HogznemM0ibOBKP9IBSAvM6ARwT1DRANYzcbuYutgdneV?= =?us-ascii?Q?b+JaIl3F9X7vukOCcM0h0gMpC5hqwUkkqzqirgtRQxfLW9e59aTCdzTv39mI?= =?us-ascii?Q?ZKXIeIn+HKE1OzCIo+fVAbm/cXycctrsl1hzNjd2P7dOLE89HlOsfDHLjnhB?= =?us-ascii?Q?9Fxmwj3J5GmdUMV59TxvUBCRUWweWHAF5cO/apnitW2R47vKpQdsu5gaB4bM?= =?us-ascii?Q?AnWI2U2b6EaXZWYy6vsIP/QMhioRaVULVT+mrr7kJ6xZ0zHYt3YpwINanodH?= =?us-ascii?Q?rISNLa42rygBAmL7Jk3Z87iucRa2/rNtMRGaRQKrMhOK0Up+SZ3T91l0FEby?= =?us-ascii?Q?FaFpmBAbYLVsTYVmY+uxd7svie/dMsBRHY6A0qZEVOuVlE7Ssc/txNdnMhE3?= =?us-ascii?Q?nYla8668F+qTjCkNfjB5wBFRU1WCrt9h4bWwMhv4ESO+g+kgdPkGMw53OrGY?= =?us-ascii?Q?oTdKt0N10HfJI55wsYBCXWeHnvK3k53XBIIrfExr5oSIZRO56FHq/PMMA9P3?= =?us-ascii?Q?YdCCUtL7xAYkUWuSJ00Ip7AkS/5sD/hBs9NB3KuThvcaemV78cG/FA9YAyk/?= =?us-ascii?Q?ALgTarx5MuFBeye05RbUy8etebvE7+yZIRy/8/3efrxREDURQ5ihrc80g=3D?= =?us-ascii?Q?=3D?= X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1709;6:XZ9yyKFxPJsLIF2Dp9PSZfItL43NAHy5PQfcZNqMNAcXfjbOXrmRk+uDeZCm5bDh2f0TyNcw+5A0FGKT03MMdVyuPHqu4jjMROmgPGeqiDvp3YKrxemL+U8s/pZEFoCWzkjVnNwi4YCaa9LPk+KMjy/prKmvE52VOWRalJ9rgcz7LIeoEjg+vCyZ1BCJipFxY/8dbYaUwRRydR3oVLYkkow0E/PJDMHMX3RcMe3bEsmzdQz2AS3JoDr/3MQp/FWmNhqKipDLuFlWaLLJheG68mMiNuxtZzrzgILBjqCE1DsUjH76XldIIKbVSR2ysfuyHmkREs+A5snDdADB47dRPg==;5:3Id+usjkUtmHpFMLbZ6Q5q2TBjjSCuTFHXFSM1ZLppK9NiH3FK9KPnGhlhRjsLdRtPazfv8OCUj+F7RrpJcArVM5xhP/fOmCPwsKhcFLqWqK1wSPeUJ2pBwNkcvMgC7Q6dDgf/r+sL8+X3ZukZE3kQ==;24:3l3eLdT3Di1MeSNF3xeyoLDzePAfQtxnXdq/cT+e9B0GDJovuxM3/8TFWUYKOAK7sTIuzUtTuw6xpFQBH/s2XNC0irhU/q7YCmHddG1Kmdo=;7:BfRFcMCrUYVwmP2ajr6ldNV22bcPNTOiWLwBg0orLpYGRGYSUlZ8v0DANui8mMuEfcxkQGELn7RrwJ7tDQBbQ3v/IpWtXgNSXjOGed4tsiUckph2WCC8Z3Zyy+4eoSSThqgQHMsGiM/pddorMR+R1cHwU3oXmmEBJpvyq9ruIbkxEz1yDp9Gpn/w4OnYlRWg+4CXTvI+PO2TujW4Lfu6BRZbBBVXRG3khDWflwZymNpdevGAbNMfI1GPQ+Ihl3I3 SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;SG2PR06MB1709;20:9v7z3T2QbBDmf/RfOURfem54r/Gdf2XMfP6hIdP+2AHI6ByKxzKwXy3E0ueNGj70GA4fSYTWrzByBQLpDx88FLG31M0XP+wzpzLUQ20gK9LEYIp1OR09A4t/9rBghWPpH4hbKdcS9M7lCa5pQaILrbQQ8QyvmxwpvIROOZxThlU= X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jul 2016 06:24:01.8443 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SG2PR06MB1709 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, Mark, again > 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 I think we can use 2 patterns ? pattern1) post "OF graph new feature" patch and "ALSA SoC use it" patch 1. OF graph new feature 1 patch 2. ALSA SoC use feature 1 patch 3. OF graph new feature 2 patch 4. ALSA SoC use feature 2 patch 5. OF graph new feature 3 patch 6. ALSA SoC use feature 3 patch ... It is easy to review, but difficult on maintainer ? pattern2) post "OF graph new feature" patch-set and "ALSA SoC use it" patch-set 1. OF graph new feature 1 patch 2. OF graph new feature 2 patch 3. OF graph new feature 3 patch 4. ALSA SoC use feature 1 patch 5. ALSA SoC use feature 2 patch 6. ALSA SoC use feature 3 patch ... It is easy on maintainer, difficult to review ? Actually, this patch-set is 1, 2, 3 part of pattern2). I can post all patch-set in next time if we use pattern2) case. Or I can post as pattern1), and maintainer will pickup as pattern2) order ? > > 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: Tue, 5 Jul 2016 06:24:01 +0000 Message-ID: <87mvlw4nje.wl%kuninori.morimoto.gx@renesas.com> References: <87k2h8g7rr.wl%kuninori.morimoto.gx@renesas.com> <20160704160736.GA25708@rob-hp-laptop> <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> Mime-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <87wpl13rlf.wl%kuninori.morimoto.gx@renesas.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Rob Herring , Mark Brown Cc: Linux-DT , Linux-ALSA , Linux-Kernel , Laurent , Grant Likely , Frank Rowand , Guennadi List-Id: devicetree@vger.kernel.org Hi Rob, Mark, again > 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 I think we can use 2 patterns ? pattern1) post "OF graph new feature" patch and "ALSA SoC use it" patch 1. OF graph new feature 1 patch 2. ALSA SoC use feature 1 patch 3. OF graph new feature 2 patch 4. ALSA SoC use feature 2 patch 5. OF graph new feature 3 patch 6. ALSA SoC use feature 3 patch ... It is easy to review, but difficult on maintainer ? pattern2) post "OF graph new feature" patch-set and "ALSA SoC use it" patch-set 1. OF graph new feature 1 patch 2. OF graph new feature 2 patch 3. OF graph new feature 3 patch 4. ALSA SoC use feature 1 patch 5. ALSA SoC use feature 2 patch 6. ALSA SoC use feature 3 patch ... It is easy on maintainer, difficult to review ? Actually, this patch-set is 1, 2, 3 part of pattern2). I can post all patch-set in next time if we use pattern2) case. Or I can post as pattern1), and maintainer will pickup as pattern2) order ? > > 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. >