[{"id":1776860,"web_url":"http://patchwork.ozlabs.org/comment/1776860/","msgid":"","list_archive_url":null,"date":"2017-09-28T08:33:18","subject":"Re: [patch v9 0/4] JTAG driver introduction","submitter":{"id":703,"url":"http://patchwork.ozlabs.org/api/people/703/","name":"Geert Uytterhoeven","email":"geert@linux-m68k.org"},"content":"Hi Oleksandr,\n\n[My attention was drawn by https://lwn.net/Articles/734440/]\n[CC linux-spi, which was never included, while linux-serial was]\n\nOn Thu, Sep 21, 2017 at 11:25 AM, Oleksandr Shamray\n wrote:\n> When a need raise up to use JTAG interface for system's devices\n> programming or CPU debugging, usually the user layer\n> application implements jtag protocol by bit-bang or using a\n> proprietary connection to vendor hardware.\n> This method can be slow and not generic.\n>\n> We propose to implement general JTAG interface and infrastructure\n> to communicate with user layer application. In such way, we can\n> have the standard JTAG interface core part and separation from\n> specific HW implementation.\n> This allow new capability to debug the CPU or program system's\n> device via BMC without additional devices nor cost.\n>\n> This patch purpose is to add JTAG master core infrastructure by\n> defining new JTAG class and provide generic JTAG interface\n> to allow hardware specific drivers to connect this interface.\n> This will enable all JTAG drivers to use the common interface\n> part and will have separate for hardware implementation.\n>\n> The JTAG (Joint Test Action Group) core driver provides minimal generic\n> JTAG interface, which can be used by hardware specific JTAG master\n> controllers. By providing common interface for the JTAG controllers,\n> user space device programing is hardware independent.\n>\n> Modern SoC which in use for embedded system' equipped with\n> internal JTAG master interface.\n> This interface is used for programming and debugging system's\n> hardware components, like CPLD, FPGA, CPU, voltage and\n> industrial controllers.\n> Firmware for such devices can be upgraded through JTAG interface during\n> Runtime. The JTAG standard support for multiple devices programming,\n> is in case their lines are daisy-chained together.\n>\n> For example, systems which equipped with host CPU, BMC SoC or/and\n> number of programmable devices are capable to connect a pin and\n> select system components dynamically for programming and debugging,\n> This is using by the BMC which is equipped with internal SoC master\n> controller.\n> For example:\n>\n> BMC JTAG master --> pin selected to CPLDs chain for programming (filed\n> upgrade, production)\n> BMC JTAG master --> pin selected to voltage monitors for programming\n> (field upgrade, production)\n> BMC JTAG master --> pin selected to host CPU (on-site debugging\n> and developers debugging)\n>\n> For example, we can have application in user space which using calls\n> to JTAG driver executes CPLD programming directly from SVF file\n>\n> The JTAG standard (IEEE 1149.1) defines the next connector pins:\n> - TDI (Test Data In);\n> - TDO (Test Data Out);\n> - TCK (Test Clock);\n> - TMS (Test Mode Select);\n> - TRST (Test Reset) (Optional);\n>\n> The SoC equipped with JTAG master controller, performs\n> device programming on command or vector level. For example\n> a file in a standard SVF (Serial Vector Format) that contains\n> boundary scan vectors, can be used by sending each vector\n> to the JTAG interface and the JTAG controller will execute\n> the programming.\n>\n> Initial version provides the system calls set for:\n> - SIR (Scan Instruction Register, IEEE 1149.1 Data Register scan);\n> - SDR (Scan Data Register, IEEE 1149.1 Instruction Register scan);\n> - RUNTEST (Forces the IEEE 1149.1 bus to a run state for a specified\n> number of clocks.\n>\n> SoC which are not equipped with JTAG master interface, can be built\n> on top of JTAG core driver infrastructure, by applying bit-banging of\n> TDI, TDO, TCK and TMS pins within the hardware specific driver.\n\nOr by using an SPI master?\n\nGr{oetje,eeting}s,\n\n Geert\n\n--\nGeert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org\n\nIn personal conversations with technical people, I call myself a hacker. But\nwhen I'm talking to journalists I just say \"programmer\" or something like that.\n -- Linus Torvalds","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"V/hr9NjP\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"X8N9zfEz\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2nz049WTz9rxl\n\tfor ;\n\tThu, 28 Sep 2017 18:33:48 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUGP-0007io-PY; Thu, 28 Sep 2017 08:33:45 +0000","from mail-wm0-x241.google.com ([2a00:1450:400c:c09::241])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUGK-0007Pb-TN for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 08:33:43 +0000","by mail-wm0-x241.google.com with SMTP id r74so619163wme.5\n\tfor ;\n\tThu, 28 Sep 2017 01:33:20 -0700 (PDT)","by 10.223.129.226 with HTTP; Thu, 28 Sep 2017 01:33:18 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=udDfQbrFblFW5lSQD8meGcSbgQcDdCQ2FsPlEb2R4qM=;\n\tb=V/hr9NjPDD4ppt\n\tSU/ACjhADphQxtNuVao2KrWRTQwKfAjxM809cEV7WUePSM+V0ez4qPh6gkjigaT0wRS5Xz7QXPc77\n\tR2N2itFP6zVka66zipjnzNPS5lKmIfVTRGun5+iQRF6mUymydL2MWqtI4jyIN9KqNBf1Vt1cvu/fA\n\toyDL8X3GYm7Hpb02QMSXQFGyk8I85HOmij7TE+Jr0WVQ4oYPAS6Hgv6rsMwqzVDUoncxiFBejhBrw\n\t2O8ctEaoyNflva3XHIjxjO4kzvAuSw6iZDgDfVkQepCoWFzWmJyxMC2ssMU55s9zfvJt+sUGckVF+\n\tFz+hcSIVJhpbwH/fkNZw==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=FJL9dm/7ZlsaQE893hGebXwGqiac4C2PcNh9ircWQ8Q=;\n\tb=X8N9zfEzucoeiJT9aT6Jdm1AKp/2v/1f8MMmQSzWCdtRSr87LGrfF4u6yfFUZBDPbh\n\t2+fgmxW1oNUBHEmylXINrnbSYMeo5pD1Bs1ZKNaIilJKlTC81lr/Hlum4Eel5TYzp8uE\n\tLXmVOQkWFGEmDm6RkMr7jAfWulivayrONt/ARnk16v5dT189D8v9kL7LTJKXltgV7WUs\n\tdD9vjpQpFlrVaZhWeof+7zlqhXrvWZcYgiYlNUwiqJPKy/HmCgGPi9OYkphm+tvWgjx/\n\tfM4b91e0ZUN2oHM2adV7vQewJ5FTJzkku4gTENnhXRPE8mb78/4Ciru/b8isan1frQpr\n\tcTAQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=FJL9dm/7ZlsaQE893hGebXwGqiac4C2PcNh9ircWQ8Q=;\n\tb=E1Qm8Pas8n0NkNQ15DSjH1rUvy1EJ/7CKBIzFgajSnXIBujP51NNra1SVisHPyr6EG\n\t+1il1S610s+2/lGAkNXazPbpNnrZarr31Xq2gmH7lTs29LeefNth6SbRlqlXDGYdG/0z\n\tSkHKRbQyEIIB/n3bObKr40CFLuD9LuzH3gQOQYan2fjZbSB4F3vylY18rUlPbHimF515\n\teJd8veI35Vo67k0KQVCOA4wJKqCSockNniJ2oKqtoiHZ9aTsSe351qLWnD+N3zuytJPq\n\tnpFHMOEg2FAHB+U6eqxHJS3Q9xaHHUx02/TXpF7rF3vbsUi+MI9Fi8L0bQonFBGQwO0Z\n\to2Lg==","X-Gm-Message-State":"AHPjjUhhAVj9bxV4vOxbZYPfzeG2nErTAsR5U+a1nDx3zgaaXcj8Uj8D\n\tqyq7AOvovhDQGDCAAdFzn/0Tl/c015rXoAksrl4=","X-Google-Smtp-Source":"AOwi7QDbfPUN4h0CanHxpn5JAospxR2VGMF9nOelwXbcDVr20BjteBp50x7wGAKon9NoeJFWWRYEWNvLVNyxXhC0Uu8=","X-Received":"by 10.28.36.212 with SMTP id k203mr260094wmk.127.1506587598760; \n\tThu, 28 Sep 2017 01:33:18 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>","From":"Geert Uytterhoeven ","Date":"Thu, 28 Sep 2017 10:33:18 +0200","X-Google-Sender-Auth":"ErALBLwzm6Pxb4jxp6dk4SBrX4o","Message-ID":"","Subject":"Re: [patch v9 0/4] JTAG driver introduction","To":"Oleksandr Shamray ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_013341_134389_8E439300 ","X-CRM114-Status":"GOOD ( 21.98 )","X-Spam-Score":"-1.7 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.7 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c09:0:0:0:241 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (geert.uytterhoeven[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level@mellanox.com,\n\t\"devicetree@vger.kernel.org\" ,\n\tJiri Pirko , Arnd Bergmann ,\n\tVadim Pasternak ,\n\tGreg KH , openbmc@lists.ozlabs.org,\n\t\"linux-kernel@vger.kernel.org\" ,\n\topenocd-devel-owner@lists.sourceforge.net,\n\tlinux-spi , mec@shout.net,\n\tRob Herring , Mark Brown ,\n\tJoel Stanley ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"linux-api@vger.kernel.org\" ,\n\tTobias Klauser ,\n\tMauro Carvalho Chehab ,\n\t\"David S. Miller\" ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776884,"web_url":"http://patchwork.ozlabs.org/comment/1776884/","msgid":"","list_archive_url":null,"date":"2017-09-28T08:53:12","subject":"RE: [patch v9 0/4] JTAG driver introduction","submitter":{"id":72099,"url":"http://patchwork.ozlabs.org/api/people/72099/","name":"Oleksandr Shamray","email":"oleksandrs@mellanox.com"},"content":"> -----Original Message-----\n> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]\n> On Behalf Of Geert Uytterhoeven\n> Sent: Thursday, September 28, 2017 11:33 AM\n> To: Oleksandr Shamray \n> Cc: Greg KH ; Arnd Bergmann\n> ; linux-kernel@vger.kernel.org; linux-arm-\n> kernel@lists.infradead.org; devicetree@vger.kernel.org;\n> openbmc@lists.ozlabs.org; Joel Stanley ; Jiri Pirko\n> ; Tobias Klauser ; linux-\n> serial@vger.kernel.org; mec@shout.net; Vadim Pasternak\n> ; system-sw-low-level level@mellanox.com>; Rob Herring ; openocd-devel-\n> owner@lists.sourceforge.net; linux-api@vger.kernel.org; David S. Miller\n> ; Mauro Carvalho Chehab ;\n> linux-spi ; Mark Brown \n> Subject: Re: [patch v9 0/4] JTAG driver introduction\n> \n> Hi Oleksandr,\n> \n> [My attention was drawn by\n> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flwn.net\n> %2FArticles%2F734440%2F&data=02%7C01%7Coleksandrs%40mellanox.com%7\n> C97b8ba88686a42daaace08d5064b92eb%7Ca652971c7d2e4d9ba6a4d149256f\n> 461b%7C0%7C0%7C636421844026854216&sdata=TeHD4a3%2FBN6a5XG3Jizf5\n> pmsyJHJjzkEzkpnqsXC6S0%3D&reserved=0]\n> [CC linux-spi, which was never included, while linux-serial was]\n> \n> On Thu, Sep 21, 2017 at 11:25 AM, Oleksandr Shamray\n> wrote:\n> > When a need raise up to use JTAG interface for system's devices\n> > programming or CPU debugging, usually the user layer application\n> > implements jtag protocol by bit-bang or using a proprietary connection\n> > to vendor hardware.\n> > This method can be slow and not generic.\n> >\n\n\n[..]\n\n> >\n> > Initial version provides the system calls set for:\n> > - SIR (Scan Instruction Register, IEEE 1149.1 Data Register scan);\n> > - SDR (Scan Data Register, IEEE 1149.1 Instruction Register scan);\n> > - RUNTEST (Forces the IEEE 1149.1 bus to a run state for a specified\n> > number of clocks.\n> >\n> > SoC which are not equipped with JTAG master interface, can be built on\n> > top of JTAG core driver infrastructure, by applying bit-banging of\n> > TDI, TDO, TCK and TMS pins within the hardware specific driver.\n> \n> Or by using an SPI master?\n> \n\nI think it depends on how flexible the SPI interface is. If you can set it to transfer \nfrom 1 to n bits at a time, and you control the TMS line in software, you should \nbe able to use it. If the SPI interface can only transfer a multiple of 8 bits \nat a time, then in general it would not be suitable for JTAG.\n\n> Gr{oetje,eeting}s,\n> \n> Geert\n> \n> --\n> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-\n> m68k.org\n> \n> In personal conversations with technical people, I call myself a hacker. But\n> when I'm talking to journalists I just say \"programmer\" or something like that.\n> -- Linus Torvalds","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"WXRnqbBQ\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"YhB68n3o\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2pQH5fctz9t2Q\n\tfor ;\n\tThu, 28 Sep 2017 18:53:59 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUZw-0006xB-L8; Thu, 28 Sep 2017 08:53:56 +0000","from mail-eopbgr10077.outbound.protection.outlook.com\n\t([40.107.1.77] helo=EUR02-HE1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUZs-0006ny-Cd for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 08:53:55 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com (10.165.82.13) by\n\tHE1PR05MB3339.eurprd05.prod.outlook.com (10.170.243.145) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.77.7; Thu, 28 Sep 2017 08:53:12 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6]) by\n\tAM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6%18]) with mapi id 15.20.0077.016;\n\tThu, 28 Sep 2017 08:53:12 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References:\n\tMessage-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=AHfl0oJYW0FiRGgimsdjFqLnCc+owRIUSMxyG1zI+GY=;\n\tb=WXRnqbBQk+Esi0\n\to0Qogmu3aUHF5EX6KLbnImEQeizFyq2Z8NW9MMqeiT/yyCUDrh4+O3Ns2hotAOFjgsDbxa9xd+yzI\n\tCte89xL3eJNue5vAvmo8QmQGbUcJcMjfHKvJxMeazFrLGpAbPp7JJxP3VilFSV3hWwYdSq7tZ0g4d\n\tQtW7LvsN2EIp3HaJrToSHHHKu6aftCYis34pHXjsXA9hDaL3jCLGDSyedby6QdnCvdtVHe9ox5XJL\n\t1dpuKG57rme9lggDJ+TcyVcV1CTVQT7EJOR/gZMJydPDBFvrSBbsmfYicvCi0nDIz54bBCY08mmaS\n\tQqM1uP07kr66+WxOwKqg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n\ts=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=ZSzKvaLqOB+8JYL84xiGjioh1+Ge986pOVACqDcoF7A=;\n\tb=YhB68n3oRQmGg9w303lbAsZU7We49vO7GEDff5oF+YcfIRCjA8P7wZNfmSoffUVNHVfPrF6PoClrOTIfrv5ilez84C/1+QRGcWk1UCFiNgq+tJqWObEG/HLeXSBv8bgYpoY3UVk+4Xg08JW+NCkrjGbcfi9LmzY82oeW197SEXk="],"From":"Oleksandr Shamray ","To":"Geert Uytterhoeven ","Subject":"RE: [patch v9 0/4] JTAG driver introduction","Thread-Topic":"[patch v9 0/4] JTAG driver introduction","Thread-Index":"AQHTMru4M5azt7XbqEuLkdSxxS/sN6LKA2cAgAAE/MA=","Date":"Thu, 28 Sep 2017 08:53:12 +0000","Message-ID":"","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t","In-Reply-To":"","Accept-Language":"en-US, uk-UA","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"WXRnqbBQ\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"YhB68n3o\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"x-originating-ip":"[80.90.224.13]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; HE1PR05MB3339;\n\t6:q+k/swSyvnXMeIADVfddUZ4/QfGRNAJI6qPB0CJ0I1OjU6p82DZm7/vOSHUzBJsUsXxtcNCmKWNUigc977ldy4l1rC0Hou/rok49BXphXP0LbtcYZNpLaNoLxT4+NaHdSCDZcMM1B2BLbRYQQigSepww3O2kCSSgX5HgkTXEu5kEjK+R4I0J4j1VEHCdp9cZjry+fyBkUV7AZZUdDVLgrdqkb4ALvLybuR5ptQ69j33AuCBAsHHNxA9buf2/j2C+9G0VI7+UGUErYnMZ6JegrYyJTLjbMJ2iB1XrBQKaGZMJoCVesukXsDZ5e7QZrziifssW1RFz2Fg1iYbzWWcvyQ==;\n\t5:ClwoVo+Fk+rAlj2RvFhpZbaZsuDNiVbgp7fgs6CnzzkV/Oh//r39zG6QNMsNgfk0LoE+sqvySpMdYt9K2b0xwnL7qCPatl8LyRCNH07PZMAbddU/oBM8fqsYsYLC7hI2Uj/hRv/vIhrbiNytJX7DGw==;\n\t24:6LRVj4bt+W3GL5mjMPHnEpIIrfwBHwVxoEJ3nQ8cLhfXrjGlZEyo/XQJZvouI+tQgmIHbaKKrRb658/7w8wM0Di0qyT8MoqIvp2epQwSwjI=;\n\t7:HuJvL1dSsp07gbzfpDyapcaFWIOPplqDMGWThQvVCREjAd7HK/CWtGnnxH23kGAkDVlz3RUnf9N/BFf0pOZ7sm2YN6W+LtrLMHX6qxitS5BkKxw+pZy/WAEV64Il07X0GUSEG8vQhgH3B6BAoHkUiTDu9PC9wiy63L2IudefHLwnyRWLpAnWa5nMrTeED0pKO9Lm0PogzB+nR+aVHVufPTN0Hmai0ZhDuwCmEeRqKxg=","x-ms-office365-filtering-correlation-id":"dee184af-35ab-403f-a408-08d5064e59c3","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075);\n\tSRVR:HE1PR05MB3339; ","x-ms-traffictypediagnostic":"HE1PR05MB3339:","x-exchange-antispam-report-test":"UriScan:(143289334528602)(9452136761055)(189930954265078)(65623756079841)(258649278758335)(42262312472803)(45079756050767);","x-microsoft-antispam-prvs":"","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(93006095)(93001095)(10201501046)(3002001)(100000703101)(100105400095)(6055026)(6041248)(20161123558100)(20161123555025)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123562025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:HE1PR05MB3339; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:HE1PR05MB3339; ","x-forefront-prvs":"0444EB1997","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(376002)(346002)(39860400002)(13464003)(24454002)(199003)(189002)(377454003)(55016002)(101416001)(14454004)(7736002)(6246003)(105586002)(7696004)(305945005)(97736004)(106356001)(5250100002)(966005)(3660700001)(6306002)(86362001)(45080400002)(9686003)(3280700002)(2950100002)(6506006)(53546010)(99286003)(189998001)(5660300001)(478600001)(2900100001)(53936002)(8676002)(81156014)(316002)(25786009)(54356999)(50986999)(76176999)(8936002)(81166006)(7416002)(33656002)(68736007)(54906003)(66066001)(6916009)(74316002)(3846002)(6436002)(2906002)(229853002)(4326008)(102836003)(6116002);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR05MB3339;\n\tH:AM4PR0501MB2194.eurprd05.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; A:1; MX:1; LANG:en; ","received-spf":"None (protection.outlook.com: mellanox.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-OriginatorOrg":"Mellanox.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"28 Sep 2017 08:53:12.4284\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"a652971c-7d2e-4d9b-a6a4-d149256f461b","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"HE1PR05MB3339","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_015352_921880_350C5F05 ","X-CRM114-Status":"GOOD ( 19.91 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [40.107.1.77 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H4 RBL: Very Good reputation (+4)\n\t[40.107.1.77 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's\n\tdomain -0.0 RCVD_IN_MSPIKE_WL Mailspike good senders","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\tJiri Pirko , Arnd Bergmann ,\n\tVadim Pasternak ,\n\tGreg KH , \n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\tlinux-spi , \"mec@shout.net\" ,\n\tRob Herring , Mark Brown ,\n\tJoel Stanley ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"linux-api@vger.kernel.org\" ,\n\tTobias Klauser , Mauro\n\tCarvalho Chehab ,\n\t\"David S. Miller\" , \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776892,"web_url":"http://patchwork.ozlabs.org/comment/1776892/","msgid":"","list_archive_url":null,"date":"2017-09-28T09:02:10","subject":"Re: [patch v9 0/4] JTAG driver introduction","submitter":{"id":703,"url":"http://patchwork.ozlabs.org/api/people/703/","name":"Geert Uytterhoeven","email":"geert@linux-m68k.org"},"content":"Hi Oleksandr,\n\nOn Thu, Sep 28, 2017 at 10:53 AM, Oleksandr Shamray\n wrote:\n>> [My attention was drawn by\n>> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flwn.net\n>> %2FArticles%2F734440%2F&data=02%7C01%7Coleksandrs%40mellanox.com%7\n>> C97b8ba88686a42daaace08d5064b92eb%7Ca652971c7d2e4d9ba6a4d149256f\n>> 461b%7C0%7C0%7C636421844026854216&sdata=TeHD4a3%2FBN6a5XG3Jizf5\n>> pmsyJHJjzkEzkpnqsXC6S0%3D&reserved=0]\n>> [CC linux-spi, which was never included, while linux-serial was]\n>>\n>> On Thu, Sep 21, 2017 at 11:25 AM, Oleksandr Shamray\n>> wrote:\n>> > When a need raise up to use JTAG interface for system's devices\n>> > programming or CPU debugging, usually the user layer application\n>> > implements jtag protocol by bit-bang or using a proprietary connection\n>> > to vendor hardware.\n>> > This method can be slow and not generic.\n>> >\n>\n>\n> [..]\n>\n>> >\n>> > Initial version provides the system calls set for:\n>> > - SIR (Scan Instruction Register, IEEE 1149.1 Data Register scan);\n>> > - SDR (Scan Data Register, IEEE 1149.1 Instruction Register scan);\n>> > - RUNTEST (Forces the IEEE 1149.1 bus to a run state for a specified\n>> > number of clocks.\n>> >\n>> > SoC which are not equipped with JTAG master interface, can be built on\n>> > top of JTAG core driver infrastructure, by applying bit-banging of\n>> > TDI, TDO, TCK and TMS pins within the hardware specific driver.\n>>\n>> Or by using an SPI master?\n>>\n>\n> I think it depends on how flexible the SPI interface is. If you can set it to transfer\n> from 1 to n bits at a time, and you control the TMS line in software, you should\n> be able to use it. If the SPI interface can only transfer a multiple of 8 bits\n> at a time, then in general it would not be suitable for JTAG.\n\nSure, that depends on the actual SPI master interface.\nBut I guess you can write a generic JTAG-over-SPI driver, and use a tms-gpios\nproperty in DT to specify how to control TMS.\n\nGr{oetje,eeting}s,\n\n Geert\n\n--\nGeert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org\n\nIn personal conversations with technical people, I call myself a hacker. But\nwhen I'm talking to journalists I just say \"programmer\" or something like that.\n -- Linus Torvalds","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"LjEdA8yq\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"FV2jeYPi\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2pcM04Pmz9t3B\n\tfor ;\n\tThu, 28 Sep 2017 19:02:42 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUiL-0004uL-FN; Thu, 28 Sep 2017 09:02:37 +0000","from mail-wm0-x243.google.com ([2a00:1450:400c:c09::243])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxUiH-0004dT-Jl for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 09:02:35 +0000","by mail-wm0-x243.google.com with SMTP id u138so775164wmu.5\n\tfor ;\n\tThu, 28 Sep 2017 02:02:12 -0700 (PDT)","by 10.223.129.226 with HTTP; Thu, 28 Sep 2017 02:02:10 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=/WqHShDgqF+pL0siahIc1vWKC3cxcHd59Wi3QUGAUKU=;\n\tb=LjEdA8yq+C29Na\n\tiThXCMo18D8au80CGt7ofbf+wmgjJKSI1M9dOVaZvrUOIwrkLfvjCJhH5PloezV9prD8oJ0fC7CtK\n\txDr5p2bZ5+yr7xMMqD6SMSC7MFT2vTKvtS7pMBUMJFmI8zw+bq1vtjuNT4Q1Smeg+KbiuHFdkB95E\n\tU39c56MPJITuOSaC3wcwncIRVELKmmRD0GAFUy6j9fPy4CyNbKnFMwrkr9dpzxplSVV9rBjaaxrBv\n\tI2dog61prkWx4ArKdNzfr2hHtEN01ywlw93Z4v2h1HPhGBAchgrckc6wUcEpFJbAcwjcnjogStVuQ\n\tQu17fYZRvHX9Ws6cgnDA==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:sender:in-reply-to:references:from:date:message-id\n\t:subject:to:cc;\n\tbh=B8OHUWY4g6nbqeNhcZNkWYkQiSc3AHzgIHIMyPyG6to=;\n\tb=FV2jeYPizZAIGwZynPVykmTDWB7/TkZdTf5MK9bZBw6AckuhoWpitCCjVe4iNAmHbQ\n\t1IErqwjvtwnSmDJN9vNG0zP6haCbqj3XoTTllboN1MGCQAe62PnUmvZHAe7ILfqsDW3E\n\tlXUaHHoWUQS7VEea4o7AAhB4BB8er0nFi2eaL/+hfatZoSxk+ZKHcJ/nEDKEPpXmrnfZ\n\tMkfeLArpcD4tHa2A/zbgxBlVlq0QdsQ74NsYlVLdCTDGb2LjcM9oWtDvbCE9Jv69jI2+\n\tG3xH6LgANEV7j6Ig/zny6eGVj7gc+1kwqjDI8PjV3cMybw/0aUHYssACM6hpWcUO4KHw\n\tX5Ig=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:in-reply-to:references:from\n\t:date:message-id:subject:to:cc;\n\tbh=B8OHUWY4g6nbqeNhcZNkWYkQiSc3AHzgIHIMyPyG6to=;\n\tb=aRJqQP383CI6bqxjQM2oImgSot9gAZfiA+Q5wh7GBnX6glraQQMF2QRWHI2PdqoYQ2\n\thDE7RuHXxMtT6Pykntar2YvfJdCoLhL52Iy6XY3iKAyv1ushAefj1vsaA/3HxIPV061a\n\t8UUa1C0PQO487q+Nkv0sJVRDYXrpKSlFKKsUM4QQG9PMv/anzjXg7i43zH2tYIbIhx9P\n\tZNBxyJuuXuytF9bLuSDPOvuEhWef5QXF5tD1sVs6wd1mbk2+Hyym52JyBtEvrzLPQMSF\n\t44/sfI3WXbyPxntHlEAVptBQAzvY6qBYp+bkKALnWtBhTOGAwlpoJNBvEgOLde9N5Mup\n\tdYGg==","X-Gm-Message-State":"AHPjjUhHxo8YhRb3oo0VQZdL5VhyfHjlcFIIxu/qHmM3ZL7qWQTzTOlH\n\txMwgAYkg6zDIckQF+upWRpTt90c1TRo3xHcBSrI=","X-Google-Smtp-Source":"AOwi7QABqUCPPd4e9SYSYljs3C62h4SmYVMqoXc6U86sfd7wiHmnNS1NiPlDSCQPLDGDDfFE5cVKdzUO+gnuEm7FGwA=","X-Received":"by 10.28.71.25 with SMTP id u25mr373899wma.22.1506589331224; Thu,\n\t28 Sep 2017 02:02:11 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t\n\t","From":"Geert Uytterhoeven ","Date":"Thu, 28 Sep 2017 11:02:10 +0200","X-Google-Sender-Auth":"yqjJszmxOFubJPBq44r3ySY7E7Y","Message-ID":"","Subject":"Re: [patch v9 0/4] JTAG driver introduction","To":"Oleksandr Shamray ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_020233_927218_767620D7 ","X-CRM114-Status":"GOOD ( 20.30 )","X-Spam-Score":"-1.7 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.7 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a00:1450:400c:c09:0:0:0:243 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (geert.uytterhoeven[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\tJiri Pirko , Arnd Bergmann ,\n\tVadim Pasternak ,\n\tGreg KH , \n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\tlinux-spi , \"mec@shout.net\" ,\n\tRob Herring , Mark Brown ,\n\tJoel Stanley ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"linux-api@vger.kernel.org\" ,\n\tTobias Klauser ,\n\tMauro Carvalho Chehab ,\n\t\"David S. Miller\" ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776966,"web_url":"http://patchwork.ozlabs.org/comment/1776966/","msgid":"","list_archive_url":null,"date":"2017-09-28T11:11:42","subject":"RE: [patch v9 0/4] JTAG driver introduction","submitter":{"id":72099,"url":"http://patchwork.ozlabs.org/api/people/72099/","name":"Oleksandr Shamray","email":"oleksandrs@mellanox.com"},"content":"> -----Original Message-----\n> From: geert.uytterhoeven@gmail.com [mailto:geert.uytterhoeven@gmail.com]\n> On Behalf Of Geert Uytterhoeven\n> Sent: Thursday, September 28, 2017 12:02 PM\n> To: Oleksandr Shamray \n> Cc: Greg KH ; Arnd Bergmann\n> ; linux-kernel@vger.kernel.org; linux-arm-\n> kernel@lists.infradead.org; devicetree@vger.kernel.org;\n> openbmc@lists.ozlabs.org; Joel Stanley ; Jiri Pirko\n> ; Tobias Klauser ; linux-\n> serial@vger.kernel.org; mec@shout.net; Vadim Pasternak\n> ; system-sw-low-level level@mellanox.com>; Rob Herring ; openocd-devel-\n> owner@lists.sourceforge.net; linux-api@vger.kernel.org; David S. Miller\n> ; Mauro Carvalho Chehab ;\n> linux-spi ; Mark Brown \n> Subject: Re: [patch v9 0/4] JTAG driver introduction\n> \n> Hi Oleksandr,\n> \n> On Thu, Sep 28, 2017 at 10:53 AM, Oleksandr Shamray\n> wrote:\n> >> [My attention was drawn by\n> >> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flw\n> >> n.net\n> >>\n> %2FArticles%2F734440%2F&data=02%7C01%7Coleksandrs%40mellanox.com%7\n> >>\n> C97b8ba88686a42daaace08d5064b92eb%7Ca652971c7d2e4d9ba6a4d149256f\n> >>\n> 461b%7C0%7C0%7C636421844026854216&sdata=TeHD4a3%2FBN6a5XG3Jizf5\n> >> pmsyJHJjzkEzkpnqsXC6S0%3D&reserved=0]\n> >> [CC linux-spi, which was never included, while linux-serial was]\n> >>\n> >> On Thu, Sep 21, 2017 at 11:25 AM, Oleksandr Shamray\n> >> wrote:\n> >> > When a need raise up to use JTAG interface for system's devices\n> >> > programming or CPU debugging, usually the user layer application\n> >> > implements jtag protocol by bit-bang or using a proprietary\n> >> > connection to vendor hardware.\n> >> > This method can be slow and not generic.\n> >> >\n> >\n> >\n> > [..]\n> >\n> >> >\n> >> > Initial version provides the system calls set for:\n> >> > - SIR (Scan Instruction Register, IEEE 1149.1 Data Register scan);\n> >> > - SDR (Scan Data Register, IEEE 1149.1 Instruction Register scan);\n> >> > - RUNTEST (Forces the IEEE 1149.1 bus to a run state for a specified\n> >> > number of clocks.\n> >> >\n> >> > SoC which are not equipped with JTAG master interface, can be built\n> >> > on top of JTAG core driver infrastructure, by applying bit-banging\n> >> > of TDI, TDO, TCK and TMS pins within the hardware specific driver.\n> >>\n> >> Or by using an SPI master?\n> >>\n> >\n> > I think it depends on how flexible the SPI interface is. If you can\n> > set it to transfer from 1 to n bits at a time, and you control the TMS\n> > line in software, you should be able to use it. If the SPI interface\n> > can only transfer a multiple of 8 bits at a time, then in general it would not be\n> suitable for JTAG.\n> \n> Sure, that depends on the actual SPI master interface.\n> But I guess you can write a generic JTAG-over-SPI driver, and use a tms-gpios\n> property in DT to specify how to control TMS.\n> \n\nYes, this is a good idea for SoC which are not equipped with JTAG hardware.\n\n> Gr{oetje,eeting}s,\n> \n> Geert\n> \n> --\n> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-\n> m68k.org\n> \n> In personal conversations with technical people, I call myself a hacker. But\n> when I'm talking to journalists I just say \"programmer\" or something like that.\n> -- Linus Torvalds","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"Op0XhiPG\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"RkJXnhJu\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2sTx2Zwfz9tXj\n\tfor ;\n\tThu, 28 Sep 2017 21:12:21 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxWjq-0005Nw-5L; Thu, 28 Sep 2017 11:12:18 +0000","from mail-ve1eur01on0071.outbound.protection.outlook.com\n\t([104.47.1.71] helo=EUR01-VE1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxWjh-0004hE-1s for linux-arm-kernel@lists.infradead.org;\n\tThu, 28 Sep 2017 11:12:16 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com (10.165.82.13) by\n\tVI1PR05MB3343.eurprd05.prod.outlook.com (10.170.238.144) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.77.7; Thu, 28 Sep 2017 11:11:42 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6]) by\n\tAM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6%18]) with mapi id 15.20.0077.016;\n\tThu, 28 Sep 2017 11:11:42 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References:\n\tMessage-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=sIzRNL1U+HKiGll2w0RxqhirGAXom8O8fQIgJwcjkyQ=;\n\tb=Op0XhiPGW19kH5\n\tq4wEsM24/hiYquVvTPn8epe5PaUFmBGHh0WbhZmaHsy0Ic0Oh9UWCp/oQ387lkXddCd8I0VrRHR8O\n\tyJjPesZMd8pS5XIngXAxF6nUsrnxn4Ho1RehpvmVRUnFz8y8/BnRi8Nu4BamowUKIBt3RG1AOhjaF\n\tN7KCgAe9hQ6yFhy2aNTiRjHXc8c4SLfVEnjcxZcdmQG8KIS3KYybSFfTdtTgITBE7OCeyxZioAYjL\n\tr8F5yC7hMkZALp6X7Prm+APuAE0JfqBgnI1jgCwELWQKZCXvtXVg4OOIWVMhBzSyF8wpSXn5eFSLq\n\tZuW7HpneDdthOvSeFepg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n\ts=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=x0nmrw6G39mJ5AFWSCC0hEmoEzScp7LObaJLgCsRl8I=;\n\tb=RkJXnhJu0FiNvzi9Df5KoPKI7hOsrGjfCYjh+MczJG7UrnW6laB/3dLobLhMjNzMsq3F+eqf1OMBhYbfW8e1lT29WsRjaioKwTuy1VoetoGhENUmFVPuodM5OnQahzbSTMEqgAT3VALpYm3/vXTX+MMzlN3OQJNSrMB+lKW4WHI="],"From":"Oleksandr Shamray ","To":"Geert Uytterhoeven ","Subject":"RE: [patch v9 0/4] JTAG driver introduction","Thread-Topic":"[patch v9 0/4] JTAG driver introduction","Thread-Index":"AQHTMru4M5azt7XbqEuLkdSxxS/sN6LKA2cAgAAE/MCAAAMUAIAAIqNQ","Date":"Thu, 28 Sep 2017 11:11:42 +0000","Message-ID":"","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t\n\t\n\t","In-Reply-To":"","Accept-Language":"en-US, uk-UA","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"Op0XhiPG\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"RkJXnhJu\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"x-originating-ip":"[80.90.224.13]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; VI1PR05MB3343;\n\t6:no5mxZ/+rBgQR4VUQ9EIsDKHlGLze6N9DQ9neeKWeCAjkm1GQK44xulUH2kOr9pQG5ZAXWQTERvbGV0gQ9eUftrmm7PvEF3fwGw4bHqar0QvDWtG9Tst9SrLlM5D5WNaDweBzyr46n55RAoY/g0CBjwxWhqBRCgJ9HUiRJnr/T8iTsE9MZQExViCwPqQ4c4YviMJ/Coz5oz9Cl03OpwO/m/GIqmkqc55PIC+uDdLHw3OjA52p3sLWc3kph0Gn/d9TyqWcAJq1QeDSXTXXMBmqkpUXhi+L1gFviSo+TcqfFJF93NWpZRW/xUQPnVd0nFrCDShDpj4EDirliny18ZjfA==;\n\t5:bSeEanLLST/NB8olvOlpyz3Jz2kAklRfiU5bmNiIJi0aA9Ma9Auf/XqNqlY31gJMMkf96R5tJD1X4DMu4xhe3tGntJieYkHJbXit+q8xbaZa7NwPyf9J7fmhqOsiUsOsoCfjrw/pJc3uWvs2/ZRA/w==;\n\t24:S8LdIrYUnF/LZ4jm2BIzpHAe2Ev2Cs9AbEwTzy1BawX7MN8sGwaH6Po5NtVLRBOwqTSFvfHLl18ujGuojsF2JBwRZFSoXVN6pb/M13A4+vo=;\n\t7:ObfCEA2lL3SNjvnn1TEJtf0N6nE+DTSV+SH+IXGflSnkgVLG8xUBn1MFD2kF1CCpVNT7AH8uUXuXasBHBFC1YnS3M9uBnO+FikzzoVgSsvfElxjGoEf1Ib/Bw7XdRTMLUNUdXiWG4K2n/dd4CDCHpK96dvKOujROZq00omPZVWpMsJrKUpeg9Q1lOOd39N4lf2D3C8WxdlCseL79DdTpkJ4h91Q/QnPBuhaaM8ueBGk=","x-ms-office365-filtering-correlation-id":"79a4c72b-661c-4866-8de8-08d50661b2ed","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075);\n\tSRVR:VI1PR05MB3343; ","x-ms-traffictypediagnostic":"VI1PR05MB3343:","x-exchange-antispam-report-test":"UriScan:(143289334528602)(9452136761055)(189930954265078)(65623756079841)(258649278758335)(42262312472803)(45079756050767);","x-microsoft-antispam-prvs":"","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(93006095)(93001095)(10201501046)(100000703101)(100105400095)(3002001)(6055026)(6041248)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123562025)(20161123555025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:VI1PR05MB3343; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:VI1PR05MB3343; ","x-forefront-prvs":"0444EB1997","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(376002)(346002)(39860400002)(189002)(13464003)(377454003)(199003)(24454002)(6306002)(2906002)(3660700001)(86362001)(14454004)(93886005)(55016002)(8676002)(8936002)(33656002)(3280700002)(25786009)(99286003)(81166006)(966005)(68736007)(81156014)(478600001)(229853002)(189998001)(4326008)(45080400002)(54906003)(5250100002)(53936002)(316002)(6506006)(97736004)(6436002)(3846002)(7736002)(53546010)(50986999)(5660300001)(7696004)(2950100002)(106356001)(105586002)(2900100001)(6246003)(9686003)(74316002)(101416001)(76176999)(6916009)(7416002)(305945005)(66066001)(54356999)(6116002)(102836003);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR05MB3343;\n\tH:AM4PR0501MB2194.eurprd05.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: mellanox.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-OriginatorOrg":"Mellanox.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"28 Sep 2017 11:11:42.4581\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"a652971c-7d2e-4d9b-a6a4-d149256f461b","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"VI1PR05MB3343","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170928_041209_649989_029138AB ","X-CRM114-Status":"GOOD ( 23.91 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [104.47.1.71 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3)\n\t[104.47.1.71 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's\n\tdomain -0.0 RCVD_IN_MSPIKE_WL Mailspike good senders","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\tJiri Pirko , Arnd Bergmann ,\n\tVadim Pasternak ,\n\tGreg KH , \n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\tlinux-spi , \"mec@shout.net\" ,\n\tRob Herring , Mark Brown ,\n\tJoel Stanley ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"linux-api@vger.kernel.org\" ,\n\tTobias Klauser , Mauro\n\tCarvalho Chehab ,\n\t\"David S. Miller\" , \n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1791442,"web_url":"http://patchwork.ozlabs.org/comment/1791442/","msgid":"<20171020115512.GA2073@kroah.com>","list_archive_url":null,"date":"2017-10-20T11:55:12","subject":"Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver","submitter":{"id":11800,"url":"http://patchwork.ozlabs.org/api/people/11800/","name":"Greg Kroah-Hartman","email":"gregkh@linuxfoundation.org"},"content":"On Thu, Sep 21, 2017 at 12:25:29PM +0300, Oleksandr Shamray wrote:\n> +struct jtag {\n> +\tstruct device *dev;\n> +\tstruct cdev cdev;\n\nWhy are you using a cdev here and not just a normal misc device? I\nforgot if this is what you were doing before, sorry...\n\n> +\tint id;\n> +\tatomic_t open;\n\nWhy do you need this?\n\n> +\tconst struct jtag_ops *ops;\n> +\tunsigned long priv[0] __aligned(ARCH_DMA_MINALIGN);\n\nHuh? Why is this needed to be dma aligned? Why not just use the\nprivate pointer in struct device?\n\n> +};\n> +\n> +static dev_t jtag_devt;\n> +static DEFINE_IDA(jtag_ida);\n> +\n> +void *jtag_priv(struct jtag *jtag)\n> +{\n> +\treturn jtag->priv;\n> +}\n> +EXPORT_SYMBOL_GPL(jtag_priv);\n> +\n> +static u8 *jtag_copy_from_user(__u64 udata, unsigned long bit_size)\n> +{\n> +\tunsigned long size;\n> +\tvoid *kdata;\n> +\n> +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> +\tkdata = memdup_user(u64_to_user_ptr(udata), size);\n\nYou only use this once, why not just open-code it?\n\n> +\n> +\treturn kdata;\n> +}\n> +\n> +static unsigned long jtag_copy_to_user(__u64 udata, u8 *kdata,\n> +\t\t\t\t unsigned long bit_size)\n> +{\n> +\tunsigned long size;\n> +\n> +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> +\n> +\treturn copy_to_user(u64_to_user_ptr(udata), (void *)(kdata), size);\n\nSame here, making this a separate function seems odd.\n\n> +}\n> +\n> +static struct class jtag_class = {\n> +\t.name = \"jtag\",\n> +\t.owner = THIS_MODULE,\n> +};\n> +\n> +static int jtag_run_test_idle_op(struct jtag *jtag,\n> +\t\t\t\t struct jtag_run_test_idle *idle)\n> +{\n> +\tif (jtag->ops->idle)\n> +\t\treturn jtag->ops->idle(jtag, idle);\n> +\telse\n> +\t\treturn -EOPNOTSUPP;\n\nWhy is this a function? Why not just do this work in the ioctl?\n\n> +}\n> +\n> +static int jtag_xfer_op(struct jtag *jtag, struct jtag_xfer *xfer, u8 *data)\n> +{\n> +\tif (jtag->ops->xfer)\n> +\t\treturn jtag->ops->xfer(jtag, xfer, data);\n> +\telse\n> +\t\treturn -EOPNOTSUPP;\n\nSame here, doesn't need to be a function.\n\n> +}\n> +\n> +static long jtag_ioctl(struct file *file, unsigned int cmd, unsigned long arg)\n> +{\n> +\tstruct jtag *jtag = file->private_data;\n> +\tstruct jtag_run_test_idle idle;\n> +\tstruct jtag_xfer xfer;\n> +\tu8 *xfer_data;\n> +\tu32 value;\n> +\tint err;\n> +\n> +\tswitch (cmd) {\n> +\tcase JTAG_GIOCFREQ:\n> +\t\tif (jtag->ops->freq_get)\n> +\t\t\terr = jtag->ops->freq_get(jtag, &value);\n> +\t\telse\n> +\t\t\terr = -EOPNOTSUPP;\n> +\t\tif (err)\n> +\t\t\tbreak;\n> +\n> +\t\terr = put_user(value, (__u32 *)arg);\n> +\t\tbreak;\n\nAre you sure the return value of put_user is correct here? Shouldn't\nyou return -EFAULT if err is not 0?\n\n> +\n> +\tcase JTAG_SIOCFREQ:\n> +\t\terr = get_user(value, (__u32 *)arg);\n> +\n\nwhy a blank line?\n\n> +\t\tif (value == 0)\n> +\t\t\terr = -EINVAL;\n\nCheck err first.\n\n> +\t\tif (err)\n> +\t\t\tbreak;\n\nAnd set it properly, again err should be -EFAULT, right?\n\n> +\n> +\t\tif (jtag->ops->freq_set)\n> +\t\t\terr = jtag->ops->freq_set(jtag, value);\n> +\t\telse\n> +\t\t\terr = -EOPNOTSUPP;\n> +\t\tbreak;\n> +\n> +\tcase JTAG_IOCRUNTEST:\n> +\t\tif (copy_from_user(&idle, (void *)arg,\n> +\t\t\t\t sizeof(struct jtag_run_test_idle)))\n> +\t\t\treturn -ENOMEM;\n> +\t\terr = jtag_run_test_idle_op(jtag, &idle);\n\nWho validates the structure fields? Is that up to the individual jtag\ndriver? Why not do it in the core correctly so that it only has to be\ndone in one place and you do not have to audit every individual driver?\n\n> +\t\tbreak;\n> +\n> +\tcase JTAG_IOCXFER:\n> +\t\tif (copy_from_user(&xfer, (void *)arg,\n> +\t\t\t\t sizeof(struct jtag_xfer)))\n> +\t\t\treturn -EFAULT;\n> +\n> +\t\tif (xfer.length >= JTAG_MAX_XFER_DATA_LEN)\n> +\t\t\treturn -EFAULT;\n> +\n> +\t\txfer_data = jtag_copy_from_user(xfer.tdio, xfer.length);\n> +\t\tif (!xfer_data)\n> +\t\t\treturn -ENOMEM;\n\nAre you sure that's the correct error value?\n\n> +\n> +\t\terr = jtag_xfer_op(jtag, &xfer, xfer_data);\n> +\t\tif (jtag_copy_to_user(xfer.tdio, xfer_data, xfer.length)) {\n> +\t\t\tkfree(xfer_data);\n> +\t\t\treturn -EFAULT;\n> +\t\t}\n> +\n> +\t\tkfree(xfer_data);\n> +\t\tif (copy_to_user((void *)arg, &xfer, sizeof(struct jtag_xfer)))\n> +\t\t\treturn -EFAULT;\n> +\t\tbreak;\n> +\n> +\tcase JTAG_GIOCSTATUS:\n> +\t\tif (jtag->ops->status_get)\n> +\t\t\terr = jtag->ops->status_get(jtag, &value);\n> +\t\telse\n> +\t\t\terr = -EOPNOTSUPP;\n> +\t\tif (err)\n> +\t\t\tbreak;\n> +\n> +\t\terr = put_user(value, (__u32 *)arg);\n\nSame put_user err issue here.\n\n> +\t\tbreak;\n> +\n> +\tdefault:\n> +\t\treturn -EINVAL;\n> +\t}\n> +\treturn err;\n> +}\n> +\n> +#ifdef CONFIG_COMPAT\n> +static long jtag_ioctl_compat(struct file *file, unsigned int cmd,\n> +\t\t\t unsigned long arg)\n> +{\n> +\treturn jtag_ioctl(file, cmd, (unsigned long)compat_ptr(arg));\n> +}\n> +#endif\n\nWhy do you need a compat callback for a new ioctl? Create your\nstructures properly and this should not be needed, right?\n\n> +\n> +static int jtag_open(struct inode *inode, struct file *file)\n> +{\n> +\tstruct jtag *jtag = container_of(inode->i_cdev, struct jtag, cdev);\n> +\n> +\tif (atomic_read(&jtag->open)) {\n> +\t\tdev_info(NULL, \"jtag already opened\\n\");\n> +\t\treturn -EBUSY;\n\nWhy do you care if multiple opens can happen?\n\n> +\t}\n> +\n> +\tatomic_inc(&jtag->open);\n\nOh look, you just raced with two different people opening at the same\ntime :(\n\nAn atomic value is never a replacement for a lock, sorry.\n\n> +\tfile->private_data = jtag;\n> +\treturn 0;\n> +}\n> +\n> +static int jtag_release(struct inode *inode, struct file *file)\n> +{\n> +\tstruct jtag *jtag = file->private_data;\n> +\n> +\tatomic_dec(&jtag->open);\n\nAgain, not needed.\n\n> +\treturn 0;\n> +}\n> +\n> +static const struct file_operations jtag_fops = {\n> +\t.owner\t\t= THIS_MODULE,\n> +\t.open\t\t= jtag_open,\n> +\t.release\t= jtag_release,\n> +\t.llseek\t\t= noop_llseek,\n> +\t.unlocked_ioctl = jtag_ioctl,\n> +#ifdef CONFIG_COMPAT\n> +\t.compat_ioctl\t= jtag_ioctl_compat,\n> +#endif\n> +};\n> +\n> +struct jtag *jtag_alloc(size_t priv_size, const struct jtag_ops *ops)\n> +{\n> +\tstruct jtag *jtag;\n> +\n> +\tjtag = kzalloc(sizeof(*jtag) + round_up(priv_size, ARCH_DMA_MINALIGN),\n> +\t\t GFP_KERNEL);\n> +\tif (!jtag)\n> +\t\treturn NULL;\n> +\n> +\tjtag->ops = ops;\n> +\treturn jtag;\n> +}\n> +EXPORT_SYMBOL_GPL(jtag_alloc);\n\nregister should allocate and register the device, why pass a structure\nback that the caller then has to do something else with to use it?\n\n> +\n> +void jtag_free(struct jtag *jtag)\n> +{\n> +\tkfree(jtag);\n> +}\n> +EXPORT_SYMBOL_GPL(jtag_free);\n> +\n> +int jtag_register(struct jtag *jtag)\n> +{\n> +\tint id;\n> +\tint err;\n> +\n> +\tid = ida_simple_get(&jtag_ida, 0, 0, GFP_KERNEL);\n> +\tif (id < 0)\n> +\t\treturn id;\n> +\n> +\tjtag->id = id;\n> +\tcdev_init(&jtag->cdev, &jtag_fops);\n> +\tjtag->cdev.owner = THIS_MODULE;\n> +\terr = cdev_add(&jtag->cdev, MKDEV(MAJOR(jtag_devt), jtag->id), 1);\n> +\tif (err)\n> +\t\tgoto err_cdev;\n\nmisc device would be so much simpler and easier here :(\n\n\n> +\n> +\t/* Register this jtag device with the driver core */\n> +\tjtag->dev = device_create(&jtag_class, NULL, MKDEV(MAJOR(jtag_devt),\n> +\t\t\t\t\t\t\t jtag->id),\n> +\t\t\t\t NULL, \"jtag%d\", jtag->id);\n> +\tif (!jtag->dev)\n> +\t\tgoto err_device_create;\n> +\n> +\tatomic_set(&jtag->open, 0);\n> +\tdev_set_drvdata(jtag->dev, jtag);\n> +\treturn err;\n> +\n> +err_device_create:\n> +\tcdev_del(&jtag->cdev);\n> +err_cdev:\n> +\tida_simple_remove(&jtag_ida, id);\n> +\treturn err;\n> +}\n> +EXPORT_SYMBOL_GPL(jtag_register);\n> +\n> +void jtag_unregister(struct jtag *jtag)\n> +{\n> +\tstruct device *dev = jtag->dev;\n> +\n> +\tcdev_del(&jtag->cdev);\n> +\tdevice_unregister(dev);\n> +\tida_simple_remove(&jtag_ida, jtag->id);\n> +}\n> +EXPORT_SYMBOL_GPL(jtag_unregister);\n> +\n> +static int __init jtag_init(void)\n> +{\n> +\tint err;\n> +\n> +\terr = alloc_chrdev_region(&jtag_devt, JTAG_FIRST_MINOR,\n> +\t\t\t\t JTAG_MAX_DEVICES, \"jtag\");\n> +\tif (err)\n> +\t\treturn err;\n> +\n> +\terr = class_register(&jtag_class);\n> +\tif (err)\n> +\t\tunregister_chrdev_region(jtag_devt, JTAG_MAX_DEVICES);\n> +\n> +\treturn err;\n> +}\n> +\n> +static void __exit jtag_exit(void)\n> +{\n> +\tclass_unregister(&jtag_class);\n> +\tunregister_chrdev_region(jtag_devt, JTAG_MAX_DEVICES);\n\nYour idr leaked memory :(\n\n\n\n> +}\n> +\n> +module_init(jtag_init);\n> +module_exit(jtag_exit);\n> +\n> +MODULE_AUTHOR(\"Oleksandr Shamray \");\n> +MODULE_DESCRIPTION(\"Generic jtag support\");\n> +MODULE_LICENSE(\"GPL v2\");\n> diff --git a/include/linux/jtag.h b/include/linux/jtag.h\n> new file mode 100644\n> index 0000000..c016fed\n> --- /dev/null\n> +++ b/include/linux/jtag.h\n> @@ -0,0 +1,48 @@\n> +/*\n> + * drivers/jtag/jtag.c\n> + *\n> + * Copyright (c) 2017 Mellanox Technologies. All rights reserved.\n> + * Copyright (c) 2017 Oleksandr Shamray \n> + *\n> + * Released under the GPLv2 only.\n> + * SPDX-License-Identifier: GPL-2.0\n> + */\n> +\n> +#ifndef __JTAG_H\n> +#define __JTAG_H\n> +\n> +#include \n> +\n> +#ifndef ARCH_DMA_MINALIGN\n> +#define ARCH_DMA_MINALIGN 1\n> +#endif\n> +\n> +#define jtag_u64_to_ptr(arg) ((void *)(uintptr_t)arg)\n> +\n> +#define JTAG_MAX_XFER_DATA_LEN 65535\n> +\n> +struct jtag;\n> +/**\n> + * struct jtag_ops - callbacks for jtag control functions:\n> + *\n> + * @freq_get: get frequency function. Filled by device driver\n> + * @freq_set: set frequency function. Filled by device driver\n> + * @status_get: set status function. Filled by device driver\n> + * @idle: set JTAG to idle state function. Filled by device driver\n> + * @xfer: send JTAG xfer function. Filled by device driver\n> + */\n> +struct jtag_ops {\n> +\tint (*freq_get)(struct jtag *jtag, u32 *freq);\n> +\tint (*freq_set)(struct jtag *jtag, u32 freq);\n> +\tint (*status_get)(struct jtag *jtag, u32 *state);\n> +\tint (*idle)(struct jtag *jtag, struct jtag_run_test_idle *idle);\n> +\tint (*xfer)(struct jtag *jtag, struct jtag_xfer *xfer);\n> +};\n> +\n> +void *jtag_priv(struct jtag *jtag);\n> +int jtag_register(struct jtag *jtag);\n> +void jtag_unregister(struct jtag *jtag);\n> +struct jtag *jtag_alloc(size_t priv_size, const struct jtag_ops *ops);\n> +void jtag_free(struct jtag *jtag);\n> +\n> +#endif /* __JTAG_H */\n> diff --git a/include/uapi/linux/jtag.h b/include/uapi/linux/jtag.h\n> new file mode 100644\n> index 0000000..180bf22\n> --- /dev/null\n> +++ b/include/uapi/linux/jtag.h\n> @@ -0,0 +1,115 @@\n> +/*\n> + * JTAG class driver\n> + *\n> + * Copyright (c) 2017 Mellanox Technologies. All rights reserved.\n> + * Copyright (c) 2017 Oleksandr Shamray \n> + *\n> + * Released under the GPLv2/BSD.\n> + * SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause\n> + */\n> +\n> +#ifndef __UAPI_LINUX_JTAG_H\n> +#define __UAPI_LINUX_JTAG_H\n> +\n> +/**\n> + * enum jtag_xfer_mode:\n> + *\n> + * @JTAG_XFER_HW_MODE: hardware mode transfer\n> + * @JTAG_XFER_SW_MODE: software mode transfer\n> + */\n> +enum jtag_xfer_mode {\n> +\tJTAG_XFER_HW_MODE,\n> +\tJTAG_XFER_SW_MODE,\n> +};\n> +\n> +/**\n> + * enum jtag_endstate:\n> + *\n> + * @JTAG_STATE_IDLE: JTAG state machine IDLE state\n> + * @JTAG_STATE_PAUSEIR: JTAG state machine PAUSE_IR state\n> + * @JTAG_STATE_PAUSEDR: JTAG state machine PAUSE_DR state\n> + */\n> +enum jtag_endstate {\n> +\tJTAG_STATE_IDLE,\n> +\tJTAG_STATE_PAUSEIR,\n> +\tJTAG_STATE_PAUSEDR,\n> +};\n> +\n> +/**\n> + * enum jtag_xfer_type:\n> + *\n> + * @JTAG_SIR_XFER: SIR transfer\n> + * @JTAG_SDR_XFER: SDR transfer\n> + */\n> +enum jtag_xfer_type {\n> +\tJTAG_SIR_XFER,\n> +\tJTAG_SDR_XFER,\n> +};\n> +\n> +/**\n> + * enum jtag_xfer_direction:\n> + *\n> + * @JTAG_READ_XFER: read transfer\n> + * @JTAG_WRITE_XFER: write transfer\n> + */\n> +enum jtag_xfer_direction {\n> +\tJTAG_READ_XFER,\n> +\tJTAG_WRITE_XFER,\n> +};\n> +\n> +/**\n> + * struct jtag_run_test_idle - forces JTAG state machine to\n> + * RUN_TEST/IDLE state\n> + *\n> + * @mode: access mode\n> + * @reset: 0 - run IDLE/PAUSE from current state\n> + * 1 - go through TEST_LOGIC/RESET state before IDLE/PAUSE\n> + * @end: completion flag\n> + * @tck: clock counter\n> + *\n> + * Structure represents interface to JTAG device for jtag idle\n> + * execution.\n> + */\n> +struct jtag_run_test_idle {\n> +\t__u8\tmode;\n> +\t__u8\treset;\n> +\t__u8\tendstate;\n> +\t__u8\ttck;\n> +};\n> +\n> +/**\n> + * struct jtag_xfer - jtag xfer:\n> + *\n> + * @mode: access mode\n> + * @type: transfer type\n> + * @direction: xfer direction\n> + * @length: xfer bits len\n> + * @tdio : xfer data array\n> + * @endir: xfer end state\n> + *\n> + * Structure represents interface to Aspeed JTAG device for jtag sdr xfer\n> + * execution.\n> + */\n> +struct jtag_xfer {\n> +\t__u8\tmode;\n> +\t__u8\ttype;\n> +\t__u8\tdirection;\n> +\t__u8\tendstate;\n> +\t__u32\tlength;\n> +\t__u64\ttdio;\n\nI don't see anything odd here that requires a compat ioctl callback, do\nyou?\n\nthanks,\n\ngreg k-h","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"qXxQ4bf6\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=infradead.org header.i=@infradead.org\n\theader.b=\"cP1XtaCx\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yJQbg73TXz9t3f\n\tfor ;\n\tFri, 20 Oct 2017 23:49:19 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5Wjj-0004Ri-7k; Fri, 20 Oct 2017 12:49:15 +0000","from merlin.infradead.org ([2001:8b0:10b:1231::1])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5Wjg-0004R7-OD for linux-arm-kernel@bombadil.infradead.org;\n\tFri, 20 Oct 2017 12:49:13 +0000","from mail.linuxfoundation.org ([140.211.169.12])\n\tby merlin.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5Vtg-0003ki-N0 for linux-arm-kernel@lists.infradead.org;\n\tFri, 20 Oct 2017 11:55:30 +0000","from localhost (LFbn-1-12253-150.w90-92.abo.wanadoo.fr\n\t[90.92.67.150])\n\tby mail.linuxfoundation.org (Postfix) with ESMTPSA id C24B6BD9;\n\tFri, 20 Oct 2017 11:55:04 +0000 (UTC)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=/Gacp3/mL/CGrv+PnGLhf/k5eKcUgDCHr8TwVcod7n4=;\n\tb=qXxQ4bf6YCZOPq\n\t1NCbY/+Z3uNqlqM40vIDKPeY5km3aDlj/dUBBw08r0zdonxHxlCNaqu1SnBJV9Lvja66lM6ah2gXq\n\tiSp5Pei16S8I0UavIXDLgKmzW1rw9GZ07RDlaCCgKQPaIuSUu1CCW0tmNzd7FccL4Q4fX/q+Xp4Cc\n\t/mPMd8vdSOQfa/vdZHfl5rP5UKMs95QD6jS3QuNyGGF/D2lZwwrSMsQFWxFeysn4Rvfj0SZYVrE9f\n\tZVrY/0XT//4TKiM2EmkcYtjhkUSm5GVRzvlE1iMZb47roenvjOVvv+Snl4F1wjuRsG7nEuEya6/2g\n\tu5DKCmEpzU8+mmSjTLWQ==;","v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=infradead.org; s=merlin.20170209;\n\th=In-Reply-To:Content-Type:MIME-Version:\n\tReferences:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:\n\tContent-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:\n\tResent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:\n\tList-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;\n\tbh=2Xx1p4KT/LNYM5xNRxAf8FcfY7+jd00zprRxbEjFdx8=;\n\tb=cP1XtaCxAZbrq+daL6yCeStIj\n\tAUTYDc0vW2ZwA3was1vdFBRp/v6+kEG9iq4halL3Cjj+lPJdSsvu6Xi+Zt6qWstgNLczxm8mBmXbl\n\tNWg/jj8qupj5soLIhkoW/geOzG5hHT0WPGwkMnuOdNHsLI3ON4l2xraaUCJ80PKKJdm2WJjS3Q+6x\n\t8aNj+ZXudkLvvn27pB9bpgbdAag5+u5S0prAjmtBQGiaFiXr0xm5romjBG5mpvhWr7B+la47/E3dH\n\tSWnIgBj55kyjpdYhcqhm1v8t1sUw0kNPanWHiastYn8uGYnEL87oDGcmmKzIz52BAnA0LkuCigwNz\n\tCgAoJte5w==;"],"Date":"Fri, 20 Oct 2017 13:55:12 +0200","From":"Greg KH ","To":"Oleksandr Shamray ","Subject":"Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver","Message-ID":"<20171020115512.GA2073@kroah.com>","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t<1505985932-27568-2-git-send-email-oleksandrs@mellanox.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<1505985932-27568-2-git-send-email-oleksandrs@mellanox.com>","User-Agent":"Mutt/1.9.1 (2017-09-22)","X-Spam-Note":"CRM114 invocation failed","X-Spam-Score":"-4.2 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on merlin.infradead.org summary:\n\tContent analysis details: (-4.2 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [140.211.169.12 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level@mellanox.com, devicetree@vger.kernel.org,\n\tjiri@resnulli.us, arnd@arndb.de, vadimp@mellanox.com,\n\tlinux-api@vger.kernel.org, openbmc@lists.ozlabs.org,\n\tlinux-kernel@vger.kernel.org, openocd-devel-owner@lists.sourceforge.net, \n\tmec@shout.net, Jiri Pirko , robh+dt@kernel.org,\n\tjoel@jms.id.au, linux-serial@vger.kernel.org, tklauser@distanz.ch,\n\tmchehab@kernel.org, davem@davemloft.net,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1791514,"web_url":"http://patchwork.ozlabs.org/comment/1791514/","msgid":"","list_archive_url":null,"date":"2017-10-20T14:34:00","subject":"RE: [patch v9 1/4] drivers: jtag: Add JTAG core driver","submitter":{"id":72099,"url":"http://patchwork.ozlabs.org/api/people/72099/","name":"Oleksandr Shamray","email":"oleksandrs@mellanox.com"},"content":"Hi Greg.\r\n\r\n> -----Original Message-----\r\n> From: Greg KH [mailto:gregkh@linuxfoundation.org]\r\n> Sent: Friday, October 20, 2017 2:55 PM\r\n> To: Oleksandr Shamray \r\n> Cc: arnd@arndb.de; linux-kernel@vger.kernel.org; linux-arm-\r\n> kernel@lists.infradead.org; devicetree@vger.kernel.org;\r\n> openbmc@lists.ozlabs.org; joel@jms.id.au; jiri@resnulli.us;\r\n> tklauser@distanz.ch; linux-serial@vger.kernel.org; mec@shout.net; Vadim\r\n> Pasternak ; system-sw-low-level level@mellanox.com>; robh+dt@kernel.org; openocd-devel-\r\n> owner@lists.sourceforge.net; linux-api@vger.kernel.org;\r\n> davem@davemloft.net; mchehab@kernel.org; Jiri Pirko \r\n> Subject: Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver\r\n> \r\n> On Thu, Sep 21, 2017 at 12:25:29PM +0300, Oleksandr Shamray wrote:\r\n> > +struct jtag {\r\n> > +\tstruct device *dev;\r\n> > +\tstruct cdev cdev;\r\n> \r\n> Why are you using a cdev here and not just a normal misc device? \r\n\r\nWhat the benefits to use misc instead of cdev?\r\n\r\n> I forgot if this is what you were doing before, sorry...\r\n> \r\n> > +\tint id;\r\n> > +\tatomic_t open;\r\n> \r\n> Why do you need this?\r\n\r\nThis counter used to avoid open at the same time by 2 or more users.\r\n> \r\n> > +\tconst struct jtag_ops *ops;\r\n> > +\tunsigned long priv[0] __aligned(ARCH_DMA_MINALIGN);\r\n> \r\n> Huh? Why is this needed to be dma aligned? Why not just use the private\r\n> pointer in struct device?\r\n> \r\n\r\nIt is critical?\r\n\r\n> > +};\r\n> > +\r\n> > +static dev_t jtag_devt;\r\n> > +static DEFINE_IDA(jtag_ida);\r\n> > +\r\n> > +void *jtag_priv(struct jtag *jtag)\r\n> > +{\r\n> > +\treturn jtag->priv;\r\n> > +}\r\n> > +EXPORT_SYMBOL_GPL(jtag_priv);\r\n> > +\r\n> > +static u8 *jtag_copy_from_user(__u64 udata, unsigned long bit_size) {\r\n> > +\tunsigned long size;\r\n> > +\tvoid *kdata;\r\n> > +\r\n> > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\r\n> > +\tkdata = memdup_user(u64_to_user_ptr(udata), size);\r\n> \r\n> You only use this once, why not just open-code it?\r\n\r\nI think it makes code more understandable.\r\n\r\n> \r\n> > +\r\n> > +\treturn kdata;\r\n> > +}\r\n> > +\r\n> > +static unsigned long jtag_copy_to_user(__u64 udata, u8 *kdata,\r\n> > +\t\t\t\t unsigned long bit_size)\r\n> > +{\r\n> > +\tunsigned long size;\r\n> > +\r\n> > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\r\n> > +\r\n> > +\treturn copy_to_user(u64_to_user_ptr(udata), (void *)(kdata), size);\r\n> \r\n> Same here, making this a separate function seems odd.\r\n\r\nSame, I think it makes code more understandable.\r\n\r\n> \r\n> > +}\r\n> > +\r\n> > +static struct class jtag_class = {\r\n> > +\t.name = \"jtag\",\r\n> > +\t.owner = THIS_MODULE,\r\n> > +};\r\n> > +\r\n> > +static int jtag_run_test_idle_op(struct jtag *jtag,\r\n> > +\t\t\t\t struct jtag_run_test_idle *idle) {\r\n> > +\tif (jtag->ops->idle)\r\n> > +\t\treturn jtag->ops->idle(jtag, idle);\r\n> > +\telse\r\n> > +\t\treturn -EOPNOTSUPP;\r\n> \r\n> Why is this a function? Why not just do this work in the ioctl?\r\n\r\nAgree. I will move it just to ioctl function body.\r\n\r\n> \r\n> > +}\r\n> > +\r\n> > +static int jtag_xfer_op(struct jtag *jtag, struct jtag_xfer *xfer, u8\r\n> > +*data) {\r\n> > +\tif (jtag->ops->xfer)\r\n> > +\t\treturn jtag->ops->xfer(jtag, xfer, data);\r\n> > +\telse\r\n> > +\t\treturn -EOPNOTSUPP;\r\n> \r\n> Same here, doesn't need to be a function.\r\n\r\nAgree.\r\n\r\n> \r\n> > +}\r\n> > +\r\n> > +static long jtag_ioctl(struct file *file, unsigned int cmd, unsigned\r\n> > +long arg) {\r\n> > +\tstruct jtag *jtag = file->private_data;\r\n> > +\tstruct jtag_run_test_idle idle;\r\n> > +\tstruct jtag_xfer xfer;\r\n> > +\tu8 *xfer_data;\r\n> > +\tu32 value;\r\n> > +\tint err;\r\n> > +\r\n> > +\tswitch (cmd) {\r\n> > +\tcase JTAG_GIOCFREQ:\r\n> > +\t\tif (jtag->ops->freq_get)\r\n> > +\t\t\terr = jtag->ops->freq_get(jtag, &value);\r\n> > +\t\telse\r\n> > +\t\t\terr = -EOPNOTSUPP;\r\n> > +\t\tif (err)\r\n> > +\t\t\tbreak;\r\n> > +\r\n> > +\t\terr = put_user(value, (__u32 *)arg);\r\n> > +\t\tbreak;\r\n> \r\n> Are you sure the return value of put_user is correct here? Shouldn't you return\r\n> -EFAULT if err is not 0?\r\n\r\nYes, thanks for point of this.\r\n\r\n> \r\n> > +\r\n> > +\tcase JTAG_SIOCFREQ:\r\n> > +\t\terr = get_user(value, (__u32 *)arg);\r\n> > +\r\n> \r\n> why a blank line?\r\n\r\nWill remove\r\n\r\n> \r\n> > +\t\tif (value == 0)\r\n> > +\t\t\terr = -EINVAL;\r\n> \r\n> Check err first.\r\n\r\nOk.\r\n\r\n> \r\n> > +\t\tif (err)\r\n> > +\t\t\tbreak;\r\n> \r\n> And set it properly, again err should be -EFAULT, right?\r\n\r\nRight 😊0\r\n\r\n> \r\n> > +\r\n> > +\t\tif (jtag->ops->freq_set)\r\n> > +\t\t\terr = jtag->ops->freq_set(jtag, value);\r\n> > +\t\telse\r\n> > +\t\t\terr = -EOPNOTSUPP;\r\n> > +\t\tbreak;\r\n> > +\r\n> > +\tcase JTAG_IOCRUNTEST:\r\n> > +\t\tif (copy_from_user(&idle, (void *)arg,\r\n> > +\t\t\t\t sizeof(struct jtag_run_test_idle)))\r\n> > +\t\t\treturn -ENOMEM;\r\n> > +\t\terr = jtag_run_test_idle_op(jtag, &idle);\r\n> \r\n> Who validates the structure fields? Is that up to the individual jtag driver? Why\r\n> not do it in the core correctly so that it only has to be done in one place and you\r\n> do not have to audit every individual driver?\r\n\r\nInput parameters validated by jtag platform driver. I think it not critical.\r\n\r\n> \r\n> > +\t\tbreak;\r\n> > +\r\n> > +\tcase JTAG_IOCXFER:\r\n> > +\t\tif (copy_from_user(&xfer, (void *)arg,\r\n> > +\t\t\t\t sizeof(struct jtag_xfer)))\r\n> > +\t\t\treturn -EFAULT;\r\n> > +\r\n> > +\t\tif (xfer.length >= JTAG_MAX_XFER_DATA_LEN)\r\n> > +\t\t\treturn -EFAULT;\r\n> > +\r\n> > +\t\txfer_data = jtag_copy_from_user(xfer.tdio, xfer.length);\r\n> > +\t\tif (!xfer_data)\r\n> > +\t\t\treturn -ENOMEM;\r\n> \r\n> Are you sure that's the correct error value?\r\n\r\nI think yes, but what you suggest?\r\n\r\n> \r\n> > +\r\n> > +\t\terr = jtag_xfer_op(jtag, &xfer, xfer_data);\r\n> > +\t\tif (jtag_copy_to_user(xfer.tdio, xfer_data, xfer.length)) {\r\n> > +\t\t\tkfree(xfer_data);\r\n> > +\t\t\treturn -EFAULT;\r\n> > +\t\t}\r\n> > +\r\n> > +\t\tkfree(xfer_data);\r\n> > +\t\tif (copy_to_user((void *)arg, &xfer, sizeof(struct jtag_xfer)))\r\n> > +\t\t\treturn -EFAULT;\r\n> > +\t\tbreak;\r\n> > +\r\n> > +\tcase JTAG_GIOCSTATUS:\r\n> > +\t\tif (jtag->ops->status_get)\r\n> > +\t\t\terr = jtag->ops->status_get(jtag, &value);\r\n> > +\t\telse\r\n> > +\t\t\terr = -EOPNOTSUPP;\r\n> > +\t\tif (err)\r\n> > +\t\t\tbreak;\r\n> > +\r\n> > +\t\terr = put_user(value, (__u32 *)arg);\r\n> \r\n> Same put_user err issue here.\r\n\r\nYes.\r\n\r\n> \r\n> > +\t\tbreak;\r\n> > +\r\n> > +\tdefault:\r\n> > +\t\treturn -EINVAL;\r\n> > +\t}\r\n> > +\treturn err;\r\n> > +}\r\n> > +\r\n> > +#ifdef CONFIG_COMPAT\r\n> > +static long jtag_ioctl_compat(struct file *file, unsigned int cmd,\r\n> > +\t\t\t unsigned long arg)\r\n> > +{\r\n> > +\treturn jtag_ioctl(file, cmd, (unsigned long)compat_ptr(arg)); }\r\n> > +#endif\r\n> \r\n> Why do you need a compat callback for a new ioctl? Create your structures\r\n> properly and this should not be needed, right?\r\n\r\n\r\nI does this because Arnd (arndbergmann@gmail.com) writed in review (On Wed, Aug 2, 2017 at 3:18 PM)\r\n\r\n[..]\r\n> + .unlocked_ioctl = jtag_ioctl,\r\n> + .open = jtag_open,\r\n> + .release = jtag_release,\r\n> +};\r\n\r\nadd a compat_ioctl pointer here, after ensuring that all ioctl commands are compatible between 32-bit and 64-bit user space.\r\n[..]\r\n\r\n\r\n> \r\n> > +\r\n> > +static int jtag_open(struct inode *inode, struct file *file) {\r\n> > +\tstruct jtag *jtag = container_of(inode->i_cdev, struct jtag, cdev);\r\n> > +\r\n> > +\tif (atomic_read(&jtag->open)) {\r\n> > +\t\tdev_info(NULL, \"jtag already opened\\n\");\r\n> > +\t\treturn -EBUSY;\r\n> \r\n> Why do you care if multiple opens can happen?\r\n\r\nJtag HW not support to using with multiple requests from different users. So we prohibit this.\r\n\r\n> \r\n> > +\t}\r\n> > +\r\n> > +\tatomic_inc(&jtag->open);\r\n> \r\n> Oh look, you just raced with two different people opening at the same time :(\r\n> \r\n> An atomic value is never a replacement for a lock, sorry.\r\n> \r\n> > +\tfile->private_data = jtag;\r\n> > +\treturn 0;\r\n> > +}\r\n> > +\r\n> > +static int jtag_release(struct inode *inode, struct file *file) {\r\n> > +\tstruct jtag *jtag = file->private_data;\r\n> > +\r\n> > +\tatomic_dec(&jtag->open);\r\n> \r\n> Again, not needed.\r\n> \r\n> > +\treturn 0;\r\n> > +}\r\n> > +\r\n> > +static const struct file_operations jtag_fops = {\r\n> > +\t.owner\t\t= THIS_MODULE,\r\n> > +\t.open\t\t= jtag_open,\r\n> > +\t.release\t= jtag_release,\r\n> > +\t.llseek\t\t= noop_llseek,\r\n> > +\t.unlocked_ioctl = jtag_ioctl,\r\n> > +#ifdef CONFIG_COMPAT\r\n> > +\t.compat_ioctl\t= jtag_ioctl_compat,\r\n> > +#endif\r\n> > +};\r\n> > +\r\n> > +struct jtag *jtag_alloc(size_t priv_size, const struct jtag_ops *ops)\r\n> > +{\r\n> > +\tstruct jtag *jtag;\r\n> > +\r\n> > +\tjtag = kzalloc(sizeof(*jtag) + round_up(priv_size,\r\n> ARCH_DMA_MINALIGN),\r\n> > +\t\t GFP_KERNEL);\r\n> > +\tif (!jtag)\r\n> > +\t\treturn NULL;\r\n> > +\r\n> > +\tjtag->ops = ops;\r\n> > +\treturn jtag;\r\n> > +}\r\n> > +EXPORT_SYMBOL_GPL(jtag_alloc);\r\n> \r\n> register should allocate and register the device, why pass a structure back that\r\n> the caller then has to do something else with to use it?\r\n> \r\n\r\nBefore registration we need to initialize JTAG device HW registers and fill private data with HW specific parameters.\r\nAnd is I see in other Linux drivers it is a common way to separate allocation device and register it\r\n\r\n> > +\r\n> > +void jtag_free(struct jtag *jtag)\r\n> > +{\r\n> > +\tkfree(jtag);\r\n> > +}\r\n> > +EXPORT_SYMBOL_GPL(jtag_free);\r\n> > +\r\n> > +int jtag_register(struct jtag *jtag)\r\n> > +{\r\n> > +\tint id;\r\n> > +\tint err;\r\n> > +\r\n> > +\tid = ida_simple_get(&jtag_ida, 0, 0, GFP_KERNEL);\r\n> > +\tif (id < 0)\r\n> > +\t\treturn id;\r\n> > +\r\n> > +\tjtag->id = id;\r\n> > +\tcdev_init(&jtag->cdev, &jtag_fops);\r\n> > +\tjtag->cdev.owner = THIS_MODULE;\r\n> > +\terr = cdev_add(&jtag->cdev, MKDEV(MAJOR(jtag_devt), jtag->id), 1);\r\n> > +\tif (err)\r\n> > +\t\tgoto err_cdev;\r\n> \r\n> misc device would be so much simpler and easier here :(\r\n> \r\n> \r\n> > +\r\n> > +\t/* Register this jtag device with the driver core */\r\n> > +\tjtag->dev = device_create(&jtag_class, NULL,\r\n> MKDEV(MAJOR(jtag_devt),\r\n> > +\t\t\t\t\t\t\t jtag->id),\r\n> > +\t\t\t\t NULL, \"jtag%d\", jtag->id);\r\n> > +\tif (!jtag->dev)\r\n> > +\t\tgoto err_device_create;\r\n> > +\r\n> > +\tatomic_set(&jtag->open, 0);\r\n> > +\tdev_set_drvdata(jtag->dev, jtag);\r\n> > +\treturn err;\r\n> > +\r\n> > +err_device_create:\r\n> > +\tcdev_del(&jtag->cdev);\r\n> > +err_cdev:\r\n> > +\tida_simple_remove(&jtag_ida, id);\r\n> > +\treturn err;\r\n> > +}\r\n> > +EXPORT_SYMBOL_GPL(jtag_register);\r\n> > +\r\n> > +void jtag_unregister(struct jtag *jtag) {\r\n> > +\tstruct device *dev = jtag->dev;\r\n> > +\r\n> > +\tcdev_del(&jtag->cdev);\r\n> > +\tdevice_unregister(dev);\r\n> > +\tida_simple_remove(&jtag_ida, jtag->id); }\r\n> > +EXPORT_SYMBOL_GPL(jtag_unregister);\r\n> > +\r\n> > +static int __init jtag_init(void)\r\n> > +{\r\n> > +\tint err;\r\n> > +\r\n> > +\terr = alloc_chrdev_region(&jtag_devt, JTAG_FIRST_MINOR,\r\n> > +\t\t\t\t JTAG_MAX_DEVICES, \"jtag\");\r\n> > +\tif (err)\r\n> > +\t\treturn err;\r\n> > +\r\n> > +\terr = class_register(&jtag_class);\r\n> > +\tif (err)\r\n> > +\t\tunregister_chrdev_region(jtag_devt, JTAG_MAX_DEVICES);\r\n> > +\r\n> > +\treturn err;\r\n> > +}\r\n> > +\r\n> > +static void __exit jtag_exit(void)\r\n> > +{\r\n> > +\tclass_unregister(&jtag_class);\r\n> > +\tunregister_chrdev_region(jtag_devt, JTAG_MAX_DEVICES);\r\n> \r\n> Your idr leaked memory :(\r\n> \r\n\r\nYes I will add ida_destroy() here.\r\n> \r\n> \r\n> > +}\r\n> > +\r\n> > +module_init(jtag_init);\r\n> > +module_exit(jtag_exit);\r\n> > +\r\n> > +MODULE_AUTHOR(\"Oleksandr Shamray \");\r\n> > +MODULE_DESCRIPTION(\"Generic jtag support\"); MODULE_LICENSE(\"GPL\r\n> v2\");\r\n> > diff --git a/include/linux/jtag.h b/include/linux/jtag.h new file mode\r\n> > 100644 index 0000000..c016fed\r\n> > --- /dev/null\r\n> > +++ b/include/linux/jtag.h\r\n> > @@ -0,0 +1,48 @@\r\n> > +/*\r\n> > + * drivers/jtag/jtag.c\r\n> > + *\r\n> > + * Copyright (c) 2017 Mellanox Technologies. All rights reserved.\r\n> > + * Copyright (c) 2017 Oleksandr Shamray \r\n> > + *\r\n> > + * Released under the GPLv2 only.\r\n> > + * SPDX-License-Identifier: GPL-2.0\r\n> > + */\r\n> > +\r\n> > +#ifndef __JTAG_H\r\n> > +#define __JTAG_H\r\n> > +\r\n> > +#include \r\n> > +\r\n> > +#ifndef ARCH_DMA_MINALIGN\r\n> > +#define ARCH_DMA_MINALIGN 1\r\n> > +#endif\r\n> > +\r\n> > +#define jtag_u64_to_ptr(arg) ((void *)(uintptr_t)arg)\r\n> > +\r\n> > +#define JTAG_MAX_XFER_DATA_LEN 65535\r\n> > +\r\n> > +struct jtag;\r\n> > +/**\r\n> > + * struct jtag_ops - callbacks for jtag control functions:\r\n> > + *\r\n> > + * @freq_get: get frequency function. Filled by device driver\r\n> > + * @freq_set: set frequency function. Filled by device driver\r\n> > + * @status_get: set status function. Filled by device driver\r\n> > + * @idle: set JTAG to idle state function. Filled by device driver\r\n> > + * @xfer: send JTAG xfer function. Filled by device driver */ struct\r\n> > +jtag_ops {\r\n> > +\tint (*freq_get)(struct jtag *jtag, u32 *freq);\r\n> > +\tint (*freq_set)(struct jtag *jtag, u32 freq);\r\n> > +\tint (*status_get)(struct jtag *jtag, u32 *state);\r\n> > +\tint (*idle)(struct jtag *jtag, struct jtag_run_test_idle *idle);\r\n> > +\tint (*xfer)(struct jtag *jtag, struct jtag_xfer *xfer); };\r\n> > +\r\n> > +void *jtag_priv(struct jtag *jtag);\r\n> > +int jtag_register(struct jtag *jtag); void jtag_unregister(struct\r\n> > +jtag *jtag); struct jtag *jtag_alloc(size_t priv_size, const struct\r\n> > +jtag_ops *ops); void jtag_free(struct jtag *jtag);\r\n> > +\r\n> > +#endif /* __JTAG_H */\r\n> > diff --git a/include/uapi/linux/jtag.h b/include/uapi/linux/jtag.h new\r\n> > file mode 100644 index 0000000..180bf22\r\n> > --- /dev/null\r\n> > +++ b/include/uapi/linux/jtag.h\r\n> > @@ -0,0 +1,115 @@\r\n> > +/*\r\n> > + * JTAG class driver\r\n> > + *\r\n> > + * Copyright (c) 2017 Mellanox Technologies. All rights reserved.\r\n> > + * Copyright (c) 2017 Oleksandr Shamray \r\n> > + *\r\n> > + * Released under the GPLv2/BSD.\r\n> > + * SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause */\r\n> > +\r\n> > +#ifndef __UAPI_LINUX_JTAG_H\r\n> > +#define __UAPI_LINUX_JTAG_H\r\n> > +\r\n> > +/**\r\n> > + * enum jtag_xfer_mode:\r\n> > + *\r\n> > + * @JTAG_XFER_HW_MODE: hardware mode transfer\r\n> > + * @JTAG_XFER_SW_MODE: software mode transfer */ enum\r\n> jtag_xfer_mode\r\n> > +{\r\n> > +\tJTAG_XFER_HW_MODE,\r\n> > +\tJTAG_XFER_SW_MODE,\r\n> > +};\r\n> > +\r\n> > +/**\r\n> > + * enum jtag_endstate:\r\n> > + *\r\n> > + * @JTAG_STATE_IDLE: JTAG state machine IDLE state\r\n> > + * @JTAG_STATE_PAUSEIR: JTAG state machine PAUSE_IR state\r\n> > + * @JTAG_STATE_PAUSEDR: JTAG state machine PAUSE_DR state */ enum\r\n> > +jtag_endstate {\r\n> > +\tJTAG_STATE_IDLE,\r\n> > +\tJTAG_STATE_PAUSEIR,\r\n> > +\tJTAG_STATE_PAUSEDR,\r\n> > +};\r\n> > +\r\n> > +/**\r\n> > + * enum jtag_xfer_type:\r\n> > + *\r\n> > + * @JTAG_SIR_XFER: SIR transfer\r\n> > + * @JTAG_SDR_XFER: SDR transfer\r\n> > + */\r\n> > +enum jtag_xfer_type {\r\n> > +\tJTAG_SIR_XFER,\r\n> > +\tJTAG_SDR_XFER,\r\n> > +};\r\n> > +\r\n> > +/**\r\n> > + * enum jtag_xfer_direction:\r\n> > + *\r\n> > + * @JTAG_READ_XFER: read transfer\r\n> > + * @JTAG_WRITE_XFER: write transfer\r\n> > + */\r\n> > +enum jtag_xfer_direction {\r\n> > +\tJTAG_READ_XFER,\r\n> > +\tJTAG_WRITE_XFER,\r\n> > +};\r\n> > +\r\n> > +/**\r\n> > + * struct jtag_run_test_idle - forces JTAG state machine to\r\n> > + * RUN_TEST/IDLE state\r\n> > + *\r\n> > + * @mode: access mode\r\n> > + * @reset: 0 - run IDLE/PAUSE from current state\r\n> > + * 1 - go through TEST_LOGIC/RESET state before IDLE/PAUSE\r\n> > + * @end: completion flag\r\n> > + * @tck: clock counter\r\n> > + *\r\n> > + * Structure represents interface to JTAG device for jtag idle\r\n> > + * execution.\r\n> > + */\r\n> > +struct jtag_run_test_idle {\r\n> > +\t__u8\tmode;\r\n> > +\t__u8\treset;\r\n> > +\t__u8\tendstate;\r\n> > +\t__u8\ttck;\r\n> > +};\r\n> > +\r\n> > +/**\r\n> > + * struct jtag_xfer - jtag xfer:\r\n> > + *\r\n> > + * @mode: access mode\r\n> > + * @type: transfer type\r\n> > + * @direction: xfer direction\r\n> > + * @length: xfer bits len\r\n> > + * @tdio : xfer data array\r\n> > + * @endir: xfer end state\r\n> > + *\r\n> > + * Structure represents interface to Aspeed JTAG device for jtag sdr\r\n> > +xfer\r\n> > + * execution.\r\n> > + */\r\n> > +struct jtag_xfer {\r\n> > +\t__u8\tmode;\r\n> > +\t__u8\ttype;\r\n> > +\t__u8\tdirection;\r\n> > +\t__u8\tendstate;\r\n> > +\t__u32\tlength;\r\n> > +\t__u64\ttdio;\r\n> \r\n> I don't see anything odd here that requires a compat ioctl callback, do you?\r\n> \r\n> thanks,\r\n> \r\n> greg k-h\r\n\r\nThanks for your review.\r\n\r\nOleksandr S.","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"KDZ+rAEv\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"p+fEfaeu\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yJSx61BzYz9t5q\n\tfor ;\n\tSat, 21 Oct 2017 01:34:34 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5YNc-0007v7-22; Fri, 20 Oct 2017 14:34:32 +0000","from mail-am5eur02on060e.outbound.protection.outlook.com\n\t([2a01:111:f400:fe07::60e]\n\thelo=EUR02-AM5-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5YNX-0007s3-5K for linux-arm-kernel@lists.infradead.org;\n\tFri, 20 Oct 2017 14:34:30 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com (10.165.82.13) by\n\tAM4PR05MB3332.eurprd05.prod.outlook.com (10.171.187.17) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.77.7; Fri, 20 Oct 2017 14:34:00 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6]) by\n\tAM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::c1dc:8bea:a5cb:bcb6%18]) with mapi id 15.20.0077.022;\n\tFri, 20 Oct 2017 14:34:00 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References:\n\tMessage-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=LT9FbN3RJrbBpaByHw5lVGRYMes2L458e5KhCHP9bC8=;\n\tb=KDZ+rAEv+cHzOF\n\tVS7LLhuIYIQP0BDZxgOsXyvTtgH7jSwppXJsA0jeaLLNQh6R5KbchQgYAhuwAs08+uaJ3Up8vdio8\n\t4mGiMsBDN/ecD18j/JuMeBLGAH2+DpBY2ww7fHSFIlBTexlviONnAnl4lJUtGs1KVySkA7vgy1yvl\n\tj/nYhM/EVMgelI11jz5CwAoR+wdPJkuU6SkIAme+qXSS2NPVLno4f4Bz7E5k5d4WwFXBp5pRmtfgW\n\tkgVXBdFNAtexozXJ71UAg4gMYiCxZf1puPbHeXykITKLdccyvfERveaBnPxrnoeKTjKY28luH3I5u\n\tCIwAhu5J7036CUDyJ9LQ==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n\ts=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=HR1pTXq4Dal8z/HiduDmP0ZZBg6Jb8dVJHKiLGvdksc=;\n\tb=p+fEfaeupFSWn66f+7n3EaYvf+yRaTfBV9JgbYM26RFnGRl5PvVNEv430YsUSUsYvsuDM/zK1FZtqOR4FqMgsg9hDyu9M3y9RR3QKQIbLvG2XlkxxqLodfXSwHldbIna5/3sE48btHgMm7dJi6Da4+S/C2m/wtWyQ2stZ/g/Qho="],"From":"Oleksandr Shamray ","To":"Greg KH ","Subject":"RE: [patch v9 1/4] drivers: jtag: Add JTAG core driver","Thread-Topic":"[patch v9 1/4] drivers: jtag: Add JTAG core driver","Thread-Index":"AQHTMru5yoecjoOKkEeYBkPR/vITOKLszxoAgAAGBCA=","Date":"Fri, 20 Oct 2017 14:34:00 +0000","Message-ID":"","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t<1505985932-27568-2-git-send-email-oleksandrs@mellanox.com>\n\t<20171020115512.GA2073@kroah.com>","In-Reply-To":"<20171020115512.GA2073@kroah.com>","Accept-Language":"en-US, uk-UA","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"KDZ+rAEv\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"p+fEfaeu\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"x-originating-ip":"[80.90.224.13]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; AM4PR05MB3332;\n\t6:xThJeGAGOkps+pu5h8Fs3xCeKMi1+LbezWtcZUeiHmsN9dLtAaTcNCbudO5H9+GifhY6cYjXeHHkk6b30R6ZtQ2h2Dcq8L98DSFuuwwjheVxMJVBW8kpveP+2z6buONMqtOKTfLcA/xtt2zLS2cspRPkirilNlhxsdAvlr70McM+cxCmwJ8A1LZsGM2YIgqfxDFSkdxXUojLGvQZx26mbw9109XW2P8BVfQVBKyHL23oa1tG+77GOb/IPLD8hIJQ+i1JZEpbhUBygCo62P84XyBf8IxwmYlxL04WBEccg63dgKGGFjqPgYtX7Oyovs784MFPiUx1LLiguoR1lbJ0TQ==;\n\t5:0ogXv3skV4Sx24eDB7nGW7xLIZataVqJMHAjpiMvAinyc/zeIAMffrzaXTv7hfllIWtMrQ9CODD9PMKhiLdWI3SMX4xFvbEu4pXw4rWjzAs9kG45XwljPMnw85tmQTO/Pky9Ro2YYdrUUxD3erD1qQ==;\n\t24:i2g7bdbKirfEHjedrJXNTLz0B7BELLnM8DElonEaP7TTf65FjTR6sm+krEXvMo8X71wdO1GpVxSP92yrlCGdsePqy30fYT+TwRJTjv7HTYM=;\n\t7:yBLUR0jgGYi8y9TGdT/orlonRpeM6jTxDv6yTB3SnYAX16Gb3IzNjUNzlcPSyG1cosjflHYzjpUB/RqNZr95kNwIrdIMha46cYtRXll6fdVPOyMKoJtO36MIBxsdRQZ8Ap2lRBlhY2WiRUXP2uobyfTFKSb/B7eFOwgT4lDuUgyrlehUFfEqfyGSvAqbHwbHXoQfaQ1ZduQj5WfIpJhEfj2oD5wjQ6LvA+GIq6pGJSo=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;SSOR;","x-ms-office365-filtering-correlation-id":"f33c2bf1-4afa-4bac-ca70-08d517c79b0a","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(22001)(4534020)(4602075)(4627075)(201703031133081)(201702281549075)(48565401081)(2017052603199);\n\tSRVR:AM4PR05MB3332; ","x-ms-traffictypediagnostic":"AM4PR05MB3332:","x-exchange-antispam-report-test":"UriScan:(143289334528602)(9452136761055)(65623756079841)(258649278758335)(42262312472803);","x-microsoft-antispam-prvs":"","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3002001)(10201501046)(100000703101)(100105400095)(3231020)(93006095)(93001095)(6055026)(6041248)(20161123560025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123562025)(20161123555025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:AM4PR05MB3332; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:AM4PR05MB3332; ","x-forefront-prvs":"0466CA5A45","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(346002)(376002)(39860400002)(24454002)(199003)(189002)(13464003)(5250100002)(86362001)(3846002)(107886003)(55016002)(33656002)(106356001)(99286003)(3660700001)(74316002)(68736007)(6436002)(6246003)(189998001)(105586002)(4326008)(9686003)(6506006)(53936002)(305945005)(3280700002)(97736004)(8936002)(66066001)(53946003)(54906003)(8676002)(2900100001)(478600001)(102836003)(76176999)(316002)(101416001)(7736002)(229853002)(54356999)(2906002)(6116002)(2950100002)(6916009)(50986999)(81166006)(14454004)(7416002)(25786009)(53546010)(7696004)(5660300001)(81156014);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM4PR05MB3332;\n\tH:AM4PR0501MB2194.eurprd05.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; A:1; MX:1; LANG:en; ","received-spf":"None (protection.outlook.com: mellanox.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-OriginatorOrg":"Mellanox.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"20 Oct 2017 14:34:00.8180\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"a652971c-7d2e-4d9b-a6a4-d149256f461b","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"AM4PR05MB3332","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171020_073427_784629_BCDFA824 ","X-CRM114-Status":"GOOD ( 23.48 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a01:111:f400:fe07:0:0:0:60e listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"jiri@resnulli.us\" , \"arnd@arndb.de\" ,\n\tVadim Pasternak ,\n\t\"linux-api@vger.kernel.org\" ,\n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\t\"mec@shout.net\" , Jiri Pirko ,\n\t\"robh+dt@kernel.org\" , \n\t\"joel@jms.id.au\" ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"tklauser@distanz.ch\" ,\n\t\"mchehab@kernel.org\" ,\n\t\"davem@davemloft.net\" ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1791527,"web_url":"http://patchwork.ozlabs.org/comment/1791527/","msgid":"<20171020145417.GD8965@kroah.com>","list_archive_url":null,"date":"2017-10-20T14:54:17","subject":"Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver","submitter":{"id":11800,"url":"http://patchwork.ozlabs.org/api/people/11800/","name":"Greg Kroah-Hartman","email":"gregkh@linuxfoundation.org"},"content":"On Fri, Oct 20, 2017 at 02:34:00PM +0000, Oleksandr Shamray wrote:\n> Hi Greg.\n> \n> > -----Original Message-----\n> > From: Greg KH [mailto:gregkh@linuxfoundation.org]\n> > Sent: Friday, October 20, 2017 2:55 PM\n> > To: Oleksandr Shamray \n> > Cc: arnd@arndb.de; linux-kernel@vger.kernel.org; linux-arm-\n> > kernel@lists.infradead.org; devicetree@vger.kernel.org;\n> > openbmc@lists.ozlabs.org; joel@jms.id.au; jiri@resnulli.us;\n> > tklauser@distanz.ch; linux-serial@vger.kernel.org; mec@shout.net; Vadim\n> > Pasternak ; system-sw-low-level > level@mellanox.com>; robh+dt@kernel.org; openocd-devel-\n> > owner@lists.sourceforge.net; linux-api@vger.kernel.org;\n> > davem@davemloft.net; mchehab@kernel.org; Jiri Pirko \n> > Subject: Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver\n> > \n> > On Thu, Sep 21, 2017 at 12:25:29PM +0300, Oleksandr Shamray wrote:\n> > > +struct jtag {\n> > > +\tstruct device *dev;\n> > > +\tstruct cdev cdev;\n> > \n> > Why are you using a cdev here and not just a normal misc device? \n> \n> What the benefits to use misc instead of cdev?\n\nLess code, simpler logic, easier to review and understand, etc.\n\nLet me ask you, why use a cdev instead of a misc?\n\n> > I forgot if this is what you were doing before, sorry...\n> > \n> > > +\tint id;\n> > > +\tatomic_t open;\n> > \n> > Why do you need this?\n> \n> This counter used to avoid open at the same time by 2 or more users.\n\nBut it isn't working :)\n\nAnd why do you care?\n\n> > > +\tconst struct jtag_ops *ops;\n> > > +\tunsigned long priv[0] __aligned(ARCH_DMA_MINALIGN);\n> > \n> > Huh? Why is this needed to be dma aligned? Why not just use the private\n> > pointer in struct device?\n> > \n> \n> It is critical?\n\nYou are saying it is, so you have to justify it. There is a pointer for\nyou to use, don't make new ones for no reason, right?\n\n> > > +};\n> > > +\n> > > +static dev_t jtag_devt;\n> > > +static DEFINE_IDA(jtag_ida);\n> > > +\n> > > +void *jtag_priv(struct jtag *jtag)\n> > > +{\n> > > +\treturn jtag->priv;\n> > > +}\n> > > +EXPORT_SYMBOL_GPL(jtag_priv);\n> > > +\n> > > +static u8 *jtag_copy_from_user(__u64 udata, unsigned long bit_size) {\n> > > +\tunsigned long size;\n> > > +\tvoid *kdata;\n> > > +\n> > > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> > > +\tkdata = memdup_user(u64_to_user_ptr(udata), size);\n> > \n> > You only use this once, why not just open-code it?\n> \n> I think it makes code more understandable.\n\nAs a reviewer, I don't :)\n\n> > > +\n> > > +\treturn kdata;\n> > > +}\n> > > +\n> > > +static unsigned long jtag_copy_to_user(__u64 udata, u8 *kdata,\n> > > +\t\t\t\t unsigned long bit_size)\n> > > +{\n> > > +\tunsigned long size;\n> > > +\n> > > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> > > +\n> > > +\treturn copy_to_user(u64_to_user_ptr(udata), (void *)(kdata), size);\n> > \n> > Same here, making this a separate function seems odd.\n> \n> Same, I think it makes code more understandable.\n\nBut it doesn't.\n\n> > > +\n> > > +\t\tif (jtag->ops->freq_set)\n> > > +\t\t\terr = jtag->ops->freq_set(jtag, value);\n> > > +\t\telse\n> > > +\t\t\terr = -EOPNOTSUPP;\n> > > +\t\tbreak;\n> > > +\n> > > +\tcase JTAG_IOCRUNTEST:\n> > > +\t\tif (copy_from_user(&idle, (void *)arg,\n> > > +\t\t\t\t sizeof(struct jtag_run_test_idle)))\n> > > +\t\t\treturn -ENOMEM;\n> > > +\t\terr = jtag_run_test_idle_op(jtag, &idle);\n> > \n> > Who validates the structure fields? Is that up to the individual jtag driver? Why\n> > not do it in the core correctly so that it only has to be done in one place and you\n> > do not have to audit every individual driver?\n> \n> Input parameters validated by jtag platform driver. I think it not critical.\n\nNot true at all. It is very critical. Remmeber, \"All Input Is Evil!\"\n\nYou have to validate this. I as a reviewer have to find where you are\nvalidating this data to ensure bad things do not happen. I can't review\nthat here, now I have to go and review all of the individual drivers,\nwhich is a major pain, don't you agree?\n\n> > > +\t\tbreak;\n> > > +\n> > > +\tcase JTAG_IOCXFER:\n> > > +\t\tif (copy_from_user(&xfer, (void *)arg,\n> > > +\t\t\t\t sizeof(struct jtag_xfer)))\n> > > +\t\t\treturn -EFAULT;\n> > > +\n> > > +\t\tif (xfer.length >= JTAG_MAX_XFER_DATA_LEN)\n> > > +\t\t\treturn -EFAULT;\n> > > +\n> > > +\t\txfer_data = jtag_copy_from_user(xfer.tdio, xfer.length);\n> > > +\t\tif (!xfer_data)\n> > > +\t\t\treturn -ENOMEM;\n> > \n> > Are you sure that's the correct error value?\n> \n> I think yes, but what you suggest?\n\nA fault happened, so -EFAULT, right?\n\n> [..]\n> > + .unlocked_ioctl = jtag_ioctl,\n> > + .open = jtag_open,\n> > + .release = jtag_release,\n> > +};\n> \n> add a compat_ioctl pointer here, after ensuring that all ioctl\n> commands are compatible between 32-bit and 64-bit user space.\n> [..]\n\nAnd if you do not, what happens? You shouldn't need it as there is no\nfixups necessary, or am I mistaken about that?\n\n> > > +static int jtag_open(struct inode *inode, struct file *file) {\n> > > +\tstruct jtag *jtag = container_of(inode->i_cdev, struct jtag, cdev);\n> > > +\n> > > +\tif (atomic_read(&jtag->open)) {\n> > > +\t\tdev_info(NULL, \"jtag already opened\\n\");\n> > > +\t\treturn -EBUSY;\n> > \n> > Why do you care if multiple opens can happen?\n> \n> Jtag HW not support to using with multiple requests from different users. So we prohibit this.\n\nWhy does the kernel care?\n\nAnd again, your implementation is broken, it's not actually doing this\nprotection. I recommend just not doing it at all, but if you really are\ninsisting on it, you have to get it correct :)\n\nthanks,\n\ngreg k-h","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"ENChbmqo\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yJTNK1pBFz9t5q\n\tfor ;\n\tSat, 21 Oct 2017 01:54:41 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5Yh3-00034s-RH; Fri, 20 Oct 2017 14:54:37 +0000","from mail.linuxfoundation.org ([140.211.169.12])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5Yh0-0002ro-BY for linux-arm-kernel@lists.infradead.org;\n\tFri, 20 Oct 2017 14:54:36 +0000","from localhost (LFbn-1-12253-150.w90-92.abo.wanadoo.fr\n\t[90.92.67.150])\n\tby mail.linuxfoundation.org (Postfix) with ESMTPSA id C4D9C88A;\n\tFri, 20 Oct 2017 14:54:08 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=jvLT9HUyyzVhmmGToHYM/3RYhWn7XYeyKmy7xxwt2PQ=;\n\tb=ENChbmqoEjmtNe\n\t67q8JRrQdceRwgEYsxy8ox27vH/4tZRONA04hX+IxheXL488673lMc+WJt0thWMtSwWolXMlrETQ4\n\tnFPEPavKBhwS0uAF1on6fidTbgnQXSY1u5yNyVdjtDxO2r0CljmqCTJUbt3m2bniOvmvaJkGqq92c\n\tPDP4SjrGz/etJGZHMjA5C4eE4ZGREelq7J08L44k9zlG+NGDmLdYNQSvuXJIAZBsjHX2HRsVWWXv3\n\tSqMstZSdpffFOsQD639G5bjjCUsjM7lcR1g5ESs+loHu14JDg1hQFO0mS4slaDln2xWVT1rZ0BstS\n\tlHSFY89NSKfNyvco0UZw==;","Date":"Fri, 20 Oct 2017 16:54:17 +0200","From":"Greg KH ","To":"Oleksandr Shamray ","Subject":"Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver","Message-ID":"<20171020145417.GD8965@kroah.com>","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t<1505985932-27568-2-git-send-email-oleksandrs@mellanox.com>\n\t<20171020115512.GA2073@kroah.com>\n\t","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"","User-Agent":"Mutt/1.9.1 (2017-09-22)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171020_075434_525252_D76F290F ","X-CRM114-Status":"GOOD ( 27.38 )","X-Spam-Score":"-4.2 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-4.2 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [140.211.169.12 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"jiri@resnulli.us\" , \"arnd@arndb.de\" ,\n\tVadim Pasternak ,\n\t\"linux-api@vger.kernel.org\" ,\n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\t\"mec@shout.net\" , Jiri Pirko ,\n\t\"robh+dt@kernel.org\" , \n\t\"joel@jms.id.au\" ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"tklauser@distanz.ch\" ,\n\t\"mchehab@kernel.org\" ,\n\t\"davem@davemloft.net\" ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1793831,"web_url":"http://patchwork.ozlabs.org/comment/1793831/","msgid":"","list_archive_url":null,"date":"2017-10-25T15:58:42","subject":"RE: [patch v9 1/4] drivers: jtag: Add JTAG core driver","submitter":{"id":72099,"url":"http://patchwork.ozlabs.org/api/people/72099/","name":"Oleksandr Shamray","email":"oleksandrs@mellanox.com"},"content":"> -----Original Message-----\n> From: Greg KH [mailto:gregkh@linuxfoundation.org]\n> Sent: Friday, October 20, 2017 5:54 PM\n> To: Oleksandr Shamray \n> Cc: arnd@arndb.de; linux-kernel@vger.kernel.org; linux-arm-\n> kernel@lists.infradead.org; devicetree@vger.kernel.org;\n> openbmc@lists.ozlabs.org; joel@jms.id.au; jiri@resnulli.us;\n> tklauser@distanz.ch; linux-serial@vger.kernel.org; mec@shout.net; Vadim\n> Pasternak ; system-sw-low-level level@mellanox.com>; robh+dt@kernel.org; openocd-devel-\n> owner@lists.sourceforge.net; linux-api@vger.kernel.org;\n> davem@davemloft.net; mchehab@kernel.org; Jiri Pirko \n> Subject: Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver\n> \n> On Fri, Oct 20, 2017 at 02:34:00PM +0000, Oleksandr Shamray wrote:\n> > Hi Greg.\n> >\n> > > -----Original Message-----\n> > > From: Greg KH [mailto:gregkh@linuxfoundation.org]\n> > > Sent: Friday, October 20, 2017 2:55 PM\n> > > To: Oleksandr Shamray \n> > > Cc: arnd@arndb.de; linux-kernel@vger.kernel.org; linux-arm-\n> > > kernel@lists.infradead.org; devicetree@vger.kernel.org;\n> > > openbmc@lists.ozlabs.org; joel@jms.id.au; jiri@resnulli.us;\n> > > tklauser@distanz.ch; linux-serial@vger.kernel.org; mec@shout.net;\n> > > Vadim Pasternak ; system-sw-low-level\n> > > ; robh+dt@kernel.org;\n> > > openocd-devel- owner@lists.sourceforge.net;\n> > > linux-api@vger.kernel.org; davem@davemloft.net; mchehab@kernel.org;\n> > > Jiri Pirko \n> > > Subject: Re: [patch v9 1/4] drivers: jtag: Add JTAG core driver\n> > >\n> > > On Thu, Sep 21, 2017 at 12:25:29PM +0300, Oleksandr Shamray wrote:\n> > > > +struct jtag {\n> > > > +\tstruct device *dev;\n> > > > +\tstruct cdev cdev;\n> > >\n> > > Why are you using a cdev here and not just a normal misc device?\n> >\n> > What the benefits to use misc instead of cdev?\n> \n> Less code, simpler logic, easier to review and understand, etc.\n> \n> Let me ask you, why use a cdev instead of a misc?\n\nAs I know misc device more applicable if we want to create one device f.e. /dev/jtag. \nBut in current case we can have more than one jtag device /dev/jtag0 ... /dev/jtagN. \nSo I decided to use cdev.\n\n> \n> > > I forgot if this is what you were doing before, sorry...\n> > >\n> > > > +\tint id;\n> > > > +\tatomic_t open;\n> > >\n> > > Why do you need this?\n> >\n> > This counter used to avoid open at the same time by 2 or more users.\n> \n> But it isn't working :)\n> \n> And why do you care?\n> \n> > > > +\tconst struct jtag_ops *ops;\n> > > > +\tunsigned long priv[0] __aligned(ARCH_DMA_MINALIGN);\n> > >\n> > > Huh? Why is this needed to be dma aligned? Why not just use the\n> > > private pointer in struct device?\n> > >\n> >\n> > It is critical?\n> \n> You are saying it is, so you have to justify it. There is a pointer for you to use,\n> don't make new ones for no reason, right?\n> \n\nYou are right. Will remove.\n\n> > > > +};\n> > > > +\n> > > > +static dev_t jtag_devt;\n> > > > +static DEFINE_IDA(jtag_ida);\n> > > > +\n> > > > +void *jtag_priv(struct jtag *jtag) {\n> > > > +\treturn jtag->priv;\n> > > > +}\n> > > > +EXPORT_SYMBOL_GPL(jtag_priv);\n> > > > +\n> > > > +static u8 *jtag_copy_from_user(__u64 udata, unsigned long bit_size) {\n> > > > +\tunsigned long size;\n> > > > +\tvoid *kdata;\n> > > > +\n> > > > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> > > > +\tkdata = memdup_user(u64_to_user_ptr(udata), size);\n> > >\n> > > You only use this once, why not just open-code it?\n> >\n> > I think it makes code more understandable.\n> \n> As a reviewer, I don't :)\n\nOk, I will fix :)\n\n> \n> > > > +\n> > > > +\treturn kdata;\n> > > > +}\n> > > > +\n> > > > +static unsigned long jtag_copy_to_user(__u64 udata, u8 *kdata,\n> > > > +\t\t\t\t unsigned long bit_size) {\n> > > > +\tunsigned long size;\n> > > > +\n> > > > +\tsize = DIV_ROUND_UP(bit_size, BITS_PER_BYTE);\n> > > > +\n> > > > +\treturn copy_to_user(u64_to_user_ptr(udata), (void *)(kdata),\n> > > > +size);\n> > >\n> > > Same here, making this a separate function seems odd.\n> >\n> > Same, I think it makes code more understandable.\n> \n> But it doesn't.\n> \n\nOk, I will fix :)\n\n> > > > +\n> > > > +\t\tif (jtag->ops->freq_set)\n> > > > +\t\t\terr = jtag->ops->freq_set(jtag, value);\n> > > > +\t\telse\n> > > > +\t\t\terr = -EOPNOTSUPP;\n> > > > +\t\tbreak;\n> > > > +\n> > > > +\tcase JTAG_IOCRUNTEST:\n> > > > +\t\tif (copy_from_user(&idle, (void *)arg,\n> > > > +\t\t\t\t sizeof(struct jtag_run_test_idle)))\n> > > > +\t\t\treturn -ENOMEM;\n> > > > +\t\terr = jtag_run_test_idle_op(jtag, &idle);\n> > >\n> > > Who validates the structure fields? Is that up to the individual\n> > > jtag driver? Why not do it in the core correctly so that it only\n> > > has to be done in one place and you do not have to audit every individual\n> driver?\n> >\n> > Input parameters validated by jtag platform driver. I think it not critical.\n> \n> Not true at all. It is very critical. Remmeber, \"All Input Is Evil!\"\n> \n> You have to validate this. I as a reviewer have to find where you are validating\n> this data to ensure bad things do not happen. I can't review that here, now I\n> have to go and review all of the individual drivers, which is a major pain, don't\n> you agree?\n\nAgree. I will add input parameter checking here before call device driver.\n\n> \n> > > > +\t\tbreak;\n> > > > +\n> > > > +\tcase JTAG_IOCXFER:\n> > > > +\t\tif (copy_from_user(&xfer, (void *)arg,\n> > > > +\t\t\t\t sizeof(struct jtag_xfer)))\n> > > > +\t\t\treturn -EFAULT;\n> > > > +\n> > > > +\t\tif (xfer.length >= JTAG_MAX_XFER_DATA_LEN)\n> > > > +\t\t\treturn -EFAULT;\n> > > > +\n> > > > +\t\txfer_data = jtag_copy_from_user(xfer.tdio, xfer.length);\n> > > > +\t\tif (!xfer_data)\n> > > > +\t\t\treturn -ENOMEM;\n> > >\n> > > Are you sure that's the correct error value?\n> >\n> > I think yes, but what you suggest?\n> \n> A fault happened, so -EFAULT, right?\n> \n\nRight.\n\n\n> > [..]\n> > > + .unlocked_ioctl = jtag_ioctl,\n> > > + .open = jtag_open,\n> > > + .release = jtag_release,\n> > > +};\n> >\n> > add a compat_ioctl pointer here, after ensuring that all ioctl\n> > commands are compatible between 32-bit and 64-bit user space.\n> > [..]\n> \n> And if you do not, what happens? You shouldn't need it as there is no fixups\n> necessary, or am I mistaken about that?\n\nYes, you are right. In code compat_ioctl called same function as in unlocked_ioctl. \nSo I can remove compat and system will always call unlocked_ioctl.\n\n> \n> > > > +static int jtag_open(struct inode *inode, struct file *file) {\n> > > > +\tstruct jtag *jtag = container_of(inode->i_cdev, struct jtag,\n> > > > +cdev);\n> > > > +\n> > > > +\tif (atomic_read(&jtag->open)) {\n> > > > +\t\tdev_info(NULL, \"jtag already opened\\n\");\n> > > > +\t\treturn -EBUSY;\n> > >\n> > > Why do you care if multiple opens can happen?\n> >\n> > Jtag HW not support to using with multiple requests from different users. So\n> we prohibit this.\n> \n> Why does the kernel care?\n> \n> And again, your implementation is broken, it's not actually doing this\n> protection. I recommend just not doing it at all, but if you really are insisting\n> on it, you have to get it correct :)\n\nI will follow your recommendations and remove it. \n\n> \n> thanks,\n> \n> greg k-h\n\nThanks. \nOleksandr S","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"XTq8QWDP\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"FqOiz4gd\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yMZbx15xrz9t2h\n\tfor ;\n\tThu, 26 Oct 2017 03:00:29 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e7O6T-00081m-Q7; Wed, 25 Oct 2017 16:00:25 +0000","from mail-db5eur01on0050.outbound.protection.outlook.com\n\t([104.47.2.50] helo=EUR01-DB5-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e7O5J-0006Db-Dv for linux-arm-kernel@lists.infradead.org;\n\tWed, 25 Oct 2017 15:59:32 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com (10.165.82.13) by\n\tHE1PR05MB1756.eurprd05.prod.outlook.com (10.169.120.26) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.156.4; Wed, 25 Oct 2017 15:58:42 +0000","from AM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::b02a:703f:24d5:1193]) by\n\tAM4PR0501MB2194.eurprd05.prod.outlook.com\n\t([fe80::b02a:703f:24d5:1193%18]) with mapi id 15.20.0156.007;\n\tWed, 25 Oct 2017 15:58:42 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References:\n\tMessage-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=NH7YmgtqIN1HWlSMRj+sAIFequM7G/009YQGGyu6pIk=;\n\tb=XTq8QWDPQ6TdN/\n\tsrCCRxcCyY0NcjJ7zLJfDkuJzdgchiDfDtASziaPmjRDSxhm3vmMzJ/8kR72d8V+9qtefRJ8Lziss\n\tyYq619xdI8xkuUCIi4TEKeDl9L0f3sRnvuq1T+dllHGI/TAEGXCmP596qbLux7ID8cnqgIQaxnnIt\n\tXdOGf4wuWX6jRbO+6etIEfrtzJMMUkk1KN7AHKyJn18WOJrx5LTBx4173sS5aTTQTzMF1usAhr1YT\n\tyKKynrt1EfpARcZIlhNvlxKBBTOOu6pGvRWVtzLTG3V5XIP5zQItb4Z2lVJV0naWAqKQYVzfTBIt0\n\tyI9nxbXeWXqUrIspdhRg==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;\n\ts=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=eNHgtBicbgOFkkBPNPWxWm+dfkXDdjiQxBKBrkoKKrY=;\n\tb=FqOiz4gd+GaM+X4AaoQ3041mwtEYGEaDEjkQvepZBrd8Nk2a+gr8LLD3UT8RYapFVKR1tjwTODKuDfkUcQuBk6YxyP6N6BhzlZuKVUAcHLzUOrx0S5XSX/8fg5t/m4F4Z9YF93pMTBQkzZesq16sSxMr1cGYX3xKbAxnb7wpx+U="],"From":"Oleksandr Shamray ","To":"Greg KH ","Subject":"RE: [patch v9 1/4] drivers: jtag: Add JTAG core driver","Thread-Topic":"[patch v9 1/4] drivers: jtag: Add JTAG core driver","Thread-Index":"AQHTMru5yoecjoOKkEeYBkPR/vITOKLszxoAgAAGBCCAACwFgIAH2fow","Date":"Wed, 25 Oct 2017 15:58:42 +0000","Message-ID":"","References":"<1505985932-27568-1-git-send-email-oleksandrs@mellanox.com>\n\t<1505985932-27568-2-git-send-email-oleksandrs@mellanox.com>\n\t<20171020115512.GA2073@kroah.com>\n\t\n\t<20171020145417.GD8965@kroah.com>","In-Reply-To":"<20171020145417.GD8965@kroah.com>","Accept-Language":"en-US, uk-UA","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"XTq8QWDP\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=Mellanox.com header.i=@Mellanox.com\n\theader.b=\"FqOiz4gd\"; dkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=oleksandrs@mellanox.com; "],"x-originating-ip":"[80.90.224.13]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; HE1PR05MB1756;\n\t6:6pH5gR+V5H6qUhYXMqJCd23d09OATvqlQjV2KB8WC8s4D54I0g3GmQAfO6fhP00lIdKZ2XfdmnKaGI8saaLJqdH7hzUJaNd61OCvjRusXofBIp81EnbOgZuM1A0derVxE8A6CQ1eq5YeRJW6K+ABrzEAGC1bZhmikU0cT8VClyEVFSQsGehKYjh4Q64cz7QabHavq+eWpcrMsRJfnoWlZGrLxLlVxvFKIGdtOr209Pf1Ho4IAw1oqa4/1swXb5S6PeJ8G/cA4D5bSgoWyI4No+Xy9fK7WjDgXrc578uyCb8cQ1jAVI8GBMf7Vp9JcgfIO6cZTmjpqp/pVQL8jt0+Kg==;\n\t5:I9Va1Ux3fo8b8NGHFmQhXUphjWkXwlvQ5dhT0VzBRsYbyXrx0WTSbIwZU4RI4T5NGbRlDCyttpewxsTKM/SUIaEmvzUN/Ckdj/pP7OZjdy+Pdp5qzoaGRtzUuI7f1N2aeDYH3uwLEkD0waxUjV0y6w==;\n\t24:ku3mxYI4JWZF0nR7+5Fin93gXi9RCgLx+AMTGIJVo+X2eFzzMV0wUYTTne/NeMP+OhsLZfiOYWKgqswXO7AJZr1jB2Yt56syBrnRVpgAcZs=;\n\t7:vrafflJEzH2R7GkQQdsi0pdy6xuWqomTSEDU5TV8JNDaAkjjftptTqSVh8gslTlbG/p5BMp9WI+Sx83rFVI7TZemnri6PYcUdkTZS93XTiDzZwA5xawFFS8+Khj+5P4oqD7oQQhvBdzHzewpEgPoi6NdnRRcOFa0MDFI5J059hkaR5jocXjn0SKUkLgHmPzaQcYsAFS7dgEBGUPGT8JiWmoqpHmRerkH0YBZXYF+UqU=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;SSOR;","x-ms-office365-filtering-correlation-id":"a93cf6bd-d920-4b6d-a2cb-08d51bc143e1","x-ms-office365-filtering-ht":"Tenant","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(22001)(48565401081)(4534020)(4602075)(4627075)(201703031133081)(201702281549075)(2017052603199);\n\tSRVR:HE1PR05MB1756; ","x-ms-traffictypediagnostic":"HE1PR05MB1756:","x-exchange-antispam-report-test":"UriScan:(143289334528602)(9452136761055)(65623756079841)(258649278758335)(42262312472803);","x-microsoft-antispam-prvs":"","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(93006095)(93001095)(100000703101)(100105400095)(3002001)(10201501046)(3231020)(6055026)(6041248)(20161123558100)(20161123555025)(20161123564025)(20161123562025)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:HE1PR05MB1756; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:HE1PR05MB1756; ","x-forefront-prvs":"0471B73328","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(39860400002)(376002)(346002)(199003)(189002)(24454002)(13464003)(105586002)(6436002)(74316002)(68736007)(2950100002)(6506006)(66066001)(54356999)(6916009)(14454004)(76176999)(25786009)(305945005)(229853002)(97736004)(3280700002)(50986999)(101416001)(2906002)(3660700001)(33656002)(86362001)(106356001)(7696004)(53546010)(5660300001)(7416002)(3846002)(102836003)(93886005)(2900100001)(7736002)(6116002)(55016002)(8676002)(9686003)(54906003)(107886003)(478600001)(53936002)(99286003)(4326008)(8936002)(81156014)(81166006)(5250100002)(6246003)(189998001)(316002);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR05MB1756;\n\tH:AM4PR0501MB2194.eurprd05.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; A:1; MX:1; LANG:en; ","received-spf":"None (protection.outlook.com: mellanox.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-OriginatorOrg":"Mellanox.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"a93cf6bd-d920-4b6d-a2cb-08d51bc143e1","X-MS-Exchange-CrossTenant-originalarrivaltime":"25 Oct 2017 15:58:42.2729\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"a652971c-7d2e-4d9b-a6a4-d149256f461b","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"HE1PR05MB1756","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171025_085914_169744_E0D271F6 ","X-CRM114-Status":"GOOD ( 29.96 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3)\n\t[104.47.2.50 listed in wl.mailspike.net]\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [104.47.2.50 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 SPF_HELO_PASS SPF: HELO matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's\n\tdomain -0.0 RCVD_IN_MSPIKE_WL Mailspike good senders","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"system-sw-low-level ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"jiri@resnulli.us\" , \"arnd@arndb.de\" ,\n\tVadim Pasternak ,\n\t\"linux-api@vger.kernel.org\" ,\n\t\"openbmc@lists.ozlabs.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"openocd-devel-owner@lists.sourceforge.net\"\n\t,\n\t\"mec@shout.net\" , Jiri Pirko ,\n\t\"robh+dt@kernel.org\" , \n\t\"joel@jms.id.au\" ,\n\t\"linux-serial@vger.kernel.org\" ,\n\t\"tklauser@distanz.ch\" ,\n\t\"mchehab@kernel.org\" ,\n\t\"davem@davemloft.net\" ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]