2010-01-30 16:37:24

by Christian Hesse

[permalink] [raw]
Subject: Samsung N220 backlight support

Hello Greg,

this adds support for Samsung N220 to your samsung-backlight. Tested on my
device, works for me.

Regards,
Chris

Signed-off-by: Christian Hesse <[email protected]>


Attachments:
samsung-backlight-n220.diff (1.39 kB)

2010-01-30 16:28:40

by Greg KH

[permalink] [raw]
Subject: Re: Samsung N220 backlight support

On Sat, Jan 30, 2010 at 05:06:18PM +0100, Christian Hesse wrote:
> Hello Greg,
>
> this adds support for Samsung N220 to your samsung-backlight. Tested on my
> device, works for me.

Hm, the driver that ended up in the kernel is very different from this
version. Can you work with that version and add the needed changes
there and generate a patch?

thanks,

greg k-h

2010-01-30 18:39:50

by Christian Hesse

[permalink] [raw]
Subject: Re: Samsung N220 backlight support

On Saturday 30 January 2010 17:28:57 you wrote:
> On Sat, Jan 30, 2010 at 05:06:18PM +0100, Christian Hesse wrote:
> > Hello Greg,
> >
> > this adds support for Samsung N220 to your samsung-backlight. Tested on
> > my device, works for me.
>
> Hm, the driver that ended up in the kernel is very different from this
> version. Can you work with that version and add the needed changes
> there and generate a patch?

Oh, did not notice it was included in mainline already.
I applied the attached patch. Insmod now gives:

insmod: error inserting 'samsung-laptop.ko': -1 Invalid parameters

and the logs contain the following:

kernel: samsung_laptop: found laptop model 'N220'
kernel: This computer does not support SABI

What is this SABI and SECLINUX thing about? Do you have a link to any more
information? Is it possible that the N220 does not have this?
The N220 uses a Pine Train platform (featuring N450 with 64bit support), so I
installed x86_64 to the device.

I would be glad to test whatever you give me.

Regards,
Chris


Attachments:
samsung-laptop-n220.diff (593.00 B)

2010-01-31 00:36:11

by Greg KH

[permalink] [raw]
Subject: Re: Samsung N220 backlight support

On Sat, Jan 30, 2010 at 07:37:34PM +0100, Christian Hesse wrote:
> Oh, did not notice it was included in mainline already.
> I applied the attached patch. Insmod now gives:
>
> insmod: error inserting 'samsung-laptop.ko': -1 Invalid parameters
>
> and the logs contain the following:
>
> kernel: samsung_laptop: found laptop model 'N220'
> kernel: This computer does not support SABI
>
> What is this SABI and SECLINUX thing about?

It is the BIOS interface that the kernel talks to.

> Do you have a link to any more information?

No, I don't know of any public information on this interface, sorry.

> Is it possible that the N220 does not have this?

Yes, it does not.

> The N220 uses a Pine Train platform (featuring N450 with 64bit support), so I
> installed x86_64 to the device.
>
> I would be glad to test whatever you give me.

I'm sorry, but I am not allowed to make any changes to the driver to
support other laptops at this time, but I can accept patches written by
others.

Yeah, it sucks, sorry.

greg k-h