From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Date: Tue, 17 Sep 2013 16:20:22 +0000 Subject: Re: [PATCH 03/15] OMAPDSS: fix DPI and SDI device ids Message-Id: <20130917162022.GC9994@atomide.com> List-Id: References: <1377783120-14001-1-git-send-email-tomi.valkeinen@ti.com> <1377783120-14001-4-git-send-email-tomi.valkeinen@ti.com> In-Reply-To: <1377783120-14001-4-git-send-email-tomi.valkeinen@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Tomi Valkeinen Cc: linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org, Archit Taneja * Tomi Valkeinen [130829 06:40]: > The DPI and SDI platform devices are currently created with the ID of > -1. The ID doesn't currently affect anything. > > However, we have added regulator supply entries for "omapdss_dpi.0" and > "omapdss_sdi.0" to the board files, although these supply entries are > not yet used. As the ID used for the devices is -1, these regulator > supply entries will not work. > > To fix the issue, assign ID of 0 to the devices. In the future there may > be more than one DPI or SDI output, so it makes sense to have a proper > ID for them. This should be safe to queue by you along with other DSS patches: Acked-by: Tony Lindgren > Signed-off-by: Tomi Valkeinen > --- > arch/arm/mach-omap2/display.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/mach-omap2/display.c b/arch/arm/mach-omap2/display.c > index ff37be1..03a0516 100644 > --- a/arch/arm/mach-omap2/display.c > +++ b/arch/arm/mach-omap2/display.c > @@ -400,7 +400,7 @@ int __init omap_display_init(struct omap_dss_board_info *board_data) > > /* Create devices for DPI and SDI */ > > - pdev = create_simple_dss_pdev("omapdss_dpi", -1, > + pdev = create_simple_dss_pdev("omapdss_dpi", 0, > board_data, sizeof(*board_data), dss_pdev); > if (IS_ERR(pdev)) { > pr_err("Could not build platform_device for omapdss_dpi\n"); > @@ -408,7 +408,7 @@ int __init omap_display_init(struct omap_dss_board_info *board_data) > } > > if (cpu_is_omap34xx()) { > - pdev = create_simple_dss_pdev("omapdss_sdi", -1, > + pdev = create_simple_dss_pdev("omapdss_sdi", 0, > board_data, sizeof(*board_data), dss_pdev); > if (IS_ERR(pdev)) { > pr_err("Could not build platform_device for omapdss_sdi\n"); > -- > 1.8.1.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html