lkml.org 
[lkml]   [2019]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] ipmi: add i2c-addr property to ipmi platform device
On Wed, Apr 24, 2019 at 09:48:36AM +0000, Kamlakant Patel wrote:
> We need 'i2c-addr' property entry for ssif to work.
> Otherwise, it thorws following error:
> "ipmi_ssif dmi-ipmi-ssif.0: ipmi_ssif: No i2c-addr property"

The patch really isn't right, you only want to add the i2c-addr
property if the interface is SSIF. But there's no easy way to
tell this in the current code. I'll work something up.

Also, your mailer has added DOS line endings to all your
patches and I can't apply them cleanly. If there's any way
you can use git-send for sending patches, that would be ideal.

-corey

>
> Signed-off-by: Kamlakant Patel <kamlakantp@marvell.com>
> ---
> drivers/char/ipmi/ipmi_plat_data.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/char/ipmi/ipmi_plat_data.c b/drivers/char/ipmi/ipmi_plat_data.c
> index 8f0ca2a..1075076 100644
> --- a/drivers/char/ipmi/ipmi_plat_data.c
> +++ b/drivers/char/ipmi/ipmi_plat_data.c
> @@ -34,6 +34,8 @@ struct platform_device *ipmi_platform_add(const char *name, unsigned int inst,
> p->regspacing = p->regsize;
>
> pr[pidx++] = PROPERTY_ENTRY_U8("ipmi-type", p->type);
> + if (p->addr)
> + pr[pidx++] = PROPERTY_ENTRY_U16("i2c-addr", p->addr);
> if (p->slave_addr)
> pr[pidx++] = PROPERTY_ENTRY_U8("slave-addr", p->slave_addr);
> pr[pidx++] = PROPERTY_ENTRY_U8("addr-source", p->addr_source);
> --
> 1.8.3.1
>

\
 
 \ /
  Last update: 2019-04-24 15:26    [W:0.044 / U:0.776 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site