From: Jun Li <jun.li@nxp.com> To: Peter Chen <hzpeterchen@gmail.com> Cc: Roger Quadros <rogerq@ti.com>, "stern@rowland.harvard.edu" <stern@rowland.harvard.edu>, "balbi@kernel.org" <balbi@kernel.org>, "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>, "peter.chen@freescale.com" <peter.chen@freescale.com>, "dan.j.williams@intel.com" <dan.j.williams@intel.com>, "jun.li@freescale.com" <jun.li@freescale.com>, "mathias.nyman@linux.intel.com" <mathias.nyman@linux.intel.com>, "tony@atomide.com" <tony@atomide.com>, "Joao.Pinto@synopsys.com" <Joao.Pinto@synopsys.com>, "abrestic@chromium.org" <abrestic@chromium.org>, "r.baldyga@samsung.com" <r.baldyga@samsung.com>, "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org> Subject: RE: [PATCH v6 07/12] usb: otg: add OTG/dual-role core Date: Tue, 26 Apr 2016 05:11:36 +0000 [thread overview] Message-ID: <AM4PR04MB2130124412E09799A2CB01BF89630@AM4PR04MB2130.eurprd04.prod.outlook.com> (raw) In-Reply-To: <20160426034711.GC2978@peterchendt> Hi > -----Original Message----- > From: Peter Chen [mailto:hzpeterchen@gmail.com] > Sent: Tuesday, April 26, 2016 11:47 AM > To: Jun Li <jun.li@nxp.com> > Cc: Roger Quadros <rogerq@ti.com>; stern@rowland.harvard.edu; > balbi@kernel.org; gregkh@linuxfoundation.org; peter.chen@freescale.com; > dan.j.williams@intel.com; jun.li@freescale.com; > mathias.nyman@linux.intel.com; tony@atomide.com; Joao.Pinto@synopsys.com; > abrestic@chromium.org; r.baldyga@samsung.com; linux-usb@vger.kernel.org; > linux-kernel@vger.kernel.org; linux-omap@vger.kernel.org > Subject: Re: [PATCH v6 07/12] usb: otg: add OTG/dual-role core > > On Tue, Apr 26, 2016 at 02:07:56AM +0000, Jun Li wrote: > > > +struct usb_otg *usb_otg_register(struct device *dev, > > > + struct usb_otg_config *config) { > > > + struct usb_otg *otg; > > > + struct otg_wait_data *wait; > > > + int ret = 0; > > > + > > > + if (!dev || !config || !config->fsm_ops) > > > + return ERR_PTR(-EINVAL); > > > + > > > + /* already in list? */ > > > + mutex_lock(&otg_list_mutex); > > > + if (usb_otg_get_data(dev)) { > > > + dev_err(dev, "otg: %s: device already in otg list\n", > > > + __func__); > > > + ret = -EINVAL; > > > + goto unlock; > > > + } > > > + > > > + /* allocate and add to list */ > > > + otg = kzalloc(sizeof(*otg), GFP_KERNEL); > > > + if (!otg) { > > > + ret = -ENOMEM; > > > + goto unlock; > > > + } > > > + > > > + otg->dev = dev; > > > + otg->caps = config->otg_caps; > > > + > > > + if ((otg->caps->hnp_support || otg->caps->srp_support || > > > + otg->caps->adp_support) && !config->otg_work) > > > + dev_info(dev, "otg: limiting to dual-role\n"); > > > > dev_err, this should be an error. > > The condition may be wrong, but it is an information to show that current > OTG is dual-role. This should not happen in any correct design, I even doubt if we should try to continue by "downgrade" it to be duel role, currently the only example user is dual role, so doing like this can't be tested by real case, this downgrade is not so easy like we image, at least for chipidea otg driver, simply replace a queue worker may not work, as we have much more difference between the 2 configs. Li Jun > > Peter > > > > > + > > > + if (config->otg_work) /* custom otg_work ? */ > > > + INIT_WORK(&otg->work, config->otg_work); > > > + else > > > + INIT_WORK(&otg->work, usb_otg_work); > > > + > > > + otg->wq = create_singlethread_workqueue("usb_otg"); > > > + if (!otg->wq) { > > > + dev_err(dev, "otg: %s: can't create workqueue\n", > > > + __func__); > > > + ret = -ENOMEM; > > > + goto err_wq; > > > + } > > > + > > > + /* set otg ops */ > > > + otg->fsm.ops = config->fsm_ops; > > > + > > > + mutex_init(&otg->fsm.lock); > > > + > > > + list_add_tail(&otg->list, &otg_list); > > > + mutex_unlock(&otg_list_mutex); > > > + > > > + /* were we in wait list? */ > > > + mutex_lock(&wait_list_mutex); > > > + wait = usb_otg_get_wait(dev); > > > + mutex_unlock(&wait_list_mutex); > > > + if (wait) { > > > + /* register pending host/gadget and flush from list */ > > > + usb_otg_flush_wait(dev); > > > + } > > > + > > > + return otg; > > > + > > > +err_wq: > > > + kfree(otg); > > > +unlock: > > > + mutex_unlock(&otg_list_mutex); > > > + return ERR_PTR(ret); > > > +} > > > +EXPORT_SYMBOL_GPL(usb_otg_register); > > > + > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-usb" > > in the body of a message to majordomo@vger.kernel.org More majordomo > > info at http://vger.kernel.org/majordomo-info.html > > -- > Best Regards, > Peter Chen
WARNING: multiple messages have this Message-ID (diff)
From: Jun Li <jun.li-3arQi8VN3Tc@public.gmane.org> To: Peter Chen <hzpeterchen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org>, "stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org" <stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org>, "balbi-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" <balbi-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, "gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org" <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>, "peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org" <peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, "dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org" <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, "jun.li-KZfg59tc24xl57MIdRCFDg@public.gmane.org" <jun.li-KZfg59tc24xl57MIdRCFDg@public.gmane.org>, "mathias.nyman-VuQAYsv1563Yd54FQh9/CA@public.gmane.org" <mathias.nyman-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>, "tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org" <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>, "Joao.Pinto-HKixBCOQz3hWk0Htik3J/w@public.gmane.org" <Joao.Pinto-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>, "abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org" <abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>, "r.baldyga-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org" <r.baldyga-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>, "linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: RE: [PATCH v6 07/12] usb: otg: add OTG/dual-role core Date: Tue, 26 Apr 2016 05:11:36 +0000 [thread overview] Message-ID: <AM4PR04MB2130124412E09799A2CB01BF89630@AM4PR04MB2130.eurprd04.prod.outlook.com> (raw) In-Reply-To: <20160426034711.GC2978@peterchendt> Hi > -----Original Message----- > From: Peter Chen [mailto:hzpeterchen-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org] > Sent: Tuesday, April 26, 2016 11:47 AM > To: Jun Li <jun.li-3arQi8VN3Tc@public.gmane.org> > Cc: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org>; stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org; > balbi-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org; gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org; peter.chen-KZfg59tc24xl57MIdRCFDg@public.gmane.org; > dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org; jun.li-KZfg59tc24xl57MIdRCFDg@public.gmane.org; > mathias.nyman-VuQAYsv1563Yd54FQh9/CA@public.gmane.org; tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org; Joao.Pinto-HKixBCOQz3hWk0Htik3J/w@public.gmane.org; > abrestic-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org; r.baldyga-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org; linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; > linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH v6 07/12] usb: otg: add OTG/dual-role core > > On Tue, Apr 26, 2016 at 02:07:56AM +0000, Jun Li wrote: > > > +struct usb_otg *usb_otg_register(struct device *dev, > > > + struct usb_otg_config *config) { > > > + struct usb_otg *otg; > > > + struct otg_wait_data *wait; > > > + int ret = 0; > > > + > > > + if (!dev || !config || !config->fsm_ops) > > > + return ERR_PTR(-EINVAL); > > > + > > > + /* already in list? */ > > > + mutex_lock(&otg_list_mutex); > > > + if (usb_otg_get_data(dev)) { > > > + dev_err(dev, "otg: %s: device already in otg list\n", > > > + __func__); > > > + ret = -EINVAL; > > > + goto unlock; > > > + } > > > + > > > + /* allocate and add to list */ > > > + otg = kzalloc(sizeof(*otg), GFP_KERNEL); > > > + if (!otg) { > > > + ret = -ENOMEM; > > > + goto unlock; > > > + } > > > + > > > + otg->dev = dev; > > > + otg->caps = config->otg_caps; > > > + > > > + if ((otg->caps->hnp_support || otg->caps->srp_support || > > > + otg->caps->adp_support) && !config->otg_work) > > > + dev_info(dev, "otg: limiting to dual-role\n"); > > > > dev_err, this should be an error. > > The condition may be wrong, but it is an information to show that current > OTG is dual-role. This should not happen in any correct design, I even doubt if we should try to continue by "downgrade" it to be duel role, currently the only example user is dual role, so doing like this can't be tested by real case, this downgrade is not so easy like we image, at least for chipidea otg driver, simply replace a queue worker may not work, as we have much more difference between the 2 configs. Li Jun > > Peter > > > > > + > > > + if (config->otg_work) /* custom otg_work ? */ > > > + INIT_WORK(&otg->work, config->otg_work); > > > + else > > > + INIT_WORK(&otg->work, usb_otg_work); > > > + > > > + otg->wq = create_singlethread_workqueue("usb_otg"); > > > + if (!otg->wq) { > > > + dev_err(dev, "otg: %s: can't create workqueue\n", > > > + __func__); > > > + ret = -ENOMEM; > > > + goto err_wq; > > > + } > > > + > > > + /* set otg ops */ > > > + otg->fsm.ops = config->fsm_ops; > > > + > > > + mutex_init(&otg->fsm.lock); > > > + > > > + list_add_tail(&otg->list, &otg_list); > > > + mutex_unlock(&otg_list_mutex); > > > + > > > + /* were we in wait list? */ > > > + mutex_lock(&wait_list_mutex); > > > + wait = usb_otg_get_wait(dev); > > > + mutex_unlock(&wait_list_mutex); > > > + if (wait) { > > > + /* register pending host/gadget and flush from list */ > > > + usb_otg_flush_wait(dev); > > > + } > > > + > > > + return otg; > > > + > > > +err_wq: > > > + kfree(otg); > > > +unlock: > > > + mutex_unlock(&otg_list_mutex); > > > + return ERR_PTR(ret); > > > +} > > > +EXPORT_SYMBOL_GPL(usb_otg_register); > > > + > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-usb" > > in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo > > info at http://vger.kernel.org/majordomo-info.html > > -- > Best Regards, > Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-04-26 5:11 UTC|newest] Thread overview: 155+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-05 14:05 [PATCH v6 00/12] USB OTG/dual-role framework Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 01/12] usb: hcd: Initialize hcd->flags to 0 Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-06 6:09 ` Felipe Balbi 2016-04-06 6:09 ` Felipe Balbi 2016-04-06 6:32 ` Roger Quadros 2016-04-06 6:32 ` Roger Quadros 2016-04-07 9:42 ` Peter Chen 2016-04-07 10:40 ` Roger Quadros 2016-04-07 10:40 ` Roger Quadros 2016-04-08 1:01 ` Peter Chen 2016-04-08 1:01 ` Peter Chen 2016-04-08 7:16 ` Roger Quadros 2016-04-08 7:16 ` Roger Quadros 2016-04-08 7:45 ` Peter Chen 2016-04-08 7:45 ` Peter Chen 2016-04-18 2:29 ` Peter Chen 2016-04-18 14:11 ` Alan Stern 2016-04-18 14:11 ` Alan Stern 2016-04-19 1:56 ` Peter Chen 2016-04-19 1:56 ` Peter Chen 2016-04-20 8:15 ` Roger Quadros 2016-04-20 8:15 ` Roger Quadros 2016-04-20 9:40 ` Peter Chen 2016-04-05 14:05 ` [PATCH v6 02/12] usb: hcd.h: Add OTG to HCD interface Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 7:41 ` Peter Chen 2016-04-05 14:05 ` [PATCH v6 03/12] usb: otg-fsm: use usb_otg wherever possible Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 7:42 ` Peter Chen 2016-04-05 14:05 ` [PATCH v6 04/12] usb: otg-fsm: move host controller operations into usb_otg->hcd_ops Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 8:00 ` Peter Chen 2016-04-18 8:00 ` Peter Chen 2016-04-05 14:05 ` [PATCH v6 05/12] usb: gadget.h: Add OTG to gadget interface Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 06/12] usb: otg: get rid of CONFIG_USB_OTG_FSM in favour of CONFIG_USB_OTG Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 8:05 ` Peter Chen 2016-04-20 8:12 ` Roger Quadros 2016-04-20 8:12 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 07/12] usb: otg: add OTG/dual-role core Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-07 8:52 ` Yoshihiro Shimoda 2016-04-07 11:45 ` Roger Quadros 2016-04-08 11:22 ` Yoshihiro Shimoda 2016-04-11 10:54 ` Roger Quadros 2016-04-14 8:36 ` Yoshihiro Shimoda 2016-04-14 10:59 ` Roger Quadros 2016-04-14 11:15 ` Yoshihiro Shimoda 2016-04-14 11:15 ` Yoshihiro Shimoda 2016-04-14 11:32 ` Roger Quadros 2016-04-14 11:32 ` Roger Quadros 2016-04-15 9:59 ` Yoshihiro Shimoda 2016-04-15 9:59 ` Yoshihiro Shimoda 2016-04-15 10:57 ` Roger Quadros 2016-04-15 10:57 ` Roger Quadros 2016-04-15 10:03 ` Yoshihiro Shimoda 2016-04-19 9:18 ` Peter Chen 2016-04-20 5:08 ` Yoshihiro Shimoda 2016-04-20 7:03 ` Roger Quadros 2016-04-22 6:05 ` Peter Chen 2016-04-22 6:05 ` Peter Chen 2016-04-22 1:26 ` Peter Chen 2016-04-22 3:34 ` Peter Chen 2016-04-22 3:34 ` Peter Chen 2016-04-22 5:57 ` Yoshihiro Shimoda 2016-04-22 5:57 ` Yoshihiro Shimoda 2016-04-15 9:25 ` Peter Chen 2016-04-15 9:25 ` Peter Chen 2016-04-15 11:00 ` Roger Quadros 2016-04-15 11:00 ` Roger Quadros 2016-04-18 2:09 ` Peter Chen 2016-04-20 6:54 ` Roger Quadros 2016-04-20 6:54 ` Roger Quadros 2016-04-20 9:26 ` Peter Chen 2016-04-19 8:06 ` Peter Chen 2016-04-20 7:02 ` Roger Quadros 2016-04-20 7:02 ` Roger Quadros 2016-04-20 9:39 ` Peter Chen 2016-04-21 6:52 ` Peter Chen 2016-04-21 6:52 ` Peter Chen 2016-04-25 14:05 ` Roger Quadros 2016-04-25 14:05 ` Roger Quadros 2016-04-26 2:07 ` Jun Li 2016-04-26 3:47 ` Peter Chen 2016-04-26 3:47 ` Peter Chen 2016-04-26 5:11 ` Jun Li [this message] 2016-04-26 5:11 ` Jun Li 2016-04-26 6:28 ` Peter Chen 2016-04-26 6:28 ` Peter Chen 2016-04-26 7:00 ` Jun Li 2016-04-26 7:00 ` Jun Li 2016-04-26 8:21 ` Peter Chen 2016-04-27 3:15 ` Peter Chen 2016-04-27 10:59 ` Roger Quadros 2016-04-27 10:59 ` Roger Quadros 2016-04-28 1:54 ` Peter Chen 2016-04-28 1:54 ` Peter Chen 2016-04-28 8:01 ` Roger Quadros 2016-04-28 8:01 ` Roger Quadros 2016-04-27 11:15 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 08/12] usb: hcd: Adapt to OTG core Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 6:29 ` Peter Chen 2016-04-19 8:14 ` Peter Chen 2016-04-20 6:47 ` Roger Quadros 2016-04-20 6:47 ` Roger Quadros 2016-04-20 6:46 ` Roger Quadros 2016-04-20 6:46 ` Roger Quadros 2016-04-27 10:16 ` Jun Li 2016-04-27 11:00 ` Roger Quadros 2016-04-27 11:11 ` Roger Quadros 2016-04-27 12:49 ` Jun Li 2016-04-27 13:18 ` Jun Li 2016-04-05 14:05 ` [PATCH v6 09/12] usb: gadget: udc: adapt " Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 6:59 ` Peter Chen 2016-04-18 6:59 ` Peter Chen 2016-04-20 6:51 ` Roger Quadros 2016-04-20 6:51 ` Roger Quadros 2016-04-21 6:38 ` Jun Li 2016-04-25 14:04 ` Roger Quadros 2016-04-26 0:07 ` Jun Li 2016-04-27 11:22 ` Roger Quadros 2016-04-27 11:22 ` Roger Quadros 2016-04-28 9:54 ` Roger Quadros 2016-04-28 9:54 ` Roger Quadros 2016-04-28 10:23 ` Jun Li 2016-04-28 12:22 ` Roger Quadros 2016-05-03 7:06 ` Jun Li 2016-05-03 15:44 ` Roger Quadros 2016-05-04 1:47 ` Peter Chen 2016-05-04 1:47 ` Peter Chen 2016-05-04 3:35 ` Peter Chen 2016-05-04 6:37 ` Roger Quadros 2016-05-04 7:53 ` Peter Chen 2016-05-04 8:03 ` Jun Li 2016-05-04 8:40 ` Roger Quadros 2016-05-04 8:40 ` Roger Quadros 2016-05-04 8:39 ` Peter Chen 2016-04-05 14:05 ` [PATCH v6 10/12] usb: doc: dt-binding: Add otg-controller property Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 11/12] usb: core: hub: Notify OTG fsm when A device sets b_hnp_enable Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-18 7:08 ` Peter Chen 2016-04-27 14:35 ` Roger Quadros 2016-04-27 14:35 ` Roger Quadros 2016-04-05 14:05 ` [PATCH v6 12/12] usb: host: xhci-plat: Add otg device to platform data Roger Quadros 2016-04-05 14:05 ` Roger Quadros 2016-04-06 3:23 ` Yoshihiro Shimoda 2016-04-06 3:23 ` Yoshihiro Shimoda 2016-04-06 6:30 ` Roger Quadros 2016-04-06 6:30 ` Roger Quadros
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=AM4PR04MB2130124412E09799A2CB01BF89630@AM4PR04MB2130.eurprd04.prod.outlook.com \ --to=jun.li@nxp.com \ --cc=Joao.Pinto@synopsys.com \ --cc=abrestic@chromium.org \ --cc=balbi@kernel.org \ --cc=dan.j.williams@intel.com \ --cc=gregkh@linuxfoundation.org \ --cc=hzpeterchen@gmail.com \ --cc=jun.li@freescale.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=mathias.nyman@linux.intel.com \ --cc=peter.chen@freescale.com \ --cc=r.baldyga@samsung.com \ --cc=rogerq@ti.com \ --cc=stern@rowland.harvard.edu \ --cc=tony@atomide.com \ /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.