linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ulf.hansson@linaro.org (Ulf Hansson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 19/33] clk: ux500: Add Device Tree support for the PRCMU clock
Date: Mon, 10 Jun 2013 23:19:58 +0200	[thread overview]
Message-ID: <CAPDyKFozMRxcpeEfaqKQrZ1_xxr=UMLw2EybdKiRM2zWcppikw@mail.gmail.com> (raw)
In-Reply-To: <1370521041-32318-20-git-send-email-lee.jones@linaro.org>

On 6 June 2013 14:17, Lee Jones <lee.jones@linaro.org> wrote:
> This patch enables clocks to be specified from Device Tree via phandles
> to the "prcmu-clock" node.
>
> Cc: Mike Turquette <mturquette@linaro.org>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
>  drivers/clk/ux500/u8500_clk.c |   50 +++++++++++++++++++++++++++++++++++++++--
>  1 file changed, 48 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/clk/ux500/u8500_clk.c b/drivers/clk/ux500/u8500_clk.c
> index 3a7040b..4f5ad4c 100644
> --- a/drivers/clk/ux500/u8500_clk.c
> +++ b/drivers/clk/ux500/u8500_clk.c
> @@ -15,6 +15,8 @@
>  #include <linux/platform_data/clk-ux500.h>
>  #include "clk.h"
>
> +static struct clk *prcmu_clk[PRCMU_NUM_CLKS];
> +
>  struct clk *ux500_twocell_get(struct of_phandle_args *clkspec, void *data)

Actually I thought ux500_twocell_get was going to be used in this
patch, since the previous one was adding this helper function, but it
isn't?

Kind regards
Ulf Hansson

