Page MenuHomeFreeBSD

Add SSP SPI driver for Intel Baytrail SoC
ClosedPublic

Authored by gonzo on Dec 23 2016, 11:05 PM.
Tags
None
Referenced Files
F105934902: D8896.diff
Sun, Dec 22, 7:12 PM
F105934863: D8896.diff
Sun, Dec 22, 7:12 PM
Unknown Object (File)
Wed, Nov 27, 2:27 AM
Unknown Object (File)
Nov 12 2024, 1:51 PM
Unknown Object (File)
Nov 12 2024, 8:58 AM
Unknown Object (File)
Nov 12 2024, 8:41 AM
Unknown Object (File)
Nov 12 2024, 7:14 AM
Unknown Object (File)
Nov 12 2024, 5:33 AM
Subscribers
None

Details

Summary

Add SPI driver for multi-functional serial port (SSP) that can
be found in Intel's SoCs. I only have Baytrail SoC so this is the only
supported hardware for now.

Test Plan

Use SSD1306 for basic tests

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

gonzo retitled this revision from to Add SSP SPI driver for Intel Baytrail SoC.
gonzo updated this object.
gonzo edited the test plan for this revision. (Show Details)
gonzo added reviewers: manu, imp, br.
br edited edge metadata.

looks good

sys/dev/intel/spi.c
79 ↗(On Diff #23229)

we usually do
​#define<tab>INTELSPI_SSPREG_SSCR0
​#define<tab><space>SSCR0_SCR(n)
instead of
​#define<tab>INTELSPI_SSPREG_SSCR0
​#define<tab><tab>SSCR0_SCR(n)

156 ↗(On Diff #23229)
        if (sssr & SSSR_TNF)
	​                return (0);

        return (1);
385 ↗(On Diff #23229)

remove empty line here

This revision is now accepted and ready to land.Dec 24 2016, 9:32 PM
manu edited edge metadata.

Wouldn't be better to directly add support for mode/clock setting in transfer ?

In D8896#184336, @manu wrote:

Wouldn't be better to directly add support for mode/clock setting in transfer ?

Yes. I'll probably do this before committing the driver

This revision was automatically updated to reflect the committed changes.
In D8896#184406, @gonzo wrote:
In D8896#184336, @manu wrote:

Wouldn't be better to directly add support for mode/clock setting in transfer ?

Yes. I'll probably do this before committing the driver

I will add mode/clock support in the next round of change.