From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> To: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, Tejun Heo <tj@kernel.org> Cc: Sekhar Nori <nsekhar@ti.com>, Kevin Hilman <khilman@baylibre.com>, Arnd Bergmann <arnd@arndb.de>, Russell King <linux@arm.linux.org.uk>, Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>, linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 1/5] ata: add Palmchip BK3710 PATA controller driver Date: Thu, 23 Mar 2017 13:37:44 +0300 [thread overview] Message-ID: <103c435e-206c-aaaa-59ba-1eb2a4f77780@cogentembedded.com> (raw) In-Reply-To: <1490206862-26065-2-git-send-email-b.zolnierkie@samsung.com> Hello! On 3/22/2017 9:20 PM, Bartlomiej Zolnierkiewicz wrote: > Add Palmchip BK3710 PATA controller driver. > > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> [...] > diff --git a/drivers/ata/pata_bk3710.c b/drivers/ata/pata_bk3710.c > new file mode 100644 > index 0000000..1a1223f > --- /dev/null > +++ b/drivers/ata/pata_bk3710.c > @@ -0,0 +1,382 @@ [...] > +static int __init pata_bk3710_probe(struct platform_device *pdev) > +{ > + struct clk *clk; > + struct resource *mem; > + struct ata_host *host; > + struct ata_port *ap; > + void __iomem *base; > + unsigned long rate; > + int irq; > + > + clk = devm_clk_get(&pdev->dev, NULL); > + if (IS_ERR(clk)) > + return -ENODEV; > + > + clk_enable(clk); > + rate = clk_get_rate(clk); > + if (!rate) > + return -EINVAL; > + > + /* NOTE: round *down* to meet minimum timings; we count in clocks */ > + ideclk_period = 1000000000UL / rate; > + > + mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); I'd put this close to devm_iorempe_resource() and would probably even get rid of 'mem' altogether... [...] Acked-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> MBR, Sergei
WARNING: multiple messages have this Message-ID (diff)
From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 1/5] ata: add Palmchip BK3710 PATA controller driver Date: Thu, 23 Mar 2017 13:37:44 +0300 [thread overview] Message-ID: <103c435e-206c-aaaa-59ba-1eb2a4f77780@cogentembedded.com> (raw) In-Reply-To: <1490206862-26065-2-git-send-email-b.zolnierkie@samsung.com> Hello! On 3/22/2017 9:20 PM, Bartlomiej Zolnierkiewicz wrote: > Add Palmchip BK3710 PATA controller driver. > > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> [...] > diff --git a/drivers/ata/pata_bk3710.c b/drivers/ata/pata_bk3710.c > new file mode 100644 > index 0000000..1a1223f > --- /dev/null > +++ b/drivers/ata/pata_bk3710.c > @@ -0,0 +1,382 @@ [...] > +static int __init pata_bk3710_probe(struct platform_device *pdev) > +{ > + struct clk *clk; > + struct resource *mem; > + struct ata_host *host; > + struct ata_port *ap; > + void __iomem *base; > + unsigned long rate; > + int irq; > + > + clk = devm_clk_get(&pdev->dev, NULL); > + if (IS_ERR(clk)) > + return -ENODEV; > + > + clk_enable(clk); > + rate = clk_get_rate(clk); > + if (!rate) > + return -EINVAL; > + > + /* NOTE: round *down* to meet minimum timings; we count in clocks */ > + ideclk_period = 1000000000UL / rate; > + > + mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); I'd put this close to devm_iorempe_resource() and would probably even get rid of 'mem' altogether... [...] Acked-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> MBR, Sergei
next prev parent reply other threads:[~2017-03-23 10:37 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <CGME20170322182127epcas1p267b13e8c92205223458863783bdea476@epcas1p2.samsung.com> 2017-03-22 18:20 ` [PATCH v3 0/5] ATA/ARM: convert ARM/DaVinci to use libata PATA drivers Bartlomiej Zolnierkiewicz 2017-03-22 18:20 ` Bartlomiej Zolnierkiewicz [not found] ` <CGME20170322182131epcas1p26b2402ec94ca08d2a6a332579a65e269@epcas1p2.samsung.com> 2017-03-22 18:20 ` [PATCH v3 1/5] ata: add Palmchip BK3710 PATA controller driver Bartlomiej Zolnierkiewicz 2017-03-22 18:20 ` Bartlomiej Zolnierkiewicz 2017-03-23 10:37 ` Sergei Shtylyov [this message] 2017-03-23 10:37 ` Sergei Shtylyov [not found] ` <CGME20170322182135epcas5p3d21d7301da84a5dcb686af4f6817ded9@epcas5p3.samsung.com> 2017-03-22 18:20 ` [PATCH v3 2/5] pata_bk3710: disable IORDY Timer on chipset initialization Bartlomiej Zolnierkiewicz 2017-03-22 18:20 ` Bartlomiej Zolnierkiewicz 2017-03-23 10:38 ` Sergei Shtylyov 2017-03-23 10:38 ` Sergei Shtylyov 2017-03-23 10:38 ` Sergei Shtylyov [not found] ` <CGME20170322182139epcas5p32d912444c0be8747565a4979f8744572@epcas5p3.samsung.com> 2017-03-22 18:21 ` [PATCH v3 3/5] pata_bk3710: clear status bits of BMISP " Bartlomiej Zolnierkiewicz 2017-03-22 18:21 ` Bartlomiej Zolnierkiewicz 2017-03-23 10:39 ` Sergei Shtylyov 2017-03-23 10:39 ` Sergei Shtylyov [not found] ` <CGME20170322182143epcas5p3ed7e25912e24ae67b5593d2baaf63791@epcas5p3.samsung.com> 2017-03-22 18:21 ` [PATCH v3 4/5] ARM: davinci: add pata_bk3710 libata driver support Bartlomiej Zolnierkiewicz 2017-03-22 18:21 ` Bartlomiej Zolnierkiewicz [not found] ` <CGME20170322182147epcas5p3037b4fd5aaf82741ae14353df7d8789d@epcas5p3.samsung.com> 2017-03-22 18:21 ` [PATCH v3 5/5] ARM: davinci_all_defconfig: convert to use libata PATA Bartlomiej Zolnierkiewicz 2017-03-22 18:21 ` Bartlomiej Zolnierkiewicz 2017-03-22 18:21 ` Bartlomiej Zolnierkiewicz 2017-03-23 13:57 ` [PATCH v3 0/5] ATA/ARM: convert ARM/DaVinci to use libata PATA drivers Sekhar Nori 2017-03-23 13:57 ` Sekhar Nori 2017-03-23 13:57 ` Sekhar Nori 2017-03-24 15:38 ` Bartlomiej Zolnierkiewicz 2017-03-24 15:38 ` Bartlomiej Zolnierkiewicz 2017-03-29 15:35 ` Tejun Heo 2017-03-29 15:35 ` Tejun Heo 2017-03-29 15:35 ` Tejun Heo 2017-03-30 11:53 ` Sekhar Nori 2017-03-30 11:53 ` Sekhar Nori 2017-03-30 11:53 ` Sekhar Nori
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=103c435e-206c-aaaa-59ba-1eb2a4f77780@cogentembedded.com \ --to=sergei.shtylyov@cogentembedded.com \ --cc=arnd@arndb.de \ --cc=b.zolnierkie@samsung.com \ --cc=dbaryshkov@gmail.com \ --cc=khilman@baylibre.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-ide@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=nsekhar@ti.com \ --cc=tj@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.