>  {
>         struct clk **clk_data = data;
> @@ -52,14 +54,17 @@ void u8500_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
>         clk = clk_reg_prcmu_gate("soc0_pll", NULL, PRCMU_PLLSOC0,
>                                 CLK_IS_ROOT|CLK_IGNORE_UNUSED);
>         clk_register_clkdev(clk, "soc0_pll", NULL);
> +       prcmu_clk[PRCMU_PLLSOC0] = clk;
>
>         clk = clk_reg_prcmu_gate("soc1_pll", NULL, PRCMU_PLLSOC1,
>                                 CLK_IS_ROOT|CLK_IGNORE_UNUSED);
>         clk_register_clkdev(clk, "soc1_pll", NULL);
> +       prcmu_clk[PRCMU_PLLSOC1] = clk;
>
>         clk = clk_reg_prcmu_gate("ddr_pll", NULL, PRCMU_PLLDDR,
>                                 CLK_IS_ROOT|CLK_IGNORE_UNUSED);
>         clk_register_clkdev(clk, "ddr_pll", NULL);
> +       prcmu_clk[PRCMU_PLLDDR] = clk;
>
>         /* FIXME: Add sys, ulp and int clocks here. */
>
> @@ -90,65 +95,84 @@ void u8500_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
>                 clk = clk_reg_prcmu_gate("sgclk", NULL,
>                                         PRCMU_SGACLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "mali");
> +       prcmu_clk[PRCMU_SGACLK] = clk;
>
>         clk = clk_reg_prcmu_gate("uartclk", NULL, PRCMU_UARTCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "UART");
> +       prcmu_clk[PRCMU_UARTCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("msp02clk", NULL, PRCMU_MSP02CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "MSP02");
> +       prcmu_clk[PRCMU_MSP02CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("msp1clk", NULL, PRCMU_MSP1CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "MSP1");
> +       prcmu_clk[PRCMU_MSP1CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("i2cclk", NULL, PRCMU_I2CCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "I2C");
> +       prcmu_clk[PRCMU_I2CCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("slimclk", NULL, PRCMU_SLIMCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "slim");
> +       prcmu_clk[PRCMU_SLIMCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per1clk", NULL, PRCMU_PER1CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH1");
> +       prcmu_clk[PRCMU_PER1CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per2clk", NULL, PRCMU_PER2CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH2");
> +       prcmu_clk[PRCMU_PER2CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per3clk", NULL, PRCMU_PER3CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH3");
> +       prcmu_clk[PRCMU_PER3CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per5clk", NULL, PRCMU_PER5CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH5");
> +       prcmu_clk[PRCMU_PER5CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per6clk", NULL, PRCMU_PER6CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH6");
> +       prcmu_clk[PRCMU_PER6CLK] = clk;
>
>         clk = clk_reg_prcmu_gate("per7clk", NULL, PRCMU_PER7CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "PERIPH7");
> +       prcmu_clk[PRCMU_PER7CLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("lcdclk", NULL, PRCMU_LCDCLK, 0,
>                                 CLK_IS_ROOT|CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, NULL, "lcd");
>         clk_register_clkdev(clk, "lcd", "mcde");
> +       prcmu_clk[PRCMU_LCDCLK] = clk;
>
>         clk = clk_reg_prcmu_opp_gate("bmlclk", NULL, PRCMU_BMLCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "bml");
> +       prcmu_clk[PRCMU_BMLCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("hsitxclk", NULL, PRCMU_HSITXCLK, 0,
>                                 CLK_IS_ROOT|CLK_SET_RATE_GATE);
> +       prcmu_clk[PRCMU_HSITXCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("hsirxclk", NULL, PRCMU_HSIRXCLK, 0,
>                                 CLK_IS_ROOT|CLK_SET_RATE_GATE);
> +       prcmu_clk[PRCMU_HSIRXCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("hdmiclk", NULL, PRCMU_HDMICLK, 0,
>                                 CLK_IS_ROOT|CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, NULL, "hdmi");
>         clk_register_clkdev(clk, "hdmi", "mcde");
> +       prcmu_clk[PRCMU_HDMICLK] = clk;
>
>         clk = clk_reg_prcmu_gate("apeatclk", NULL, PRCMU_APEATCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "apeat");
> +       prcmu_clk[PRCMU_APEATCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("apetraceclk", NULL, PRCMU_APETRACECLK,
>                                 CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "apetrace");
> +       prcmu_clk[PRCMU_APETRACECLK] = clk;
>
>         clk = clk_reg_prcmu_gate("mcdeclk", NULL, PRCMU_MCDECLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "mcde");
> @@ -156,76 +180,92 @@ void u8500_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
>         clk_register_clkdev(clk, "dsisys", "dsilink.0");
>         clk_register_clkdev(clk, "dsisys", "dsilink.1");
>         clk_register_clkdev(clk, "dsisys", "dsilink.2");
> +       prcmu_clk[PRCMU_MCDECLK] = clk;
>
>         clk = clk_reg_prcmu_opp_gate("ipi2cclk", NULL, PRCMU_IPI2CCLK,
>                                 CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "ipi2");
> +       prcmu_clk[PRCMU_IPI2CCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("dsialtclk", NULL, PRCMU_DSIALTCLK,
>                                 CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "dsialt");
> +       prcmu_clk[PRCMU_DSIALTCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("dmaclk", NULL, PRCMU_DMACLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "dma40.0");
> +       prcmu_clk[PRCMU_DMACLK] = clk;
>
>         clk = clk_reg_prcmu_gate("b2r2clk", NULL, PRCMU_B2R2CLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "b2r2");
>         clk_register_clkdev(clk, NULL, "b2r2_core");
>         clk_register_clkdev(clk, NULL, "U8500-B2R2.0");
> +       prcmu_clk[PRCMU_B2R2CLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("tvclk", NULL, PRCMU_TVCLK, 0,
>                                 CLK_IS_ROOT|CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, NULL, "tv");
>         clk_register_clkdev(clk, "tv", "mcde");
> +       prcmu_clk[PRCMU_TVCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("sspclk", NULL, PRCMU_SSPCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "SSP");
> +       prcmu_clk[PRCMU_SSPCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("rngclk", NULL, PRCMU_RNGCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "rngclk");
> +       prcmu_clk[PRCMU_RNGCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("uiccclk", NULL, PRCMU_UICCCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "uicc");
> +       prcmu_clk[PRCMU_UICCCLK] = clk;
>
>         clk = clk_reg_prcmu_gate("timclk", NULL, PRCMU_TIMCLK, CLK_IS_ROOT);
>         clk_register_clkdev(clk, NULL, "mtu0");
>         clk_register_clkdev(clk, NULL, "mtu1");
> +       prcmu_clk[PRCMU_TIMCLK] = clk;
>
>         clk = clk_reg_prcmu_opp_volt_scalable("sdmmcclk", NULL, PRCMU_SDMMCCLK,
>                                         100000000,
>                                         CLK_IS_ROOT|CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, NULL, "sdmmc");
> +       prcmu_clk[PRCMU_SDMMCCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi_pll", "hdmiclk",
>                                 PRCMU_PLLDSI, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsihs2", "mcde");
>         clk_register_clkdev(clk, "dsihs2", "dsilink.2");
> -
> +       prcmu_clk[PRCMU_PLLDSI] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi0clk", "dsi_pll",
>                                 PRCMU_DSI0CLK, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsihs0", "mcde");
>         clk_register_clkdev(clk, "dsihs0", "dsilink.0");
> +       prcmu_clk[PRCMU_DSI0CLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi1clk", "dsi_pll",
>                                 PRCMU_DSI1CLK, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsihs1", "mcde");
>         clk_register_clkdev(clk, "dsihs1", "dsilink.1");
> +       prcmu_clk[PRCMU_DSI1CLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi0escclk", "tvclk",
>                                 PRCMU_DSI0ESCCLK, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsilp0", "dsilink.0");
>         clk_register_clkdev(clk, "dsilp0", "mcde");
> +       prcmu_clk[PRCMU_DSI0ESCCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi1escclk", "tvclk",
>                                 PRCMU_DSI1ESCCLK, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsilp1", "dsilink.1");
>         clk_register_clkdev(clk, "dsilp1", "mcde");
> +       prcmu_clk[PRCMU_DSI1ESCCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable("dsi2escclk", "tvclk",
>                                 PRCMU_DSI2ESCCLK, 0, CLK_SET_RATE_GATE);
>         clk_register_clkdev(clk, "dsilp2", "dsilink.2");
>         clk_register_clkdev(clk, "dsilp2", "mcde");
> +       prcmu_clk[PRCMU_DSI2ESCCLK] = clk;
>
>         clk = clk_reg_prcmu_scalable_rate("armss", NULL,
>                                 PRCMU_ARMSS, 0, CLK_IS_ROOT|CLK_IGNORE_UNUSED);
> @@ -556,6 +596,12 @@ void u8500_clk_init(u32 clkrst1_base, u32 clkrst2_base, u32 clkrst3_base,
>                 return;
>
>         for_each_child_of_node(np, child) {
> -               /* Place holder for supported nodes. */
> +               static struct clk_onecell_data clk_data;
> +
> +               if (!of_node_cmp(child->name, "prcmu-clock")) {
> +                       clk_data.clks = prcmu_clk;
> +                       clk_data.clk_num = ARRAY_SIZE(prcmu_clk);
> +                       of_clk_add_provider(child, of_clk_src_onecell_get, &clk_data);
> +               }
>         }
>  }
> --
> 1.7.10.4
>

  reply	other threads:[~2013-06-10 21:19 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-06 12:16 [PATCH 00/32] ARM: ux500: Enable clocks for Device Tree Lee Jones
2013-06-06 12:16 ` [PATCH 01/33] mfd: dbx500-prcmu: Provide PRCMU numerical clock identifiers Lee Jones
2013-06-06 12:16 ` [PATCH 02/33] ARM: ux500: Add PRCMU clock node to DBx500 Device Tree Lee Jones
2013-06-06 12:16 ` [PATCH 03/33] ARM: ux500: Supply the DMA clock lookup to the DBX500 DT Lee Jones
2013-06-06 12:16 ` [PATCH 04/33] ARM: ux500: Add PRCC Peripheral clock node to DBx500 Device Tree Lee Jones
2013-06-06 12:16 ` [PATCH 05/33] ARM: ux500: Supply the GPIO clocks lookup to the DBX500 DT Lee Jones
2013-06-06 12:16 ` [PATCH 06/33] ARM: ux500: Supply the USB clock " Lee Jones
2013-06-06 12:16 ` [PATCH 07/33] ARM: ux500: Supply the Ethernet clock lookup to Snowball's DT Lee Jones
2013-06-06 12:16 ` [PATCH 08/33] ARM: ux500: Add PRCC Kernel clock node to DBx500 Device Tree Lee Jones
2013-06-06 12:16 ` [PATCH 09/33] ARM: ux500: Supply the I2C clocks lookup to the DBX500 DT Lee Jones
2013-08-20  9:11   ` Linus Walleij
2013-08-20  9:30     ` Sascha Hauer
2013-08-22 13:37       ` Mark Rutland
2013-08-22 13:49         ` Lee Jones
2013-08-22 14:19         ` Lee Jones
2013-08-22 15:17           ` Mark Rutland
2013-08-22 15:41             ` Lee Jones
2013-08-22 16:04               ` Mark Rutland
2013-08-22 21:19               ` Sascha Hauer
2013-08-23  7:56                 ` Lee Jones
2013-08-23 16:55                   ` Mark Rutland
2013-08-27  8:06                     ` Lee Jones
2013-08-27 13:46                       ` Mark Rutland
2013-08-27 14:08                         ` Lee Jones
2013-08-27 15:51                           ` Rob Herring
2013-08-27 16:15                             ` Pawel Moll
2013-08-21  8:28     ` Lee Jones
2013-08-21 22:44       ` Linus Walleij
2013-08-22  9:23     ` Lee Jones
2013-06-06 12:16 ` [PATCH 10/33] ARM: ux500: Supply the UART " Lee Jones
2013-06-06 12:16 ` [PATCH 11/33] ARM: ux500: Supply the SDI (MMC) " Lee Jones
2013-06-06 12:17 ` [PATCH 12/33] ARM: ux500: Supply the MSP (Audio) " Lee Jones
2013-06-06 12:17 ` [PATCH 13/33] ARM: ux500: Add RTC (fixed-frequency) clock node to DBx500 Device Tree Lee Jones
2013-06-06 12:17 ` [PATCH 14/33] ARM: ux500: Supply the RTC clock lookup to the DBX500 DT Lee Jones
2013-06-06 12:17 ` [PATCH 15/33] ARM: ux500: Add TWD (fixed-factor) clock node to DBx500 Device Tree Lee Jones
2013-06-06 12:17 ` [PATCH 16/33] ARM: ux500: Supply the TWD Timer clock lookup to the DBX500 DT Lee Jones
2013-06-06 12:17 ` [PATCH 17/33] clk: ux500: Provide u8500_clk with skeleton Device Tree support Lee Jones
2013-06-06 12:17 ` [PATCH 18/33] clk: ux500: Add a 2-cell Device Tree parser for obtaining PRCC clocks Lee Jones
2013-06-10 20:54   ` Ulf Hansson
2013-06-11  9:12     ` Lee Jones
2013-06-11 11:07   ` [PATCH 18/33 v2] " Lee Jones
2013-06-06 12:17 ` [PATCH 19/33] clk: ux500: Add Device Tree support for the PRCMU clock Lee Jones
2013-06-10 21:19   ` Ulf Hansson [this message]
2013-06-11 11:10     ` Lee Jones
2013-06-11 11:12       ` Lee Jones
2013-08-21  8:23   ` Linus Walleij
2013-08-21 10:10     ` Lee Jones
2013-06-06 12:17 ` [PATCH 20/33] clk: ux500: Add Device Tree support for the PRCC Peripheral clock Lee Jones
2013-06-11 11:51   ` [PATCH 20/33 v2] " Lee Jones
2013-06-06 12:17 ` [PATCH 21/33] clk: ux500: Add Device Tree support for the PRCC Kernel clock Lee Jones
2013-06-10 21:24   ` Ulf Hansson
2013-06-11  9:10     ` Lee Jones
2013-06-11 11:09   ` [PATCH 21/33 v2] " Lee Jones
2013-06-12 14:46     ` Arnd Bergmann
2013-06-18 21:17       ` Mike Turquette
2013-06-19  7:42         ` Lee Jones
2013-06-21 18:20           ` Mike Turquette
2013-08-21  8:17   ` [PATCH 21/33] " Linus Walleij
2013-08-21 10:14     ` Lee Jones
2013-08-21 22:46       ` Linus Walleij
2013-08-22  9:21         ` Lee Jones
2013-08-23 18:01           ` Linus Walleij
2013-08-24  8:00             ` Arnd Bergmann
2013-08-24 21:19               ` Linus Walleij
2013-08-27  8:23             ` Lee Jones
2013-09-12 12:50               ` Linus Walleij
2013-09-12 14:56                 ` Lee Jones
2013-09-13  7:20                   ` Linus Walleij
2013-06-06 12:17 ` [PATCH 22/33] clk: ux500: Add Device Tree support for the RTC clock Lee Jones
2013-06-06 12:17 ` [PATCH 23/33] clk: ux500: Add Device Tree support for the TWD clock Lee Jones
2013-06-06 12:17 ` [PATCH 24/33] ARM: ux500: Remove AUXDATA relating to GPIO clock-name bindings Lee Jones
2013-06-06 12:17 ` [PATCH 25/33] ARM: ux500: Remove AUXDATA relating to UART " Lee Jones
2013-06-06 12:17 ` [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) " Lee Jones
2013-08-23 13:31   ` Linus Walleij
2013-08-23 14:45     ` Lee Jones
2013-08-24  7:57     ` Arnd Bergmann
2013-08-27  8:11       ` Lee Jones
2013-06-06 12:17 ` [PATCH 27/33] ARM: ux500: Remove AUXDATA relating to I2C " Lee Jones
2013-06-06 12:17 ` [PATCH 28/33] ARM: ux500: Remove AUXDATA relating to MSP (Audio) " Lee Jones
2013-08-21  8:08   ` Linus Walleij
2013-08-21  8:17     ` Lee Jones
2013-06-06 12:17 ` [PATCH 29/33] ARM: ux500: Remove AUXDATA relating to USB " Lee Jones
2013-06-06 12:17 ` [PATCH 30/33] ARM: ux500: Remove AUXDATA relating to Ethernet " Lee Jones
2013-06-06 12:17 ` [PATCH 31/33] ARM: ux500: Remove AUXDATA relating to DMA " Lee Jones
2013-06-06 12:17 ` [PATCH 32/33] ARM: ux500: Reclassify PRCMU AUXDATA entry Lee Jones
2013-06-06 12:17 ` [PATCH 33/33] ARM: ux500: Remove SSP AUXDATA pertaining to DMA bindings Lee Jones
2013-06-12 13:27 ` [PATCH 00/32] ARM: ux500: Enable clocks for Device Tree Lee Jones
2013-06-13  8:41   ` Linus Walleij
2013-06-13  9:34     ` Lee Jones

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='CAPDyKFozMRxcpeEfaqKQrZ1_xxr=UMLw2EybdKiRM2zWcppikw@mail.gmail.com' \
    --to=ulf.hansson@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).