diff for duplicates of <20160927201628.GB9084@lukather> diff --git a/a/1.txt b/N1/1.txt index 2251bb6..f9b1d35 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -20,14 +20,14 @@ spin_lock_irqsave, if possible, yes. > thought it through yet, let alone tested it: > > -> + if (!(sun4i_pwm->data->has_rdy)) -> + ndelay(pwm_get_period(pwm)); -> + else -> + do { -> + spin_lock(&sun4i_pwm->ctrl_lock); -> + val = sun4i_pwm_readl(sun4i_pwm, PWM_CTRL_REG); -> + spin_unlock(&sun4i_pwm->ctrl_lock); -> + } while (!(val & PWM_RDY(pwm->hwpwm))) +> +???????if (!(sun4i_pwm->data->has_rdy)) +> +???????????????ndelay(pwm_get_period(pwm)); +> +???????else +> +???????????????do { +> +???????????????????????spin_lock(&sun4i_pwm->ctrl_lock); +> +???????????????????????val = sun4i_pwm_readl(sun4i_pwm, PWM_CTRL_REG); +> +???????????????????????spin_unlock(&sun4i_pwm->ctrl_lock); +> +???????????????} while (!(val & PWM_RDY(pwm->hwpwm))) > > Here I assumed the spin_lock is cheap to make, expensive to hold for > long, e.g. reducing the length the spin-lock is active for. the @@ -45,3 +45,10 @@ Maxime Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com +-------------- next part -------------- +A non-text attachment was scrubbed... +Name: signature.asc +Type: application/pgp-signature +Size: 819 bytes +Desc: not available +URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160927/72cc3f3f/attachment.sig> diff --git a/a/2.bin b/a/2.bin deleted file mode 100644 index f727a3f..0000000 --- a/a/2.bin +++ /dev/null @@ -1,17 +0,0 @@ ------BEGIN PGP SIGNATURE----- -Version: GnuPG v1 - -iQIcBAEBAgAGBQJX6tOcAAoJEBx+YmzsjxAg+0IP/31GX3LiF4mrYkdrg3g6ZcYN -OUHFEXXQXxWErXwJYfq2Q/RwwBEk4f6IgH4dKar73ULjvZTua6dx65s1fNG+CVkU -wbAc4SXv/OrB6meLf0rI1ylWJCADJgNEyzCJuDsPZNx76GcdL3v5jtEOJXpg3/+f -CpJirvKanf5wAxw0W8xTnvNsYrHosMquP585idKYZhArxB8O0L9GLbWWP/qE1mzQ -fhxY7S+um1F+pp1I6+35J2tFE58W5QxmFsW7O+q4wu1dVD3d4wZqMXy5lY1RKmeZ -bItAmwyt9gLvPTzXrGseRtiKMvYsxWYHzI8srJWdeRinwtL+MTxZJ3pFPLu9QDnR -rchP80+kGDFFsDotq3Yui+RfxLhHztFk7WNTNRewoECnnYYjDt/Xwpie+R4TBIIa -ctx8/jQ8trH+bJvGOGHbAphptszEv5j8et/3aAYLDcjKAtGWQhCVdCNL2sWI0mvp -kYk/SPHKFrZ7W/sRX7Jpnid5tScaAKjt97owwmxhhqhtXabAN944WJT3Y51+gCrm -5b6wBsAuIwnSK9YL70LSfuz6PDLjghrc98DZBsgejPnbpCzylNVUXOg41+arCx3D -hxsyLN3NTCD3X3jX/gb5ryoOEw35CBzsrwRzLOmPS1u/j1QCuikam+5dMf0DRhLy -FSA8yJd6DdowZOlLBmFQ -=AiHG ------END PGP SIGNATURE----- diff --git a/a/2.hdr b/a/2.hdr deleted file mode 100644 index 5e5352c..0000000 --- a/a/2.hdr +++ /dev/null @@ -1 +0,0 @@ -Content-Type: application/pgp-signature; name="signature.asc" diff --git a/a/content_digest b/N1/content_digest index 2f73640..cfc8206 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,17 +6,11 @@ "ref\01473411668.731.75.camel@schinagl.nl\0" "ref\020160924202502.GF16901@lukather\0" "ref\01474879585.6096.33.camel@schinagl.nl\0" - "From\0Maxime Ripard <maxime.ripard@free-electrons.com>\0" - "Subject\0Re: [PATCH 1/2] pwm: sunxi: allow the pwm to finish its pulse before disable\0" + "From\0maxime.ripard@free-electrons.com (Maxime Ripard)\0" + "Subject\0[PATCH 1/2] pwm: sunxi: allow the pwm to finish its pulse before disable\0" "Date\0Tue, 27 Sep 2016 22:16:28 +0200\0" - "To\0Olliver Schinagl <oliver@schinagl.nl>\0" - "Cc\0Alexandre Belloni <alexandre.belloni@free-electrons.com>" - Thierry Reding <thierry.reding@gmail.com> - Chen-Yu Tsai <wens@csie.org> - linux-pwm@vger.kernel.org - linux-arm-kernel@lists.infradead.org - " linux-kernel@vger.kernel.org\0" - "\01:1\0" + "To\0linux-arm-kernel@lists.infradead.org\0" + "\00:1\0" "b\0" "Hi,\n" "\n" @@ -40,14 +34,14 @@ "> thought it through yet, let alone tested it:\n" "> \n" "> \n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240if (!(sun4i_pwm->data->has_rdy))\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240ndelay(pwm_get_period(pwm));\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240else\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240do {\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240spin_lock(&sun4i_pwm->ctrl_lock);\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240val = sun4i_pwm_readl(sun4i_pwm, PWM_CTRL_REG);\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240spin_unlock(&sun4i_pwm->ctrl_lock);\n" - "> +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240} while (!(val & PWM_RDY(pwm->hwpwm)))\n" + "> +???????if (!(sun4i_pwm->data->has_rdy))\n" + "> +???????????????ndelay(pwm_get_period(pwm));\n" + "> +???????else\n" + "> +???????????????do {\n" + "> +???????????????????????spin_lock(&sun4i_pwm->ctrl_lock);\n" + "> +???????????????????????val = sun4i_pwm_readl(sun4i_pwm, PWM_CTRL_REG);\n" + "> +???????????????????????spin_unlock(&sun4i_pwm->ctrl_lock);\n" + "> +???????????????} while (!(val & PWM_RDY(pwm->hwpwm)))\n" "> \n" "> Here I assumed the spin_lock is cheap to make, expensive to hold for\n" "> long, e.g. reducing the length the spin-lock is active for. the\n" @@ -64,26 +58,13 @@ "-- \n" "Maxime Ripard, Free Electrons\n" "Embedded Linux and Kernel engineering\n" - http://free-electrons.com - "\01:2\0" - "fn\0signature.asc\0" - "b\0" - "-----BEGIN PGP SIGNATURE-----\n" - "Version: GnuPG v1\n" - "\n" - "iQIcBAEBAgAGBQJX6tOcAAoJEBx+YmzsjxAg+0IP/31GX3LiF4mrYkdrg3g6ZcYN\n" - "OUHFEXXQXxWErXwJYfq2Q/RwwBEk4f6IgH4dKar73ULjvZTua6dx65s1fNG+CVkU\n" - "wbAc4SXv/OrB6meLf0rI1ylWJCADJgNEyzCJuDsPZNx76GcdL3v5jtEOJXpg3/+f\n" - "CpJirvKanf5wAxw0W8xTnvNsYrHosMquP585idKYZhArxB8O0L9GLbWWP/qE1mzQ\n" - "fhxY7S+um1F+pp1I6+35J2tFE58W5QxmFsW7O+q4wu1dVD3d4wZqMXy5lY1RKmeZ\n" - "bItAmwyt9gLvPTzXrGseRtiKMvYsxWYHzI8srJWdeRinwtL+MTxZJ3pFPLu9QDnR\n" - "rchP80+kGDFFsDotq3Yui+RfxLhHztFk7WNTNRewoECnnYYjDt/Xwpie+R4TBIIa\n" - "ctx8/jQ8trH+bJvGOGHbAphptszEv5j8et/3aAYLDcjKAtGWQhCVdCNL2sWI0mvp\n" - "kYk/SPHKFrZ7W/sRX7Jpnid5tScaAKjt97owwmxhhqhtXabAN944WJT3Y51+gCrm\n" - "5b6wBsAuIwnSK9YL70LSfuz6PDLjghrc98DZBsgejPnbpCzylNVUXOg41+arCx3D\n" - "hxsyLN3NTCD3X3jX/gb5ryoOEw35CBzsrwRzLOmPS1u/j1QCuikam+5dMf0DRhLy\n" - "FSA8yJd6DdowZOlLBmFQ\n" - "=AiHG\n" - "-----END PGP SIGNATURE-----\n" + "http://free-electrons.com\n" + "-------------- next part --------------\n" + "A non-text attachment was scrubbed...\n" + "Name: signature.asc\n" + "Type: application/pgp-signature\n" + "Size: 819 bytes\n" + "Desc: not available\n" + URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160927/72cc3f3f/attachment.sig> -376a44eabce25b773906b76b0e14e0c3d99f8c3434c5767a16ec6e8ce0f62f9f +9edbb8be093cd946cb325613c894835e2fef9b9888553dbd81f779ea0ffd5f01
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.